Skip to content
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

Deglob stdlib #12074

Closed
wants to merge 15 commits into from
Closed

Deglob stdlib #12074

wants to merge 15 commits into from

Conversation

chromatic
Copy link
Contributor

This is another attempt to address GH #12006. I've rebased against master and only two commits appear. Let's hope the Windows build works now.

adrientetar and others added 15 commits February 6, 2014 13:25
I have a hunch this just deadlocked the windows bots. Due to UDP being a lossy
protocol, I don't think we can guarantee that the server can receive both
packets, so just listen for one of them.
This complements `usage` by auto-generating a short one-liner summary
of the options.
A weak pointer inside itself will have its destructor run when the last
strong pointer to that data disappears, so we need to make sure that the
Weak and Rc destructors don't duplicate work (i.e. freeing).

By making the Rcs effectively take a weak pointer, we ensure that no
Weak destructor will free the pointer while still ensuring that Weak
pointers can't be upgraded to strong ones as the destructors run.

This approach of starting weak at 1 is what libstdc++ does.

Fixes #12046.
This replaces the imports from the prelude with the re-exported symbols.
@chromatic
Copy link
Contributor Author

Yet more commits brought in. They're going to revoke my Git license. One more try.

@chromatic chromatic closed this Feb 6, 2014
@chromatic chromatic deleted the deglob_stdlib branch February 6, 2014 21:42
@adrientetar
Copy link
Contributor

@chromatic I helped another user in #11457 if it can be of help.

bors added a commit to rust-lang-ci/rust that referenced this pull request Jul 25, 2022
…t, r=jonas-schievink

Reduce priority of flyimport completions

Fixes rust-lang/rust-analyzer#12068
flip1995 pushed a commit to flip1995/rust that referenced this pull request Jan 25, 2024
…hile-checking-duplicate-code, r=Jarcho

Make `HirEqInterExpr::eq_block` take comments into account while checking if two blocks are equal

This PR:
- now makes `HirEqInterExpr::eq_block` take comments into account. Identical code with varying comments will no longer be considered equal.
- makes necessary adjustments to UI tests.

Closes rust-lang#12044

**Lintcheck Changes**
- `match_same_arms` 53 => 52
- `if_same_then_else` 3 => 0

changelog: [`if_same_then_else`]: Blocks with different comments will no longer trigger this lint.
changelog: [`match_same_arms`]: Arms with different comments will no longer trigger this lint.
```
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.

9 participants