Skip to content

Don't allow duplicate methods. - #7064

Merged
bors merged 4 commits into
rust-lang:incomingfrom
luqmana:issue-7062
Jun 12, 2013
Merged

Don't allow duplicate methods.#7064
bors merged 4 commits into
rust-lang:incomingfrom
luqmana:issue-7062

Conversation

@luqmana

Copy link
Copy Markdown
Contributor

Fixes#7062

bors added a commit that referenced this pull request Jun 12, 2013
@borsbors closed this Jun 12, 2013
@bors
bors merged commit aa9a992 into rust-lang:incomingJun 12, 2013
@luqmana
luqmana deleted the issue-7062 branch June 20, 2013 23:18
flip1995 pushed a commit to flip1995/rust that referenced this pull request Apr 22, 2021
Fix FP in `wrong_self_convention` lint
Previously, this lint didn't check into impl block when it was implementing a trait.
Recent improvements (rust-lang#6924) have moved this check and some impl blocks are now checked but they shouldn't, such as in rust-lang#7032.
Fixesrust-lang#7032
changelog: Fix FP when not taking `self` in impl block for `wrong_self_convention` lint
U007D pushed a commit to U007D/rust-mos that referenced this pull request Aug 21, 2026
7064: Ignore qualifiers when doing autoimport completions lookup r=lnicola a=SomeoneToIgnore
A follow-up of rust-lang/rust-analyzer#6918 (comment) and the PR itself.
Tweaks the `import_map` query api to be more flexible with the ways to match against the import path and now fuzzy imports search in names only.
This had improved the completion speed for me locally in ~5 times for `fuzzy_completion` span time, but please recheck me here.
IMO we're fast and presice enough now, so I've added the modules back to the fuzzy search output.
Also tweaks the the expect tests to display functions explicitly, to avoid confusing "duplicate" results.
Co-authored-by: Kirill Bulatov <mail4score@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.

3 participants

@luqmana@sanxiyn@bors