-
Notifications
You must be signed in to change notification settings - Fork 1.7k
feat: Visualize compiler inserted reborrows via inlay hints #11771
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
bors r+ |
It would be nice to be able to toggle visualizations like this and the implicit lifetimes, I guess. Probably very few people will want to turn them on permanently, but would sometimes want to see them. |
Ye hence why I've disabled them by default. |
Ah nvm, we added an extra thing for that #4601 |
So, I personally don't feel too good about adding a |
Can we drop #4601 then? 😆 |
I'd say yes :) |
I wouldn't want to add a toggle command for all the things. I do think though that if we want to provide a good user experience for this, we should provide some way of showing it temporarily and not require a hack with another extension. (Is it even possible to toggle the setting without restarting the server?) |
Yes changing inlay hint settings retriggers them. |
Disabled by default.
Closes #11275