Update crates/ide-completion/src/context.rs

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

fix formatting

refactor hover

change struct_rest_pat to guarentee a HoverResult

Move token and node matching out of struct_rest_pat into hover

Auto merge of #13290 - poliorcetics:multiple-targets, r=Veykril

Support multiple targets for checkOnSave (in conjunction with cargo 1.64.0+)

This PR adds support for the ability to pass multiple `--target` flags when using
`cargo` 1.64.0+.

## Questions

I needed to change the type of two configurations options, but I did not plurialize the names to
avoid too much churn, should I ?

## Zulip thread

https://rust-lang.zulipchat.com/#narrow/stream/185405-t-compiler.2Frust-analyzer/topic/Issue.2013282.20.28supporting.20multiple.20targets.20with.201.2E64.2B.29

## Example

To see it working, on a macOS machine:

```sh
$ cd /tmp
$ cargo new cargo-multiple-targets-support-ra-test
$ cd !$
$ mkdir .cargo
$ echo '
[build]
target = [
    "aarch64-apple-darwin",
    "x86_64-apple-darwin",
]
' > .cargo/config.toml
$ echo '
fn main() {
    #[cfg(all(target_arch = "aarch64", target_os = "macos"))]
    {
        let a = std::fs::read_to_string("/tmp/test-read");
    }

    #[cfg(all(target_arch = "x86_64", target_os = "macos"))]
    {
        let a = std::fs::read_to_string("/tmp/test-read");
    }

    #[cfg(all(target_arch = "x86_64", target_os = "windows"))]
    {
        let a = std::fs::read_to_string("/tmp/test-read");
    }
}
' > src/main.rs
# launch your favorite editor with the version of RA from this PR
#
# You should see warnings under the first two `let a = ...` but not the third
```

## Screen


Helps with #13282

Auto merge of #13641 - DesmondWillowbrook:fix-move-format-string, r=Veykril

fix: format expression parsing edge-cases

- Handle positional arguments with formatting options (i.e. `{:b}`). Previously copied `:b` as an argument, producing broken code.

