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
Recover from incorrectly ordered/duplicated function keywords #117282
Conversation
r? @TaKO8Ki (rustbot has picked a reviewer for you, use r? to override) |
b769520
to
82644aa
Compare
This comment has been minimized.
This comment has been minimized.
82644aa
to
be0b42f
Compare
LL | impl S { | ||
| - while parsing this item list starting here | ||
LL | #[cfg(FALSE)] | ||
LL | unsafe async fn g() {} | ||
| -------^^^^^ | ||
| | | | ||
| | expected one of `extern` or `fn` | ||
| help: `async` must come before `unsafe`: `async unsafe` | ||
LL | } | ||
| - the item list ends here |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ideally, I think it's best to leave this as it is.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, this message indicates that it wasn't able to recover the rest of the item list if I understand right? I think this is an improvement as it's able to continue parsing other items in the list, so the message isn't really necessary
@bors r+ rollup |
…eader, r=TaKO8Ki Recover from incorrectly ordered/duplicated function keywords Fixes rust-lang#115714
…llaumeGomez Rollup of 5 pull requests Successful merges: - rust-lang#117263 (handle the case when the change-id isn't found) - rust-lang#117282 (Recover from incorrectly ordered/duplicated function keywords) - rust-lang#117679 (tests/rustdoc-json: Avoid needless use of `no_core` and `lang_items`) - rust-lang#117702 (target: move base and target specifications) - rust-lang#117713 (Add test for reexported hidden item with `--document-hidden-items`) r? `@ghost` `@rustbot` modify labels: rollup
Rollup merge of rust-lang#117282 - clubby789:recover-wrong-function-header, r=TaKO8Ki Recover from incorrectly ordered/duplicated function keywords Fixes rust-lang#115714
Fixes #115714