Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Fixed #5129 | petr-tik | 2020-07-27 | 1 | -8/+23 |
| | | | | | | | | | | | | | | | | | | Addresses two issues: - keep the parens from dbg!() in case the call is chained or there is semantic difference if parens are excluded - Exclude the semicolon after the dbg!(); by checking if it was accidentally included in the macro_call investigated, but decided against: fix ast::MacroCall extraction to never include semicolons at the end - this logic lives in rowan. Defensively shorten the macro_range if there is a semicolon token. Deleted unneccessary temp variable macro_args Renamed macro_content to "paste_instead_of_dbg", because it isn't a simple extraction of text inside dbg!() anymore | ||||
* | Make all test fn names consistent in remove_dbg | petr-tik | 2020-07-27 | 1 | -3/+29 |
| | |||||
* | Added failing tests | petr-tik | 2020-07-27 | 1 | -0/+21 |
| | |||||
* | assists: change_return_type_to_result: clarify assist description | Matthias Krüger | 2020-07-22 | 1 | -1/+1 |
| | | | | | | | | I had a -> Option<PathBuf> fn, which I wanted to change to Result<PathBuf, _>, but despite advertising to do so, the assist did not change the result type to Result<PathBuf, _> but instead just wrapped it in a Result: <Result<Option<PathBuf>, _>. I changed the assist description to "Wrap return type in Result" to clarify that the assist only wraps the preexisting type and does not do any deep Option-to-Result refactoring. | ||||
* | Replace existing visibility modifier in fix_visibility | Timo Freiberg | 2020-07-21 | 1 | -15/+75 |
| | |||||
* | Cleanup extact variable | Aleksey Kladov | 2020-07-21 | 1 | -48/+55 |
| | |||||
* | minor | Aleksey Kladov | 2020-07-21 | 1 | -2/+2 |
| | |||||
* | Minor perf tweaks per clippy | Jeremy Kolb | 2020-07-19 | 2 | -2/+2 |
| | |||||
* | Add turbo-fish works after `()` | Aleksey Kladov | 2020-07-18 | 1 | -1/+28 |
| | |||||
* | More precise ranges in remove hashes assist | Aleksey Kladov | 2020-07-17 | 1 | -79/+52 |
| | |||||
* | Move remove_bounds to edit.rs | Jonas Schievink | 2020-07-14 | 1 | -9/+1 |
| | |||||
* | missing impl members: remove assoc. type bounds | Jonas Schievink | 2020-07-14 | 1 | -0/+33 |
| | |||||
* | change vis works on statics | Aleksey Kladov | 2020-07-12 | 1 | -3/+13 |
| | |||||
* | Better caret placement when filling match arms | Aleksey Kladov | 2020-07-10 | 1 | -124/+86 |
| | |||||
* | Don't mess with cursor position when adding hashes | Aleksey Kladov | 2020-07-09 | 1 | -9/+22 |
| | |||||
* | Add AssistKind::Generate | Aleksey Kladov | 2020-07-03 | 5 | -25/+30 |
| | |||||
* | Unify naming of generating assists | Aleksey Kladov | 2020-07-03 | 5 | -87/+96 |
| | |||||
* | Don't categorize things we don't care about | kjeremy | 2020-07-02 | 3 | -62/+52 |
| | |||||
* | Move AssistKind into AssistId | kjeremy | 2020-07-02 | 38 | -118/+72 |
| | |||||
* | Recategorize a few | kjeremy | 2020-07-02 | 3 | -3/+3 |
| | |||||
* | Categorize assists | Jeremy Kolb | 2020-07-02 | 38 | -497/+682 |
| | |||||
* | Record and suggest trait items via ImportMap | Jonas Schievink | 2020-07-02 | 1 | -0/+140 |
| | |||||
* | Reuse Semantics instances | Laurențiu Nicola | 2020-07-01 | 2 | -2/+2 |
| | |||||
* | Use the existing Semantics in auto_import | Laurențiu Nicola | 2020-07-01 | 1 | -6/+6 |
| | |||||
* | Remove db from AssistsContext | Laurențiu Nicola | 2020-07-01 | 10 | -32/+32 |
| | |||||
* | Use more of FxHash* | Laurențiu Nicola | 2020-06-29 | 1 | -3/+3 |
| | |||||
* | Get rid of unused imports | Veetaha | 2020-06-28 | 2 | -10/+3 |
| | |||||
* | Follow matklad suggestions | Veetaha | 2020-06-28 | 3 | -17/+3 |
| | | | | | | - Move vis_offset() to utils.rs - Shorten explicit ra_syntax::ast -> ast - Undo refactoring exhaustive pattern to non-exhaustive | ||||
* | Simlify with matches!() | Veetaha | 2020-06-28 | 3 | -17/+5 |
| | |||||
* | introduce_variable -> extract_variable | Aleksey Kladov | 2020-06-27 | 1 | -53/+53 |
| | |||||
* | Merge #4945 | bors[bot] | 2020-06-26 | 1 | -2/+31 |
|\ | | | | | | | | | | | | | | | 4945: do not suggest assist for return type to result in bad case r=matklad a=bnjjj close #4826 Co-authored-by: Benjamin Coenen <[email protected]> | ||||
| * | do not suggest assist for return type to result in bad case #4826 | Benjamin Coenen | 2020-06-21 | 1 | -0/+3 |
| | | | | | | | | Signed-off-by: Benjamin Coenen <[email protected]> | ||||
| * | do not suggest assist for return type to result in bad case #4826 | Benjamin Coenen | 2020-06-18 | 1 | -2/+28 |
| | | | | | | | | Signed-off-by: Benjamin Coenen <[email protected]> | ||||
* | | Smarter introduce variable | Aleksey Kladov | 2020-06-25 | 1 | -8/+49 |
| | | | | | | | | Use field init shorthand | ||||
* | | Centralize fixture parsing for assists | Aleksey Kladov | 2020-06-23 | 4 | -35/+40 |
| | | |||||
* | | More principled indentation trimming in fixtures | Aleksey Kladov | 2020-06-23 | 3 | -123/+127 |
| | | |||||
* | | Fix panic in split and merge import assists | Laurențiu Nicola | 2020-06-23 | 2 | -1/+21 |
| | | |||||
* | | Remove RelativePathBuf from fixture | Aleksey Kladov | 2020-06-22 | 3 | -3/+3 |
|/ | | | | | The paths in fixture are not really relative (the default one is `/main.rs`), so it doesn't make sense to use `RelativePathBuf` here. | ||||
* | Merge #4878 | bors[bot] | 2020-06-16 | 1 | -50/+230 |
|\ | | | | | | | | | | | | | | | 4878: Make "Replace qualified name with use" replace *all* mentions of the path r=matklad a=jonas-schievink Fixes https://github.com/rust-analyzer/rust-analyzer/issues/4836 Co-authored-by: Jonas Schievink <[email protected]> | ||||
| * | Operate only on AST paths instead of HIR | Jonas Schievink | 2020-06-15 | 1 | -30/+36 |
| | | |||||
| * | Simplify | Jonas Schievink | 2020-06-15 | 1 | -13/+6 |
| | | |||||
| * | It's fookin' raw | Jonas Schievink | 2020-06-15 | 1 | -48/+48 |
| | | |||||
| * | Shorten *all* qualified paths when adding use | Jonas Schievink | 2020-06-13 | 1 | -10/+191 |
| | | |||||
* | | fill_match_arms bind pattern implementation | Toby Dimmick | 2020-06-14 | 1 | -2/+14 |
| | | |||||
* | | fill_match_arms bind pattern test | Toby Dimmick | 2020-06-14 | 1 | -0/+34 |
|/ | |||||
* | Respect casing when searching for imports | Jonas Schievink | 2020-06-10 | 1 | -0/+25 |
| | |||||
* | Add test for macro generated items | Jonas Schievink | 2020-06-10 | 1 | -0/+31 |
| | |||||
* | ImportsLocator: use ImportMap for non-local crates | Jonas Schievink | 2020-06-10 | 1 | -1/+46 |
| | |||||
* | Use correct indent when replacing with match | Aleksey Kladov | 2020-06-09 | 2 | -3/+36 |
| | |||||
* | Unwrap block works with match arms | Aleksey Kladov | 2020-06-09 | 1 | -4/+40 |
| |