You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Nope, just leaving a historical note for anyone else who was confused. Although if anyone knows when this was fixed (and if a test was added?) that would be good to note.
Activity
Centril commentedon Dec 29, 2017
Dumping this link here for future reference: https://github.com/search?utf8=%E2%9C%93&q=leak+extension%3A.rs+language%3ARust&type=Code&ref=advsearch&l=Rust&l=
stabilize (version: 1.26.0) Box::leak, cc rust-lang#46179
Rollup merge of rust-lang#48110 - Centril:stabilize/box_leak, r=alexc…
Centril commentedon Mar 18, 2018
Closing since
Box::leakis stable in nightly and will make its way to stable in 1.26.0.Gankra commentedon May 8, 2018
I was confused why this was allowed to stabilize, but it appears that the bug that made this API unsound was fixed?
Centril commentedon May 9, 2018
@gankro https://play.rust-lang.org/?gist=66d0329787975630e4f0cc62da61ed46&version=stable&mode=debug
The
dupemechanism doesn't work anymore at least.Should we escalate this to a new issue with P-high?
Gankra commentedon May 9, 2018
Nope, just leaving a historical note for anyone else who was confused. Although if anyone knows when this was fixed (and if a test was added?) that would be good to note.