aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge #7431bors[bot]2021-01-256-74/+158
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | 7431: Handle `super` paths inside blocks correctly r=jonas-schievink a=jonas-schievink We now intern `BlockLoc` and use `BlockId` to refer to block expressions. This is needed to keep `ModuleId` simple, since it would otherwise have to store an arbitrarily long chain of blocks and couldn't be `Copy`. The `DefMap` hierarchy is now created as the caller descends into an item body. This is necessary to link the correct module as the block's parent, which is important for correct name resolution. As a result, we can now resolve `super` paths inside block expressions by climbing the `DefMap` chain. bors r+ Co-authored-by: Jonas Schievink <[email protected]>
| * Traverse parent DefMap for `super` pathsJonas Schievink2021-01-252-9/+54
| |
| * Intern block locations and handle them in ModuleIdJonas Schievink2021-01-255-65/+104
|/
* Merge #7426bors[bot]2021-01-2511-87/+48
|\ | | | | | | | | | | | | | | | | | | | | 7426: Create all `ModuleId`s through a `DefMap` method r=jonas-schievink a=jonas-schievink `ModuleId` needs to be able to represent blocks, and only the associated `DefMap` will know how to construct that `ModuleId` bors r+ Co-authored-by: Jonas Schievink <[email protected]>
| * Create all `ModuleId`s through a `DefMap` methodJonas Schievink2021-01-2511-87/+48
| | | | | | | | | | `ModuleId` needs to be able to represent blocks, and only the associated `DefMap` will know how to construct that `ModuleId`
* | Merge #7424bors[bot]2021-01-252-12/+3
|\ \ | |/ |/| | | | | | | | | | | 7424: Derive Default on ItemTree r=lnicola a=lnicola Co-authored-by: Laurențiu Nicola <[email protected]>
| * Derive Default on ItemTreeLaurențiu Nicola2021-01-252-12/+3
| |
* | Merge #7425bors[bot]2021-01-251-3/+4
|\ \ | |/ |/| | | | | | | | | | | 7425: Fix typo in style guide r=lnicola a=lnicola bors r+ Co-authored-by: Laurențiu Nicola <[email protected]>
| * Fix typo in style guideLaurențiu Nicola2021-01-251-3/+4
|/
* Merge #7422bors[bot]2021-01-251-2/+4
|\ | | | | | | | | | | | | | | 7422: Disallow non-boolean literals in concat! r=lnicola a=lnicola Co-authored-by: Laurențiu Nicola <[email protected]>
| * Disallow non-boolean literals in concat!Laurențiu Nicola2021-01-251-2/+4
| |
| |
| \
*-. \ Merge #7409 #7421bors[bot]2021-01-256-34/+53
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7409: Add References CodeLens. r=matklad a=vsrs Closes #5836 7421: Fix RA_LOG example in dev docs r=lnicola a=lnicola bors r+ Co-authored-by: vsrs <[email protected]> Co-authored-by: Laurențiu Nicola <[email protected]>
| | * Fix RA_LOG example in dev docsLaurențiu Nicola2021-01-251-1/+1
| | |
| * | Add References code lens.vsrs2021-01-235-33/+52
| | | | | | | | | | | | For Struct, Enum, Union and Trait symbols.
* | | Merge #7419bors[bot]2021-01-251-16/+8
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | | | | 7419: Unquote strings when expanding concat! r=matklad a=lnicola Fixes #7417. Co-authored-by: Laurențiu Nicola <[email protected]>
| * | Unquote strings and handle boolean literals in concat!Laurențiu Nicola2021-01-251-16/+8
|/ /
* | Merge #7418bors[bot]2021-01-253-6/+6
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | 7418: :arrow_up: rowan r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | :arrow_up: rowanAleksey Kladov2021-01-253-6/+6
|/ /
* | Merge #7415bors[bot]2021-01-251-7/+9
|\ \ | | | | | | | | | | | | | | | | | | | | | 7415: Code: reduce progress notification spam r=lnicola a=lnicola Co-authored-by: Laurențiu Nicola <[email protected]>
| * | Code: reduce progress notification spamLaurențiu Nicola2021-01-251-7/+9
|/ /
* | Merge #7414bors[bot]2021-01-246-25/+40
|\ \ | | | | | | | | | | | | | | | | | | | | | 7414: Add validation for mutable const items r=Veykril a=Veykril bors r+ Co-authored-by: Lukas Wirth <[email protected]>
| * | Add validation for mutable const itemsLukas Wirth2021-01-246-25/+40
| | |
* | | Merge #7413bors[bot]2021-01-243-7/+11
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | 7413: Handle unions in symbol search r=Veykril a=Veykril bors r+ Co-authored-by: Lukas Wirth <[email protected]>
| * | Support unions in symbol searchLukas Wirth2021-01-243-7/+11
|/ /
* | Merge #7411bors[bot]2021-01-231-7/+7
|\ \ | |/ |/| | | | | | | | | | | 7411: cargo update r=kjeremy a=kjeremy Co-authored-by: Jeremy Kolb <[email protected]>
| * cargo updateJeremy Kolb2021-01-231-7/+7
|/
* Merge #7062bors[bot]2021-01-237-19/+224
|\ | | | | | | | | | | | | | | 7062: Add diagnostic for filter_map followed by next r=theotherphil a=theotherphil Fixes https://github.com/rust-analyzer/rust-analyzer/issues/1725 Co-authored-by: Phil Ellison <[email protected]>
| * Remove use of SourceFileEditPhil Ellison2021-01-231-2/+1
| |
| * Update tests to register the required standard library typesPhil Ellison2021-01-232-16/+48
| |
| * Fix test namesPhil Ellison2021-01-231-4/+4
| |
| * Identify methods using functions ids rather than string namesPhil Ellison2021-01-233-7/+31
| |
| * cargo fmtPhil Ellison2021-01-231-1/+5
| |
| * Address review commentsPhil Ellison2021-01-231-9/+4
| |
| * cargo fmtPhil Ellison2021-01-231-2/+7
| |
| * Implement fix, add testsPhil Ellison2021-01-234-55/+66
| |
| * Add diagnostic for filter_map followed by nextPhil Ellison2021-01-235-15/+150
|/
* Merge #7391bors[bot]2021-01-232-0/+24
|\ | | | | | | | | | | | | | | 7391: Fix error when using "extern crate self as" r=kazatsuyu a=kazatsuyu Fix #6957 Co-authored-by: kazatsuyu <[email protected]>
| * Add mark::check! and mark::hit!kazatsuyu2021-01-222-0/+2
| |
| * cargo fmtkazatsuyu2021-01-221-2/+3
| |
| * Fix error when using "extern crate self as"kazatsuyu2021-01-222-0/+21
| |
* | Merge #7399bors[bot]2021-01-2210-38/+62
|\ \ | | | | | | | | | | | | | | | | | | | | | 7399: Make `ModuleId`'s `krate` field private and audit uses r=jonas-schievink a=jonas-schievink bors r+ Co-authored-by: Jonas Schievink <[email protected]>
| * | Make `ModuleId`'s `krate` field privateJonas Schievink2021-01-2210-38/+62
| | |
* | | Merge #7398bors[bot]2021-01-221-0/+1
|\ \ \ | |/ / |/| | | | | | | | | | | | | | | | | | | | 7398: stave off zombies r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | stave off zombiesAleksey Kladov2021-01-221-0/+1
|/ /
* | Merge #7396bors[bot]2021-01-2210-139/+128
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | 7396: More annoying asserts r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
| * | Normalize testsAleksey Kladov2021-01-221-45/+37
| | |
| * | More useful fn detail in completionAleksey Kladov2021-01-229-93/+85
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Detail should be rendered as shtort one line, just dumping fn header there is not useful, despite the fact that TS does this. The fact that this is a function should be indicated by the icon, the same goes for pub/const/async etc qualitfiers name is already present in the lable (and arg list should be a part of that, as in idea) But the return type is the small genuinerlly useful bit of info we can show here
| * | Make assertion failures more annoying for maintainersAleksey Kladov2021-01-221-1/+6
|/ /
| |
| \
*-. \ Merge #7310 #7395bors[bot]2021-01-2217-32/+292
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7310: Add assist: add lifetime to type r=matklad a=bnjjj close #7200 7395: Obtain `ModuleId`'s `DefMap` through a method r=jonas-schievink a=jonas-schievink Once `ModuleId` can represent modules inside block expressions, this will be required in order to obtain the correct `DefMap`. Otherwise existing code might use a `LocalModuleId` from the wrong `DefMap` and cause a panic. bors r+ Co-authored-by: Benjamin Coenen <[email protected]> Co-authored-by: Jonas Schievink <[email protected]>
| | * | Obtain `ModuleId`'s `DefMap` through a methodJonas Schievink2021-01-2214-32/+43
| | | |