aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* | | Merge #419bors[bot]2019-01-033-35/+34
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 419: file-id-to-symbol r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | | embed file_id into FileSymbolAleksey Kladov2019-01-033-35/+34
| | | |
* | | | Merge #418bors[bot]2019-01-031-18/+37
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | 418: field-vis r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | | add pub(crate) works for named fieldsAleksey Kladov2019-01-031-18/+37
| | | |
* | | | Merge #417bors[bot]2019-01-031-1/+2
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | 417: travis-compile r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | | dont build PRs on windowsAleksey Kladov2019-01-031-1/+1
| | | |
| * | | measure compile-time separately on travisAleksey Kladov2019-01-031-0/+1
| | | |
* | | | Merge #416bors[bot]2019-01-0312-214/+286
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 416: assist-builder r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | fix the testAleksey Kladov2019-01-032-6/+1
| | | | |
| * | | | more enterprisey assists APIAleksey Kladov2019-01-0311-210/+287
| |/ / /
* | | | Merge #415bors[bot]2019-01-034-78/+61
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 415: use LSP file system operations r=matklad a=vemoo implements #131 I've replaced `source_file_edits` and `file_system_edits` with `workspace_edit` because [`WorkspacEdit`](https://docs.rs/languageserver-types/0.53.1/languageserver_types/struct.WorkspaceEdit.html) can represent both. I only use `document_changes` because `changes` cannot represent file system operations. But if the client doesn't have the `workspace.workspaceEdit.resourceOperations` capability `WorkspaceEdit` cannot replace the current `FileSystemEdit`. Can we assume that the client will support it? I also adapted the extension code to make use of the new response type, but only for vscode, i don't know if changes have to be made for the emacs part. Co-authored-by: Bernardo <[email protected]>
| * | | fix testsBernardo2019-01-031-9/+14
| | | |
| * | | use lsp WorkspaceEdit instead of custom source_file_edits and file_system_editsBernardo2019-01-033-69/+47
|/ / /
* | | Merge #413bors[bot]2019-01-0312-428/+573
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 413: add visibility owner, refactor assists r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | | split assists over several filesAleksey Kladov2019-01-038-403/+488
| | | |
| * | | rename code-actions -> assistsAleksey Kladov2019-01-033-8/+8
| | | |
| * | | visibility ownerAleksey Kladov2019-01-033-4/+64
|/ / /
* | | Merge #412bors[bot]2019-01-032-1/+27
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | | | | 412: vec-macro r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | hardcode vec macroAleksey Kladov2019-01-032-1/+27
|/ /
* | Merge #409bors[bot]2019-01-031-0/+79
|\ \ | | | | | | | | | | | | | | | | | | | | | 409: Add Analysis#teype_of test r=matklad a=h-michael Co-authored-by: Hirokazu Hata <[email protected]>
| * | Deive type_of test from testsHirokazu Hata2019-01-032-17/+80
| | |
| * | Remove unnecessary mock functionsHirokazu Hata2019-01-031-27/+1
| | |
| * | Add Analysis#teype_of testHirokazu Hata2019-01-032-2/+44
| | |
* | | Merge #410bors[bot]2019-01-036-171/+221
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 410: Detangle symbols r=matklad a=matklad Previously, we used `FileSymbol` both to represent bytes which are stored in the index and as an API of `ra_analysis`. Mixing internal storage format and an API is not a really bright idea, so we introduce `NavigationTarget` to handle API part. Co-authored-by: Aleksey Kladov <[email protected]>
| * | | docsAleksey Kladov2019-01-032-3/+33
| | | |
| * | | use LocalPtr in navigation targetAleksey Kladov2019-01-034-7/+13
| | | |
| * | | use LocalSyntaxPtr for file symbolAleksey Kladov2019-01-033-53/+49
| | | |
| * | | move some logic to navigation targetAleksey Kladov2019-01-033-113/+127
| | | |
| * | | add kind to LocalSyntaxPtrAleksey Kladov2019-01-031-0/+4
|/ / /
* / / explain that completion shouldn't do filteringAleksey Kladov2019-01-031-1/+15
|/ /
* | Merge #408bors[bot]2019-01-021-14/+50
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 408: vscode problem matcher improvements r=matklad a=vemoo The problem matcher wasn't working properly and looking at the rustc errors i realized it could be simplified. I also added a new problem matcher that can be used with https://github.com/passcod/cargo-watch to get the errors in the editor on save. To use it one can create a tasks.json file with: ```json { "version": "2.0.0", "tasks": [ { "type": "shell", "label": "cargo watch", "command": "cargo", "isBackground": true, "args": [ "watch", "-c" ], "problemMatcher": [ "$rustc-watch" ] } ] } ``` I initially implemented it like this: https://github.com/rust-analyzer/rust-analyzer/commit/cff9f62d321a90c45e622f5304e60a248cbcf4f2 but i think there's a bug in vscode so i worked around it by copying the pattern for both problem matchers. The first commit can be used if https://github.com/Microsoft/vscode/pull/65840 is merged. Co-authored-by: Bernardo <[email protected]>
| * | named multiline problem patterns are not parsed properly in vscode at the momentBernardo2019-01-011-2/+29
| | |
| * | fix regex and add rustc-watch problem matcherBernardo2019-01-011-23/+32
| | |
* | | Merge #407bors[bot]2019-01-027-124/+65
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 407: run r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | | simplify runnablesAleksey Kladov2019-01-027-124/+65
| | | |
* | | | Merge #406bors[bot]2019-01-024-218/+203
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 406: Simplify r=matklad a=matklad Get rid of `AnalysisImpl` wrapper around salsa database. It was useful before we migrated by salsa, but it's long have been just a useless boilerplate. Co-authored-by: Aleksey Kladov <[email protected]>
| * | | fix compilationAleksey Kladov2019-01-022-2/+2
| | | |
| * | | doctringsAleksey Kladov2019-01-021-28/+62
| | | |
| * | | get rid of AnalysisImplAleksey Kladov2019-01-022-101/+80
| | | |
| * | | remove AnalysisHostImplAleksey Kladov2019-01-022-45/+26
| | | |
| * | | remove some methods from analysis implAleksey Kladov2019-01-022-22/+13
| | | |
| * | | move world-symbols to file_symbolsAleksey Kladov2019-01-023-42/+42
| | | |
* | | | Merge #405bors[bot]2019-01-022-5/+21
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | 405: extend selection inside a string literal should select a word first r=matklad a=gfreezy fixed #402 Co-authored-by: gfreezy <[email protected]>
| * | | extend selection inside a string literal should select a word firstgfreezy2019-01-022-5/+21
|/ / /
* | | Merge #404bors[bot]2019-01-029-298/+329
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 404: Move FileSymbol to ra_analysis r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | | fix testsAleksey Kladov2019-01-021-5/+5
| | | |
| * | | make FileSymbol privateAleksey Kladov2019-01-022-8/+7
| | | |
| * | | use navigation target in APIAleksey Kladov2019-01-024-26/+44
| | | |
| * | | introduce navigation targetAleksey Kladov2019-01-023-13/+28
| | | |
| * | | move symbols to ra_analysisAleksey Kladov2019-01-026-256/+255
| |/ /