Mirrors/rust-clippy
2
0
Fork 0
mirror of https://github.com/rust-lang/rust-clippy synced 2024-11-10 07:04:18 +00:00
Code Issues Projects Releases Packages Wiki Activity

Not trigger duplicated_attributes on duplicate reasons

Browse source
This commit is contained in:
Alexey Semenyuk 2024-09-12 11:49:16 +05:00
parent a53614a910
commit 7097830a9b
2 changed files with 5 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
clippy_lints/src/attrs/duplicated_attributes.rs
View file

@ -36,7 +36,7 @@ fn check_duplicated_attr(
} }
let Some(ident) = attr.ident() else { return }; let Some(ident) = attr.ident() else { return };
let name = ident.name; let name = ident.name;
if name == sym::doc || name == sym::cfg_attr || name == sym::rustc_on_unimplemented { if name == sym::doc || name == sym::cfg_attr || name == sym::rustc_on_unimplemented || name == sym::reason {
// FIXME: Would be nice to handle `cfg_attr` as well. Only problem is to check that cfg // FIXME: Would be nice to handle `cfg_attr` as well. Only problem is to check that cfg
// conditions are the same. // conditions are the same.
// `#[rustc_on_unimplemented]` contains duplicated subattributes, that's expected. // `#[rustc_on_unimplemented]` contains duplicated subattributes, that's expected.

4
tests/ui/duplicated_attributes.rs
View file

@ -27,4 +27,8 @@ trait Abc {}
#[proc_macro_attr::duplicated_attr()] // Should not warn! #[proc_macro_attr::duplicated_attr()] // Should not warn!
fn babar() {} fn babar() {}
#[allow(missing_docs, reason = "library for internal use only")]
#[allow(exported_private_dependencies, reason = "library for internal use only")]
fn duplicate_reason() {}
fn main() {} fn main() {}