Skip to content

Make inlay hints work in attributed items#10231

Merged
bors[bot] merged 1 commit into
rust-lang:masterfrom
Veykril:inlayattr
Sep 14, 2021
Merged

Make inlay hints work in attributed items#10231
bors[bot] merged 1 commit into
rust-lang:masterfrom
Veykril:inlayattr

Conversation

@Veykril

@Veykril Veykril commented Sep 14, 2021

Copy link
Copy Markdown
Member

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 #10043

bors r+

@bors

bors Bot commented Sep 14, 2021

Copy link
Copy Markdown
Contributor

@bors bors Bot merged commit e08b3bf into rust-lang:master Sep 14, 2021
@Veykril Veykril deleted the inlayattr branch September 14, 2021 00:17
@matklad

matklad commented Sep 14, 2021

Copy link
Copy Markdown
Contributor

✨ magical ✨

bors Bot added a commit that referenced this pull request Sep 14, 2021
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>
@Veykril Veykril changed the title feat: Make inlay hints work in attributed items ~feat~: Make inlay hints work in attributed items Sep 14, 2021
@Veykril Veykril changed the title ~feat~: Make inlay hints work in attributed items Make inlay hints work in attributed items Sep 14, 2021
@Veykril Veykril changed the title Make inlay hints work in attributed items ~~feat~~:Make inlay hints work in attributed items Sep 14, 2021
@Veykril Veykril changed the title ~~feat~~:Make inlay hints work in attributed items Make inlay hints work in attributed items Sep 14, 2021
@lnicola

lnicola commented Sep 14, 2021

Copy link
Copy Markdown
Member

changelog skip

bors Bot added a commit that referenced this pull request Sep 18, 2021
10274: feat: Make inlay hints work in attributed items r=Veykril a=Veykril

Attempt at #10043(previous attempt #10231)
![image](https://user-images.githubusercontent.com/3757771/133887500-091675db-f473-44f8-9bf5-8313ecfbc9e7.png)
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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Inlay hints should work in attributed items with bodies

3 participants