Skip to content

Commit 63388cb

Browse files
AurelienFTAurelienFT
committed
add MSRV for manual_pattern_char_comparison
1 parent 0dc265f commit 63388cb

7 files changed

+53
-6
lines changed

‎book/src/lint_configuration.md

+1
Original file line numberDiff line numberDiff line change
@@ -693,6 +693,7 @@ The minimum rust version that the project supports. Defaults to the `rust-versio
693693
* [`manual_is_ascii_check`](https://rust-lang.github.io/rust-clippy/master/index.html#manual_is_ascii_check)
694694
* [`manual_let_else`](https://rust-lang.github.io/rust-clippy/master/index.html#manual_let_else)
695695
* [`manual_non_exhaustive`](https://rust-lang.github.io/rust-clippy/master/index.html#manual_non_exhaustive)
696+
* [`manual_pattern_char_comparison`](https://rust-lang.github.io/rust-clippy/master/index.html#manual_pattern_char_comparison)
696697
* [`manual_range_contains`](https://rust-lang.github.io/rust-clippy/master/index.html#manual_range_contains)
697698
* [`manual_rem_euclid`](https://rust-lang.github.io/rust-clippy/master/index.html#manual_rem_euclid)
698699
* [`manual_retain`](https://rust-lang.github.io/rust-clippy/master/index.html#manual_retain)

‎clippy_config/src/conf.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -265,7 +265,7 @@ define_Conf! {
265265
///
266266
/// Suppress lints whenever the suggested change would cause breakage for other crates.
267267
(avoid_breaking_exported_api: bool = true),
268-
/// Lint: MANUAL_SPLIT_ONCE, MANUAL_STR_REPEAT, CLONED_INSTEAD_OF_COPIED, REDUNDANT_FIELD_NAMES, OPTION_MAP_UNWRAP_OR, REDUNDANT_STATIC_LIFETIMES, FILTER_MAP_NEXT, CHECKED_CONVERSIONS, MANUAL_RANGE_CONTAINS, USE_SELF, MEM_REPLACE_WITH_DEFAULT, MANUAL_NON_EXHAUSTIVE, OPTION_AS_REF_DEREF, MAP_UNWRAP_OR, MATCH_LIKE_MATCHES_MACRO, MANUAL_STRIP, MISSING_CONST_FOR_FN, UNNESTED_OR_PATTERNS, FROM_OVER_INTO, PTR_AS_PTR, IF_THEN_SOME_ELSE_NONE, APPROX_CONSTANT, DEPRECATED_CFG_ATTR, INDEX_REFUTABLE_SLICE, MAP_CLONE, BORROW_AS_PTR, MANUAL_BITS, ERR_EXPECT, CAST_ABS_TO_UNSIGNED, UNINLINED_FORMAT_ARGS, MANUAL_CLAMP, MANUAL_LET_ELSE, UNCHECKED_DURATION_SUBTRACTION, COLLAPSIBLE_STR_REPLACE, SEEK_FROM_CURRENT, SEEK_REWIND, UNNECESSARY_LAZY_EVALUATIONS, TRANSMUTE_PTR_TO_REF, ALMOST_COMPLETE_RANGE, NEEDLESS_BORROW, DERIVABLE_IMPLS, MANUAL_IS_ASCII_CHECK, MANUAL_REM_EUCLID, MANUAL_RETAIN, TYPE_REPETITION_IN_BOUNDS, TUPLE_ARRAY_CONVERSIONS, MANUAL_TRY_FOLD, MANUAL_HASH_ONE, ITER_KV_MAP, MANUAL_C_STR_LITERALS, ASSIGNING_CLONES, LEGACY_NUMERIC_CONSTANTS.
268+
/// Lint: MANUAL_SPLIT_ONCE, MANUAL_STR_REPEAT, CLONED_INSTEAD_OF_COPIED, REDUNDANT_FIELD_NAMES, OPTION_MAP_UNWRAP_OR, REDUNDANT_STATIC_LIFETIMES, FILTER_MAP_NEXT, CHECKED_CONVERSIONS, MANUAL_RANGE_CONTAINS, USE_SELF, MEM_REPLACE_WITH_DEFAULT, MANUAL_NON_EXHAUSTIVE, OPTION_AS_REF_DEREF, MAP_UNWRAP_OR, MATCH_LIKE_MATCHES_MACRO, MANUAL_STRIP, MISSING_CONST_FOR_FN, UNNESTED_OR_PATTERNS, FROM_OVER_INTO, PTR_AS_PTR, IF_THEN_SOME_ELSE_NONE, APPROX_CONSTANT, DEPRECATED_CFG_ATTR, INDEX_REFUTABLE_SLICE, MAP_CLONE, BORROW_AS_PTR, MANUAL_BITS, ERR_EXPECT, CAST_ABS_TO_UNSIGNED, UNINLINED_FORMAT_ARGS, MANUAL_CLAMP, MANUAL_LET_ELSE, UNCHECKED_DURATION_SUBTRACTION, COLLAPSIBLE_STR_REPLACE, SEEK_FROM_CURRENT, SEEK_REWIND, UNNECESSARY_LAZY_EVALUATIONS, TRANSMUTE_PTR_TO_REF, ALMOST_COMPLETE_RANGE, NEEDLESS_BORROW, DERIVABLE_IMPLS, MANUAL_IS_ASCII_CHECK, MANUAL_REM_EUCLID, MANUAL_RETAIN, TYPE_REPETITION_IN_BOUNDS, TUPLE_ARRAY_CONVERSIONS, MANUAL_TRY_FOLD, MANUAL_HASH_ONE, ITER_KV_MAP, MANUAL_C_STR_LITERALS, ASSIGNING_CLONES, LEGACY_NUMERIC_CONSTANTS, MANUAL_PATTERN_CHAR_COMPARISON.
269269
///
270270
/// The minimum rust version that the project supports. Defaults to the `rust-version` field in `Cargo.toml`
271271
#[default_text = ""]

‎clippy_lints/src/lib.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -1168,7 +1168,7 @@ pub fn register_lints(store: &mut rustc_lint::LintStore, conf: &'static Conf) {
11681168
..Default::default()
11691169
})
11701170
});
1171-
store.register_late_pass(|_| Box::new(string_patterns::StringPatterns));
1171+
store.register_late_pass(move |_| Box::new(string_patterns::StringPatterns::new(msrv())));
11721172
// add lints here, do not remove this comment, it's used in `new_lint`
11731173
}
11741174

‎clippy_lints/src/string_patterns.rs

+19-3
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
use std::ops::ControlFlow;
22

3+
use clippy_config::msrvs::{self, Msrv};
34
use clippy_utils::diagnostics::{span_lint_and_sugg, span_lint_and_then};
45
use clippy_utils::eager_or_lazy::switch_to_eager_eval;
56
use clippy_utils::macros::matching_root_macro_call;
@@ -12,7 +13,7 @@ use rustc_errors::Applicability;
1213
use rustc_hir::{Expr, ExprKind, PatKind};
1314
use rustc_lint::{LateContext, LateLintPass};
1415
use rustc_middle::ty;
15-
use rustc_session::declare_lint_pass;
16+
use rustc_session::impl_lint_pass;
1617
use rustc_span::{sym, Span};
1718

1819
declare_clippy_lint! {
@@ -69,7 +70,18 @@ declare_clippy_lint! {
6970
"using a single-character str where a char could be used, e.g., `_.split(\"x\")`"
7071
}
7172

72-
declare_lint_pass!(StringPatterns => [MANUAL_PATTERN_CHAR_COMPARISON, SINGLE_CHAR_PATTERN]);
73+
pub struct StringPatterns {
74+
msrv: Msrv,
75+
}
76+
77+
impl StringPatterns {
78+
#[must_use]
79+
pub fn new(msrv: Msrv) -> Self {
80+
Self { msrv }
81+
}
82+
}
83+
84+
impl_lint_pass!(StringPatterns => [MANUAL_PATTERN_CHAR_COMPARISON, SINGLE_CHAR_PATTERN]);
7385

7486
const PATTERN_METHODS: [(&str, usize); 22] = [
7587
("contains", 0),
@@ -220,8 +232,12 @@ impl<'tcx> LateLintPass<'tcx> for StringPatterns {
220232
&& let Some(arg) = args.get(pos)
221233
{
222234
check_single_char_pattern_lint(cx, arg);
223-
235+
if !self.msrv.meets(msrvs::PATTERN_TRAIT_CHAR_ARRAY) {
236+
return;
237+
}
224238
check_manual_pattern_char_comparison(cx, arg);
225239
}
226240
}
241+
242+
extract_msrv_attr!(LateContext);
227243
}

‎tests/ui/manual_pattern_char_comparison.fixed

+12
Original file line numberDiff line numberDiff line change
@@ -47,3 +47,15 @@ fn main() {
4747
}
4848
"".find(|c| m!(c));
4949
}
50+
51+
#[clippy::msrv = "1.57"]
52+
fn msrv_1_57() {
53+
let sentence = "Hello, world!";
54+
sentence.trim_end_matches(|c: char| c == '.' || c == ',' || c == '!' || c == '?');
55+
}
56+
57+
#[clippy::msrv = "1.58"]
58+
fn msrv_1_58() {
59+
let sentence = "Hello, world!";
60+
sentence.trim_end_matches(['.', ',', '!', '?']);
61+
}

‎tests/ui/manual_pattern_char_comparison.rs

+12
Original file line numberDiff line numberDiff line change
@@ -47,3 +47,15 @@ fn main() {
4747
}
4848
"".find(|c| m!(c));
4949
}
50+
51+
#[clippy::msrv = "1.57"]
52+
fn msrv_1_57() {
53+
let sentence = "Hello, world!";
54+
sentence.trim_end_matches(|c: char| c == '.' || c == ',' || c == '!' || c == '?');
55+
}
56+
57+
#[clippy::msrv = "1.58"]
58+
fn msrv_1_58() {
59+
let sentence = "Hello, world!";
60+
sentence.trim_end_matches(|c: char| c == '.' || c == ',' || c == '!' || c == '?');
61+
}

‎tests/ui/manual_pattern_char_comparison.stderr

+7-1
Original file line numberDiff line numberDiff line change
@@ -55,5 +55,11 @@ error: this manual char comparison can be written more succinctly
5555
LL | sentence.find(|c| c == '🎈');
5656
| ^^^^^^^^^^^^^ help: consider using a `char`: `'🎈'`
5757

58-
error: aborting due to 9 previous errors
58+
error: this manual char comparison can be written more succinctly
59+
--> tests/ui/manual_pattern_char_comparison.rs:60:31
60+
|
61+
LL | sentence.trim_end_matches(|c: char| c == '.' || c == ',' || c == '!' || c == '?');
62+
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider using an array of `char`: `['.', ',', '!', '?']`
63+
64+
error: aborting due to 10 previous errors
5965

0 commit comments

Comments
 (0)