Skip to content

[5.3][CodeCompletion] Avoid suggesting duplicated module names #31917

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

Conversation

rintaro
Copy link
Member

@rintaro rintaro commented May 20, 2020

Cherry-pick of #31892 into release/5.3

  • Explanation: Overlay modules have the same name as the shadowed modules. For global symbol completions, we should not list both names because they are identical. Maintain a set of seen module names to avoid suggesting duplicated names.
  • Scope: Code completion for global symbols
  • Risk: Very low. The change is pretty simple.
  • Issue: rdar://problem/63370253
  • Testing: Added regression test cases
  • Reviewers: Nathan Hawes (@nathawes)

in global completion result. Overly modules have the same name as the
shadowed modules. We should not list both names because they are
identical.

Maintain a set of seen module names to avoid suggesting duplicated
names.

rdar://problem/63370253
(cherry picked from commit af27e8f)
@rintaro rintaro added the r5.3 label May 20, 2020
@rintaro rintaro requested a review from a team as a code owner May 20, 2020 16:19
@rintaro
Copy link
Member Author

rintaro commented May 20, 2020

@swift-ci Please test

@rintaro
Copy link
Member Author

rintaro commented May 20, 2020

@swift-ci Please nominate

@tkremenek tkremenek merged commit ac783dd into swiftlang:release/5.3 May 20, 2020
@AnthonyLatsis AnthonyLatsis added swift 5.3 🍒 release cherry pick Flag: Release branch cherry picks labels Jan 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍒 release cherry pick Flag: Release branch cherry picks swift 5.3
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants