Don't allocate common visibilities

Remove file id from item tree

It's not needed, and `source` is only used by tests anyways

Slightly reduce ItemTree memory footprint

Merge item tree traits

The Source trait isn't needed anymore since we no longer merge
extern crate items with use items.

Generate ModItem via macro

Address more comments

if let else -> match

Unify and test attribute handling

Collect inner items in expression macros

Measure ItemTree query memory usage

Make generics and attr queries use ItemTree

Now it's fast

Record HirFileId's in profiler

Make remaining item data queries use item tree

draw the rest of the owl

ItemTree: Make clonable and add AST access

Lower generics

ItemTree: use a newtyped ID

Move ItemTree lowering into its own module

Remove now-broken incremental test

The ItemTree tracks impl and trait items, so the CrateDefMap now needs
recomputing when those change

Remove raw item query

Fix test

Move collector

Draw the rest of the owl

collector changes WIP

lower traits

Hook up query, add impls, lower moar

Start item tree

Merge #5034

5034: Cleanup project.json deserialization r=matklad a=matklad

bors r+
🤖

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

Cleanup project.json deserialization

Rename json_project -> project_json

Merge #5015 #5027 #5028

5015: Account for updated module ids when determining whether a resolution is changed r=matklad a=Nashenas88

Fixes #4943

5027: Make Debug less verbose for VfsPath and use Display in analysis-stats r=matklad a=lnicola

5028: Remove namedExports config r=matklad a=lnicola

Fixes a warning:

```
(!) Plugin commonjs: The namedExports option from "@rollup/plugin-commonjs" is deprecated. Named exports are now handled automatically.
```

Co-authored-by: Paul Daniel Faria <Nashenas88@users.noreply.github.com>
Co-authored-by: Laurențiu Nicola <lnicola@dend.ro>

Merge #5029

5029: Be more explicit about absolute paths at various places r=matklad a=matklad

bors r+
🤖

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

Fix formatting issue with glob shadow test

Be more explicit about absolute paths at various places

Use a macro in push_res to simplify code, simplify glob_shadowed_def assertion

Remove namedExports config

Make Debug less verbose for VfsPath and use Display in analysis-stats

Merge #5023

5023: Fix string literal inference in match r=flodiebold a=lnicola

Fixes #5020

r? @flodiebold

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

Merge #5024 #5026

5024: Simplify r=matklad a=matklad

bors r+
🤖

5026: Disable file watching when running slow tests r=matklad a=matklad

This should rid us of the intermittent test failure

https://github.com/rust-analyzer/rust-analyzer/pull/5017#issuecomment-648717983

bors r+
🤖

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

Simplify

Disable file watching when running slow tests

This should rid us of the intermittent test failure

https://github.com/rust-analyzer/rust-analyzer/pull/5017#issuecomment-648717983

Fix string literal inference in match

Simplify

Merge #5022

5022: Simplify r=matklad a=matklad

bors r+
🤖

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

Simplify

Simplify

Simplify

Merge #5021

5021: Reduce code dupication around fixtures r=matklad a=matklad

bors r+
🤖

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

Remove duplication

More consistent usage of fixtures

Use fixtures more

Merge #5018

5018: Simplify r=matklad a=matklad

bors r+
🤖

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

Simplify

Merge #4940

4940: Add support for marking doctest items as distinct from normal code r=ltentrup a=Nashenas88

This adds `HighlightTag::Generic | HighlightModifier::Injected` as the default highlight for all elements within a doctest. Please feel free to suggest that a new tag be created or a different one used.


Fixes #4929
Fixes #4939

Co-authored-by: Paul Daniel Faria <Nashenas88@users.noreply.github.com>
Co-authored-by: Paul Daniel Faria <nashenas88@users.noreply.github.com>

Account for updated module ids when determining whether a resolution is "changed"

Improve readability be replacing hard-to-read if-else branches with a match

Merge #5014

5014: Use only one code-path for parsing fixtures r=matklad a=matklad

This removes leading newlines everywhere, shifting all ranges in tests
by one

bors r+
🤖

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

Use only one code-path for parsing fixtures

This removes leading newlines everywhere, shifting all ranges in tests
by one

Centralize fixture parsing for assists

Merge #5008

