-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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.
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
Rollup of 6 pull requests #91775
Rollup of 6 pull requests #91775
Conversation
As suggested in rust-lang#88564. This adds a `try_clone()` to `OwnedFd` by refactoring the code out of the existing `File`/`Socket` code.
This commit also updates `stdarch` git submodule.
Keep track of the origin of a `T: Future` obligation when caused by an `.await` expression. Address rust-lang#66731.
…" suggestion Keep the `HirId` of `.await`ed expressions so in the case of a `fn` call on on a sync `fn`, we can suggest maybe turning it into an `async fn`.
The spans no longer overlap, so we no longer need to specialize the output depending on whether they would.
…` point ``` error[E0759]: `self` has an anonymous lifetime `'_` but it needs to satisfy a `'static` lifetime requirement --> $DIR/issue-72312.rs:10:24 | LL | pub async fn start(&self) { | ^^^^^ this data with an anonymous lifetime `'_`... ... LL | require_static(async move { | -------------- ...is required to live as long as `'static` here... LL | &self; | ----- ...and is captured here | note: `'static` lifetime requirement introduced by this trait bound --> $DIR/issue-72312.rs:2:22 | LL | fn require_static<T: 'static>(val: T) -> T { | ^^^^^^^ error: aborting due to previous error For more information about this error, try `rustc --explain E0759`. ``` Fix rust-lang#72312.
* take diagnostic logic out of happy-path * sort/dedup once * add more comments
More accurate filtering still needed.
@bors r+ rollup=never p=6 |
📌 Commit 1305bc5 has been approved by |
⌛ Testing commit 1305bc5 with merge edb8af266eb17db69b7760b377465172f4ef4b22... |
💔 Test failed - checks-actions |
The job Click to see the possible cause of the failure (guessed by this bot)
|
Looks like a github connection problem :( |
mmh wait now, maybe we are referencing the wrong commit in a git dep somewhere... |
Hmm, but the commit exists and is part of master apparently ... |
@bors r+ |
📌 Commit 1305bc5 has been approved by |
⌛ Testing commit 1305bc5 with merge ba1d81a523e0a2510a7a51016d01dacb0e5f7802... |
The job Click to see the possible cause of the failure (guessed by this bot)
|
💔 Test failed - checks-actions |
Successful merges:
try_clone()
function toOwnedFd
. #88794 (Add atry_clone()
function toOwnedFd
.)'static
reference crossing ayield
point #89734 (Point at capture points for non-'static
reference crossing ayield
point)Borrow
andBorrowMut
implsconst
#90270 (MakeBorrow
andBorrowMut
implsconst
)for
-loop,?
and.await
desugaring #90939 (Tweak errors coming fromfor
-loop,?
and.await
desugaring)Failed merges:
r? @ghost
@rustbot modify labels: rollup
Create a similar rollup