aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | Remove outdated FIXMELaurențiu Nicola2020-12-211-3/+2
|/ / / / /
* | | | | Merge #6985bors[bot]2020-12-211-1/+10
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6985: Use /etc/os-release to check for NixOS r=matklad a=lnicola Closes #5641 The motivation in #5641 isn't too strong, but /etc/os-release exists on pretty much every Linux distro, while /etc/nixos sounds like an implementation detail. Co-authored-by: Laurențiu Nicola <[email protected]>
| * | | | | Use /etc/os-release to check for NixOSLaurențiu Nicola2020-12-211-1/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The motivation in #5641 isn't too strong, but /etc/os-release exists on pretty much every Linux distro, while /etc/nixos sounds like an implementation detail.
* | | | | | Merge #6987bors[bot]2020-12-211-7/+74
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6987: Delete related whitespace when removing unused param r=matklad a=rherrmann Include adjacent whitespace in the text edits to remove the parameter declaration and its occurences in calling code. https://github.com/rust-analyzer/rust-analyzer/issues/6663 Co-authored-by: Rüdiger Herrmann <[email protected]>
| * | | | | Delete related whitespace when removing unused paramRüdiger Herrmann2020-12-211-7/+74
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Include adjacent whitespace in the text edits to remove the parameter declaration and its occurences in calling code. https://github.com/rust-analyzer/rust-analyzer/issues/6663
* | | | | Merge #6982bors[bot]2020-12-212-1/+16
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6982: Remove parentheses when inverting `!(cond)` r=matklad a=Jesse-Bakker Followup to #6894 When inverting a composite condition twice, the parentheses were left. This also removes those unnecessary parentheses when applying the invert-if assist. Co-authored-by: Jesse Bakker <[email protected]>
| * | | | Remove parentheses when inverting `!(cond)`Jesse Bakker2020-12-212-1/+16
| | |_|/ | |/| |
* | | | Merge #6981bors[bot]2020-12-211-6/+42
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6981: Acquire original text range when searching for references r=Veykril a=Veykril Fixes #6976 Co-authored-by: Lukas Wirth <[email protected]>
| * | | | Acquire original text range when searching for referencesLukas Wirth2020-12-211-6/+42
|/ / / /
* | | | Merge #6978bors[bot]2020-12-211-0/+4
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6978: Filter out empty rustc spans r=lnicola a=jonas-schievink Hopefully, this fixes https://github.com/rust-analyzer/rust-analyzer/issues/6892 (I couldn't test it since r-a currently requires a too recent version of VS Code) Co-authored-by: Jonas Schievink <[email protected]>
| * | | | Filter out empty rustc spansJonas Schievink2020-12-211-0/+4
| |/ / /
* | | | Merge #6977bors[bot]2020-12-211-1/+1
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6977: Revert "Stop setting CompletionItem::deprecated" r=lnicola a=lnicola We should keep setting it, according to https://github.com/rust-analyzer/rust-analyzer/pull/6974#issuecomment-748983789. Co-authored-by: Laurențiu Nicola <[email protected]>
| * | | | Revert "Stop setting CompletionItem::deprecated"Laurențiu Nicola2020-12-211-1/+1
|/ / / / | | | | | | | | | | | | This reverts commit 299ce609952dfaa6bef86e967c5632e8e42b8fca.
* | | | Merge #6974bors[bot]2020-12-211-1/+1
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6974: Stop setting CompletionItem::deprecated r=matklad a=lnicola Closes #2042 We're now using the `CompletionItem::tags` field to mark `CompletionItem`s as deprecated, and `CompletionItem::deprecated` is gone from LSP. Co-authored-by: Laurențiu Nicola <[email protected]>
| * | | Stop setting CompletionItem::deprecatedLaurențiu Nicola2020-12-211-1/+1
|/ / /
* | | Merge #6965bors[bot]2020-12-217-147/+199
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6965: Properly attach attributes to Param instead of parent ParamList r=matklad a=Veykril Fixes #2783, fixes #2781 The problem with `let _a = [0,#[cfg(feature = "L")]0];` has already been fixed some time ago it seems: <details> <summary>Syntax Tree for the const item</summary> ``` [email protected] [email protected] "let" [email protected] " " [email protected] [email protected] [email protected] "_a" [email protected] " " [email protected] "=" [email protected] " " [email protected] [email protected] "[" [email protected] [email protected] "0" [email protected] "," [email protected] [email protected] [email protected] "#" [email protected] "[" [email protected] [email protected] [email protected] [email protected] "cfg" [email protected] [email protected] "(" [email protected] "feature" [email protected] " " [email protected] "=" [email protected] " " [email protected] "\"L\"" [email protected] ")" [email protected] "]" [email protected] "0" [email protected] "]" [email protected] ";" ``` </details> Co-authored-by: Lukas Wirth <[email protected]>
| * | | Parse attributes in tuple expressionsLukas Wirth2020-12-203-3/+56
| | | |
| * | | Properly attach attributes to Param instead of parent ParamListLukas Wirth2020-12-204-144/+143
| | |/ | |/|
* | | Merge #6967bors[bot]2020-12-214-9/+63
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | | | | 6967: Correctly parse legacy trait objects with leading ForType r=matklad a=Veykril Fixes #1422 Co-authored-by: Lukas Wirth <[email protected]>
| * | Properly parse legacy trait objects with leading ForTypeLukas Wirth2020-12-204-9/+63
| |/
* | Merge #6968bors[bot]2020-12-201-4/+4
|\ \ | | | | | | | | | | | | | | | | | | | | | 6968: Update syn and quote r=kjeremy a=kjeremy Co-authored-by: Jeremy Kolb <[email protected]>
| * | Update syn and quoteJeremy Kolb2020-12-201-4/+4
| | |
* | | Merge #6963bors[bot]2020-12-201-15/+27
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6963: Split lines between sentences r=lnicola a=lnicola bors r+ Co-authored-by: Laurențiu Nicola <[email protected]>
| * | | Split lines between sentencesLaurențiu Nicola2020-12-201-15/+27
| | | |
* | | | Merge #6961bors[bot]2020-12-201-1/+1
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6961: Document the version requirements for building from source r=lnicola a=lnicola Fixes #6667 bors r+ Co-authored-by: Laurențiu Nicola <[email protected]>
| * | | Document the version requirements for building from sourceLaurențiu Nicola2020-12-201-1/+1
| | | |
* | | | Merge #6959bors[bot]2020-12-203-2/+14
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6959: Use pattern_single instead of pattern in mbe pat r=jonas-schievink a=edwin0cheng Fix #6882 Co-authored-by: Edwin Cheng <[email protected]>
| * | | | Use pattern_single instead of pattern in mbe patEdwin Cheng2020-12-203-2/+14
| | |_|/ | |/| |
* | | | Merge #6921bors[bot]2020-12-205-19/+74
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | 6921: Higher-ranked trait bounds for where clauses r=flodiebold a=Veykril There is a slight problem with this which is also noted in a FIXME now but `LifetimeParameters` of these ForLifetime where clauses allocate the lifetimes in the corresponding arena as if they were lifetimes of the item itself and not just the clause they belong to. I wasn't entirely sure what I could do about this but given nothing really uses lifetimes like that currently I figured it might be fine? Open to suggestions for that problem. Co-authored-by: Lukas Wirth <[email protected]>
| * | | Higher-ranked trait bounds for where clausesLukas Wirth2020-12-174-18/+74
| | | |
| * | | Remove obsolete FIXMELukas Wirth2020-12-171-1/+0
| | | |
* | | | Merge #6958bors[bot]2020-12-2022-102/+88
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6958: Align code_model name with ungrammar r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | Align code_model name with ungrammarAleksey Kladov2020-12-2022-102/+88
|/ / / /
* | | | Merge #6929bors[bot]2020-12-204-3/+18
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | 6929: Handle $_ in mbe r=edwin0cheng a=lnicola Fixes #6926 Co-authored-by: Laurențiu Nicola <[email protected]>
| * | | mbe: treat _ as identLaurențiu Nicola2020-12-204-3/+18
| | | |
* | | | Merge #6954bors[bot]2020-12-191-10/+10
|\ \ \ \ | | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | 6954: cargo update r=kjeremy a=kjeremy Co-authored-by: Jeremy Kolb <[email protected]>
| * | | cargo updateJeremy Kolb2020-12-191-10/+10
| | | |
* | | | Merge #6953bors[bot]2020-12-191-0/+35
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | 6953: Add test_rename_bind_pat r=bjorn3 a=bjorn3 Fixes #2976 Co-authored-by: bjorn3 <[email protected]>
| * | | Add test_rename_bind_patbjorn32020-12-191-0/+35
| | | |
* | | | Merge #6946bors[bot]2020-12-1912-216/+263
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6946: Better fuzzy heuristics r=matklad a=SomeoneToIgnore Continuation of the https://github.com/rust-analyzer/rust-analyzer/pull/6922, mainly created for a test. Turns out our current completions tests were sorting the completions by label, I had to remove that to test the order properly and update this order in a bunch of tests (ergo the changes) Co-authored-by: Kirill Bulatov <[email protected]>
| * | | Fewer allocationsKirill Bulatov2020-12-191-5/+7
| | | |
| * | | Tidy upKirill Bulatov2020-12-192-32/+26
| | | |
| * | | Keep the original completion order in testsKirill Bulatov2020-12-1912-194/+193
| | | |
| * | | Add a slightly better fuzzy search heuristicsKirill Bulatov2020-12-191-10/+62
| | | |
* | | | Merge #6951bors[bot]2020-12-196-36/+39
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6951: Make sure that HighlightModifier::ALL is synchronized with enum r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | Make sure that HighlightModifier::ALL is synchronized with enumAleksey Kladov2020-12-196-36/+39
|/ / / /
* | | | Merge #6948bors[bot]2020-12-191-23/+55
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6948: Add API for mapping `Attr` back to its syntax node r=jonas-schievink a=jonas-schievink This will be useful for emitting diagnostics pertaining to a specific attribute Co-authored-by: Jonas Schievink <[email protected]>
| * | | | Add API for mapping `Attr` back to its syntax nodeJonas Schievink2020-12-191-23/+55
| |/ / /
* | | | Merge #6949bors[bot]2020-12-194-15/+14
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6949: Clarify the meaning of no-op highlight tag r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | Clarify the meaning of no-op highlight tagAleksey Kladov2020-12-194-15/+14
| | | | |