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

Make visible types public in rustc #12662

Closed
wants to merge 1 commit into from

Conversation

sfackler
Copy link
Member

@sfackler sfackler commented Mar 2, 2014

No description provided.

@bors bors closed this Mar 3, 2014
@sfackler sfackler deleted the unexported-type branch May 15, 2014 05:03
bors added a commit to rust-lang-ci/rust that referenced this pull request Jul 25, 2022
…on-duplicate-name, r=DorianListens

fix: Extract Function produces duplicate fn names

This change fixes rust-lang#10037, in more or less the most naive fashion
possible.

We continue to start with the hardcoded default of "fun_name", and now append a
counter to the end of it if that name is already in scope.

In the future, we can probably apply more heuristics here to wind up with more
useful names by default, but for now this resolves the immediate problem.
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