Skip to content

don't leak internal temporaries from dbg!#154994

Open
dianne wants to merge 1 commit intorust-lang:mainfrom
dianne:no-dbg-temp
Open

don't leak internal temporaries from dbg!#154994
dianne wants to merge 1 commit intorust-lang:mainfrom
dianne:no-dbg-temp

Conversation

@dianne
Copy link
Copy Markdown
Contributor

@dianne dianne commented Apr 8, 2026

Fixes #154988

r? @Mark-Simulacrum as the reviewer of #154074

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Apr 8, 2026
@dianne
Copy link
Copy Markdown
Contributor Author

dianne commented Apr 8, 2026

I'd feel most comfortable reverting #154074 and #149869 on beta, but I'll beta-nominate this in case there's time to vet it and a preference to go ahead with it. It's a small diff, but macro changes are subtle.

@dianne dianne added the beta-nominated Nominated for backporting to the compiler in the beta channel. label Apr 8, 2026
@cuviper
Copy link
Copy Markdown
Member

cuviper commented Apr 8, 2026

We discussed this today in the libs meeting. We agree that reverting is the safer option for 1.95, given the late timing. However, this PR might also miss the 1.96 branch if it's not merged quickly, and then we probably will want a backport there. So, we're going to leave the nomination for now and see where things stand next week.

@cuviper
Copy link
Copy Markdown
Member

cuviper commented Apr 8, 2026

I've posted reverts in #154999

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

beta-nominated Nominated for backporting to the compiler in the beta channel. S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-libs Relevant to the library team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1.95 beta regression: "lua does not live long enough"

4 participants