aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Move the test markKirill Bulatov2021-01-051-2/+4
|
* Small helpersKirill Bulatov2021-01-042-1/+5
|
* Code review fixesKirill Bulatov2021-01-043-42/+50
|
* Do not collect trait type aliasesKirill Bulatov2021-01-041-5/+6
|
* Ignore associated items during unqialified path fuzzy completionsKirill Bulatov2021-01-041-1/+9
|
* SimplifyKirill Bulatov2021-01-042-27/+37
|
* Properly check assoc items lookupKirill Bulatov2021-01-042-46/+73
|
* Add associated data into fstKirill Bulatov2021-01-041-49/+45
|
* Add a basic test for the trait fuzzy importKirill Bulatov2021-01-041-0/+24
|
* Merge #7154bors[bot]2021-01-042-45/+150
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7154: Show goto type actions for Const and TypeParams r=matklad a=Veykril Shows goto type actions for type parameters: ![Code_6hn3rowu9M](https://user-images.githubusercontent.com/3757771/103547890-42aaeb00-4ea5-11eb-8ac7-f166869af5f8.png) Shows goto type actions for const parameters: ![Code_8UFCcbZL3z](https://user-images.githubusercontent.com/3757771/103547891-43438180-4ea5-11eb-91e8-50681e4d831e.png) Also shows implementations for `Self`: ![Code_eQj1pWfser](https://user-images.githubusercontent.com/3757771/103547892-43438180-4ea5-11eb-9122-461f2e0fdd01.png) Co-authored-by: Lukas Wirth <[email protected]>
| * Show GotoTypeAction for TypeParamLukas Wirth2021-01-042-17/+66
| |
| * Show GotoTypeAction for ConstParamLukas Wirth2021-01-042-37/+72
| |
| * Show implementations when hovering over SelfTypeLukas Wirth2021-01-041-8/+29
| |
* | Merge #7155bors[bot]2021-01-041-1/+1
|\ \ | | | | | | | | | | | | | | | | | | | | | 7155: Fix x86_64-pc-windows-msvc artifact name r=lnicola a=lnicola bors r+ Co-authored-by: Laurențiu Nicola <[email protected]>
| * | Fix x86_64-pc-windows-msvc artifact nameLaurențiu Nicola2021-01-041-1/+1
| | |
* | | Merge #7151bors[bot]2021-01-041-10/+9
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | 7151: Allow download of aarch64-pc-windows-msvc binaries r=matklad a=lnicola Co-authored-by: Laurențiu Nicola <[email protected]>
| * | Allow download of aarch64-pc-windows-msvc binariesLaurențiu Nicola2021-01-041-10/+9
| | |
* | | Merge #7153bors[bot]2021-01-043-60/+60
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | | | | | | | 7153: rename exrtract_assignment -> pull r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | rename exrtract_assignment -> pullAleksey Kladov2021-01-043-60/+60
| | | | | | | | | | | | | | | | | | | | | Vertical code motions are conventionally called "pull up" / "push down". "extract" is used for introducing new names.
* | | Merge #7152bors[bot]2021-01-043-15/+41
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | 7152: Implement hover for ConstParam r=Veykril a=Veykril bors r+ Co-authored-by: Lukas Wirth <[email protected]>
| * | Fix HoverAction::Implementation typoLukas Wirth2021-01-042-11/+12
| | |
| * | Implement hover for ConstParamLukas Wirth2021-01-042-4/+29
|/ /
* | Merge #7149bors[bot]2021-01-043-7/+67
|\ \ | |/ |/| | | | | | | | | | | 7149: Implement hovering for TypeParams r=matklad a=Veykril Co-authored-by: Lukas Wirth <[email protected]>
| * Impl hovering for TypeParamsLukas Wirth2021-01-043-7/+67
| |
* | Merge #7148bors[bot]2021-01-041-17/+51
|\ \ | | | | | | | | | | | | | | | | | | | | | 7148: Build aarch64-pc-windows-msvc releases r=matklad a=lnicola Co-authored-by: Laurențiu Nicola <[email protected]>
| * | Build aarch64-pc-windows-msvc releasesLaurențiu Nicola2021-01-041-17/+51
| |/
* | Merge #6587bors[bot]2021-01-0434-0/+278
|\ \ | |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6587: SSR: Support statement matching and replacing r=davidlattimore a=MarijnS95 For #3186 Hi! This is a smaller initial patchset that came up while working on support for statement lists (and my first time working on RA :grin:). It has me stuck on trailing semicolons for which I hope to receive some feedback. Matching (and replacing) `let` bindings with a trailing semicolon works fine, but trying to omit these (to make patterns more ergonomic) turns out more complex than expected. The "optional trailing semicolon solution" implemented in this PR is ugly because `Matcher::attempt_match_token` should only consume a trailing `;` when parsing `let` bindings to prevent other code from breaking. That at the same time has a nasty side-effect of `;` ending up in the matched code: any replacements on that should include the trailing semicolon as well even if it was not in the pattern. A better example is in the tests: https://github.com/rust-analyzer/rust-analyzer/blob/3ae1649c24a689473b874c331f5f176e5839978e/crates/ssr/src/tests.rs#L178-L184 The end result to achieve is (I guess) allowing replacement of let bindings without trailing semicolon like `let x = $a ==>> let x = 1` (but including them on both sides is still fine), and should make replacement in a macro call (where `foo!(let a = 2;)` for a `$x:stmt` is invalid syntax) possible as well. That should allow to enable/fix these tests: https://github.com/rust-analyzer/rust-analyzer/blob/3ae1649c24a689473b874c331f5f176e5839978e/crates/ssr/src/tests.rs#L201-L214 A possible MVP of this PR might be to drop this optional `;' handling entirely and only allow an SSR pattern/template with semicolons on either side. Co-authored-by: Marijn Suijten <[email protected]>
| * ssr: Add tests for raw LetStmt matchingMarijn Suijten2021-01-031-0/+44
| |
| * parser,syntax: Add separate parser for stmt with optional semicolonMarijn Suijten2021-01-0315-5/+72
| | | | | | | | | | Adjusting `grammar::fragments::stmt` to Optional or Yes will break original functionality and tests.
| * syntax,ssr: Implement statement parsingMarijn Suijten2021-01-0331-0/+167
| |
* | Merge #7113bors[bot]2021-01-032-0/+7
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | 7113: Manual updates r=matklad a=tekul Add some details on how to build the manual and some clarification on how to deal with "proc macro2 warnings. For context, this arose from [this question](https://users.rust-lang.org/t/how-to-disable-rust-analyzer-proc-macro-warnings-in-neovim/53150) on users.rust-lang.org. Co-authored-by: Luke Taylor <[email protected]>
| * | Expand doc on dealing with "proc macro not expanded"Luke Taylor2021-01-011-0/+4
| | | | | | | | | | | | | | | | | | Added the error message to the doc for the UnresolvedProcMacro diagnostic, explaining that either enabling the procMacro setting or disabling this diagnostic should make the warnings go away.
| * | Expand contributing "tip" in manual.adocLuke Taylor2020-12-311-0/+3
| | | | | | | | | | | | | | | | | | - Add link to asciidoc website - Explain how to create the generated adoc files from the source code - How to run asciidoctor to generate the manual
* | | Merge #7138bors[bot]2021-01-031-14/+89
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7138: Support assignment to FieldExpr for extract_assignment assist r=matklad a=Jesse-Bakker Co-authored-by: Jesse Bakker <[email protected]>
| * | | Support assignment to FieldExpr for extract_assignment assistJesse Bakker2021-01-031-14/+89
| | | |
* | | | Merge #7139bors[bot]2021-01-0310-12/+12
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | 7139: Upgrade expect-test to 1.1 r=matklad a=Jesse-Bakker Co-authored-by: Jesse Bakker <[email protected]>
| * | | Upgrade expect-test to 1.1Jesse Bakker2021-01-0310-12/+12
|/ / /
* | | Merge #7137bors[bot]2021-01-037-176/+52
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7137: Revert "Proper handling $crate and local_inner_macros" r=jonas-schievink a=jonas-schievink Reverts rust-analyzer/rust-analyzer#7133 It caused a fairly significant performance regression. bors r+ Co-authored-by: Jonas Schievink <[email protected]>
| * | Revert "Proper handling $crate and local_inner_macros"Jonas Schievink2021-01-037-176/+52
|/ /
* | Merge #7136bors[bot]2021-01-032-0/+52
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | 7136: Fixed nested eager macro bug r=edwin0cheng a=edwin0cheng fixes #7126 bors r+ Co-authored-by: Edwin Cheng <[email protected]>
| * | Fixed nested eager macro bugEdwin Cheng2021-01-032-0/+52
| | |
* | | Merge #7068bors[bot]2021-01-0312-2/+143
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7068: Add VSCode command to view the hir of a function body r=theotherphil a=theotherphil Will fix https://github.com/rust-analyzer/rust-analyzer/issues/7061. Very rough initial version just to work out where I needed to wire everything up. @matklad would you be happy merging a hir visualiser of some kind? If so, do you have any thoughts on what you'd like it show, and how? I've spent very little time on this thus far, so I'm fine with throwing away the contents of this PR, but I want to avoid taking the time to make this more polished/interactive/useful only to discover that no-one else has any interest in this functionality. ![image](https://user-images.githubusercontent.com/1974256/103236081-bb58f700-493b-11eb-9d12-55ae1b870f8f.png) Co-authored-by: Phil Ellison <[email protected]>
| * | | Use stdx::format_to instead of writelnPhil Ellison2021-01-031-4/+4
| | | |
| * | | Remove some unnecessary code copied from the Syntax Tree commandPhil Ellison2021-01-011-17/+3
| | | |
| * | | Address review suggestion, fix tidy testsPhil Ellison2021-01-014-23/+34
| | | |
| * | | Initial implementation of view-hir commandPhil Ellison2020-12-2810-1/+145
| | | |
* | | | Merge #7115bors[bot]2021-01-0320-225/+239
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7115: Migrate HasSource::source to return Option r=matklad a=nick96 I've made a start on fixing #6913 based on the provided work plan, migrating `HasSource::source` to return an `Option`. The simple cases are migrated but there are a few that I'm unsure exactly how they should be handled: - Logging the processing of functions in `AnalysisStatsCmd::run`: In verbose mode it includes the path to the module containing the function and the syntax range. I've handled this with an if-let but would it be better to blow up here with `expect`? I'm not 100% on the code paths but if we're processing a function definition then the source should exist. I've handled `source()` in all code paths as `None` being a valid return value but are there some cases where we should just blow up? Also, all I've done is bubble up the returned `None`s, there may be some places where we can recover and still provide something. Co-authored-by: Nick Spain <[email protected]> Co-authored-by: Nick Spain <[email protected]>
| * | | | Fix ConstParam HasSource impl and implement TryToNav not NavNick Spain2021-01-022-8/+8
| | | | |
| * | | | Remove old_source now we've fully migratedNick Spain2021-01-021-68/+0
| | | | | | | | | | | | | | | | | | | | Fixes #6913
| * | | | Handle case where detail doesn't exist without giving up on completionNick Spain2021-01-021-1/+1
| | | | | | | | | | | | | | | Co-authored-by: Aleksey Kladov <[email protected]>