Skip to content

refactor: remove Ty::pinned_ref in favor of Ty::maybe_pinned_ref #151849

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
Merged
rust-bors merged 1 commit into from
Jan 31, 2026
Merged

refactor: remove Ty::pinned_ref in favor of Ty::maybe_pinned_ref #151849

rust-bors merged 1 commit into from
Jan 31, 2026
+12 −20

Conversation

@frank-king
Copy link
Contributor

@frank-king frank-king commented Jan 30, 2026 •
edited
Loading

Also returns the Region of the reference type in Ty::maybe_pinned_Ref.

Part of #149130.

r? jackh726

@rustbot
Copy link
Collaborator

rustbot commented Jan 30, 2026

Some changes occurred in exhaustiveness checking

cc @Nadrieril

@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 30, 2026
@frank-king frank-king mentioned this pull request Jan 30, 2026
Open
Zalathar
Zalathar reviewed Jan 30, 2026
View reviewed changes
@frank-king
refactor: remove Ty::pinned_ref in favor of Ty::maybe_pinned_ref
4ae5b43 
Also returns the `Region` of the reference type.
@frank-king frank-king force-pushed the refactor/pin-coerce-1 branch from a91b8db to 4ae5b43 Compare January 30, 2026 06:23
@jackh726
Copy link
Member

jackh726 commented Jan 31, 2026

@bors r+ rollup

@rust-bors
Copy link
Contributor

rust-bors bot commented Jan 31, 2026

📌 Commit 4ae5b43 has been approved by jackh726

It is now in the queue for this repository.

@rust-bors rust-bors bot added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jan 31, 2026
@matthiaskrgr matthiaskrgr mentioned this pull request Jan 31, 2026
Merged
rust-bors bot pushed a commit that referenced this pull request Jan 31, 2026
@bors
Auto merge of #151897 - matthiaskrgr:rollup-eTQjYr5, r=matthiaskrgr
78865ca 
Rollup of 4 pull requests

Successful merges:

 - #151262 (Introducing clap on tidy)
 - #151896 (Revert enabling `outline-atomics` on various platforms)
 - #151849 (refactor: remove `Ty::pinned_ref` in favor of `Ty::maybe_pinned_ref`)
 - #151892 (Document enum types used as values for E0423)
@rust-bors rust-bors bot merged commit 29fcd63 into rust-lang:main Jan 31, 2026
11 checks passed
@rustbot rustbot added this to the 1.95.0 milestone Jan 31, 2026
rust-timer added a commit that referenced this pull request Jan 31, 2026
@rust-timer
Unrolled build for #151849
c267f51 
Rollup merge of #151849 - frank-king:refactor/pin-coerce-1, r=jackh726

refactor: remove `Ty::pinned_ref` in favor of `Ty::maybe_pinned_ref`

Also returns the `Region` of the reference type in `Ty::maybe_pinned_Ref`.

Part of #149130.

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

Reviewers

@Zalathar Zalathar Zalathar left review comments

Assignees

@jackh726 jackh726

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

Milestone

1.95.0

Development

Successfully merging this pull request may close these issues.

4 participants

@frank-king @rustbot @jackh726 @Zalathar