aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Rename the method to avoid false promisesKirill Bulatov2020-09-092-17/+15
|
* Add VirtualPath testsKirill Bulatov2020-09-091-3/+38
|
* Fix the testsKirill Bulatov2020-09-095-7/+9
|
* Add testsKirill Bulatov2020-09-091-0/+153
|
* Properly handle mod.rs importsKirill Bulatov2020-09-091-47/+40
|
* Complete semicolon when neededKirill Bulatov2020-09-096-26/+30
|
* Properly reacto to keywordsKirill Bulatov2020-09-096-4/+28
|
* Less false positive completion candidatesKirill Bulatov2020-09-091-44/+59
|
* Properly handle nested submodules in the same fileKirill Bulatov2020-09-091-70/+79
|
* Move most of the logic into the completion moduleKirill Bulatov2020-09-098-118/+107
|
* Finally cretae the mod completion moduleKirill Bulatov2020-09-094-18/+43
|
* Move rust-related logic from vfs to base_db levelKirill Bulatov2020-09-092-61/+74
|
* Properly handle special cases (binaries, mod.rs)Kirill Bulatov2020-09-093-52/+60
|
* Exclude special filesKirill Bulatov2020-09-092-3/+5
|
* Implement file name & extension retrieval method for VirtualPathKirill Bulatov2020-09-091-2/+20
|
* Small refactoringKirill Bulatov2020-09-092-32/+26
|
* Happy path implementedKirill Bulatov2020-09-098-26/+38
|
* Properly use FileSet APIKirill Bulatov2020-09-093-38/+49
|
* Better APIKirill Bulatov2020-09-098-24/+64
|
* First steps for mod<|> completionKirill Bulatov2020-09-094-6/+65
|
* Merge #5968bors[bot]2020-09-093-14/+30
|\ | | | | | | | | | | | | | | | | | | 5968: Lookup ADT and associated type names for chalk debugging / tweak chalk interner r=flodiebold a=nathanwhit This PR improves the chalk program writing integration by looking up the names for ADTs and associated types, making the output much more readable. There are also a few small changes to the interner, which gives some nice performance improvements. We clone `Ty`s and `ProgramClause`s relatively often in chalk, so wrapping them in `Arc`s is a perf win. This takes the time for performing type inference on the rust-analyzer codebase from 40s to 33s on my machine. Co-authored-by: Nathan Whitaker <[email protected]>
| * Tweak interner for chalkNathan Whitaker2020-09-091-9/+9
| |
| * Lookup ADT and assoc. type names for chalk debugNathan Whitaker2020-09-092-5/+21
| |
* | Merge #5965bors[bot]2020-09-081-16/+13
|\ \ | | | | | | | | | | | | | | | | | | | | | 5965: Update crates r=kjeremy a=kjeremy Moves past pulled crossbeam-channel crate Co-authored-by: kjeremy <[email protected]>
| * | Update crateskjeremy2020-09-081-16/+13
|/ / | | | | | | Gets passed pulled crossbeam crate
* | Merge #5940bors[bot]2020-09-045-1/+267
|\ \ | | | | | | | | | | | | | | | | | | | | | 5940: Implement "Replace `impl Trait` function argument with the named generic" assist. r=matklad a=alekseysidorov Fixes #5085 Co-authored-by: Aleksei Sidorov <[email protected]>
| * | Remove unnecessary commaAleksei Sidorov2020-09-043-11/+10
| | |
| * | Fix testsAleksei Sidorov2020-09-043-18/+25
| | |
| * | Update codegenAleksei Sidorov2020-09-041-0/+13
| | |
| * | Fix nitpicksAleksei Sidorov2020-09-042-24/+14
| | |
| * | Resolve most of corner casesAleksei Sidorov2020-09-032-12/+124
| | |
| * | Initial implementation of the #5085 issueAleksei Sidorov2020-09-034-0/+145
| | |
* | | Merge #5935bors[bot]2020-09-046-535/+744
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 5935: Rewrite import insertion r=matklad a=Veykril This is my attempt at refactoring the import insertion #3947. I hope what I created here is somewhat in line with what was requested, it wouldn't surprise me . `common_prefix` is a copy from `merge_imports.rs` so those should be unified somewhere, `try_merge_trees` is also copied from there but slighly modified to take the `MergeBehaviour` enum into account. `MergeBehaviour` should in the end become a configuration option, and the order if `ImportGroup` probably as well? I'm not too familiar with the assist stuff and the like which is why I dont know what i have to do with `insert_use_statement` and `find_insert_use_container` for now. I will most likely add more test cases in the end as well as I currently only tried to hit every path in `find_insert_position`. Some of the merge tests also fail atm due to them not sorting what they insert. There is also this test case I'm not sure if we want to support it. I would assume we want to? https://github.com/rust-analyzer/rust-analyzer/pull/5935/files#diff-6923916dd8bdd2f1ab4b984adacd265fR540-R547 The entire module was rewritten so looking at the the file itself is probably better than looking at the diff. Regarding the sub issues of #3947: - #3301: This is fixed with the rewrite, what this implementation does is that it scans through the first occurence of groupings and picks the appropriate one out. This means the user can actually rearrange the groupings on a per file basis to their liking. If a group isnt being found it is inserted according to the `ImportGroup` variant order(Would be nice if this was configurable I imagine). - #3831: This should be fixed with the introduced `MergeBehaviour` enum and it's `Last` variant. - #3946: This should also be [fixed](https://github.com/rust-analyzer/rust-analyzer/pull/5935/files#diff-6923916dd8bdd2f1ab4b984adacd265fR87) - #5795: This is fixed in the sense that the grouping search picks the first group that is of the same kind as the import that is being added. So if there is a random import in the middle of the program it should only be considered if there is no group of the same kind in the file already present. - the last point in the list I havent checked yet, tho I got the feeling that it's not gonna be too simple as that will require knowledge of whether in this example `ast` is a crate or the module that is already imported. Co-authored-by: Lukas Wirth <[email protected]>
| * | | Add extra insert_use test for pub(crate) re-export handlingLukas Wirth2020-09-032-12/+19
| | | |
| * | | Disable insert_import in extract_struct_from_enum_variant until its fixedLukas Wirth2020-09-031-0/+4
| | | |
| * | | Fix import insertion breaking nested modulesLukas Wirth2020-09-035-62/+117
| | | |
| * | | Fix inserting imports in front of inner attributesLukas Wirth2020-09-032-3/+54
| | | |
| * | | Replace insert_use_statement with the new insert_useLukas Wirth2020-09-034-54/+68
| | | |
| * | | Impl make::blank_lineLukas Wirth2020-09-033-27/+20
| | | |
| * | | Use mark to check that paths that are too long will not be mergedLukas Wirth2020-09-031-0/+13
| | | |
| * | | Tidy up tests and apply suggested changesLukas Wirth2020-09-031-72/+107
| | | |
| * | | Add more import insertion testsLukas Wirth2020-09-031-3/+68
| | | |
| * | | Begin refactor of import insertionLukas Wirth2020-09-031-468/+440
| |/ /
* | | Merge #5948bors[bot]2020-09-033-5/+5
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | 5948: Up proc-macro2 to 1.20 r=matklad a=kjeremy This changes the way Display is implemented Co-authored-by: kjeremy <[email protected]>
| * | Up proc-macro2 to 1.20kjeremy2020-09-033-5/+5
|/ / | | | | | | This changes the way Display is implemented
* | Merge #5942bors[bot]2020-09-031-3/+6
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | 5942: Actually assert disjointness r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | Actually assert disjointnessAleksey Kladov2020-09-031-3/+6
|/ /
* | Merge #5941bors[bot]2020-09-032-5/+5
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | 5941: Unify naming r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | Unify namingAleksey Kladov2020-09-032-5/+5
| | |
* | | Merge #5937bors[bot]2020-09-032-11/+11
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | 5937: Chalk 0.25 r=matklad a=kjeremy Picks up flodiebold's infinite loop fix Co-authored-by: kjeremy <[email protected]>
generated by cgit v1.2.3 (git 2.25.1) at 2024-11-25 13:33:11 +0000