Skip to content

fix issue #3535 and add colon between mode and type when dumping function - #3595

Closed
Vincent-Belliard wants to merge 1 commit into
rust-lang:incomingfrom
Vincent-Belliard:issue_3535
Closed

fix issue #3535 and add colon between mode and type when dumping function#3595
Vincent-Belliard wants to merge 1 commit into
rust-lang:incomingfrom
Vincent-Belliard:issue_3535

Conversation

@Vincent-Belliard

Copy link
Copy Markdown
Contributor

Now, an unnamed argument "&int" is parsed "x: int" and not "&x: int" anymore.

When dumping functions prototypes, a colon is added between mode and type.

fn (&a: int)

will be dumped:

fn (&:int)

The ambiguity between "&x: int" and "x: &int" is removed

@nikomatsakis

Copy link
Copy Markdown
Contributor

Thanks!

RalfJung pushed a commit to RalfJung/rust that referenced this pull request Sep 8, 2024
Fixesrust-lang#3595 by using -fvisibility=hidden and the visibility attribute supported by both gcc and clang rather than the previous gcc-only mechanism for symbol hiding. Also brings over cfg changes from rust-lang#3594 which enable native-lib functionality on all unixes.
RalfJung pushed a commit to RalfJung/rust that referenced this pull request Sep 8, 2024
Enable native libraries on macOS
Fixesrust-lang#3595 by using `-fvisibility=hidden` and the visibility attribute supported by both gcc and clang rather than the previous gcc-only mechanism for symbol hiding. Also brings over cfg changes from rust-lang#3594 which enable native-lib functionality on all unixes.
Thanks for taking a look, feedback very welcome!
cc `@RalfJung`
shrirambalaji pushed a commit to shrirambalaji/rust that referenced this pull request Oct 6, 2024
Fixesrust-lang#3595 by using -fvisibility=hidden and the visibility attribute supported by both gcc and clang rather than the previous gcc-only mechanism for symbol hiding. Also brings over cfg changes from rust-lang#3594 which enable native-lib functionality on all unixes.
U007D pushed a commit to U007D/rust-mos that referenced this pull request Aug 21, 2026
3595: Fix completion of trait items r=matklad a=flodiebold
Trait items should be public by default.
Co-authored-by: Florian Diebold <flodiebold@gmail.com>
Sign up for freeto 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

@Vincent-Belliard@nikomatsakis