Skip to content

std::rt: Try stealing from all schedulers - #9353

Merged
bors merged 1 commit into
rust-lang:masterfrom
brson:sched
Sep 22, 2013
Merged

std::rt: Try stealing from all schedulers#9353
bors merged 1 commit into
rust-lang:masterfrom
brson:sched

Conversation

@brson

@brson brson commented Sep 20, 2013

Copy link
Copy Markdown
Contributor

This guarantees that if there is work to do it will be found

This guarantees that if there is work to do it will be found
@brson

brson commented Sep 21, 2013

Copy link
Copy Markdown
Contributor Author

Added another commit to paper over more scheduler problems.

bors added a commit that referenced this pull request Sep 21, 2013
This guarantees that if there is work to do it will be found
@brson

brson commented Sep 22, 2013

Copy link
Copy Markdown
Contributor Author

This deadlocked twice in a row :-/. Going to try to land one commit at a time.

bors added a commit that referenced this pull request Sep 22, 2013
This guarantees that if there is work to do it will be found
@bors bors closed this Sep 22, 2013
@bors
bors merged commit c767643 into rust-lang:master Sep 22, 2013
flip1995 pushed a commit to flip1995/rust that referenced this pull request Nov 16, 2023
Replace if_chain with let chains

Closes rust-lang#9353

Let chains are now supported by rustfmt 🎉

The PR is split into two commits
1. The result of running [`if-to-let-chain clippy*/**/*.rs`](https://github.com/Alexendoo/if-to-let-chain)
2. The manual clean up: fixing some errors/formatting, dogfood lints, removing the if_chain internal lint

r? `@flip1995`

changelog: none
U007D pushed a commit to U007D/rust-mos that referenced this pull request Aug 21, 2026
9353: Include extra targets when the pkg_root is not the same as the target root. r=matklad a=rezural

Fixes rust-lang#7715 

For example, if a sub-crate includes sets the path='../somewhere-else/lib.rs', the files will not be in pkg_root , but in the target root's parent.

It may actually be in root.parent().parent(), I'm not sure about that.

At the moment it is just a fix, are there any relevant tests that this could go in? I've got about 1 brain cell left... but im happy to add tests where appropriate.


Co-authored-by: rezural <rezural@protonmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants