Skip to content

Commit ab1da8f

Browse files
hkmatsumotohkmatsumoto
committed
Add new lint upper_case_acronyms 
1 parent 6c830ff commit ab1da8f

24 files changed

+256
-88
lines changed

‎CHANGELOG.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1877,7 +1877,6 @@ Released 2018-09-13
18771877
[`box_vec`]: https://rust-lang.github.io/rust-clippy/master/index.html#box_vec
18781878
[`boxed_local`]: https://rust-lang.github.io/rust-clippy/master/index.html#boxed_local
18791879
[`builtin_type_shadow`]: https://rust-lang.github.io/rust-clippy/master/index.html#builtin_type_shadow
1880-
[`capitalized_acronyms`]: https://rust-lang.github.io/rust-clippy/master/index.html#capitalized_acronyms
18811880
[`cargo_common_metadata`]: https://rust-lang.github.io/rust-clippy/master/index.html#cargo_common_metadata
18821881
[`case_sensitive_file_extension_comparisons`]: https://rust-lang.github.io/rust-clippy/master/index.html#case_sensitive_file_extension_comparisons
18831882
[`cast_lossless`]: https://rust-lang.github.io/rust-clippy/master/index.html#cast_lossless
@@ -2275,6 +2274,7 @@ Released 2018-09-13
22752274
[`unusual_byte_groupings`]: https://rust-lang.github.io/rust-clippy/master/index.html#unusual_byte_groupings
22762275
[`unwrap_in_result`]: https://rust-lang.github.io/rust-clippy/master/index.html#unwrap_in_result
22772276
[`unwrap_used`]: https://rust-lang.github.io/rust-clippy/master/index.html#unwrap_used
2277+
[`upper_case_acronyms`]: https://rust-lang.github.io/rust-clippy/master/index.html#upper_case_acronyms
22782278
[`use_debug`]: https://rust-lang.github.io/rust-clippy/master/index.html#use_debug
22792279
[`use_self`]: https://rust-lang.github.io/rust-clippy/master/index.html#use_self
22802280
[`used_underscore_binding`]: https://rust-lang.github.io/rust-clippy/master/index.html#used_underscore_binding

‎clippy_lints/src/capitalized_acronyms.rs‎

Lines changed: 0 additions & 28 deletions
This file was deleted.

