aboutsummaryrefslogtreecommitdiff
path: root/crates/assists/src
Commit message (Collapse)AuthorAgeFilesLines
* use `&T` for non copy params of extracted functionVladyslav Katasonov2021-02-041-2/+55
| | | | Use shared ref if param is not `T: Copy` and is used after body
* split extract_function into pieces and order themVladyslav Katasonov2021-02-041-380/+510
|
* remove ignored test for downgrading mut to sharedVladyslav Katasonov2021-02-031-30/+0
|
* allow calling `&mut` methods on outer vars when extracing functionVladyslav Katasonov2021-02-031-0/+116
|
* allow `&mut param` when extracting functionVladyslav Katasonov2021-02-031-3/+107
| | | | | | Recognise &mut as variable modification. This allows extracting functions with `&mut var` with `var` being in outer scope
* allow modifications of vars from outer scope inside extracted functionVladyslav Katasonov2021-02-031-45/+336
| | | | | It currently allows only directly setting variable. No `&mut` references or methods.
* allow local variables to be used after extracted bodyVladyslav Katasonov2021-02-031-41/+183
| | | | | when variable is defined inside extracted body export this variable to original scope via return value(s)
* change TODO to FIXMEVladyslav Katasonov2021-02-031-2/+2
|
* disable test for downgrading mutability on extractVladyslav Katasonov2021-02-031-0/+3
|
* convert IdentPat to Pat via IntoVladyslav Katasonov2021-02-031-5/+5
| | | | before child getter was used
* support extracting methods; no mut loweringVladyslav Katasonov2021-02-031-37/+191
| | | | | currently mut refernce will *not* be downgraded to shared if it is sufficient(see relevant test for example)
* initial version of extract function assistVladyslav Katasonov2021-02-033-0/+848
| | | | | | | | there are a few currently limitations: * no modifications of function body * does not handle mutability and references * no method support * may produce incorrect results
* Enable fill_match_arms in macrosLukas Wirth2021-01-271-3/+35
|
* Add failing test caseDániel Buga2021-01-261-0/+28
|
* Deduplicate variant matchingDániel Buga2021-01-223-42/+25
|
* replace_if_let_with_match: don't assume sad patternDániel Buga2021-01-221-2/+88
|
* Merge #7310 #7395bors[bot]2021-01-223-0/+249
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7310: Add assist: add lifetime to type r=matklad a=bnjjj close #7200 7395: Obtain `ModuleId`'s `DefMap` through a method r=jonas-schievink a=jonas-schievink Once `ModuleId` can represent modules inside block expressions, this will be required in order to obtain the correct `DefMap`. Otherwise existing code might use a `LocalModuleId` from the wrong `DefMap` and cause a panic. bors r+ Co-authored-by: Benjamin Coenen <[email protected]> Co-authored-by: Jonas Schievink <[email protected]>
| * Add assist: add lifetime to type #7200Benjamin Coenen2021-01-212-18/+23
| | | | | | | | Signed-off-by: Benjamin Coenen <[email protected]>
| * Add assist: add lifetime to type #7200Benjamin Coenen2021-01-181-0/+6
| | | | | | | | Signed-off-by: Benjamin Coenen <[email protected]>
| * Add assist: add lifetime to type #7200Benjamin Coenen2021-01-173-0/+238
| | | | | | | | Signed-off-by: Benjamin Coenen <[email protected]>
* | Treat BlockExpr as a potential module originJonas Schievink2021-01-201-5/+14
| |
* | .Aleksey Kladov2021-01-195-19/+23
| |
* | Show deprecated completions for deprecated traitsKirill Bulatov2021-01-181-1/+1
| |
* | Merge #7297 #7338bors[bot]2021-01-182-42/+40
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7297: Propose trait associated items and autoimport traits on completion r=matklad a=SomeoneToIgnore ![trait_imports](https://user-images.githubusercontent.com/2690773/104819998-6faeb480-583a-11eb-8b45-b7351b51b90e.gif) Closes #7248 7338: Parse `impl const Trait` r=Veykril a=Veykril Closes #7313 bors r+ Co-authored-by: Kirill Bulatov <[email protected]> Co-authored-by: Lukas Wirth <[email protected]>
| * | Add flyimport completion for trait assoc itemsKirill Bulatov2021-01-162-42/+40
| |/
* | Merge #7291bors[bot]2021-01-181-1/+1
|\ \ | |/ |/| | | | | | | | | | | 7291: Wrap remaining self/super/crate in Name{Ref} r=matklad a=Veykril That should be the remaining special casing for `self` 🎉 Co-authored-by: Lukas Wirth <[email protected]>
| * Wrap remaining self/super/crate in Name{Ref}Lukas Wirth2021-01-151-1/+1
| |
* | Share import_assets and related entitiesKirill Bulatov2021-01-167-300/+34
| |
* | Add test mark for skipping single use itemunexge2021-01-151-0/+3
| |
* | Create new nodes only when applying changesunexge2021-01-151-16/+31
| |
* | Add Unmerge Use assistunexge2021-01-153-0/+229
|/
* Merge #7286bors[bot]2021-01-151-3/+2
|\ | | | | | | | | | | | | | | | | 7286: Remove useless wrapper r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * Remove useless wrapperAleksey Kladov2021-01-151-3/+2
| |
* | Add support for yiled keywordDaiki Ihara2021-01-151-22/+23
|/
* Phase out SourceFileEdits in favour of a plain HashMapLukas Wirth2021-01-142-34/+19
|
* Group file source edits by FileIdLukas Wirth2021-01-142-23/+12
|
* Sort impls by trait definitionFisher Darling2021-01-133-0/+238
| | | | Closes #6110
* Ensure uniqueness of file ids in reference search via hashmapLukas Wirth2021-01-123-20/+18
|
* Group references by FileIdLukas Wirth2021-01-123-90/+107
|
* Replace SyntaxKind usage with T! macro where applicableLukas Wirth2021-01-101-6/+6
|
* Better fixture highlightAleksey Kladov2021-01-071-3/+3
|
* Change <|> to $0 - RebaseKevaundray Wedderburn2021-01-0752-816/+804
|
* Remove some stale deprecationsAleksey Kladov2021-01-071-1/+0
|
* Align config's API with usageAleksey Kladov2021-01-066-55/+37
| | | | The config now is mostly immutable, optimize for that.
* Better target for move moduleAleksey Kladov2021-01-063-35/+47
|
* Rename expr -> tail_exprAleksey Kladov2021-01-0510-18/+22
|
* Created an inline-function assist (fixes #6863)Michael-F-Bryan2021-01-043-0/+227
|
* rename exrtract_assignment -> pullAleksey Kladov2021-01-043-60/+60
| | | | | | | Vertical code motions are conventionally called "pull up" / "push down". "extract" is used for introducing new names.
* Support assignment to FieldExpr for extract_assignment assistJesse Bakker2021-01-031-14/+89
|
* Merge #7115bors[bot]2021-01-033-18/+23
|\ | | | | | | | | | | | | | | | | | | | | | | | | 7115: Migrate HasSource::source to return Option r=matklad a=nick96 I've made a start on fixing #6913 based on the provided work plan, migrating `HasSource::source` to return an `Option`. The simple cases are migrated but there are a few that I'm unsure exactly how they should be handled: - Logging the processing of functions in `AnalysisStatsCmd::run`: In verbose mode it includes the path to the module containing the function and the syntax range. I've handled this with an if-let but would it be better to blow up here with `expect`? I'm not 100% on the code paths but if we're processing a function definition then the source should exist. I've handled `source()` in all code paths as `None` being a valid return value but are there some cases where we should just blow up? Also, all I've done is bubble up the returned `None`s, there may be some places where we can recover and still provide something. Co-authored-by: Nick Spain <[email protected]> Co-authored-by: Nick Spain <[email protected]>
generated by cgit v1.2.3 (git 2.25.1) at 2024-11-20 10:39:32 +0000