5008: Remove useless test code r=matklad a=matklad

bors r+
🤖

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

Remove useless test code

Merge #5013

5013: More principled indentation trimming in fixtures r=matklad a=matklad

bors r+
🤖

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

More principled indentation trimming in fixtures

Minor, rename

Merge #5012

5012: Slightly better name r=matklad a=matklad

bors r+
🤖

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

Slightly better name

Merge #5011

5011: Simplify fixtures r=matklad a=matklad

bors r+
🤖

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

Complicate

Fixing test fallout unfortunately requires more work, we need to do
it, but let's merge something at least!

Docs for Fixture

Cut problematic dependency

Nicer API

Simplify

Remove unrelated change

Simplify

Simplify

Drop rarely used fixture functionality

Move fixtures to a separate file

Update comment for pop_and_inject

Co-authored-by: Leander Tentrup <ltentrup@users.noreply.github.com>

Update injection mechanism and stop injecting through highlight element, switch to more general new highlight tag, generic

Add punctuation highlighting for highlighting punctuation in doctests, fix highlighting in doctests

Add default color and opacity for documentation and injected, respectively, in the html generator

Add support for marking doctest items as distinct from normal code, add default tag to all doctest elements

Merge #4976

4976: New VFS r=matklad a=matklad

Still a draft, but mostly working already.

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

New VFS

Merge #5010

5010: Remove relative-path dependency r=matklad a=matklad

bors r+
🤖

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

Remove relative-path dependency

Merge #5004

5004: Fix panic in split/merge import assists r=matklad a=lnicola

Fixes #4368 #4905

Not sure if this is the best solution here. Maybe the `make` functions should be fallible? We generally seem to be playing whack-a-mole with panics in assists, although most of them are `unwrap`s in the assist code.

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

Fix panic in split and merge import assists

Merge #5002

5002: Fix underflow panic when doctests are at top of file r=Nashenas88 a=Nashenas88

While debugging a comment at the top of a test string, I discovered that the offset calculations could underflow and panic. This only seemed to occur in tests, I assume because it's running a debug mode. The wrapping is quickly fixed later on in release mode, which is why this seems to have gone unnoticed. The new checks ensure the value is always positive or zero.

Co-authored-by: Paul Daniel Faria <nashenas88@users.noreply.github.com>

Merge #4992 #4993 #4994 #4997 #4998

4992: Never disable error logging on the frontend r=matklad a=Veetaha

4993: Make bootstrap error message more informative and better-fitting r=matklad a=Veetaha

Now this better fits standard vscode extension activation failure message and suggests enabling verbose logs.


4994: Decouple http file stream logic from temp dir logic r=matklad a=Veetaha

Followup for #4989

4997: Update manual.adoc r=matklad a=gwutz

GNOME Builder (Nightly) supports now rust-analyzer

4998: Disrecommend trace.server: "verbose" for regular users r=matklad a=Veetaha

This option has never been useful for me, I wonder if anyone finds regular users can use this for sending logs

Co-authored-by: Veetaha <veetaha2@gmail.com>
Co-authored-by: Günther Wagner <info@gunibert.de>

Merge #4999

4999: SSR: Allow matching of whole macro calls r=matklad a=davidlattimore

Matching within macro calls is to come later and matching of macro calls within macro calls later still.

Co-authored-by: David Lattimore <dml@google.com>

Update docs/user/manual.adoc

Co-authored-by: Laurențiu Nicola <lnicola@users.noreply.github.com>

Fix underflow panic when doctests are at top of file

Merge #5000

5000: Remove RelativePathBuf from fixture r=matklad a=matklad

The paths in fixture are not really relative (the default one is
`/main.rs`), so it doesn't make sense to use `RelativePathBuf` here.

bors r+
🤖

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

Remove RelativePathBuf from fixture

The paths in fixture are not really relative (the default one is
`/main.rs`), so it doesn't make sense to use `RelativePathBuf` here.

SSR: Allow matching of whole macro calls

Matching within macro calls is to come later and matching of macro calls within macro calls later still.

Disrecommend trace.server: "verbose" for regular users

Update manual.adoc

GNOME Builder (Nightly) supports now rust-analyzer

Decouple http file stream logic from temp dir logic

Never disable error logging on the frontend