minor: improve readability

minor: reduce duplication

Merge #10276

10276: internal: parser cleanup r=matklad a=matklad

bors r+
🤖

Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

fix: correctly handle jointness

minor: improve readability

internal: better naming

internal: parser cleanup

internal: minimize use_tree parser tests

The code here is intentionally dense and does exactly what is written.
Explaining semantic difference between Rust 2015 and 2018 doesn't help
with understanding syntax. Better to just add more targeted tests.

Merge #10275

10275: internal: parser cleanups r=Veykril a=matklad

bors r+
🤖

Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

internal: parser cleanups

Merge #10274

10274: feat: Make inlay hints work in attributed items r=Veykril a=Veykril

Attempt at #10043(previous attempt #10231)
Slight difference to the first approach, chaining hints now work but attribute inputs do not as they are basically the same as normal macros in that they take a token tree.

Fixes #10043

Co-authored-by: Lukas Wirth <lukastw97@gmail.com>

internal: more focused trait item parsing tests

Make inlay hints work in attributed items

Merge #10273

10273: minor: Add multi-token mapping test for goto_definition r=Veykril a=Veykril

bors r+

Co-authored-by: Lukas Wirth <lukastw97@gmail.com>

Add multi-token mapping test for goto_definition

Merge #10252

10252: fix: make `goto_definition` multi-token mapping aware r=Veykril a=XFFXFF

Implement #10070 in  [`goto_definition`](https://github.com/rust-analyzer/rust-analyzer/blob/master/crates/ide/src/goto_definition.rs)

Co-authored-by: zhoufan <1247714429@qq.com>

do comment handling on the original token

Merge #10267

10267: Narrow "if-let to match" assist available range r=Veykril a=rainy-me

ref: https://github.com/rust-analyzer/rust-analyzer/issues/10220

Co-authored-by: rainy-me <github@rainy.me>

Merge #10270

10270: intenral: more item parsing cleanup r=matklad a=matklad

bors r+
🤖

Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

intenral: more local tests for statics

minor: cleanup const parsing

minor: better test placement

internal: cleanup adt parsing

Merge #10269

10269: internal: cleanup item parsing r=matklad a=matklad

bors r+
🤖

Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

internal: cleanup item parsing

minor: parser cleanup

Merge #10268

10268: intenral: simplify parser r=matklad a=matklad

bors r+
🤖

Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

minor: simplify

Narrow "if-let to match" assist available range

minor: cleanup

Merge #10265

10265: internal: parser cleanups r=matklad a=matklad

bors r+
🤖

Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

minor: simplify

internal: more local parsing tests for macros

internal: more focused parsing tests

internal: more local test for mod item

internal: more local parsing tests

internal: parser cleanups

Merge #10260

10260: fix: fix names generation in `Generate function` r=Veykril a=iDawer

