Skip to content

librustc: Mark all type implementations public. rs=impl-publicity - #5136

Closed
pcwalton wants to merge 3 commits into
rust-lang:incomingfrom
pcwalton:impl-publicizing
Closed

librustc: Mark all type implementations public. rs=impl-publicity #5136
pcwalton wants to merge 3 commits into
rust-lang:incomingfrom
pcwalton:impl-publicizing

Conversation

@pcwalton

Copy link
Copy Markdown
Contributor

No description provided.

@bors bors closed this Feb 27, 2013
RalfJung added a commit to RalfJung/rust that referenced this pull request Jun 27, 2026
rust-bors Bot pushed a commit that referenced this pull request Jul 21, 2026
Various whitespace-related changes did not so far respect
`--file-lines`, see also [#5136](tests/target/issue-5136-4.rs). This PR
fixes this in all of the cases reported in #5136:

*  Newlines at the beginning of the file
*  Spaces at the beginning of the file
*  Missing newline at the end of the file
*  Space after a doc comment
*  Space before a comment in the last line of the file (without trailing
   newline)

The PR includes tests that fail without the fixes.
U007D pushed a commit to U007D/rust-mos that referenced this pull request Aug 21, 2026
5136: Split namespace maps in `ItemScope` r=jonas-schievink a=jonas-schievink

Reduces memory usage of the CrateDefMap query by ~130 MB (50%) on r-a.

I was also looking into handling glob imports more efficiently (storing scope chains instead of always duplicating everything into the glob-importing module's scope), but it seems that this already gives the most significant wins.

Co-authored-by: Jonas Schievink <jonas.schievink@ferrous-systems.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.

2 participants