Skip to content

Conversation

cramertj
Copy link
Member

@cramertj cramertj commented Jan 7, 2019

Fixes #57339.

This makes it possible to call .set multiple times without
using .as_mut() first to reborrow the pointer.

r? @withoutboats
cc @rust-lang/libs

This makes it possible to call `.set` multiple times without
using `.as_mut()` first to reborrow the pointer.
@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jan 7, 2019
@withoutboats
Copy link
Contributor

@bors r=me

This needs to be backported to beta, I'm not sure about the procedure here.

@alexcrichton
Copy link
Member

@bors: r=withouboats

@bors
Copy link
Collaborator

bors commented Jan 7, 2019

📌 Commit 68e98a2 has been approved by withouboats

@bors bors 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 7, 2019
@alexcrichton alexcrichton added beta-nominated Nominated for backporting to the compiler in the beta channel. beta-accepted Accepted for backporting to the compiler in the beta channel. S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Jan 7, 2019
@dtolnay
Copy link
Member

dtolnay commented Jan 7, 2019

@bors r=withoutboats

@Centril
Copy link
Contributor

Centril commented Jan 9, 2019

@bors p=1

Giving prio to make sure this lands since it's beta-nominated/accepted and beta=>stable happens soon.

@bors
Copy link
Collaborator

bors commented Jan 9, 2019

⌛ Testing commit 68e98a2 with merge 6ecad33...

bors added a commit that referenced this pull request Jan 9, 2019
Reborrow Pin<P> using &mut in `Pin::set`

Fixes #57339.

This makes it possible to call `.set` multiple times without
using `.as_mut()` first to reborrow the pointer.

r? @withoutboats
cc @rust-lang/libs
@bors
Copy link
Collaborator

bors commented Jan 9, 2019

☀️ Test successful - status-appveyor, status-travis
Approved by: withouboats
Pushing 6ecad33 to master...

@bors bors merged commit 68e98a2 into rust-lang:master Jan 9, 2019
@cramertj cramertj deleted the pin-set branch January 9, 2019 16:56
@Centril Centril removed beta-accepted Accepted for backporting to the compiler in the beta channel. beta-nominated Nominated for backporting to the compiler in the beta channel. labels Jan 9, 2019
@Centril
Copy link
Contributor

Centril commented Jan 9, 2019

I think there was some confusion here.. Pin<T> is stabilized in 1.33 but the master=>beta promotion for 1.33 hasn't happened yet (on the 15th of January), thus there's nothing to backport as far as I can see.

@alexcrichton
Copy link
Member

Oops my bad, can confirm nothing needed for a backport!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants