aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge #2850bors[bot]2020-01-153-31/+9
|\ | | | | | | | | | | | | | | 2850: Use types from vscode-languageclient r=matklad a=kiljacken Now that we're running with 3.15 of the LSP for VSCode we don't need to define these interfaces ourselves. Yay! Co-authored-by: Emil Lauridsen <[email protected]>
| * Use types from vscode-langaugeclientEmil Lauridsen2020-01-153-31/+9
|/
* Merge #2843bors[bot]2020-01-156-13/+203
|\ | | | | | | | | | | | | | | | | | | | | | | 2843: Add inlay parameter name hints for call expr r=matklad a=imtsuki This patch adds Intellij-like parameter name hints for literal values in function calls. <img width="624" alt="Screenshot" src="https://user-images.githubusercontent.com/8423594/72366533-68d7f800-3735-11ea-9279-cf193ca8ca2f.png"> Signed-off-by: imtsuki <[email protected]> Co-authored-by: imtsuki <[email protected]>
| * FnSignature: use unwrap_or_default for parameter_name_listimtsuki2020-01-151-5/+3
| | | | | | | | Signed-off-by: imtsuki <[email protected]>
| * Update test snapshotimtsuki2020-01-141-2/+2
| | | | | | | | Signed-off-by: imtsuki <[email protected]>
| * Add inlay parameter name hints for function callsimtsuki2020-01-146-13/+205
| | | | | | | | Signed-off-by: imtsuki <[email protected]>
* | Merge #2849bors[bot]2020-01-151-0/+16
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2849: Display vscode message after changing cargo-watch options r=edwin0cheng a=memoryruins Currently, changed cargo-watch settings do not go into effect until after a reload. This PR checks for changed `cargoWatchOptions` in the same way as the current `cargoFeatures` check. ![2020-01-14_20-52-20](https://user-images.githubusercontent.com/6868531/72398362-b5f5a300-3710-11ea-9bc1-9943bef08447.gif) Co-authored-by: memoryruins <[email protected]>
| * | Improve readabilitymemoryruins2020-01-151-14/+11
| | |
| * | Display vscode message after changing cargo-watch optionsmemoryruins2020-01-151-0/+19
|/ /
* | Merge #2846bors[bot]2020-01-142-16/+16
|\ \ | | | | | | | | | | | | | | | | | | | | | 2846: Language Server Protocol 3.15 is now stable r=kjeremy a=kjeremy Update the client Co-authored-by: kjeremy <[email protected]>
| * | Language Server Protocol 3.15 is now stablekjeremy2020-01-142-16/+16
|/ / | | | | | | Update the client
* | Merge #2845bors[bot]2020-01-141-5/+1
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | 2845: Cleanup assert r=kjeremy a=kjeremy Re: https://github.com/rust-analyzer/rust-analyzer/pull/2841#discussion_r366510725 thanks @bjorn3! Co-authored-by: kjeremy <[email protected]>
| * | Cleanup assertkjeremy2020-01-141-5/+1
|/ /
* | Merge #2841bors[bot]2020-01-1426-42/+59
|\ \ | | | | | | | | | | | | | | | | | | | | | 2841: More UI friendly labels r=kjeremy a=kjeremy Co-authored-by: Jeremy Kolb <[email protected]>
| * | unwrapJeremy Kolb2020-01-141-2/+2
| | |
| * | Assert that first letter is capitalizedJeremy Kolb2020-01-141-0/+6
| | |
| * | Fix casingJeremy Kolb2020-01-1421-30/+33
| | |
| * | More UI friendly labelsJeremy Kolb2020-01-1423-35/+43
| | |
* | | Merge #2844bors[bot]2020-01-142-85/+15
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2844: Use dummy value for line! and column! macro r=matklad a=edwin0cheng Use dummy value `0` for line! and column! macro. Co-authored-by: Edwin Cheng <[email protected]>
| * | | Use dummy value for line! and column! macroEdwin Cheng2020-01-142-85/+15
| | | |
* | | | Merge #2834bors[bot]2020-01-141-35/+26
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2834: refactor(ra_syntax.validation): removed code duplication from validate_literal() r=kiljacken a=Veetaha Hi! This is my first ever contribution to this project. I've taken some dirty job from issue #223 This is a simple atomic PR to remove code duplication according to FIXME comment in the function that is the main focus of the further development. I just didn't want to mix refactoring with the implementation of new features... I am not sure whether you prefer such atomic PRs here or you'd rather have a single PR that contains all atomic commits inside of it? So if you want me to add all that validation in one PR I'll mark this one as WIP and update it when the work is finished, otherwise, I'll go with the option of creating separate PRs per each feature of validation of strings, numbers, and comments respectively. ### Comments about refactoring Yeah, reducing the duplication is quite hard here, extracting into stateless functions could be another option but the number of their arguments would be very big and repeated across char and string implementations so that just writing their types and names would become cumbersome. I tried the option of having everything captured implicitly in the closure but failed since rust doesn't have templated (or generic) closures as C++ does, this is needed because `unescape_byte*()` and `unescape_char|str()` have different return types... Maybe I am missing something here? I may be wrong because I am not enough experienced in Rust... Well, I am awaiting any kind of feedback! Co-authored-by: Veetaha <[email protected]>
| * | | refactor(ra_syntax.validation): removed code duplication from ↵Veetaha2020-01-141-35/+26
| | |/ | |/| | | | | | | validate_literal() function
* | | Merge #2839bors[bot]2020-01-142-1/+5
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2839: Tweak add_custom_impl r=matklad a=kjeremy Display a nicer label and categorize Co-authored-by: Jeremy Kolb <[email protected]>
| * | | Tweak add_custom_implJeremy Kolb2020-01-142-1/+5
| | | |
* | | | Merge #2838bors[bot]2020-01-147-112/+192
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | 2838: Move Type API to type r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | | Move Type API to typeAleksey Kladov2020-01-144-85/+102
| | | |
| * | | Push resolver further upAleksey Kladov2020-01-144-27/+60
| | | |
| * | | Push resolver upAleksey Kladov2020-01-141-21/+51
|/ / /
| | |
| \ \
*-. \ \ Merge #2828 #2831bors[bot]2020-01-143-19/+19
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2828: Update backtrace r=matklad a=kjeremy Has some fixes for WinXP 2831: vscode-languageclient 6.0.0-next.10 r=matklad a=kjeremy The main change is that it removes progress from the proposed section. Co-authored-by: kjeremy <[email protected]>
| | * | | vscode-languageclient 6.0.0-next.10kjeremy2020-01-132-16/+16
| | | | |
| * | | | Update backtracekjeremy2020-01-131-3/+3
| |/ / /
* | | | Merge #2836bors[bot]2020-01-145-89/+82
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2836: Move utility functions down r=matklad a=matklad Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | Move impls_future to Type, where it belongsAleksey Kladov2020-01-144-28/+21
| | | | |
| * | | | Use lang-items to resolve future traitAleksey Kladov2020-01-142-10/+10
| | | | |
| * | | | Move utility functions downAleksey Kladov2020-01-141-58/+58
| | | | |
| * | | | Rename fileAleksey Kladov2020-01-142-2/+2
|/ / / /
* | | | Merge pull request #2826 from matklad/dont-gate-on-windowsAleksey Kladov2020-01-141-1/+1
|\ \ \ \ | |_|/ / |/| | | Dont gate PRs on windows build
| * | | Dont gate PRs on windows buildAleksey Kladov2020-01-131-1/+1
| | | |
* | | | Merge #2829bors[bot]2020-01-131-0/+4
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | 2829: Log client info if present r=matklad a=kjeremy Co-authored-by: kjeremy <[email protected]>
| * | | Log client info if presentkjeremy2020-01-131-0/+4
|/ / /
* | | Merge #2818bors[bot]2020-01-131-9/+35
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2818: Don't panic if chalk panics r=matklad a=matklad r? @flodiebold Trying to paper-over panicking chalk. Not sure if this'll make situation better or worse, but I hope it'll be better, as we won't be tearing down type-inference as a whole Co-authored-by: Aleksey Kladov <[email protected]>
| * | | Don't panic if chalk panicsAleksey Kladov2020-01-131-9/+35
|/ / /
* | | Merge #2825bors[bot]2020-01-139-45/+44
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2825: Some clippy lints r=matklad a=kjeremy Co-authored-by: kjeremy <[email protected]>
| * | | Readabilitykjeremy2020-01-132-6/+6
| | | |
| * | | Some clippy lintskjeremy2020-01-1310-46/+45
| |/ /
* | | Merge #2824bors[bot]2020-01-132-2/+12
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | 2824: Defer cargo check until after workspace load r=kiljacken a=kiljacken Fixes #2822 Co-authored-by: Emil Lauridsen <[email protected]>
| * | Schedule check update regardless of notification settingEmil Lauridsen2020-01-131-1/+1
| | |
| * | Defer cargo check until after workspace loadEmil Lauridsen2020-01-132-2/+12
|/ /
* | Merge #2823bors[bot]2020-01-131-1/+0
|\ \ | |/ |/| | | | | | | | | | | 2823: Dropping a reference does nothing. r=matklad a=kjeremy Allows clippy to continue compilation Co-authored-by: kjeremy <[email protected]>
| * Dropping a reference does nothing.kjeremy2020-01-131-1/+0
|/ | | | Allows clippy to continue compilation