Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
| * | allow calling `&mut` methods on outer vars when extracing function | Vladyslav Katasonov | 2021-02-03 | 1 | -0/+116 | |
| | | ||||||
| * | allow `&mut param` when extracting function | Vladyslav Katasonov | 2021-02-03 | 1 | -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 function | Vladyslav Katasonov | 2021-02-03 | 1 | -45/+336 | |
| | | | | | | | | | | It currently allows only directly setting variable. No `&mut` references or methods. | |||||
| * | allow local variables to be used after extracted body | Vladyslav Katasonov | 2021-02-03 | 1 | -41/+183 | |
| | | | | | | | | | | when variable is defined inside extracted body export this variable to original scope via return value(s) | |||||
| * | change TODO to FIXME | Vladyslav Katasonov | 2021-02-03 | 1 | -2/+2 | |
| | | ||||||
| * | disable test for downgrading mutability on extract | Vladyslav Katasonov | 2021-02-03 | 1 | -0/+3 | |
| | | ||||||
| * | convert IdentPat to Pat via Into | Vladyslav Katasonov | 2021-02-03 | 1 | -5/+5 | |
| | | | | | | | | before child getter was used | |||||
| * | support extracting methods; no mut lowering | Vladyslav Katasonov | 2021-02-03 | 1 | -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 assist | Vladyslav Katasonov | 2021-02-03 | 3 | -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 | |||||
* | | Make `ModPath`'s representation private | Jonas Schievink | 2021-02-04 | 1 | -2/+2 | |
|/ | ||||||
* | Enable fill_match_arms in macros | Lukas Wirth | 2021-01-27 | 1 | -3/+35 | |
| | ||||||
* | Add failing test case | Dániel Buga | 2021-01-26 | 1 | -0/+28 | |
| | ||||||
* | Deduplicate variant matching | Dániel Buga | 2021-01-22 | 3 | -42/+25 | |
| | ||||||
* | replace_if_let_with_match: don't assume sad pattern | Dániel Buga | 2021-01-22 | 1 | -2/+88 | |
| | ||||||
* | Merge #7310 #7395 | bors[bot] | 2021-01-22 | 3 | -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 #7200 | Benjamin Coenen | 2021-01-21 | 2 | -18/+23 | |
| | | | | | | | | Signed-off-by: Benjamin Coenen <[email protected]> | |||||
| * | Add assist: add lifetime to type #7200 | Benjamin Coenen | 2021-01-18 | 1 | -0/+6 | |
| | | | | | | | | Signed-off-by: Benjamin Coenen <[email protected]> | |||||
| * | Add assist: add lifetime to type #7200 | Benjamin Coenen | 2021-01-17 | 3 | -0/+238 | |
| | | | | | | | | Signed-off-by: Benjamin Coenen <[email protected]> | |||||
* | | Treat BlockExpr as a potential module origin | Jonas Schievink | 2021-01-20 | 1 | -5/+14 | |
| | | ||||||
* | | . | Aleksey Kladov | 2021-01-19 | 5 | -19/+23 | |
| | | ||||||
* | | Show deprecated completions for deprecated traits | Kirill Bulatov | 2021-01-18 | 1 | -1/+1 | |
| | | ||||||
* | | Merge #7297 #7338 | bors[bot] | 2021-01-18 | 2 | -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 items | Kirill Bulatov | 2021-01-16 | 2 | -42/+40 | |
| |/ | ||||||
* | | Merge #7291 | bors[bot] | 2021-01-18 | 1 | -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 Wirth | 2021-01-15 | 1 | -1/+1 | |
| | | ||||||
* | | Share import_assets and related entities | Kirill Bulatov | 2021-01-16 | 7 | -300/+34 | |
| | | ||||||
* | | Add test mark for skipping single use item | unexge | 2021-01-15 | 1 | -0/+3 | |
| | | ||||||
* | | Create new nodes only when applying changes | unexge | 2021-01-15 | 1 | -16/+31 | |
| | | ||||||
* | | Add Unmerge Use assist | unexge | 2021-01-15 | 3 | -0/+229 | |
|/ | ||||||
* | Merge #7286 | bors[bot] | 2021-01-15 | 1 | -3/+2 | |
|\ | | | | | | | | | | | | | | | | | 7286: Remove useless wrapper r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]> | |||||
| * | Remove useless wrapper | Aleksey Kladov | 2021-01-15 | 1 | -3/+2 | |
| | | ||||||
* | | Add support for yiled keyword | Daiki Ihara | 2021-01-15 | 1 | -22/+23 | |
|/ | ||||||
* | Phase out SourceFileEdits in favour of a plain HashMap | Lukas Wirth | 2021-01-14 | 2 | -34/+19 | |
| | ||||||
* | Group file source edits by FileId | Lukas Wirth | 2021-01-14 | 2 | -23/+12 | |
| | ||||||
* | Sort impls by trait definition | Fisher Darling | 2021-01-13 | 3 | -0/+238 | |
| | | | | Closes #6110 | |||||
* | Ensure uniqueness of file ids in reference search via hashmap | Lukas Wirth | 2021-01-12 | 3 | -20/+18 | |
| | ||||||
* | Group references by FileId | Lukas Wirth | 2021-01-12 | 3 | -90/+107 | |
| | ||||||
* | Replace SyntaxKind usage with T! macro where applicable | Lukas Wirth | 2021-01-10 | 1 | -6/+6 | |
| | ||||||
* | Better fixture highlight | Aleksey Kladov | 2021-01-07 | 1 | -3/+3 | |
| | ||||||
* | Change <|> to $0 - Rebase | Kevaundray Wedderburn | 2021-01-07 | 52 | -816/+804 | |
| | ||||||
* | Remove some stale deprecations | Aleksey Kladov | 2021-01-07 | 1 | -1/+0 | |
| | ||||||
* | Align config's API with usage | Aleksey Kladov | 2021-01-06 | 6 | -55/+37 | |
| | | | | The config now is mostly immutable, optimize for that. | |||||
* | Better target for move module | Aleksey Kladov | 2021-01-06 | 3 | -35/+47 | |
| | ||||||
* | Rename expr -> tail_expr | Aleksey Kladov | 2021-01-05 | 10 | -18/+22 | |
| | ||||||
* | Created an inline-function assist (fixes #6863) | Michael-F-Bryan | 2021-01-04 | 3 | -0/+227 | |
| | ||||||
* | rename exrtract_assignment -> pull | Aleksey Kladov | 2021-01-04 | 3 | -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 assist | Jesse Bakker | 2021-01-03 | 1 | -14/+89 | |
| | ||||||
* | Merge #7115 | bors[bot] | 2021-01-03 | 3 | -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]> | |||||
| * | Handle missing source in target_data_for_def | Nick Spain | 2021-01-02 | 1 | -13/+12 | |
| | | ||||||
| * | Handle missing source in filter_assoc_items | Nick Spain | 2021-01-02 | 1 | -7/+8 | |
| | |