-
Notifications
You must be signed in to change notification settings - Fork 883
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
feature(non_lifetime_binder)
syntax gets removed
#5721
Comments
Here's the tracking issue for @calebcartwright I'm not sure if |
We actually coincidentally discussed this at the t-style meeting earlier today, but fortunately this is just a case where rustfmt was operating under the assumption that it would never see a type binder in this particular position vs. there not being formatting rules for how to handle the syntax. I.e., I don't think there's anything net new at the AST level that we need to derive something for, we probably just have a logic assumption somewhere that needs to be tweaked.
If this is particularly problematic for rustc then, assuming a fix is as trivial as I suspect, then I wouldn't mind in this instance letting the fix be applied directly in-tree in r-l/rust, but otherwise yes, to fix here we'd need to do a sync and I think I'm still a few PR reviews out from that (so probably at least 2 weeks) |
the following code:
when formatted is:
The text was updated successfully, but these errors were encountered: