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.

Sign up for GitHub

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

Jump to bottom

Leak with do loops #1257

Closed
brson opened this issue Dec 4, 2011 · 2 comments
Closed

Leak with do loops #1257

brson opened this issue Dec 4, 2011 · 2 comments
Milestone
0.1 first support...

Comments

@brson
Copy link
Contributor

brson commented Dec 4, 2011 

fn main () {
    let line = "";
    let i = 0;
    do {
        line = if i == 3 { "exit" } else { "notexit" };
        i += 1;
    } while line != "exit";
}

This leaks the string.
lht added a commit to lht/rust that referenced this issue Dec 4, 2011
@lht
rustc: Fix memory leak in do-while loop
96b0881 
Issue rust-lang#1257
@lht
Copy link
Contributor

lht commented Dec 4, 2011

Root cause is the condition BB missed a cleanup.

@brson
Copy link
Contributor Author

brson commented Dec 4, 2011

Merged. Thanks.

@brson brson closed this as completed Dec 4, 2011
bjorn3 added a commit to bjorn3/rust that referenced this issue Aug 24, 2022
@bjorn3
Merge pull request rust-lang#1257 from uweigand/memcpy-align
64c73d0 
Fix alignment flag for emit_small_memory_copy
coastalwhite pushed a commit to coastalwhite/rust that referenced this issue Aug 5, 2023
@Amanieu
Add missing vtst_p16 and vtstq_p16 intrinsics (rust-lang#1257)
f4fd7ef
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.1 first supported public release
Development

No branches or pull requests
2 participants
@lht @brson