Skip to content

Conversation

@Veykril
Copy link
Member

@Veykril Veykril commented Sep 14, 2021

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
Copy link
Contributor

bors bot commented Sep 14, 2021

@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
Copy link
Contributor

matklad commented Sep 14, 2021

✨ 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
Copy link
Member

lnicola commented Sep 14, 2021

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