-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Closed
Labels
A-documentationArea: Adding or improving documentationArea: Adding or improving documentationC-enhancementCategory: Enhancement of lints, like adding more cases or adding help messagesCategory: Enhancement of lints, like adding more cases or adding help messagesgood first issueThese issues are a good way to get started with ClippyThese issues are a good way to get started with Clippy
Description
I found the doc is wrong (https://github.com/Manishearth/rust-clippy/blob/5f09020e90b485481fde984cc2e31c0112ebf1f1/clippy_lints/src/matches.rs#L25 , https://github.com/Manishearth/rust-clippy/blob/5f09020e90b485481fde984cc2e31c0112ebf1f1/clippy_lints/src/matches.rs#L46 , https://github.com/l1048576/rust-clippy/blob/a878916ad54eba5e92a3e6f0906e5099b26815d1/src/matches.rs#L23 , etc...).
The examples are as follows:
match x {
Some(ref foo) -> bar(foo),
_ => bar(other_ref),
}but it is wrong because the former arms should be Some(ref foo) => bar(foo).
I tried to write pull request, but there are so many branches and I don't know which branch to fix.
Metadata
Metadata
Assignees
Labels
A-documentationArea: Adding or improving documentationArea: Adding or improving documentationC-enhancementCategory: Enhancement of lints, like adding more cases or adding help messagesCategory: Enhancement of lints, like adding more cases or adding help messagesgood first issueThese issues are a good way to get started with ClippyThese issues are a good way to get started with Clippy