- Handle indexed positional arguments (`{0}`) ([reference](https://doc.rust-lang.org/std/fmt/#positional-parameters)). Previously copied over `0` as an argument.

Note: the assist also breaks when named arguments are used (`"{name}$0", name = 2 + 2` is converted to `"{}"$0, name`. I'm working on fix for that as well.

Auto merge of #13642 - bvanjoi:fix-13292, r=Veykril

fix(assists): remove `item_const` which had default value when implement missing members

Fixed https://github.com/rust-lang/rust-analyzer/issues/13292

fix(assists): remove `item_const` which had default value when implement missing members`

run cargo fmt

Auto merge of #13639 - Veykril:macro-diags, r=Veykril

fix: Fix proc-macro-srv search paths for Arch Linux

Fixes https://github.com/rust-lang/rust-analyzer/issues/13616

fix: Fix nested macro diagnostics pointing at macro expansion files

fix: format expression parsing edge-cases

handle positional arg with formatting

handle indexed positional args

fix formatting
and remove unnecessary check

simplify ancestor climbing to not consider macros

feat: adds hover hint to ".." in record pattern

currently only works with struct pattern

fix: Fix proc-macro-srv search paths for Arch Linux

Auto merge of #13633 - Veykril:vscode-full-diagnostics, r=Veykril

feat: Allow viewing the full compiler diagnostic in a readonly textview


Also adds a VSCode only config that replaces the split diagnostic message with the first relevant part of the diagnostic output


This only affects diagnostics generated by primary spans and has no effect on other clients than VSCode.

Fixes https://github.com/rust-lang/rust-analyzer/issues/13574

Fix tests checking the data value

feat: Allow viewing the full compiler diagnostic in a readonly textview

Auto merge of #13632 - Veykril:scip, r=Veykril

Make it more obvious which SCIP features we do not yet emit in code

Make it more obvious which SCIP features we do not yet emit in code

Auto merge of #13629 - jonas-schievink:remove-dbg-selection, r=jonas-schievink

feat: Make "Remove dbg!()" assist work on selections

Fixes https://github.com/rust-lang/rust-analyzer/issues/12114

Make "Remove dbg!()" assist work on selections

Auto merge of #13625 - lowr:fix/proc-macro-src-test, r=lnicola

internal: Update proc-macro-srv tests

Should have been included in #13548, but I didn't notice as those tests aren't run in our CI.

cc rust-lang/rust#104454

Update proc-macro-srv tests

Auto merge of #13624 - lowr:fix/unsize-array-inference-variable, r=lnicola

fix: resolve inference variable before applying adjustments

Fixes #13619

fix: resolve inference variable before applying adjustments

Auto merge of #13615 - mati865:miow-update, r=jonas-schievink

Update several crates to bring support for the new Tier 3 Windows tar…

`cargo t` has passed on Windows 11 with both `x86_64-pc-windows-gnu` and `x86_64-pc-windows-gnullvm` targets.

Auto merge of #13623 - jonas-schievink:strip-trait-item-completions, r=jonas-schievink

fix: Strip comments and attributes off of all trait item completions

Previously, this was done in several places redundantly, but not for all items. It was also untested. This PR fixes that.

Strip comments and attributes off of all trait item completions

Auto merge of #13622 - jonas-schievink:fix-gat-completions, r=jonas-schievink

fix: include generic parameter in GAT completions

Fixes https://github.com/rust-lang/rust-analyzer/issues/13586

Fix GAT completion not including generic parameters

Update several crates to bring support for the new Tier 3 Windows targets

fix: filter unnecessary completions after colon

Auto merge of #13607 - Veykril:proc-macro-error, r=Veykril

internal: Add version info to unsupported proc macro abi error

cc https://github.com/rust-lang/rust-analyzer/issues/13589#issuecomment-1311824473

internal: Add version info to unsupported proc macro abi error

Auto merge of #13606 - Veykril:trait-alias, r=Veykril

fix: Add trait alias grammar to rust.ungram

We already parse them, but the grammar was never updated to reflect that

fix: Add trait alias grammar to rust.ungram

Auto merge of #13605 - Veykril:empty-ws-error, r=Veykril

fix: Fix r-a eagerly showing no discovered workspace errors

fix: Fix r-a eagerly showing no discovered workspace errors

feat: Support passing multiple targets to cargo (for Rust 1.64.0+)

chore: Align config property

Auto merge of #13604 - Veykril:hover-attr, r=Veykril

fix: Fix hover in attributed items not preferring similar kinded tokens

fix: Fix hover in attributed items not preferring similar kinded tokens

Auto merge of #13602 - lowr:fix/nameres-transitive-visibility, r=Veykril

fix: check visibility of each path segment

Upon path resolution, we have not been checking if every def pointed to by each segment of the path is visible from the original module. This leads to incorrect import resolutions, in particular when one uses glob imports and names collide.

There is decent amount of changes in this PR because:
- some of our tests were not correct in terms of visibility
  - I left several basic nameres tests as-is (with expect test updated) since I thought it would be nice to ensure we don't resolve defs that are not visible.
- `fix_visibility` assist relied on `Semantics::resolve_path()`, which uses the name resolution procedure I'm fixing and wouldn't be able to "see through" the items with strict visibility with this patch

The first commit is the gist of the fix itself.

Fixes #10991
Fixes #11473
Fixes #13252

Auto merge of #13548 - lowr:fix/tt-punct-spacing, r=Veykril

Fix `tt::Punct`'s spacing calculation

Fixes #13499

We currently set a `tt::Punct`'s spacing to `Spacing::Joint` unless its next token is a trivia (i.e. whitespaces or comment). As I understand it, rustc only [sets `Spacing::Joint` if the next token is an operator](https://github.com/rust-lang/rust/blob/5b3e9090757da9a95b22f589fe39b6a4b5455b96/compiler/rustc_parse/src/lexer/tokentrees.rs#L77-L78) and we should follow it to guarantee the consistent behavior of proc macros.

Auto merge of #13603 - Veykril:no-workspaces, r=Veykril

fix: Send status notification if there are no found workspaces

Closes https://github.com/rust-lang/rust-analyzer/issues/5829

Send status notification if there are no found workspaces

Fix tests that depended on loose visibility restriction

Resolve invisible defs in `fix_visibility` assist

fix: check visibility of each segment in path resolution

Test `TokenTree`s' equality modulo `Punct`s' spacing

Ignore outermost non-delimited `Subtree` when reversing fixups

Rollup merge of #104211 - lnicola:rust-analyzer-2022-11-09, r=lnicola

:arrow_up: rust-analyzer

r? ``@ghost``

Auto merge of #13590 - Veykril:proc-macro-rustc-src, r=Veykril

internal: Add proc-macro dependency to rustc_private crates

Add proc-macro dependency to rustc_private crates

:arrow_up: rust-analyzer

Rollup merge of #103919 - nnethercote:unescaping-cleanups, r=matklad

Unescaping cleanups

Some code improvements, and some error message improvements.

Best reviewed one commit at a time.

r? ````@matklad````

Auto merge of #13584 - jonas-schievink:fix-signature-panic, r=jonas-schievink

fix: fix panic when computing signature of generic `FnOnce` callable

Fixes https://github.com/rust-lang/rust-analyzer/issues/13579

Don't canonicalize self type when querying FnOnce signature

Auto merge of #13582 - lnicola:nest-cargo-toml, r=lnicola

feat: Nest Cargo.lock under Cargo.toml in Code

Closes #13580

Nest Cargo.lock under Cargo.toml in Code

Auto merge of #13581 - Veykril:unit-struct-compl, r=Veykril

fix: Fix item completions not working properly after unit structs and outline modules

Fixes https://github.com/rust-lang/rust-analyzer/issues/13578

fix: Fix item completions not working properly after unit structs and outline modules

Auto merge of #13573 - Veykril:invalid-file-range, r=Veykril

internal: error instead of panic on invalid file range

Fixes the panic in https://github.com/rust-lang/rust-analyzer/issues/13170

internal: error instead of panic on invalid file range

Auto merge of #13572 - Veykril:cancellable, r=Veykril

internal: Use Cancellable in favor of Result for clarity

internal: Use Cancellable in favor of Result for clarity

Auto merge of #13571 - Veykril:unique-references, r=Veykril

minor: Deduplicate reference search results

Fixes https://github.com/rust-lang/rust-analyzer/issues/13407

Deduplicate reference search results

Auto merge of #13568 - noritada:fix/len-of-byte-string-with-escaped-newlines, r=Veykril

Fix the length displayed for byte string literals with escaped newlines

This is a fix for the problem I reported earlier: "the length of byte strings containing escaped newlines is displayed two bytes longer when the first escaped character is a newline".

I would appreciate it if you could review the fix.
Many thanks.

Closes #13567

Auto merge of #13570 - Veykril:dedup-crates-for, r=Veykril

minor: Remove code duplication

Remove code duplication

Add test code for unescaping byte strings

Auto merge of #13545 - Veykril:adjustment-hints, r=Veykril

Generalize reborrow hints as adjustment hints

Like reborrow hints, these are still mainly useful for teaching/learning


Auto merge of #13569 - Veykril:rust-version, r=Veykril

Bump Cargo rust-version fields to latest stable

We already depend on `let ... else` now anyways

Closes https://github.com/rust-lang/rust-analyzer/issues/13562

Clarify the intent

Thanks to Lukas Wirth for a suggestion.

Bump Cargo rust-version fields to latest stable

Don't trigger adjustment hints in all inlay hint tests

Auto merge of #13565 - Veykril:sysroot, r=Veykril

Update sysroot crates

Fix the length displayed for byte string literals with escaped newlines

The length of byte strings containing escaped newlines is displayed two
bytes longer when the first escaped character is a newline.

This is due to a small bug in handling the first escaped newline in
string literals.

Closes #13567

Auto merge of #12991 - TiddoLangerak:extract-method-from-trait-into-impl-root, r=Veykril

Feat: extracted method from trait impl is placed in existing impl

**Before**

https://user-images.githubusercontent.com/1759192/183872883-3b0eafd2-d1dc-440e-9e66-38e3372f8b64.mp4

**After**

https://user-images.githubusercontent.com/1759192/183875769-87f34c7d-52f0-4dfc-9766-f591ee738ebb.mp4

Previously, when triggering a method extraction from within an impl trait block, then this would always create a new impl block for
the struct, even if there already is one. Now, if there is already an existing trait-less impl block, then it'll put the extracted method in there.

**Caveats**:
- It currently requires the target impl block to be non-empty. This limitation is because the current architecture takes a `node_to_insert_after` as reference for where to insert the extracted function. An empty impl block doesn't have such a reference node, since it's empty. It seems that supporting this requires a much larger and more complex change.
- This is my first contribution in rust, so apologies for any beginner mistakes.

Update sysroot crates

rustfmt

Auto merge of #13566 - lnicola:therefor, r=lnicola

minor: Fix typos

Fix typos

Auto merge of #13547 - Veykril:line-index, r=Veykril

internal: Optimize `apply_document_changes` a bit

cc https://github.com/rust-lang/rust-analyzer/issues/13538

Auto merge of #13552 - Veykril:flycheck-process-group, r=Veykril

internal: Use a process group for flycheck

Should fix https://github.com/rust-lang/rust-analyzer/issues/13348

Auto merge of #13556 - pd4d10:patch-1, r=lnicola

docs: fix adoc links

Correct #13536 with adoc link syntax

docs: fix adoc links

internal: Use a process group for flycheck

Auto merge of #13550 - Veykril:issue-template, r=lnicola

minor: Update github issue templates

Auto merge of #13536 - pd4d10:patch-1, r=Veykril

docs: add crates section to the manual

closes #13533

Added a section to the user manual, to make it easier for users to find the correct crate.

Auto merge of #13435 - DropDemBits:assists-format-args-capture-pt3, r=Veykril

Migrate assists to format args captures, part 3

Continuation of https://github.com/rust-lang/rust-analyzer/pull/13379

Migrates:

- `inline_call`
- `inline_local_variable`
- `introduce_named_lifetime`
- `merge_match_arms`
- `move_from_mod_rs`
- `move_guard`
- `move_module_to_file`
- `move_to_mod_rs`
- `number_representation`
- `qualify_method_call`
- `qualify_path`
- `raw_string`
- `remove_dbg`
- `replace_derive_with_manual_impl`
- `replace_or_with_or_else`
- `replace_turbofish_with_explicit_type`
- `unwrap_tuple`
- `wrap_return_type_in_result`

minor: Update github issue templates

Auto merge of #13379 - DropDemBits:ide-assists-format-args-capture, r=Veykril

internal: Migrate `ide_assists::utils` and `ide_assists::handlers` to use format arg captures (part 1)

This not only serves as making future migration to mutable syntax trees easier, it also finds out what needs to be migrated in the first place.

~~Aside from the first commit, subsequent commits are structured to only deal with one file/handler at a time.~~

This is the first of 3 PRs, migrating:

Utils:

- `gen_trait_fn_body`
- `render_snippet`
- `ReferenceConversion`
  - `convert_type`
  - `getter`

Handlers:

- `add_explicit_type`
- `add_return_type`
- `add_turbo_fish`
- `apply_demorgan`
- `auto_import`
- `convert_comment_block`
- `convert_integer_literal`
- `convert_into_to_from`
- `convert_iter_for_each_to_for`
- `convert_let_else_to_match`
- `convert_tuple_struct_to_named_struct`
- `convert_two_arm_bool_match_to_matches_macro`
- `destructure_tuple_binding`
- `extract_function`
- `extract_module`
- `extract_struct_from_enum_variant`
- `extract_type_alias`
- `extract_variable`
- `fix_visibility`

Auto merge of #13549 - Veykril:search-fix, r=Veykril

fix: Fix reference searching only accounting substrings instead of whole identifiers

Fixes https://github.com/rust-lang/rust-analyzer/issues/13498

fix: Fix reference searching only accounting substrings instead of whole identifiers

Fix `tt::Punct`'s spacing calculation

Rename convertor -> converter

Auto merge of #13454 - justinmmott:master, r=flodiebold

Fixed local shadowing the caller's argument issue

fix https://github.com/rust-lang/rust-analyzer/issues/12536