-
Notifications
You must be signed in to change notification settings - Fork 13.3k
procs should implement Finally #10655
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
Comments
cc @bjz |
Can we replace |
@thestinger: @nikomatsakis mentioned on #9842 a month or so ago that there are some destructor soudness issues that need to be worked out to make scope guards work properly. |
If we're not going to have stack-based |
Procs no longer exist, and it would seem that |
~fn
used to implementstd::unstable::finally::Finally
, butproc
doesn't.A test should also be added so this doesn't break again.
It appears that one workaround for this is to switch to
extern "Rust" fn
which does implementFinally
.The text was updated successfully, but these errors were encountered: