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

Tracking issue for borrows outliving owners with destructors #33206

Closed
pnkfelix opened this issue Apr 25, 2016 · 3 comments
Closed

Tracking issue for borrows outliving owners with destructors #33206

pnkfelix opened this issue Apr 25, 2016 · 3 comments
Labels
C-tracking-issue Category: A tracking issue for an RFC or an unstable feature. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Comments

@pnkfelix
Copy link
Member

This is a tracking issue for a planned fix for issue #31567

Any fix for #31567 is going to be a breaking change; thus, in accordance with existing policy (and draft RFC rust-lang/rfcs#1589 ) I am planning to have the change emit warnings at first, and then after a warning cycle, we can upgrade it to an error.

So, I'm grabbing this issue now while I am working on the fix, so that my warning/error messages can reference a stable URL.
@brson brson added T-lang Relevant to the language team, which will review and decide on the PR/issue. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. and removed T-lang Relevant to the language team, which will review and decide on the PR/issue. labels Mar 1, 2017
@brson
Copy link
Contributor

brson commented Mar 1, 2017

@pnkfelix Any updates?

@Mark-Simulacrum Mark-Simulacrum added the C-tracking-issue Category: A tracking issue for an RFC or an unstable feature. label Jul 25, 2017
@pnkfelix
Copy link
Member Author

I think this is fixed by NLL

@pnkfelix
Copy link
Member Author

But also note PR #52782 which hasn’t landed bur does include tests that illustrate the breaking change I discussed here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C-tracking-issue Category: A tracking issue for an RFC or an unstable feature. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@steveklabnik @brson @pnkfelix @Mark-Simulacrum