aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | Fix LSP extensions documentationvsrs2021-03-111-7/+2
| | | | |
| * | | | Add `Feature: Related Tests` commentvsrs2021-03-111-0/+13
| | | | |
| * | | | Apply review suggestionsvsrs2021-03-115-18/+11
| | | | |
| * | | | Remove erroneous commentvsrs2021-02-272-11/+1
| | | | |
| * | | | Update lsp-extensions.mdvsrs2021-02-271-1/+22
| | | | |
| * | | | Add progress reportingvsrs2021-02-272-17/+25
| | | | |
| * | | | Add LSP request and VSCode commandvsrs2021-02-277-9/+96
| | | | |
| * | | | Remove unnecessary file_id argumentvsrs2021-02-272-10/+7
| | | | |
| * | | | Add runnables::related_testsvsrs2021-02-274-7/+346
| | | | |
* | | | | Merge #7981bors[bot]2021-03-131-11/+76
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7981: Allow applying De Morgan's law to multiple terms at once r=matklad a=shepmaster Co-authored-by: Jake Goulding <[email protected]>
| * | | | | Allow applying De Morgan's law to multiple terms at onceJake Goulding2021-03-121-11/+76
| | |/ / / | |/| | |
* | | | | Merge #7984bors[bot]2021-03-132-6/+4
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7984: Improve version display r=matklad a=lnicola Maybe closes #7854 The version string for unreleased builds looks like this now: ``` $ rust-analyzer --version rust-analyzer 2021-03-08-159-gc0459c535 ``` Release builds should only have the tag name (`2021-03-15`). Co-authored-by: Laurențiu Nicola <[email protected]>
| * | | | | Improve version displayLaurențiu Nicola2021-03-122-6/+4
| |/ / / /
* | | | | Merge #7994bors[bot]2021-03-135-67/+270
|\ \ \ \ \ | |_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7994: Speed up mbe matching in heavy recursive cases r=edwin0cheng a=edwin0cheng In some cases (e.g. #4186), mbe matching is very slow due to a lot of copy and allocation for bindings, this PR try to solve this problem by introduce a semi "link-list" approach for bindings building. I used this [test case](https://github.com/weiznich/minimal_example_for_rust_81262) (for `features(32-column-tables)`) to run following command to benchmark: ``` time rust-analyzer analysis-stats --load-output-dirs ./ ``` Before this PR : 2 mins After this PR: 3 seconds. However, for 64-column-tables cases, we still need 4 mins to complete. I will try to investigate in the following weeks. bors r+ Co-authored-by: Edwin Cheng <[email protected]>
| * | | | Add bindings builder for speed up matchingEdwin Cheng2021-03-133-67/+221
| | | | |
| * | | | add expand logEdwin Cheng2021-03-132-0/+49
| | | | |
* | | | | Merge #7991bors[bot]2021-03-131-7/+2
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7991: Simplify hir_def TestDB r=jonas-schievink a=jonas-schievink bors r+ Co-authored-by: Jonas Schievink <[email protected]>
| * | | | | Simplify hir_def TestDBJonas Schievink2021-03-131-7/+2
|/ / / / /
* | | | | Merge #7989bors[bot]2021-03-122-18/+4
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7989: Remove `ItemTree::source` r=jonas-schievink a=jonas-schievink `HasSource` should be used instead bors r+ Co-authored-by: Jonas Schievink <[email protected]>
| * | | | | Remove `ItemTree::source`Jonas Schievink2021-03-122-18/+4
|/ / / / / | | | | | | | | | | | | | | | `HasSource` should be used instead
* | | | | Merge #7986bors[bot]2021-03-121-10/+5
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7986: Simplify a bit r=flodiebold a=flodiebold Co-authored-by: Florian Diebold <[email protected]>
| * | | | | Simplify a bitFlorian Diebold2021-03-121-10/+5
|/ / / / /
* | | | | Merge #7985bors[bot]2021-03-125-55/+44
|\ \ \ \ \ | |_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7985: Use Chalk Environment more directly r=flodiebold a=flodiebold Co-authored-by: Florian Diebold <[email protected]>
| * | | | Use Chalk Environment more directlyFlorian Diebold2021-03-125-55/+44
|/ / / /
* | | | Merge #7956bors[bot]2021-03-124-0/+283
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7956: Add assist to convert for_each into for loops r=Veykril a=SaiintBrisson This PR resolves #7821. Adds an assist to that converts an `Iterator::for_each` into a for loop: ```rust fn main() { let vec = vec![(1, 2), (2, 3), (3, 4)]; x.iter().for_each(|(x, y)| { println!("x: {}, y: {}", x, y); }) } ``` becomes ```rust fn main() { let vec = vec![(1, 2), (2, 3), (3, 4)]; for (x, y) in x.iter() { println!("x: {}, y: {}", x, y); }); } ``` Co-authored-by: Luiz Carlos Mourão Paes de Carvalho <[email protected]> Co-authored-by: Luiz Carlos <[email protected]> Co-authored-by: Lukas Wirth <[email protected]>
| * | | | Fix convert_iter_for_each_to_for doctestLukas Wirth2021-03-122-15/+56
| | | | |
| * | | | fix: generated test fixtureLuiz Carlos Mourão Paes de Carvalho2021-03-122-2/+25
| | | | |
| * | | | fix: replace doc-comments with normal commentsLuiz Carlos2021-03-121-20/+20
| | | | | | | | | | | | | | | Co-authored-by: Lukas Wirth <[email protected]>
| * | | | refactor: refactored and reduced assist codeLuiz Carlos Mourão Paes de Carvalho2021-03-121-36/+21
| | | | |
| * | | | fix: remove semicolonLuiz Carlos Mourão Paes de Carvalho2021-03-101-29/+56
| | | | |
| * | | | fix: code formattingLuiz Carlos Mourão Paes de Carvalho2021-03-101-10/+20
| | | | |
| * | | | fix: tests should work for convert_iter_for_each_to_forLuiz Carlos Mourão Paes de Carvalho2021-03-101-11/+44
| | | | |
| * | | | refactor: create block expressions and for loops using makeLuiz Carlos Mourão Paes de Carvalho2021-03-101-29/+50
| | | | |
| * | | | feat: add expr_for_loop to make in syntaxLuiz Carlos Mourão Paes de Carvalho2021-03-101-0/+3
| | | | |
| * | | | feat: add assist to conver for_each into for loopsLuiz Carlos Mourão Paes de Carvalho2021-03-102-0/+140
| | | | |
* | | | | Merge #7904bors[bot]2021-03-125-46/+140
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7904: Improved completion sorting r=JoshMcguigan a=JoshMcguigan I was working on extending #3954 to apply completion scores in more places (I'll have another PR open for that soon) when I discovered that actually completion sorting was not working for me at all in `coc.nvim`. This led me down a bit of a rabbit hole of how coc and vs code each sort completion items. Before this PR, rust-analyzer was setting the `sortText` field on completion items to `None` if we hadn't applied any completion score for that item, or to the label of the item with a leading whitespace character if we had applied any completion score. Completion score is defined in rust-analyzer as an enum with two variants, `TypeMatch` and `TypeAndNameMatch`. In vs code the above strategy works, because if `sortText` isn't set [they default it to the label](https://github.com/microsoft/vscode/commit/b4ead4ed665e1ce2e58d8329c682f78da2d4e89d). However, coc [does not do this](https://github.com/neoclide/coc.nvim/blob/e211e361475a38b146a903b9b02343551c6cd372/src/completion/complete.ts#L245). I was going to file a bug report against coc, but I read the [LSP spec for the `sortText` field](https://microsoft.github.io/language-server-protocol/specifications/specification-current/#textDocument_completion) and I feel like it is ambiguous and coc could claim what they do is a valid interpretation of the spec. Further, the existing rust-analyzer behavior of prepending a leading whitespace character for completion items with any completion score does not handle sorting `TypeAndNameMatch` completions above `TypeMatch` completions. They were both being treated the same. The first change this PR makes is to set the `sortText` field to either "1" for `TypeAndNameMatch` completions, "2" for `TypeMatch` completions, or "3" for completions which are neither of those. This change works around the potential ambiguity in the LSP spec and fixes completion sorting for users of coc. It also allows `TypeAndNameMatch` items to be sorted above just `TypeMatch` items (of course both of these will be sorted above completion items without a score). The second change this PR makes is to use the actual completion scores for ref matches. The existing code ignored the actual score and always assumed these would be a high priority completion item. #### Before Here coc just sorts based on how close the items are in the file. ![image](https://user-images.githubusercontent.com/22216761/110249880-46063580-7f2d-11eb-9233-91a2bbd48238.png) #### After Here we correctly get `zzz` first, since that is both a type and name match. Then we get `ccc` which is just a type match. ![image](https://user-images.githubusercontent.com/22216761/110249883-4e5e7080-7f2d-11eb-9269-a3bc133fdee7.png) Co-authored-by: Josh Mcguigan <[email protected]>
| * | | | | update relevance score u8 -> u32Josh Mcguigan2021-03-122-10/+10
| | | | | |
| * | | | | add relevance score testJosh Mcguigan2021-03-121-0/+60
| | | | | |
| * | | | | remove unused CompletionScore enumJosh Mcguigan2021-03-123-14/+3
| | | | | |
| * | | | | add completion relevance scoreJosh Mcguigan2021-03-125-37/+82
| | | | | |
* | | | | | Merge #7980bors[bot]2021-03-125-7/+7
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7980: Fix remaining references to `cargo xtask codegen` r=matklad a=Veykril Co-authored-by: Lukas Wirth <[email protected]>
| * | | | | Fix remaining references to `cargo xtask codegen`Lukas Wirth2021-03-125-7/+7
|/ / / / /
* | | | | Merge #7978bors[bot]2021-03-1213-103/+90
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7978: Unify naming r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | | Unify namingAleksey Kladov2021-03-1213-103/+90
|/ / / / /
* | | | | Merge #7974bors[bot]2021-03-1217-166/+183
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7974: use references in CompletionItem's builder r=matklad a=yonip23 @matklad This is a follow up to [this pr](https://github.com/rust-analyzer/rust-analyzer/pull/7973) Co-authored-by: yonip23 <[email protected]>
| * | | | | use references in CompletionItem's builderyonip232021-03-1117-166/+183
|/ / / / /
* | | | | Merge #7972bors[bot]2021-03-111-3/+30
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7972: fix: add semicolon after type ascription r=matklad a=conradludgate Fixes #7971 Co-authored-by: Conrad Ludgate <[email protected]>
| * | | | | fix: add semicolon after type ascriptionConrad Ludgate2021-03-111-3/+30
|/ / / / /
* | | | | Merge #7969bors[bot]2021-03-101-7/+47
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7969: Return original text range in PrepareRename responses when inside macro r=Veykril a=Veykril bors r+ Issue found in #7968 Co-authored-by: Lukas Wirth <[email protected]>
| * | | | | Return original text range in PrepareRename responses when inside macroLukas Wirth2021-03-101-7/+47
|/ / / / /