- Improve fn name computation (close #10176).
- Handle tuple indexing expressions in argument position (should close  #10241)

Co-authored-by: Dawer <7803845+iDawer@users.noreply.github.com>

Merge #10264

10264: fix: Don't seek outside of character boundaries in completion handler r=Veykril a=iDawer

Fixes #10219 panic when entering `:` in `$0` places:
```rust
fn main() {
    "⊞$0";
}
```

```rust
struct S {
    д$0 u8
}
```

Co-authored-by: Dawer <7803845+iDawer@users.noreply.github.com>

fix completion handler trying to seek outside of character boundaries.

With this patch, in these examples
```rust
fn main() {
    "⊞$0";
}
```

```rust
struct S {
д$0 u8
}
```
entering ':' character in `$0` places shouldn't cause panics.

Merge #10257

10257: assists: turn while into loop r=Veykril a=agluszak

Implements an assist to turn a `while` loop into a `loop` loop, as requested in #10214.
I'm not sure sure what the guidelines are regarding naming assists convert_x_to_y vs replace_x_with_y.

This is my first commit to rust-analyzer :D Thank you `@matklad` for your awesome Explaining rust-analyzer series <3

Closes #10214

Co-authored-by: Andrzej Głuszak <gluszak.andrzej@gmail.com>

assists: turn while into loop

Handle tuple indexing expression in argument position in `Generate function`

Imrove fn name computation in `Generate function`

Merge #10254

10254: Fix a comment typo in autoderef.rs r=lnicola a=teor2345

There's a minor typo in a comment in autoderef.rs, although the meaning is clear if your pronounce the words out loud

Co-authored-by: teor <teor@riseup.net>

Merge #10251

10251: Fix typescript syntax errors r=lnicola a=Wilfred

Remove unwanted `:` and fix the naming of the boolean type.

Co-authored-by: Wilfred Hughes <me@wilfred.me.uk>

Fix a comment typo in autoderef.rs

fix: multi-token mapping aware goto definition

Fix typescript syntax errors

Remove unwanted `:` and fix the naming of the boolean type.

Merge #10250

10250: internal: Improve call_hierarchy and incoming/outgoing_calls r=Veykril a=Veykril

bors r+

Co-authored-by: Lukas Wirth <lukastw97@gmail.com>

Improve call_hierarchy and incoming/outgoing_calls

Merge #10248

10248: internal: remove dead code r=matklad a=matklad

bors r+
🤖

Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

internal: remove dead code

Merge #10048

10048: fix: correctly complete macro call if cursor at `!` r=Veykril a=unexge

Fixes https://github.com/rust-analyzer/rust-analyzer/issues/9904

Co-authored-by: unexge <unexge@gmail.com>

Merge #10233

10233: fix: add multi-token mapping support to hovers r=Veykril a=spookyvision

implement #10070 in [`hover`](https://github.com/rust-analyzer/rust-analyzer/blob/master/crates/ide/src/hover.rs)

Co-authored-by: Anatol Ulrich <anatol.ulrich@ferrous-systems.com>

add multi-token mappings for hover

# This is a combination of 3 commits.
# This is the 1st commit message:

add multi-token mappings for hover

# The commit message #2 will be skipped:

# make fallback an Option instead of vec

# The commit message #3 will be skipped:

# fix indentation

Merge #10247

10247: fix: Don't highlight unsafe traits as unsafe in safe impls r=Veykril a=Veykril

bors r+

Co-authored-by: Lukas Wirth <lukastw97@gmail.com>

Don't highlight unsafe traits as unsafe in safe impls

Merge #10240

10240: internal: Revert attributed items inlay hints r=Veykril a=Veykril

Reverts #10231 as the implementation there is actually wrong
bors r+

Co-authored-by: Lukas Wirth <lukastw97@gmail.com>

Revert attributed items inlay hints

Merge #10237

10237: fix: Do not wrap inlined local in parens when at block tail position r=Veykril a=Veykril

bors r+

Co-authored-by: Lukas Wirth <lukastw97@gmail.com>

fix: Do not wrap inlined local in parens when at block tail position

Merge #10232

10232: internal: Add more tests for ide functionality in attributed items r=Veykril a=Veykril

cc https://github.com/rust-analyzer/rust-analyzer/issues/9868

Co-authored-by: Lukas Wirth <lukastw97@gmail.com>

fix source_to_def trying to use attribute macro calls as containers

Pick smaller node ancestors first when descending at offset

Add some more attribute ide tests

fix: Use original definition ranges for ide_db::search instead of the expanded ranges

Merge #10204

10204: feat: Show the type of what is being dereferenced in a deref expression. r=theo-lw a=theo-lw

Addresses issue #10106.

In-progress - I'm trying to figure out why `hover_deref_expr_with_coercion` is failing.

Co-authored-by: Teddy_Wang <wangtheo662@gmail.com>

Improve hover over derference message

Merge #10231

10231: feat: Make inlay hints work in attributed items r=Veykril a=Veykril

Note the lack of chaining hints, this is currently due to macro expansion lacking the input whitespace. We might be able to recover this from the input somehow in the future.
Fixes https://github.com/rust-analyzer/rust-analyzer/issues/10043
bors r+

Co-authored-by: Lukas Wirth <lukastw97@gmail.com>

feat: Make inlay hints work in attributed items

Fix hover tests

Merge #10230

10230: fix: fix expansion order for fn-like macros and attributes in token descending r=Veykril a=Veykril

Second attempt(now with a test 🎉).
Fixes https://github.com/rust-analyzer/rust-analyzer/issues/10115

Co-authored-by: Lukas Wirth <lukastw97@gmail.com>

Enable attribte macros in ide fixtures

fix: fix expansion order for fn-like macros and attributes in token descending

Fix spelling

Merge #10229

10229: fix: do not complete builtin attributes for qualified paths r=Veykril a=Veykril

bors r+

Co-authored-by: Lukas Wirth <lukastw97@gmail.com>

fix: do not complete builtin attributes for qualified paths

Merge #10126

10126: feat: Speculatively expand attributes in completions r=Veykril a=Veykril


Fixes #9866

Co-authored-by: Lukas Wirth <lukastw97@gmail.com>

Exclude test file from rust tidy check

Downmap the token in attribute inputs when expanding speculatively

Merge #10199

10199: fix: Fix Cargo.toml change detection r=lnicola a=lnicola

Maybe fixes #9546

Co-authored-by: Laurențiu Nicola <lnicola@dend.ro>

Shuffle code around to avoid an allocation

Speculatively expand attributes in completions

Merge #10225

10225: internal: Add proc-macro fixture directive r=Veykril a=Veykril

Co-authored-by: Lukas Wirth <lukastw97@gmail.com>

Merge #10226

10226: minor: Simplify r=Veykril a=Veykril

bors r+

Co-authored-by: Lukas Wirth <lukastw97@gmail.com>

Simplify

Add proc-macro fixture directive

Fix Cargo.toml change detection

Merge #10222

10222: internal: Add workaround for rename error reporting r=lnicola a=lnicola

Fixes #9875

bors r+

Co-authored-by: Laurențiu Nicola <lnicola@dend.ro>

Add workaround for rename error reporting

Merge #10221

10221: internal: prevent possible bugs when adding magical comments r=matklad a=matklad

bors r+
🤖

Co-authored-by: Aleksey Kladov <aleksey.kladov@gmail.com>

internal: document tribal knowledge of how to assist

internal: prevent possible bugs when adding magical comments

internal: simplify

internal: fix bugs in tests by simplifying code

minore: improve readability

Tuples are hard to understand

Merge #10218

10218: minor: Don't spam the manual with warnings r=lnicola a=lnicola

A single warning on the first line should™ be enough. Currently we put that warning on every section, while also breaking the table of contents in the process:


Fixes #9657

Co-authored-by: Laurențiu Nicola <lnicola@dend.ro>

Don't spam the manual with warnings