‎clippy_lints/src/cognitive_complexity.rs‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -57,9 +57,9 @@ impl CognitiveComplexity {
5757

5858
let expr = &body.value;
5959

60-
let mut helper = CCHelper { cc: 1, returns: 0 };
60+
let mut helper = CcHelper { cc: 1, returns: 0 };
6161
helper.visit_expr(expr);
62-
let CCHelper { cc, returns } = helper;
62+
let CcHelper { cc, returns } = helper;
6363
let ret_ty = cx.typeck_results().node_type(expr.hir_id);
6464
let ret_adjust = if is_type_diagnostic_item(cx, ret_ty, sym::result_type) {
6565
returns
@@ -136,12 +136,12 @@ impl<'tcx> LateLintPass<'tcx> for CognitiveComplexity {
136136
}
137137
}
138138

139-
struct CCHelper {
139+
struct CcHelper {
140140
cc: u64,
141141
returns: u64,
142142
}
143143

144-
impl<'tcx> Visitor<'tcx> for CCHelper {
144+
impl<'tcx> Visitor<'tcx> for CcHelper {
145145
type Map = Map<'tcx>;
146146

147147
fn visit_expr(&mut self, e: &'tcx Expr<'_>) {

‎clippy_lints/src/int_plus_one.rs‎

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -46,8 +46,8 @@ declare_lint_pass!(IntPlusOne => [INT_PLUS_ONE]);
4646

4747
#[derive(Copy, Clone)]
4848
enum Side {
49-
LHS,
50-
RHS,
49+
Lhs,
50+
Rhs,
5151
}
5252

5353
impl IntPlusOne {
@@ -66,11 +66,11 @@ impl IntPlusOne {
6666
match (lhskind.node, &lhslhs.kind, &lhsrhs.kind) {
6767
// `-1 + x`
6868
(BinOpKind::Add, &ExprKind::Lit(ref lit), _) if Self::check_lit(lit, -1) => {
69-
Self::generate_recommendation(cx, binop, lhsrhs, rhs, Side::LHS)
69+
Self::generate_recommendation(cx, binop, lhsrhs, rhs, Side::Lhs)
7070
},
7171
// `x - 1`
7272
(BinOpKind::Sub, _, &ExprKind::Lit(ref lit)) if Self::check_lit(lit, 1) => {
73-
Self::generate_recommendation(cx, binop, lhslhs, rhs, Side::LHS)
73+
Self::generate_recommendation(cx, binop, lhslhs, rhs, Side::Lhs)
7474
},
7575
_ => None,
7676
}
@@ -82,10 +82,10 @@ impl IntPlusOne {
8282
match (&rhslhs.kind, &rhsrhs.kind) {
8383
// `y + 1` and `1 + y`
8484
(&ExprKind::Lit(ref lit), _) if Self::check_lit(lit, 1) => {
85-
Self::generate_recommendation(cx, binop, rhsrhs, lhs, Side::RHS)
85+
Self::generate_recommendation(cx, binop, rhsrhs, lhs, Side::Rhs)
8686
},
8787
(_, &ExprKind::Lit(ref lit)) if Self::check_lit(lit, 1) => {
88-
Self::generate_recommendation(cx, binop, rhslhs, lhs, Side::RHS)
88+
Self::generate_recommendation(cx, binop, rhslhs, lhs, Side::Rhs)
8989
},
9090
_ => None,
9191
}
@@ -97,10 +97,10 @@ impl IntPlusOne {
9797
match (&lhslhs.kind, &lhsrhs.kind) {
9898
// `1 + x` and `x + 1`
9999
(&ExprKind::Lit(ref lit), _) if Self::check_lit(lit, 1) => {
100-
Self::generate_recommendation(cx, binop, lhsrhs, rhs, Side::LHS)
100+
Self::generate_recommendation(cx, binop, lhsrhs, rhs, Side::Lhs)
101101
},
102102
(_, &ExprKind::Lit(ref lit)) if Self::check_lit(lit, 1) => {
103-
Self::generate_recommendation(cx, binop, lhslhs, rhs, Side::LHS)
103+
Self::generate_recommendation(cx, binop, lhslhs, rhs, Side::Lhs)
104104
},
105105
_ => None,
106106
}
@@ -110,11 +110,11 @@ impl IntPlusOne {
110110
match (rhskind.node, &rhslhs.kind, &rhsrhs.kind) {
111111
// `-1 + y`
112112
(BinOpKind::Add, &ExprKind::Lit(ref lit), _) if Self::check_lit(lit, -1) => {
113-
Self::generate_recommendation(cx, binop, rhsrhs, lhs, Side::RHS)
113+
Self::generate_recommendation(cx, binop, rhsrhs, lhs, Side::Rhs)
114114
},
115115
// `y - 1`
116116
(BinOpKind::Sub, _, &ExprKind::Lit(ref lit)) if Self::check_lit(lit, 1) => {
117-
Self::generate_recommendation(cx, binop, rhslhs, lhs, Side::RHS)
117+
Self::generate_recommendation(cx, binop, rhslhs, lhs, Side::Rhs)
118118
},
119119
_ => None,
120120
}
@@ -138,8 +138,8 @@ impl IntPlusOne {
138138
if let Some(snippet) = snippet_opt(cx, node.span) {
139139
if let Some(other_side_snippet) = snippet_opt(cx, other_side.span) {
140140
let rec = match side {
141-
Side::LHS => Some(format!("{} {} {}", snippet, binop_string, other_side_snippet)),
142-
Side::RHS => Some(format!("{} {} {}", other_side_snippet, binop_string, snippet)),
141+
Side::Lhs => Some(format!("{} {} {}", snippet, binop_string, other_side_snippet)),
142+
Side::Rhs => Some(format!("{} {} {}", other_side_snippet, binop_string, snippet)),
143143
};
144144
return rec;
145145
}

‎clippy_lints/src/lib.rs‎

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
#![feature(concat_idents)]
77
#![feature(crate_visibility_modifier)]
88
#![feature(drain_filter)]
9+
#![feature(split_inclusive)]
910
#![feature(in_band_lifetimes)]
1011
#![feature(once_cell)]
1112
#![feature(or_patterns)]
@@ -169,7 +170,6 @@ mod blacklisted_name;
169170
mod blocks_in_if_conditions;
170171
mod booleans;
171172
mod bytecount;
172-
mod capitalized_acronyms;
173173
mod cargo_common_metadata;
174174
mod case_sensitive_file_extension_comparisons;
175175
mod checked_conversions;
@@ -342,6 +342,7 @@ mod unused_self;
342342
mod unused_unit;
343343
mod unwrap;
344344
mod unwrap_in_result;
345+
mod upper_case_acronyms;
345346
mod use_self;
346347
mod useless_conversion;
347348
mod vec;
@@ -560,7 +561,6 @@ pub fn register_plugins(store: &mut rustc_lint::LintStore, sess: &Session, conf:
560561
&booleans::LOGIC_BUG,
561562
&booleans::NONMINIMAL_BOOL,
562563
&bytecount::NAIVE_BYTECOUNT,
563-
&capitalized_acronyms::CAPITALIZED_ACRONYMS,
564564
&cargo_common_metadata::CARGO_COMMON_METADATA,
565565
&case_sensitive_file_extension_comparisons::CASE_SENSITIVE_FILE_EXTENSION_COMPARISONS,
566566
&checked_conversions::CHECKED_CONVERSIONS,
@@ -946,6 +946,7 @@ pub fn register_plugins(store: &mut rustc_lint::LintStore, sess: &Session, conf:
946946
&unwrap::PANICKING_UNWRAP,
947947
&unwrap::UNNECESSARY_UNWRAP,
948948
&unwrap_in_result::UNWRAP_IN_RESULT,
949+
&upper_case_acronyms::UPPER_CASE_ACRONYMS,
949950
&use_self::USE_SELF,
950951
&useless_conversion::USELESS_CONVERSION,
951952
&vec::USELESS_VEC,
@@ -985,7 +986,7 @@ pub fn register_plugins(store: &mut rustc_lint::LintStore, sess: &Session, conf:
985986
}
986987
store.register_late_pass(|| box utils::author::Author);
987988
store.register_late_pass(|| box await_holding_invalid::AwaitHolding);
988-
store.register_late_pass(|| box serde_api::SerdeAPI);
989+
store.register_late_pass(|| box serde_api::SerdeApi);
989990
let vec_box_size_threshold = conf.vec_box_size_threshold;
990991
store.register_late_pass(move || box types::Types::new(vec_box_size_threshold));
991992
store.register_late_pass(|| box booleans::NonminimalBool);
@@ -1176,6 +1177,7 @@ pub fn register_plugins(store: &mut rustc_lint::LintStore, sess: &Session, conf:
11761177
let enum_variant_name_threshold = conf.enum_variant_name_threshold;
11771178
store.register_early_pass(move || box enum_variants::EnumVariantNames::new(enum_variant_name_threshold));
11781179
store.register_early_pass(|| box tabs_in_doc_comments::TabsInDocComments);
1180+
store.register_early_pass(|| box upper_case_acronyms::UpperCaseAcronyms);
11791181
store.register_late_pass(|| box default::Default::default());
11801182
store.register_late_pass(|| box unused_self::UnusedSelf);
11811183
store.register_late_pass(|| box mutable_debug_assertion::DebugAssertWithMutCall);
@@ -1661,6 +1663,7 @@ pub fn register_plugins(store: &mut rustc_lint::LintStore, sess: &Session, conf:
16611663
LintId::of(&unused_unit::UNUSED_UNIT),
16621664
LintId::of(&unwrap::PANICKING_UNWRAP),
16631665
LintId::of(&unwrap::UNNECESSARY_UNWRAP),
1666+
LintId::of(&upper_case_acronyms::UPPER_CASE_ACRONYMS),
16641667
LintId::of(&useless_conversion::USELESS_CONVERSION),
16651668
LintId::of(&vec::USELESS_VEC),
16661669
LintId::of(&vec_init_then_push::VEC_INIT_THEN_PUSH),
@@ -1778,6 +1781,7 @@ pub fn register_plugins(store: &mut rustc_lint::LintStore, sess: &Session, conf:
17781781
LintId::of(&types::FN_TO_NUMERIC_CAST_WITH_TRUNCATION),
17791782
LintId::of(&unsafe_removed_from_name::UNSAFE_REMOVED_FROM_NAME),
17801783
LintId::of(&unused_unit::UNUSED_UNIT),
1784+
LintId::of(&upper_case_acronyms::UPPER_CASE_ACRONYMS),
17811785
LintId::of(&write::PRINTLN_EMPTY_STRING),
17821786
LintId::of(&write::PRINT_LITERAL),
17831787
LintId::of(&write::PRINT_WITH_NEWLINE),

‎clippy_lints/src/serde_api.rs‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,9 +18,9 @@ declare_clippy_lint! {
1818
"various things that will negatively affect your serde experience"
1919
}
2020

21-
declare_lint_pass!(SerdeAPI => [SERDE_API_MISUSE]);
21+
declare_lint_pass!(SerdeApi => [SERDE_API_MISUSE]);
2222

23-
impl<'tcx> LateLintPass<'tcx> for SerdeAPI {
23+
impl<'tcx> LateLintPass<'tcx> for SerdeApi {
2424
fn check_item(&mut self, cx: &LateContext<'tcx>, item: &'tcx Item<'_>) {
2525
if let ItemKind::Impl(Impl {
2626
of_trait: Some(ref trait_ref),

‎clippy_lints/src/upper_case_acronyms.rs‎

Lines changed: 93 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,93 @@
1+
use crate::utils::span_lint_and_sugg;
2+
use if_chain::if_chain;
3+
use itertools::Itertools;
4+
use rustc_ast::ast::{Item, ItemKind, Variant};
5+
use rustc_errors::Applicability;
6+
use rustc_lint::{EarlyContext, EarlyLintPass, LintContext};
7+
use rustc_middle::lint::in_external_macro;
8+
use rustc_session::{declare_lint_pass, declare_tool_lint};
9+
use rustc_span::symbol::Ident;
10+
11+
declare_clippy_lint! {
12+
/// **What it does:** Checks for camel case name containing a capitalized acronym.
13+
///
14+
/// **Why is this bad?** In CamelCase, acronyms count as one word.
15+
/// See [naming conventions](https://rust-lang.github.io/api-guidelines/naming.html#casing-conforms-to-rfc-430-c-case)
16+
/// for more.
17+
///
18+
/// **Known problems:** When two acronyms are contiguous, the lint can't tell where
19+
/// the first acronym ends and the second starts, so it suggests to lowercase all of
20+
/// the letters in the second acronym.
21+
///
22+
/// **Example:**
23+
///
24+
/// ```rust
25+
/// struct HTTPResponse;
26+
/// ```
27+
/// Use instead:
28+
/// ```rust
29+
/// struct HttpResponse;
30+
/// ```
31+
pub UPPER_CASE_ACRONYMS,
32+
style,
33+
"capitalized acronyms are against the naming convention"
34+
}
35+
36+
declare_lint_pass!(UpperCaseAcronyms => [UPPER_CASE_ACRONYMS]);
37+
38+
fn correct_ident(ident: &str) -> String {
39+
let ident = ident.chars().rev().collect::<String>();
40+
let fragments = ident
41+
.split_inclusive(|x: char| !x.is_ascii_lowercase())
42+
.rev()
43+
.map(|x| x.chars().rev().collect::<String>());
44+
45+
let mut ident = fragments.clone().next().unwrap();
46+
for (ref prev, ref curr) in fragments.tuple_windows() {
47+
if [prev, curr]
48+
.iter()
49+
.all(|s| s.len() == 1 && s.chars().next().unwrap().is_ascii_uppercase())
50+
{
51+
ident.push_str(&curr.to_ascii_lowercase());
52+
} else {
53+
ident.push_str(curr);
54+
}
55+
}
56+
ident
57+
}
58+
59+
fn check_ident(cx: &EarlyContext<'_>, ident: &Ident) {
60+
let span = ident.span;
61+
let ident = &ident.as_str();
62+
let corrected = correct_ident(ident);
63+
if ident != &corrected {
64+
span_lint_and_sugg(
65+
cx,
66+
UPPER_CASE_ACRONYMS,
67+
span,
68+
&format!("name `{}` contains a capitalized acronym", ident),
69+
"consider making the acronym lowercase, except the initial letter",
70+
corrected,
71+
Applicability::MaybeIncorrect,
72+
)
73+
}
74+
}
75+
76+
impl EarlyLintPass for UpperCaseAcronyms {
77+
fn check_item(&mut self, cx: &EarlyContext<'_>, it: &Item) {
78+
if_chain! {
79+
if !in_external_macro(cx.sess(), it.span);
80+
if matches!(
81+
it.kind,
82+
ItemKind::TyAlias(..) | ItemKind::Enum(..) | ItemKind::Struct(..) | ItemKind::Trait(..)
83+
);
84+
then {
85+
check_ident(cx, &it.ident);
86+
}
87+
}
88+
}
89+
90+
fn check_variant(&mut self, cx: &EarlyContext<'_>, v: &Variant) {
91+
check_ident(cx, &v.ident);
92+
}
93+
}

‎tests/ui/capitalized_acronyms.rs‎

Lines changed: 0 additions & 5 deletions
This file was deleted.

‎tests/ui/complex_types.rs‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ struct S {
1111
f: Vec<Vec<Box<(u32, u32, u32, u32)>>>,
1212
}
1313

14-
struct TS(Vec<Vec<Box<(u32, u32, u32, u32)>>>);
14+
struct Ts(Vec<Vec<Box<(u32, u32, u32, u32)>>>);
1515

1616
enum E {
1717
Tuple(Vec<Vec<Box<(u32, u32, u32, u32)>>>),

‎tests/ui/complex_types.stderr‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ LL | f: Vec<Vec<Box<(u32, u32, u32, u32)>>>,
2121
error: very complex type used. Consider factoring parts into `type` definitions
2222
--> $DIR/complex_types.rs:14:11
2323
|
24-
LL | struct TS(Vec<Vec<Box<(u32, u32, u32, u32)>>>);
24+
LL | struct Ts(Vec<Vec<Box<(u32, u32, u32, u32)>>>);
2525
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
2626

2727
error: very complex type used. Consider factoring parts into `type` definitions

0 commit comments

Comments
 (0)