aboutsummaryrefslogtreecommitdiff
path: root/crates
Commit message (Collapse)AuthorAgeFilesLines
* Create TraitEnvironment through a queryFlorian Diebold2021-03-136-50/+76
|
* Use chalk_ir::OpaqueTyIdFlorian Diebold2021-03-137-63/+53
|
* Use chalk_ir::PlaceholderIndexFlorian Diebold2021-03-138-44/+53
|
* Use chalk_ir::ClosureIdFlorian Diebold2021-03-135-23/+17
|
* Handle `cfg_attr` gating multiple attributesJonas Schievink2021-03-131-29/+19
|
* Extend cfg_attr testJonas Schievink2021-03-131-0/+3
|
* Use chalk_ir::FnDefIdFlorian Diebold2021-03-1310-30/+48
|
* Use chalk_ir::AssocTypeIdFlorian Diebold2021-03-1310-76/+87
|
* Use chalk_ir::ForeignDefIdFlorian Diebold2021-03-136-38/+32
|
* fix folding range kindAleksey Kladov2021-03-131-1/+2
|
* Separate `Ty` and `TyKind` like in ChalkFlorian Diebold2021-03-1320-619/+755
| | | | | Currently `Ty` just wraps `TyKind`, but this allows us to change most places to already use `intern` / `interned`.
* Merge #7799bors[bot]2021-03-138-17/+391
|\ | | | | | | | | | | | | | | | | | | | | | | | | 7799: Related tests r=matklad a=vsrs ![tests](https://user-images.githubusercontent.com/62505555/109397453-a9013680-7947-11eb-8b11-ac03079f7645.gif) This adds an ability to look for tests for the item under the cursor: function, constant, data type, etc The LSP part is bound to change. But the feature itself already works and I'm looking for a feedback :) Co-authored-by: vsrs <[email protected]>
| * Add `Feature: Related Tests` commentvsrs2021-03-111-0/+13
| |
| * Apply review suggestionsvsrs2021-03-114-14/+10
| |
| * Remove erroneous commentvsrs2021-02-271-10/+0
| |
| * Add LSP request and VSCode commandvsrs2021-02-273-0/+39
| |
| * 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-131-3/+2
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-121-3/+2
| |/ /
* | | 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
| | | |
* | | | Simplify hir_def TestDBJonas Schievink2021-03-131-7/+2
| | | |
* | | | Remove `ItemTree::source`Jonas Schievink2021-03-122-18/+4
| | | | | | | | | | | | | | | | `HasSource` should be used instead
* | | | Simplify a bitFlorian Diebold2021-03-121-10/+5
| | | |
* | | | 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
| | | | |
* | | | | Fix remaining references to `cargo xtask codegen`Lukas Wirth2021-03-121-1/+1
|/ / / /
* | | | Unify namingAleksey Kladov2021-03-1213-103/+90
| | | |
* | | | use references in CompletionItem's builderyonip232021-03-1117-166/+183
| | | |
* | | | fix: add semicolon after type ascriptionConrad Ludgate2021-03-111-3/+30
| | | |
* | | | Return original text range in PrepareRename responses when inside macroLukas Wirth2021-03-101-7/+47
| | | |