aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | vscode-postrefactor: compare dates by value, not by referenceVeetaha2020-03-141-2/+2
| | | | |
| * | | | vscode-postrefactor: eliminate my-mistake floating promise @matkladVeetaha2020-03-141-1/+1
| | | | |
| * | | | vscode-postrefactor: global storagesVeetaha2020-03-141-20/+19
| | | | |
| * | | | vscode: put comma backVeetaha2020-03-141-1/+1
| | | | |
| * | | | vscode: make bailing out on custom serverPath more evidentVeetaha2020-03-142-10/+8
| | | | |
| * | | | vscode: npm run fixVeetaha2020-03-142-4/+4
| | | | |
| * | | | vscode: amend server installation logic to account for nightliesVeetaha2020-03-143-52/+57
| | | | |
| * | | | vscode: add nightly extension installation logicVeetaha2020-03-141-0/+131
| | | | |
| * | | | vscode: prepare config for nightliesVeetaha2020-03-141-15/+96
| | | | |
| * | | | vscode: prepare package.json for nightliesVeetaha2020-03-142-3/+16
| | | | |
| * | | | vscode: get release date from release infoVeetaha2020-03-142-14/+7
| | | | |
| * | | | vscode-prerefactor: add some utility functionsVeetaha2020-03-141-6/+63
| | | | |
| * | | | vscode-prerefactor: merge two files into downloads.tsVeetaha2020-03-142-50/+46
| | | | |
* | | | | Merge #3540bors[bot]2020-03-167-20/+63
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3540: Swtches to rust SSR query check r=matklad a=mikhail-m1 related to #3186 Co-authored-by: Mikhail Modin <[email protected]>
| * | | | | Swtches to rust SSR query checkMikhail Modin2020-03-157-20/+63
| | | | | |
* | | | | | Merge #3598bors[bot]2020-03-161-17/+3
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3598: ra_hir_expand: migrate to impl_intern_key!() r=matklad a=Veetaha Co-authored-by: Veetaha <[email protected]> Co-authored-by: Veetaha <[email protected]>
| * | | | | ra_hir_expand: change the ordering of imports as per the formatterVeetaha2020-03-151-1/+1
| | | | | |
| * | | | | ra_hir_expand: migrate to impl_intern_key!()Veetaha2020-03-151-17/+3
| | | | | |
* | | | | | Merge #3602bors[bot]2020-03-162-66/+33
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3602: ra_ide: remove dead code, migrate from readonly String -> &str r=matklad a=Veetaha https://rust-lang.zulipchat.com/#narrow/stream/185405-t-compiler.2Fwg-rls-2.2E0/topic/hover/near/190671355 Co-authored-by: veetaha <[email protected]>
| * | | | | ra_ide: refactor readonly String -> &strveetaha2020-03-162-25/+28
| | | | | |
| * | | | | ra_ide: remove dead code in HoverResultveetaha2020-03-151-41/+5
| | | | | |
* | | | | | Merge #3597bors[bot]2020-03-151-2/+2
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3597: Update anyhow r=kjeremy a=kjeremy Co-authored-by: Jeremy Kolb <[email protected]>
| * | | | | | Update anyhowJeremy Kolb2020-03-151-2/+2
|/ / / / / /
* | | | | | Merge #3591bors[bot]2020-03-156-16/+96
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3591: Support local macro_rules r=matklad a=edwin0cheng This PR implement local `macro_rules` in function body, by adding following things: 1. While lowering, add a `MacroDefId` in body's `ItemScope` as a textual legacy macro. 2. Make `Expander::enter_expand` search with given `ItemScope`. 3. Make `Resolver::resolve_path_as_macro` search with `LocalItemScope`. Fix #2181 Co-authored-by: Edwin Cheng <[email protected]>
| * | | | | | Support local macro_rulesEdwin Cheng2020-03-146-16/+96
| | | | | | |
* | | | | | | Merge #3595bors[bot]2020-03-153-11/+80
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3595: Fix completion of trait items r=matklad a=flodiebold Trait items should be public by default. Co-authored-by: Florian Diebold <[email protected]>
| * | | | | | Fix completion of trait itemsFlorian Diebold2020-03-143-11/+80
|/ / / / / / | | | | | | | | | | | | | | | | | | Trait items should be public by default.
* | | | | | Merge #3583bors[bot]2020-03-135-37/+33
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3583: Simplify r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | | SimplifyAleksey Kladov2020-03-132-8/+5
| | | | | |
| * | | | | Simplify testsAleksey Kladov2020-03-131-20/+20
| | | | | |
| * | | | | Don't use generic DB where a concrete one will doAleksey Kladov2020-03-132-9/+8
|/ / / / /
| | | | |
| \ \ \ \
*-. \ \ \ \ Merge #3561 #3577bors[bot]2020-03-1312-45/+122
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3561: feat: add debug code lens r=matklad a=hdevalke Refs #3539 3577: Protect against infinite macro expansion in def collector r=edwin0cheng a=flodiebold Something I noticed while trying to make macro expansion more resilient against errors. There was a test for this, but it wasn't actually working because the first recursive expansion failed. (The comma...) Even with this limit, that test (when fixed) still takes some time to pass because of the exponential growth of the expansions, so I disabled it and added a different one without growth. CC @edwin0cheng Co-authored-by: Hannes De Valkeneer <[email protected]> Co-authored-by: hdevalke <[email protected]> Co-authored-by: Florian Diebold <[email protected]>
| | * | | | | Protect against infinite macro expansion in def collectorFlorian Diebold2020-03-131-9/+39
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There was a test for this, but it wasn't actually working because the first recursive expansion failed. (The comma...) Even with this limit, that test (when fixed) still takes some time to pass because of the exponential growth of the expansions, so I disabled it and added a different one without growth.
| * | | | | | Update editors/code/src/config.tshdevalke2020-03-121-1/+0
| | | | | | | | | | | | | | | | | | | | | Co-Authored-By: Veetaha <[email protected]>
| * | | | | | fixup! feat: add debug code lensHannes De Valkeneer2020-03-127-19/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | autodetect vscode-lldb
| * | | | | | fixup! feat: add debug code lensHannes De Valkeneer2020-03-123-18/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | avoid repetition of `--no-run`
| * | | | | | fixup! feat: add debug code lensHannes De Valkeneer2020-03-121-2/+1
| | | | | | |
| * | | | | | Update crates/rust-analyzer/src/main_loop/handlers.rs hdevalke2020-03-121-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | use `Vec::new` instead of `Vec::with_capacity(0)` Co-Authored-By: Veetaha <[email protected]>
| * | | | | | feat: add debug code lensHannes De Valkeneer2020-03-118-30/+77
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Refs #3539
* | | | | | | Merge #3579bors[bot]2020-03-131-10/+10
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3579: Update deps r=kjeremy a=kjeremy Co-authored-by: kjeremy <[email protected]>
| * | | | | | | Update depskjeremy2020-03-131-10/+10
|/ / / / / / /
* | | | | | | Merge #3576bors[bot]2020-03-137-14/+178
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3576: Parse variadics correctly r=matklad a=matklad closes #3571 bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | | | Parse variadics correctlyAleksey Kladov2020-03-137-14/+178
|/ / / / / / | | | | | | | | | | | | | | | | | | closes #3571
* | | | | | Merge #3574bors[bot]2020-03-135-6/+62
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3574: Fix completion of HashMap::new r=matklad a=flodiebold The `ty` function in code_model returned the type with placeholders for type parameters. That's nice for printing, but not good for completion, because placeholders won't unify with anything else: So the type we got for `HashMap` was `HashMap<K, V, T>`, which doesn't unify with `HashMap<?, ?, RandomState>`, so the `new` method wasn't shown. Now we instead return `HashMap<{unknown}, {unknown}, {unknown}>`, which does unify with the impl type. Maybe we should just expose this properly as variables though, i.e. we'd return something like `exists<type, type, type> HashMap<?0, ?1, ?2>` (in Chalk notation). It'll make the API more complicated, but harder to misuse. (And it would handle cases like `type TypeAlias<T> = HashMap<T, T>` more correctly.) The `ty` function for fields was used for signature help, so there we want placeholders so that it looks nicer, I think. Hence I renamed it to `signature_ty`. Co-authored-by: Florian Diebold <[email protected]>
| * | | | | | Fix completion of HashMap::newFlorian Diebold2020-03-135-6/+62
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `ty` function in code_model returned the type with placeholders for type parameters. That's nice for printing, but not good for completion, because placeholders won't unify with anything else: So the type we got for `HashMap` was `HashMap<K, V, T>`, which doesn't unify with `HashMap<?, ?, RandomState>`, so the `new` method wasn't shown. Now we instead return `HashMap<{unknown}, {unknown}, {unknown}>`, which does unify with the impl type. Maybe we should just expose this properly as variables though, i.e. we'd return something like `exists<type, type, type> HashMap<?0, ?1, ?2>` (in Chalk notation). It'll make the API more complicated, but harder to misuse. (And it would handle cases like `type TypeAlias<T> = HashMap<T, T>` more correctly.)
* | | | | | Merge #3575bors[bot]2020-03-1314-96/+103
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3575: Restore cargo-fmt gating r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | | | Restore cargo-fmt gatingAleksey Kladov2020-03-134-19/+12
| | | | | | |
| * | | | | | Move verbose tests out of lineAleksey Kladov2020-03-1310-77/+91
|/ / / / / /
* | | | | | Merge #3553bors[bot]2020-03-136-11/+108
|\ \ \ \ \ \ | |_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3553: Completions do not show for function with same name as mod r=matklad a=JoshMcguigan fixes #3444 I've added a test case in `crates/ra_ide/src/completion/complete_path.rs` which verifies the described behavior in #3444. Digging in, I found that [the module scope iterator](https://github.com/JoshMcguigan/rust-analyzer/blob/ba62d8bd1ce8a68b8d21aaf89ae1ea6787f18366/crates/ra_ide/src/completion/complete_path.rs#L22) only provides the module `z`, and does not provide the function `z` (although if I name the function something else then it does show up here). I thought perhaps the name wasn't being properly resolved, but I added a test in `crates/ra_hir_def/src/nameres/tests.rs` which seems to suggest that it is? I've tried to figure out how to bridge the gap between these two tests (one passing, one failing) to see where the function `z` is being dropped, but to this point I haven't been able to track it down. Any pointers on where I might look for this? Co-authored-by: Josh Mcguigan <[email protected]>
| * | | | | fix issue 3444Josh Mcguigan2020-03-136-11/+108
| |/ / / /