aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | Merge #7110bors[bot]2021-01-143-4/+10
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7110: Deduplicate macros when offering completion r=matklad a=AdnoC Closes https://github.com/rust-analyzer/rust-analyzer/issues/7081 When iterating over the names within the `hir_def::resolver::Scope` for a module, track what macros are in the `hir_def::item_scope::ItemScope::legacy_macros` collection for the module. When iterating over names from the prelude, do not proccess the name if it had been in the `legacy_macros` collection. This is implemented with a `FxHashSet` in the `Scope::process_names` function that is populated when iterating over `legacy_macros` and checked when iterating over the prelude. Alternative implementation could instead query the `legacy_macros` `FxHashMap` directly when processing names in the prelude. Also, I'd like to add a test for this, but I'm not sure where it could be added. Co-authored-by: AdnoC <[email protected]>
| * | | | we can have one less call to name.clone()AdnoC2020-12-311-2/+3
| | | | |
| * | | | deduplicate macro completions from legacy macros and preludeAdnoC2020-12-313-4/+9
| | | | |
* | | | | Merge #7260bors[bot]2021-01-147-118/+253
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7260: Render Fn* trait objects and impl types as rust does r=matklad a=Veykril Also fixes raw ptr impl types being rendered ambiguously with multiple predicates. This moves out the `FamousDefs::Fixture` into its own file as well, cause I figured it is big enough to get its own file at this point + we also get highlighting this way when editing it. Fixes #3012 Co-authored-by: Lukas Wirth <[email protected]>
| * | | | | Render Fn* trait objects and impl types as rust doesLukas Wirth2021-01-136-30/+133
| | | | | |
| * | | | | Move FamousDefs fixture out into its own fileLukas Wirth2021-01-122-88/+120
| | |/ / / | |/| | |
* | | | | Merge #6809bors[bot]2021-01-133-0/+238
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6809: Add quick fix to sort impl methods by trait definition r=fisherdarling a=fisherdarling Closes #6110 Co-authored-by: Fisher Darling <[email protected]>
| * | | | | Sort impls by trait definitionFisher Darling2021-01-133-0/+238
|/ / / / / | | | | | | | | | | | | | | | Closes #6110
* | | | | Merge #7263bors[bot]2021-01-133-17/+5
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7263: Use upstream TextSize API r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | | Use upstream TextSize APIAleksey Kladov2021-01-133-17/+5
| | | | | |
* | | | | | Merge #7220bors[bot]2021-01-133-14/+178
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7220: same level folder rename for will_rename_files r=kjeremy a=ShuiRuTian use tricky way to support folder rename. Another step after #7009 and for #4471 Co-authored-by: ShuiRuTian <[email protected]> Co-authored-by: Song Gao <[email protected]>
| * | | | | fix windows test.ShuiRuTian2021-01-131-2/+2
| | | | | |
| * | | | | unwrap.ShuiRuTian2021-01-131-2/+2
| | | | | |
| * | | | | skip slow tests.ShuiRuTian2021-01-121-3/+4
| | | | | |
| * | | | | use path rather than hard code.ShuiRuTian2021-01-121-4/+4
| | | | | |
| * | | | | fix and add tests.ShuiRuTian2021-01-122-8/+142
| | | | | |
| * | | | | formatShuiRuTian2021-01-111-3/+5
| | | | | |
| * | | | | fix as suggestion.ShuiRuTian2021-01-111-9/+5
| | | | | |
| * | | | | Update crates/rust-analyzer/src/handlers.rsSong Gao2021-01-111-3/+1
| | | | | | | | | | | | | | | | | | Co-authored-by: Jeremy Kolb <[email protected]>
| * | | | | Update crates/rust-analyzer/src/handlers.rsSong Gao2021-01-111-4/+2
| | | | | | | | | | | | | | | | | | Co-authored-by: Jeremy Kolb <[email protected]>
| * | | | | move logic from client to server.ShuiRuTian2021-01-112-21/+9
| | | | | |
| * | | | | update rs formatShuiRuTian2021-01-101-1/+2
| | | | | |
| * | | | | fix condition.ShuiRuTian2021-01-101-1/+1
| | | | | |
| * | | | | update format.ShuiRuTian2021-01-101-8/+9
| | | | | |
| * | | | | fix issue.ShuiRuTian2021-01-102-14/+21
| | | | | |
| * | | | | not track my analyticsShuiRuTian2021-01-091-42/+0
| | | | | |
| * | | | | beta version for folder renameShuiRuTian2021-01-093-4/+34
| | | | | |
| * | | | | support folder rename for initial request.ShuiRuTian2021-01-081-8/+16
| | | | | |
| * | | | | more thoughts.ShuiRuTian2021-01-061-0/+14
| | | | | |
| * | | | | add analytics md.ShuiRuTian2021-01-061-0/+28
| | | | | |
* | | | | | Merge #7262bors[bot]2021-01-133-14/+10
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7262: Don't show internal server error on rename r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | | | Don't show internal server error on renameAleksey Kladov2021-01-133-14/+10
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Doesn't quite work due to https://github.com/microsoft/vscode-languageserver-node/issues/730 Note that this intentionally removes `impl std::Error for RenameError` -- we nether want to blindly bubble the rename error.
* | | | | | Merge #7251bors[bot]2021-01-1310-251/+319
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7251: Group references by FileId r=matklad a=Veykril Fixes #4901 This doesn't address https://github.com/rust-analyzer/rust-analyzer/pull/7032/files#diff-a7e1e771e911237bb893e1b0f5e0f2c2a856b54ca06f95ef0818a922f1a8b5ebR266 Co-authored-by: Lukas Wirth <[email protected]>
| * | | | | Rename FileReferences -> UsageSearchResultLukas Wirth2021-01-123-15/+14
| | | | | |
| * | | | | Ensure uniqueness of file ids in reference search via hashmapLukas Wirth2021-01-129-121/+122
| | | | | |
| * | | | | Group references by FileIdLukas Wirth2021-01-1210-256/+324
|/ / / / /
* | | | | Merge #7250bors[bot]2021-01-111-20/+17
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7250: Improve analysis stats legibility r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | | Improve analysis stats legibilityAleksey Kladov2021-01-111-20/+17
| | | | | |
* | | | | | Merge #7249bors[bot]2021-01-111-2/+10
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7249: Add build benchmark xtask command r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | | Add build benchmark xtask commandAleksey Kladov2021-01-111-2/+10
|/ / / / /
* | | | | Merge #7247bors[bot]2021-01-111-14/+14
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7247: cargo update r=kjeremy a=kjeremy Co-authored-by: kjeremy <[email protected]>
| * | | | | cargo updatekjeremy2021-01-111-14/+14
|/ / / / /
| | | | |
| \ \ \ \
| \ \ \ \
| \ \ \ \
| \ \ \ \
| \ \ \ \
*-----. \ \ \ \ Merge #7051 #7219 #7245 #7246bors[bot]2021-01-119-99/+215
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7051: Check dbg! macro in tidy_test r=matklad a=edwin0cheng Same as `check_todo` but for dbg! macro r? @matklad 7219: Refactor rename name checking r=matklad a=Veykril Improves the user facing error messages a bit and prevents renaming to `_` when the name is referenced as this would change source to not compile anymore since `_` is only a pattern, not a proper identifier. 7245: Encourage gifs r=matklad a=matklad bors r+ 🤖 7246: Unfreeze cargo_metadata r=matklad a=kjeremy It now pulls in a newer version of semver-parser. This does add a dependency on `cargo-platform` in the interest of correctness. Co-authored-by: Edwin Cheng <[email protected]> Co-authored-by: Lukas Wirth <[email protected]> Co-authored-by: Aleksey Kladov <[email protected]> Co-authored-by: kjeremy <[email protected]>
| | | | * | | | | Unfreeze cargo_metadatakjeremy2021-01-114-5/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It now pulls in a newer version of semver-parser.
| | | * | | | | | Encourage gifsAleksey Kladov2021-01-111-0/+2
| | | |/ / / / /
| | * | | | | | Add some more marks to rename testsLukas Wirth2021-01-101-17/+58
| | | | | | | |
| | * | | | | | Use local error macros in references/renameLukas Wirth2021-01-101-39/+30
| | | | | | | |
| | * | | | | | Refactor rename name checkingLukas Wirth2021-01-092-59/+104
| | | | | | | |
| * | | | | | | Add check for dbg! macroEdwin Cheng2021-01-081-1/+28
| | | | | | | |
| * | | | | | | Remove unused dbgEdwin Cheng2021-01-081-4/+3
| | | | | | | |