Skip to content

Conversation

@BoxyUwU
Copy link
Member

@BoxyUwU BoxyUwU commented Jan 7, 2026

Fixes #150648

I don't think this matters for any non 'static lifetimes as defining the struct as Wrap<'a> would mean the lifetime argument gets replaced with an nll var before MIR type checking meaning the struct tail is dyn Trait + '?x

r? types

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Jan 7, 2026
@rust-log-analyzer

This comment has been minimized.

@lcnr
Copy link
Contributor

lcnr commented Jan 7, 2026

r=me after CI

@BoxyUwU BoxyUwU force-pushed the map_regions_for_outlives branch from 5257d68 to 3d965bd Compare January 7, 2026 13:27
@BoxyUwU
Copy link
Member Author

BoxyUwU commented Jan 7, 2026

r? lcnr

@rustbot rustbot assigned lcnr and unassigned jackh726 Jan 7, 2026
@BoxyUwU
Copy link
Member Author

BoxyUwU commented Jan 7, 2026

@bors r=lcnr rollup

@bors
Copy link
Collaborator

bors commented Jan 7, 2026

📌 Commit 3d965bd has been approved by lcnr

It is now in the queue for this repository.

@rust-bors rust-bors bot added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Jan 7, 2026
@bors
Copy link
Collaborator

bors commented Jan 7, 2026

🌲 The tree is currently closed for pull requests below priority 1000. This pull request will be tested once the tree is reopened.

@rust-bors
Copy link
Contributor

rust-bors bot commented Jan 7, 2026

📌 Commit 3d965bd has been approved by lcnr

It is now in the queue for this repository.

🌲 The tree is currently closed for pull requests below priority 50. This pull request will be tested once the tree is reopened.

@rust-bors rust-bors bot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 7, 2026
rust-bors bot added a commit that referenced this pull request Jan 7, 2026
Rollup of 4 pull requests

Successful merges:

 - #150738 (Factorize `triagebot.toml` float parsing mentions with a glob matching)
 - #150761 (rustc book: fix grammar)
 - #150764 (Convert static lifetime to an nll var)
 - #150775 (Move issue-12660 to 'ui/cross-crate/ with a descriptive name)

r? @ghost
@rust-bors rust-bors bot merged commit 6b262f7 into rust-lang:main Jan 8, 2026
11 checks passed
@rustbot rustbot added this to the 1.94.0 milestone Jan 8, 2026
rust-timer added a commit that referenced this pull request Jan 8, 2026
Rollup merge of #150764 - map_regions_for_outlives, r=lcnr

Convert static lifetime to an nll var

Fixes #150648

I don't think this matters for any non 'static lifetimes as defining the struct as `Wrap<'a>` would mean the lifetime argument gets replaced with an nll var before MIR type checking meaning the struct tail is `dyn Trait + '?x`

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

Labels

S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ICE expected region 'static to be of kind ReVar

6 participants