Fix repetition operator mismatch in macro definitions#46
Fix repetition operator mismatch in macro definitions#46tailhook merged 1 commit intotailhook:masterfrom ia0:rust_issues_61053
Conversation
| ) => { | ||
| quick_error!(SORT [$( $def )*] | ||
| enum [$( $(#[$emeta])* => $eitem $(( $($etyp),* ))* )* | ||
| $(#[$bmeta])* => $bitem: $bmode $(( $($btyp),* ))*] |
There was a problem hiding this comment.
I'm not sure I understand this deletion. Is this just the unused code that sneaked into the codebase?
There was a problem hiding this comment.
I am not familiar with the code, but this rule is very broken. Here is a similar fix: https://github.com/rust-lang-nursery/error-chain/pull/269/files#r305615630. Ideally, someone who understands the code should review this. I'm also fine just reverting the pull request if it can't be reviewed.
There was a problem hiding this comment.
Aha, this is left from @colin-kiegel's simplification commit: 94f0680
The code that matches that rule is:
pub enum Test {
#[cfg(feature = "foo")]
Variant1
#[cfg(feature = "foo")]
Variant2
}
Will try to fix it soon.
How did you find these issues?
There was a problem hiding this comment.
Nice find!
I've found those issues with a crater run when adding the meta_variable_misuse lint. You can try it with #![deny(meta_variable_misuse)] (or warn instead of deny).
There was a problem hiding this comment.
I see. Thanks! I'm going to merge it as is and tackle the bug afterwards.
2 participants
No description provided.