aboutsummaryrefslogtreecommitdiff
path: root/crates
Commit message (Collapse)AuthorAgeFilesLines
* Merge #4562bors[bot]2020-05-303-0/+330
|\ | | | | | | | | | | | | | | 4562: Assist: replace anonymous lifetime with a named one r=matklad a=jbalint (fixes #4523) Co-authored-by: Jess Balint <[email protected]>
| * add support for naming anon lifetimes in function return typeJess Balint2020-05-281-50/+210
| |
| * tweak syntaxJess Balint2020-05-231-49/+40
| |
| * use char rangeJess Balint2020-05-221-2/+2
| |
| * fix generated docs issueJess Balint2020-05-222-23/+48
| |
| * handle the case of conflicting lifetime param nameJess Balint2020-05-221-7/+36
| | | | | | | | - and clean/format code
| * Assist: replace anonymous lifetime with a named oneJess Balint2020-05-222-0/+125
| | | | | | | | (fixes #4523)
* | Merge #4653bors[bot]2020-05-302-5/+61
|\ \ | | | | | | | | | | | | | | | | | | | | | 4653: Fix match ergonomics in closure parameters r=matklad a=flodiebold Fixes #4476. Co-authored-by: Florian Diebold <[email protected]>
| * | Fix match ergonomics in closure parametersFlorian Diebold2020-05-292-5/+61
| | | | | | | | | | | | Fixes #4476.
* | | Merge #4651bors[bot]2020-05-292-4/+2
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 4651: Use first match branch in case of type mismatch, not last r=kjeremy a=flodiebold The comment says this was intentional, but I do agree with #4304 that it makes more sense the other way around (for if/else as well). Fixes #4304. Co-authored-by: Florian Diebold <[email protected]>
| * | | Use first match branch in case of type mismatch, not lastFlorian Diebold2020-05-292-4/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | The comment says this was intentional, but I do agree with #4304 that it makes more sense the other way around (for if/else as well). Fixes #4304.
* | | | Merge #4652bors[bot]2020-05-292-7/+51
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | 4652: Upgrade Chalk r=kjeremy a=flodiebold Fixes #4072. Co-authored-by: Florian Diebold <[email protected]>
| * | | Upgrade ChalkFlorian Diebold2020-05-292-7/+51
| |/ / | | | | | | | | | Fixes #4072.
* / / Support raw_ref_op's raw reference operatorrobojumper2020-05-2811-71/+256
|/ /
* | Merge #4641bors[bot]2020-05-284-55/+54
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | 4641: Upgrade Chalk r=matklad a=flodiebold Chalk newly added TypeName::Never and Array; I implemented the conversion for Never, but not Array since that expects a const argument. Co-authored-by: Florian Diebold <[email protected]>
| * | Upgrade ChalkFlorian Diebold2020-05-274-55/+54
| | | | | | | | | | | | | | | Chalk newly added TypeName::Never and Array; I implemented the conversion for Never, but not Array since that expects a const argument.
* | | Fixed missing newline on each field on "Missing structure fields".Cadu2020-05-272-3/+3
|/ /
* | Merge #4592bors[bot]2020-05-273-1/+49
|\ \ | | | | | | | | | | | | | | | | | | | | | 4592: fix textedit range returned for completion when left token is a keyword r=bnjjj a=bnjjj close #4545 Co-authored-by: Benjamin Coenen <[email protected]>
| * | fix textedit range returned for completion when left token is a keywordBenjamin Coenen2020-05-272-0/+3
| | | | | | | | | | | | Signed-off-by: Benjamin Coenen <[email protected]>
| * | fix textedit range returned for completion when left token is a keyword #4545Benjamin Coenen2020-05-243-1/+46
| | | | | | | | | | | | Signed-off-by: Benjamin Coenen <[email protected]>
* | | Merge #4596bors[bot]2020-05-271-1/+59
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 4596: Strip leading underscores of argument names in function/method r=matklad a=kuy Closes #4510 ### Goal When I select a function/method from completions, I get a snippet that doesn't contain leading underscores of argument names. ### Solution - Option 1: All signatures don't contain underscores - Option 2: Keep same signature, but inserted snippet doesn't contain underscores I choose Option 2 because I think that leading underscores is a part of "signature". Users should get correct signatures. On the other hand, trimming underscores is an assist by IDE. ### Other impls. rls: Complete argument names with underscores (same as actual ra) IntelliJ Rust: Doesn't complete argument names VSCode (TypeScript): Doesn't complete argument names ### Working example ![Screen Shot 2020-05-25 at 0 03 21](https://user-images.githubusercontent.com/151614/82757771-a05e5b80-9e1d-11ea-9dbc-1263c960e2ae.png) Co-authored-by: Yuki Kodama <[email protected]>
| * | | Trim at presentation layerYuki Kodama2020-05-252-4/+2
| | | |
| * | | Fix test to consider multiple underscoresYuki Kodama2020-05-251-7/+7
| | | |
| * | | Use built-in methodYuki Kodama2020-05-251-11/+5
| | | |
| * | | Separate assertionsYuki Kodama2020-05-251-7/+65
| | | |
| * | | Reflect test caseYuki Kodama2020-05-251-7/+7
| | | |
| * | | Strip leading underscoreYuki Kodama2020-05-251-2/+10
| | | |
| | | |
| \ \ \
*-. \ \ \ Merge #4625 #4629bors[bot]2020-05-271-3/+3
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 4625: Partially fix displaying inlay hints in Github PR diff views r=matklad a=Veetaha See the comment in https://github.com/rust-analyzer/rust-analyzer/issues/4608#issuecomment-63424257 It partially fixes the left side of diff view (the one where old code is displayed), but the diff editor with new code changes still has `file` scheme and will proceed displaying inlay hints... 4629: Fix the `should_panic` snippet r=matklad a=eminence Closes #4628 Co-authored-by: veetaha <[email protected]> Co-authored-by: Andrew Chin <[email protected]>
| | * | | | Fix the `should_panic` snippetAndrew Chin2020-05-261-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | Closes #4628
* | | | | | Merge #4622bors[bot]2020-05-274-43/+40
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 4622: Pass trivially copy types as copy r=matklad a=kjeremy Co-authored-by: kjeremy <[email protected]>
| * | | | | Pass trivially copy types as copykjeremy2020-05-264-43/+40
| |/ / / /
* / / / / Update lexerkjeremy2020-05-261-1/+1
|/ / / /
* | | | Merge #4534bors[bot]2020-05-261-0/+59
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 4534: Add call postfix completion r=matklad a=vain0x To make it easier to wrap an expression with Ok/Some/Rc::new etc. Note I agree with conclusion of the discussion in #1431 that adding many completions is not the way to go. However, this PR still could be justified due to versatility of use. Co-authored-by: vain0x <[email protected]>
| * | | | Add call postfix completionvain0x2020-05-201-0/+59
| | | | | | | | | | | | | | | | | | | | To make it easier to wrap an expression with Ok/Some/Rc::new etc.
* | | | | Hover tooltip module name is monospace once againAaron Loucks2020-05-263-9/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The line separator is moved below the function signature to split regions between the docs. This is very similar to how IntelliJ displays tooltips. Adding an additional separator between the module name and function signature currently has rendering issues. Fixes #4594 Alternative to #4615
* | | | | Fix some clippy perf warningsJeremy Kolb2020-05-254-9/+8
| |/ / / |/| | |
| | | |
| \ \ \
*-. \ \ \ Merge #4602 #4603bors[bot]2020-05-255-4/+9
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 4602: Add boolean literal semantic token type to package.json r=matklad a=lnicola Closes #4583. CC @GrayJack 4603: Add self keyword semantic token type r=matklad a=lnicola Not sure if this is warranted a new token type or just a modifier. --- CC #4583, @GrayJack Co-authored-by: Laurențiu Nicola <[email protected]>
| | * | | | Add self keyword semantic token typeLaurențiu Nicola2020-05-255-4/+9
| |/ / / /
* | | | | Document `parentModule` experimental LSP requestAleksey Kladov2020-05-254-29/+25
| | | | |
* | | | | Cleanup lsp extensions on the client sideAleksey Kladov2020-05-252-9/+4
| | | | |
* | | | | Less rust-analyzer specific onEnterAleksey Kladov2020-05-257-48/+43
|/ / / /
* | | | Merge #4593bors[bot]2020-05-243-20/+18
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 4593: Document some rust-analyzer specific protocol extensions r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | | | Fix formattingAleksey Kladov2020-05-242-4/+1
| | | | |
| * | | | Document some rust-analyzer specific protocol extensionsAleksey Kladov2020-05-241-7/+8
| | | | |
| * | | | Document matchingBrace LSP requestAleksey Kladov2020-05-243-12/+12
| | | | |
* | | | | Handle more cases in `highlight_name_by_syntax`Matthew Jasper2020-05-241-11/+19
| | | | |
* | | | | Highlight `true` and `false` as literalsMatthew Jasper2020-05-249-1/+11
|/ / / /
* | | | Fix testsAleksey Kladov2020-05-241-25/+25
| | | |
* | | | Merge pull request #4548 from bnjjj/fix_4464Aleksey Kladov2020-05-248-19/+231
|\ \ \ \ | | | | | | | | | | add support of feature flag for runnables
| * | | | add support of feature flag for runnables #4464Benjamin Coenen2020-05-242-15/+1
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Benjamin Coenen <[email protected]>