Propagate accept-comment-above-attributes to statements#15213
Conversation
Turns out that things like `#[attr]return unsafe { func(); }` didn't work
because we weren't checking above attributes on statements, only on blocks!
a8937b8 to
e0e881f
Compare
|
Thanks Alejandra! If I understand correctly, then all the cases in the issue now will pass because the reason was just that they were statements? Nice, this will help avoid confusion :) |
I'll double-check, but yes, now we should check above attributes for all statements (So items, expressions that end in a Also, I'll make it a priority for the feature freeze to fix all the issues you mentioned in that tracking issue. If Rust-For-Linux encounters a really annoying bug (or even worse, a workflow-breaking bug) ping me/dm me on Zulip and I'll either add it to our Feature Freeze tracking issue, or fix it myself. (Also, if you have a not-yet-resolved issue that you'd like to highlight, now it's the moment to link it) |
Turns out that things like
#[attr]return unsafe { func(); }didn't work because we weren't checking above attributes on statements, only on blocks!fixes #13189
Useful for Rust-For-Linux
changelog:[
undocumented_unsafe_blocks]: Make sure to propagateaccept-comment-above-attributesto all statements.