Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Fix GenericArgs grammar | Aleksey Kladov | 2020-07-31 | 1 | -5/+11 |
| | |||||
* | Rename TypeArgList -> GenericArgList | Aleksey Kladov | 2020-07-31 | 1 | -2/+2 |
| | |||||
* | Minor | Aleksey Kladov | 2020-07-31 | 1 | -1/+0 |
| | |||||
* | Rename LambdaExpr -> ClosureExpr | Aleksey Kladov | 2020-07-31 | 1 | -2/+2 |
| | |||||
* | Item is a Stmt | Aleksey Kladov | 2020-07-31 | 1 | -0/+2 |
| | |||||
* | Reame PlaceholderType -> InferType | Aleksey Kladov | 2020-07-31 | 1 | -1/+1 |
| | |||||
* | Rename TypeRef -> Type | Aleksey Kladov | 2020-07-31 | 3 | -8/+8 |
| | | | | | | | | | | The TypeRef name comes from IntelliJ days, where you often have both type *syntax* as well as *semantical* representation of types in scope. And naming both Type is confusing. In rust-analyzer however, we use ast types as `ast::Type`, and have many more semantic counterparts to ast types, so avoiding name clash here is just confusing. | ||||
* | Use ty to access most TypeRefs | Aleksey Kladov | 2020-07-30 | 2 | -2/+2 |
| | |||||
* | Remove TypeAscriptionOwner | Aleksey Kladov | 2020-07-30 | 3 | -17/+12 |
| | |||||
* | Finalize attribute grammar | Aleksey Kladov | 2020-07-30 | 1 | -2/+2 |
| | |||||
* | Finalize impl Grammar | Aleksey Kladov | 2020-07-30 | 6 | -11/+11 |
| | |||||
* | Finalize Trait grammar | Aleksey Kladov | 2020-07-30 | 2 | -3/+3 |
| | |||||
* | Finalize const&static grammar | Aleksey Kladov | 2020-07-30 | 3 | -5/+5 |
| | |||||
* | Rename EnumVariant -> Variant | Aleksey Kladov | 2020-07-30 | 2 | -3/+3 |
| | |||||
* | Rename EnumDef -> Enum | Aleksey Kladov | 2020-07-30 | 2 | -3/+3 |
| | |||||
* | Rename StructDef -> Struct | Aleksey Kladov | 2020-07-30 | 3 | -6/+6 |
| | |||||
* | Rename FieldDef -> Field | Aleksey Kladov | 2020-07-30 | 2 | -3/+3 |
| | |||||
* | Rename RecordLit -> RecordExpr | Aleksey Kladov | 2020-07-30 | 4 | -6/+6 |
| | |||||
* | Rename TypeParamList -> GenericParamList | Aleksey Kladov | 2020-07-30 | 5 | -13/+14 |
| | |||||
* | Rename TypeAliasDef -> TypeAlias | Aleksey Kladov | 2020-07-30 | 3 | -9/+5 |
| | |||||
* | Rename FnDef -> Fn | Aleksey Kladov | 2020-07-30 | 9 | -17/+17 |
| | |||||
* | Rename UseItem -> Use | Aleksey Kladov | 2020-07-30 | 4 | -8/+8 |
| | |||||
* | Rename Rename | Aleksey Kladov | 2020-07-30 | 1 | -1/+1 |
| | |||||
* | Split ItemList & AssocItemList | Aleksey Kladov | 2020-07-30 | 4 | -5/+7 |
| | |||||
* | Merge #5554 | bors[bot] | 2020-07-29 | 1 | -9/+85 |
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 5554: Fix remove_dbg r=matklad a=petr-tik Closes #5129 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 Co-authored-by: petr-tik <[email protected]> | ||||
| * | Addressed code review | petr-tik | 2020-07-29 | 1 | -27/+41 |
| | | | | | | | | | | | | | | replaced match with let-if variable assignment removed the unnecessary semicolon_on_end variable converted all code and expected test variables to raw strings and inlined them in asserts | ||||
| * | 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 |
| | | |||||
* | | Rename NomialDef -> AdtDef | Aleksey Kladov | 2020-07-29 | 2 | -3/+3 |
|/ | |||||
* | 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 |
| | |||||
* | Merge #5350 | bors[bot] | 2020-07-15 | 4 | -5/+98 |
|\ | | | | | | | | | | | | | | | | | | | | | 5350: Filter assists r=matklad a=kjeremy Uses the `CodeActionContext::only` field to compute only those assists the client cares about. It works but I don't really like the implementation. Co-authored-by: kjeremy <[email protected]> Co-authored-by: Jeremy Kolb <[email protected]> | ||||
| * | Move allow list into AssistConfig | Jeremy Kolb | 2020-07-15 | 4 | -26/+24 |
| | | |||||
| * | Filter assists | kjeremy | 2020-07-13 | 3 | -13/+108 |
| | | |||||
* | | 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 |
| | |||||
* | Refactor AssistBuilder to manage a SourceChange | Christoph Herzog | 2020-07-07 | 1 | -7/+7 |
| | | | | | | | `AssistBuilder`` now managaes a full `SourceChange` instead of a Vec<SourceFileEdit>. This prepares AssistBuilder to handle creation of new files. | ||||
* | Add AssistKind::Generate | Aleksey Kladov | 2020-07-03 | 6 | -25/+31 |
| | |||||
* | Unify naming of generating assists | Aleksey Kladov | 2020-07-03 | 7 | -206/+215 |
| | |||||
* | Don't categorize things we don't care about | kjeremy | 2020-07-02 | 4 | -64/+52 |
| | |||||
* | Move AssistKind into AssistId | kjeremy | 2020-07-02 | 40 | -139/+89 |
| |