Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

leakage in append! functions for two MutableLinkedList's #794

Open
algorithmx opened this issue Mar 24, 2022 · 6 comments
Open

leakage in append! functions for two MutableLinkedList's #794

algorithmx opened this issue Mar 24, 2022 · 6 comments
Labels

Comments

@algorithmx
Copy link

function Base.append!(l1::MutableLinkedList{T}, l2::MutableLinkedList{T}) where T

Code

A = MutableLinkedList{Int}(1,2,3)
B = MutableLinkedList{Int}(1,2,3)
append!(A,B) # endless loop

Version

julia> versioninfo()
Julia Version 1.7.2
Commit bf53498635 (2022-02-06 15:21 UTC)
Platform Info:
  OS: Linux (x86_64-pc-linux-gnu)
  CPU: Intel(R) Core(TM) i7-7700HQ CPU @ 2.80GHz
  WORD_SIZE: 64
  LIBM: libopenlibm
  LLVM: libLLVM-12.0.1 (ORCJIT, skylake)
Environment:
  JULIA_EDITOR = code
  JULIA_NUM_THREADS = 

julia> 

@algorithmx
Copy link
Author

related issue
#795

@algorithmx
Copy link
Author

The local files on my computer is different from the files on GitHub, but the version number is the same ...

@algorithmx algorithmx reopened this Mar 24, 2022
@algorithmx
Copy link
Author

I have examined the local files, I found that the fix wasn't merged into the master branch correctly. Can anyone check the status of the repo?
I have installed v0.18.10 first, this gives me a folder nBjdy in the package folder. Then I update to v0.18.11, got one more foledr with name vSp4s. The file mutable_list.jl in both versions are different from the file on GitHub, leaving the bug unfixed ...

@algorithmx
Copy link
Author

@grahamas

@algorithmx
Copy link
Author

@oxinabox

@synthfi
Copy link
Contributor

synthfi commented Sep 1, 2023

this is also what I encountered in #739

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants