aboutsummaryrefslogtreecommitdiff
path: root/crates
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | Merge #8620bors[bot]2021-04-221-14/+22
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8620: Remove unnecessary braces for extracted block expression r=Veykril a=brandondong This change addresses the first bullet point of https://github.com/rust-analyzer/rust-analyzer/issues/7839. Specifically, when extracting block expressions, remove the unneeded extra braces inside the generated function. Co-authored-by: Brandon <[email protected]>
| * | | | Remove unnecessary braces for extracted block expressionBrandon2021-04-221-14/+22
| | | | |
* | | | | Code review feedback.James Leitch2021-04-213-8/+10
| | | | |
* | | | | Diagnostic Remap Path Prefixes added.James Leitch2021-04-213-36/+41
| | | | |
* | | | | Diagnostic paths attempt to use VSCode's path join behavior before ↵James Leitch2021-04-211-3/+30
| | | | | | | | | | | | | | | | | | | | defaulting to Rust's path join behavior.
* | | | | Merge #8570bors[bot]2021-04-212-22/+47
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8570: Flycheck tries to parse both Cargo and Rustc messages. r=rickvanprim a=rickvanprim This change allows non-Cargo build systems to be used for Flycheck provided they call `rustc` with `--error-format=json` and emit those JSON messages to `stdout`. Co-authored-by: James Leitch <[email protected]>
| * | | | | Code review feedback.James Leitch2021-04-191-15/+16
| | | | | |
| * | | | | Flycheck tries to parse both Cargo and Rustc messages.James Leitch2021-04-192-22/+46
| | | | | |
* | | | | | Merge #8616bors[bot]2021-04-212-4/+4
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8616: minor: Bump deps r=lnicola a=lnicola bors r+ Co-authored-by: Laurențiu Nicola <[email protected]>
| * | | | | | Bump chalkLaurențiu Nicola2021-04-211-3/+3
| | | | | | |
| * | | | | | Bump rustc_lexerLaurențiu Nicola2021-04-211-1/+1
| | |_|/ / / | |/| | | |
* / | | | | Remove `ItemTree::all_inner_items`Jonas Schievink2021-04-211-7/+0
|/ / / / /
* | | | | Add failing local items testJonas Schievink2021-04-213-0/+46
| | | | |
* | | | | Add support for fill match arms of boolean valuesComonad2021-04-212-23/+207
| | | | | | | | | | | | | | | | | | | | - Add support for boolean inside tuple
* | | | | Parse outer atttributes for RecordPatFieldLukas Wirth2021-04-213-8/+57
| | | | |
* | | | | fix: no more Registering progress handler for token rustAnalyzer/Indexing ↵Aleksey Kladov2021-04-202-2/+13
| | | | | | | | | | | | | | | | | | | | failed.
* | | | | Merge #8600bors[bot]2021-04-204-65/+327
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8600: fix: no longer get stuck on windows r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | | fix: no longer get stuck on windowsAleksey Kladov2021-04-204-65/+327
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | reading both stdout & stderr is a common gotcha, you need to drain them concurrently to avoid deadlocks. Not sure why I didn't do the right thing from the start. Seems like I assumed the stderr is short? That's not the case when cargo spams `compiling xyz` messages
* | | | | Merge #8602bors[bot]2021-04-201-28/+45
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8602: Fix panic in `replace_derive_with_manual_impl` r=jonas-schievink a=jonas-schievink bors r+ Co-authored-by: Jonas Schievink <[email protected]>
| * | | | | Follow testing style guideJonas Schievink2021-04-201-28/+28
| | | | | |
| * | | | | Fix panic in `replace_derive_with_manual_impl`Jonas Schievink2021-04-201-0/+17
| |/ / / /
* / / / / "Inline variable" when on a use of the variableJonas Schievink2021-04-201-38/+185
|/ / / /
| | | * Don't filter equal nodes in reorder assistsLukas Wirth2021-04-212-10/+6
| | | |
| | | * Add cov_marks to insert_use testsLukas Wirth2021-04-207-42/+77
| | | |
| | | * Rewrite extract_struct_from_enum_variant assistLukas Wirth2021-04-201-85/+91
| | | |
| | | * Remove SyntaxRewriter usage in insert_use in favor of tedLukas Wirth2021-04-208-243/+185
| | | |
| | | * Replace SyntaxRewriter usage with ted in reorder_impl assistLukas Wirth2021-04-201-9/+13
| |_|/ |/| |
* | | Merge #8586bors[bot]2021-04-192-23/+18
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8586: Replace SyntaxRewriter usage with ted in eager::eager_macro_recur r=Veykril a=Veykril Co-authored-by: Lukas Wirth <[email protected]>
| * | | Replace SyntaxRewriter with ted in exppand_macro::expand_macro_recurLukas Wirth2021-04-191-14/+9
| | | |
| * | | Remove SyntaxRewriter usage in eager::eager_macro_recurLukas Wirth2021-04-191-9/+9
| | | |
* | | | Add autoimport test with inner itemsJonas Schievink2021-04-191-0/+33
| | | |
* | | | Fix some find_path bugs around inner itemsJonas Schievink2021-04-191-5/+58
| | | |
* | | | Merge #8584bors[bot]2021-04-191-1/+1
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8584: internal: fix slightly broken test r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | internal: fix slightly broken testAleksey Kladov2021-04-191-1/+1
| | | | |
* | | | | Merge #8583bors[bot]2021-04-191-23/+14
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8583: Simplify r=Veykril a=Veykril bors r+ Co-authored-by: Lukas Wirth <[email protected]>
| * | | | SimplifyLukas Wirth2021-04-191-23/+14
| |/ / /
* | | | Merge #8582bors[bot]2021-04-194-5/+5
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8582: Fix typo: comparision -> comparison r=kjeremy a=stanciuadrian Closes #8576 Co-authored-by: Adrian Stanciu <[email protected]>
| * | | | Fix typo: comparision -> comparisonAdrian Stanciu2021-04-194-5/+5
| |/ / /
* | | | Merge #8578bors[bot]2021-04-196-182/+238
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8578: fix: false positive about inner attrs in docs r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | fix: false positive about inner attrs in docsAleksey Kladov2021-04-196-182/+238
| | | | | | | | | | | | | | | | | | | | closes #8541
* | | | | Merge #8577bors[bot]2021-04-191-17/+41
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8577: Support crates/module roots in external_docs r=Veykril a=Veykril Fixes #8575 bors r+ Co-authored-by: Lukas Wirth <[email protected]>
| * | | | | Support crates/module roots in external_docsLukas Wirth2021-04-191-17/+41
| | |/ / / | |/| | |
| | | | |
| \ \ \ \
*-. \ \ \ \ Merge #8524 #8527bors[bot]2021-04-194-38/+298
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8524: Fix extract function with partial block selection r=matklad a=brandondong **Reproduction:** ```rust fn foo() { let n = 1; let mut v = $0n * n;$0 v += 1; } ``` 1. Select the snippet ($0) and use the "Extract into function" assist. 2. Extracted function is incorrect and does not compile: ```rust fn foo() { let n = 1; let mut v = fun_name(n); v += 1; } fn fun_name(n: i32) {} ``` 3. Omitting the ending semicolon from the selection fixes the extracted function: ```rust fn fun_name(n: i32) -> i32 { n * n } ``` **Cause:** - When `extraction_target` uses a block extraction (semicolon case) instead of an expression extraction (no semicolon case), the user selection is directly used as the TextRange. - However, the existing function extraction logic for blocks requires that the TextRange spans from start to end of complete statements to work correctly. - For example: ```rust fn foo() { let m = 2; let n = 1; let mut v = m $0* n; let mut w = 3;$0 v += 1; w += 1; } ``` produces ```rust fn foo() { let m = 2; let n = 1; let mut v = m let mut w = fun_name(n); v += 1; w += 1; } fn fun_name(n: i32) -> i32 { let mut w = 3; w } ``` - The user selected TextRange is directly replaced by the function call which is now in the middle of another statement. The extracted function body only contains statements that were fully covered by the TextRange and so the `* n` code is deleted. The logic for calculating variable usage and outlived variables for the function parameters and return type respectively search within the TextRange and so do not include `m` or `v`. **Fix:** - Only extract full statements when using block extraction. If a user selected part of a statement, extract that full statement. 8527: Switch introduce_named_lifetime assist to use mutable syntax tree r=matklad a=iDawer This extends `GenericParamsOwnerEdit` trait with `get_or_create_generic_param_list` method Co-authored-by: Brandon <[email protected]> Co-authored-by: Dawer <[email protected]>
| | * | | | Finish GenericParamsOwnerEdit implsDawer2021-04-151-5/+94
| | | | | |
| | * | | | Switch introduce_named_lifetime assist to use mutable syntax treeDawer2021-04-143-36/+147
| | | | | |
| * | | | | Fix extract function with partial block selectionBrandon2021-04-151-2/+62
| | | | | |
* | | | | | Merge #8462bors[bot]2021-04-1916-76/+391
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8462: Expand macros at type position r=jonas-schievink a=cynecx Co-authored-by: cynecx <[email protected]>
| * | | | | hir_ty: cleanups and extend infinitely_recursive_macro_type testcynecx2021-04-182-8/+14
| | | | | |
| * | | | | hir_ty: keep body::Expander in TyLoweringContextcynecx2021-04-184-59/+94
| | | | | |
| * | | | | hir_def: various cleanupscynecx2021-04-183-11/+9
| | | | | |