aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| | * | | | | Add some more marks to rename testsLukas Wirth2021-01-101-17/+58
| | | | | | |
| | * | | | | Use local error macros in references/renameLukas Wirth2021-01-101-39/+30
| | | | | | |
| | * | | | | Refactor rename name checkingLukas Wirth2021-01-092-59/+104
| | | | | | |
| * | | | | | Add check for dbg! macroEdwin Cheng2021-01-081-1/+28
| | | | | | |
| * | | | | | Remove unused dbgEdwin Cheng2021-01-081-4/+3
| | | | | | |
| * | | | | | Use another name instead of dbg for testEdwin Cheng2021-01-081-4/+4
| | |_|_|_|/ | |/| | | |
* | | | | | Merge #7244bors[bot]2021-01-112-40/+54
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7244: Goto for inner doc links works for module inner doc comments r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | | Goto for inner doc links works for module inner doc commentsAleksey Kladov2021-01-112-40/+54
|/ / / / /
* | | | | Merge #7235bors[bot]2021-01-111-1/+25
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7235: Postfix completions like ifl now works with references r=Veykril a=Maan2003 Closes #7230 Co-authored-by: Manmeet <[email protected]> Co-authored-by: Manmeet Maan <[email protected]>
| * | | | | Fix formattingManmeet Maan2021-01-111-1/+1
| | | | | |
| * | | | | Apply Suggestions from code reviewManmeet Maan2021-01-111-9/+6
| | | | | |
| * | | | | Postfix like ifl now works with referencesManmeet2021-01-101-2/+29
| | | | | |
* | | | | | Merge #7241bors[bot]2021-01-103-52/+71
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7241: Honor client's dynamic registration caps r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | | | Honor client's dynamic registration capsAleksey Kladov2021-01-103-52/+71
| | | | | | | | | | | | | | | | | | | | | | | | | | | | cc https://github.com/rust-analyzer/rust-analyzer/pull/5516#issuecomment-757520828
* | | | | | | Merge #7218bors[bot]2021-01-1032-118/+114
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7218: Fix typos r=Veykril a=regexident Apart from the very last commit on this PR (which fixes a public type's name) all changes are non-breaking. Co-authored-by: Vincent Esche <[email protected]>
| * | | | | | | Updated tests via `cargo xtask codegen`Vincent Esche2021-01-102-50/+50
| | | | | | | |
| * | | | | | | Fixed typos in public type namesVincent Esche2021-01-091-2/+2
| | | | | | | |
| * | | | | | | Fixed typos in private type namesVincent Esche2021-01-091-5/+5
| | | | | | | |
| * | | | | | | Fixed typos in local bindingsVincent Esche2021-01-093-7/+7
| | | | | | | |
| * | | | | | | Fixed typos in testsVincent Esche2021-01-093-6/+6
| | | | | | | |
| * | | | | | | Fixed typos in code assertsVincent Esche2021-01-091-1/+1
| | | | | | | |
| * | | | | | | Fixed typos in `log::warn!` messageVincent Esche2021-01-091-5/+1
| | | | | | | |
| * | | | | | | Fixed typos in code commentsVincent Esche2021-01-0923-42/+42
| | |_|_|/ / / | |/| | | | |
* | | | | | | Merge #7240bors[bot]2021-01-101-1/+1
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7240: Make default memory stats less verbose r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | | | | Make default memory stats less verboseAleksey Kladov2021-01-101-1/+1
| | |/ / / / / | |/| | | | |
* | | | | | | Merge #7239bors[bot]2021-01-1012-44/+50
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7239: Replace SyntaxKind usage with T! macro where applicable r=lnicola a=Veykril https://github.com/rust-analyzer/rust-analyzer/blob/master/docs/dev/style.md#token-names Co-authored-by: Lukas Wirth <[email protected]>
| * | | | | | | Replace SyntaxKind usage with T! macro where applicableLukas Wirth2021-01-1012-44/+50
| | | | | | | |
* | | | | | | | Merge #7238bors[bot]2021-01-1010-135/+135
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7238: Fix typo, parentheses is plural r=lnicola a=lnicola bors r+ Co-authored-by: Laurențiu Nicola <[email protected]>
| * | | | | | | Fix typo, parentheses is pluralLaurențiu Nicola2021-01-1010-135/+135
| |/ / / / / /
* | | | | | | Merge #7237bors[bot]2021-01-102-1/+22
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7237: Use T! for bool keywords r=matklad a=lnicola Co-authored-by: Laurențiu Nicola <[email protected]>
| * | | | | | Use T! for bool keywordsLaurențiu Nicola2021-01-102-1/+22
|/ / / / / /
* | | | | | Merge #7236bors[bot]2021-01-105-12/+57
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7236: Fix progress token is already registered crash r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | | Fix progress token is already registered crashAleksey Kladov2021-01-105-12/+57
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | After we started reporting progress when running cargo check during loading, it is possible to crash the client with two identical progress tokens. This points to a deeper issue: we might be running several cargo checks concurrently, which doesn't make sense. This commit linearizes all workspace fetches, making sure no updates are lost. As an additional touch, it also normalizes progress & result reporting, to make sure they stand in sync.
* | | | | | Merge #7234bors[bot]2021-01-103-11/+25
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7234: Allow `true` and `false` keywords in const generics r=lnicola a=callym This should fix #7232 - hopefully I've done it right, it passes the tests and the rast file looks like it's picking up the true and false in the test Co-authored-by: Callym <[email protected]>
| * | | | | Allow `true` and `false` keywords in const genericsCallym2021-01-103-11/+25
|/ / / / /
* | | | | Merge #6238bors[bot]2021-01-1014-307/+368
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6238: Split punctuation semantic highlighting up into more tags r=matklad a=Veykril Open question would be the name of the delimiter modifiers. I chose them this was as I see them this way but from what I remember people tend to mix the names however they like. So maybe using `delimSquare`, `delimCurly`, `delimRound` would be better. That would also go well with `angle` becoming `delimAngle`? Closes #6152 Co-authored-by: Lukas Wirth <[email protected]>
| * | | | | Split punctuation semantic highlighting up into more tagsLukas Wirth2021-01-1014-307/+368
| | | | | |
| | | | | |
| \ \ \ \ \
*-. \ \ \ \ \ Merge #7206 #7231bors[bot]2021-01-1012-51/+75
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7206: Use hir::GenericParam in ide_db::Definition instead of relisting all 3 r=Veykril a=Veykril Basically just this: ```diff pub enum Definition { Macro(MacroDef), Field(Field), ModuleDef(ModuleDef), SelfType(Impl), Local(Local), - TypeParam(TypeParam), - LifetimeParam(LifetimeParam), - ConstParam(ConstParam), + GenericParam(GenericParam), Label(Label), } ``` 7231: Cleaner API r=matklad a=matklad bors r+ 🤖 Co-authored-by: Lukas Wirth <[email protected]> Co-authored-by: Aleksey Kladov <[email protected]>
| | * | | | | | Cleaner APIAleksey Kladov2021-01-103-9/+10
| | | | | | | |
| * | | | | | | Use hir::GenericParam in ide_db::Definition instead of relisting all 3Lukas Wirth2021-01-109-42/+65
| | |/ / / / / | |/| | | | |
* | | | | | | Merge #7228bors[bot]2021-01-101-6/+8
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7228: Avoid string copy in complete_attribute r=Veykril a=lnicola Co-authored-by: Laurențiu Nicola <[email protected]>
| * | | | | | | Avoid string copy in complete_attributeLaurențiu Nicola2021-01-101-6/+8
| |/ / / / / /
* | | | | | | Merge #7229bors[bot]2021-01-102-37/+42
|\ \ \ \ \ \ \ | |/ / / / / / |/| / / / / / | |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7229: Cleanup r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | | CleanupAleksey Kladov2021-01-102-37/+42
| | | | | |
* | | | | | Merge #6980bors[bot]2021-01-102-5/+111
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6980: Implement to support intra-doc link r=matklad a=sasurau4 Helps with #6168 This PR is very limited implementation to support intra-doc. It only support links indicate same file function. I want someone to feedback me about this implementation. If the approach is good, I will continue this PR to support other symbols like enum and struct. Co-authored-by: Daiki Ihara <[email protected]>
| * | | | | | Support intra_doc linksDaiki Ihara2021-01-082-5/+111
| | |_|/ / / | |/| | | |
* | | | | | Merge #7224bors[bot]2021-01-101-7/+19
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7224: Remove unnecessary allocation when checking whether to hide argument name hint r=jhpratt a=jhpratt The case-insensitive prefix/suffix check can be performed character-by-character. This allows the check to be done without having to allocate a new string. As a side effect, it's also no longer necessary to convert the entire string to lowercase, as it's done as needed. As the only case equality we're handling is ASCII, this operation can be further optimized by using byte equality, rather than character equality. cc @SomeoneToIgnore, as it's an update on my PR from yesterday. Co-authored-by: Jacob Pratt <[email protected]>
| * | | | | | Skip leading underscores unconditionallyJacob Pratt2021-01-101-2/+5
| | | | | | |
| * | | | | | Short-circuit boolean operationJacob Pratt2021-01-101-4/+8
| | | | | | |
| * | | | | | Remove unnecessary allocationJacob Pratt2021-01-101-7/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The case-insensitive prefix/suffix check can be performed character-by-character. This allows the check to be done without having to allocate a new string. As a side effect, it's also no longer necessary to convert the entire string to lowercase, as it's done as needed. As the only case equality we're handling is ASCII, this operation can be further optimized by using byte equality, rather than character equality.