clippy_config/
conf.rs

1use crate::ClippyConfiguration;
2use crate::types::{
3    DisallowedPath, DisallowedPathWithoutReplacement, MacroMatcher, MatchLintBehaviour, PubUnderscoreFieldsBehaviour,
4    Rename, SourceItemOrdering, SourceItemOrderingCategory, SourceItemOrderingModuleItemGroupings,
5    SourceItemOrderingModuleItemKind, SourceItemOrderingTraitAssocItemKind, SourceItemOrderingTraitAssocItemKinds,
6    SourceItemOrderingWithinModuleItemGroupings,
7};
8use clippy_utils::msrvs::Msrv;
9use itertools::Itertools;
10use rustc_errors::Applicability;
11use rustc_session::Session;
12use rustc_span::edit_distance::edit_distance;
13use rustc_span::{BytePos, Pos, SourceFile, Span, SyntaxContext};
14use serde::de::{IgnoredAny, IntoDeserializer, MapAccess, Visitor};
15use serde::{Deserialize, Deserializer, Serialize};
16use std::collections::HashMap;
17use std::fmt::{Debug, Display, Formatter};
18use std::ops::Range;
19use std::path::PathBuf;
20use std::str::FromStr;
21use std::sync::OnceLock;
22use std::{cmp, env, fmt, fs, io};
23
24#[rustfmt::skip]
25const DEFAULT_DOC_VALID_IDENTS: &[&str] = &[
26    "KiB", "MiB", "GiB", "TiB", "PiB", "EiB",
27    "MHz", "GHz", "THz",
28    "AccessKit",
29    "CoAP", "CoreFoundation", "CoreGraphics", "CoreText",
30    "DevOps",
31    "Direct2D", "Direct3D", "DirectWrite", "DirectX",
32    "ECMAScript",
33    "GPLv2", "GPLv3",
34    "GitHub", "GitLab",
35    "IPv4", "IPv6",
36    "ClojureScript", "CoffeeScript", "JavaScript", "PostScript", "PureScript", "TypeScript",
37    "WebAssembly",
38    "NaN", "NaNs",
39    "OAuth", "GraphQL",
40    "OCaml",
41    "OpenAL", "OpenDNS", "OpenGL", "OpenMP", "OpenSSH", "OpenSSL", "OpenStreetMap", "OpenTelemetry",
42    "OpenType",
43    "WebGL", "WebGL2", "WebGPU", "WebRTC", "WebSocket", "WebTransport",
44    "WebP", "OpenExr", "YCbCr", "sRGB",
45    "TensorFlow",
46    "TrueType",
47    "iOS", "macOS", "FreeBSD", "NetBSD", "OpenBSD",
48    "TeX", "LaTeX", "BibTeX", "BibLaTeX",
49    "MinGW",
50    "CamelCase",
51];
52const DEFAULT_DISALLOWED_NAMES: &[&str] = &["foo", "baz", "quux"];
53const DEFAULT_ALLOWED_IDENTS_BELOW_MIN_CHARS: &[&str] = &["i", "j", "x", "y", "z", "w", "n"];
54const DEFAULT_ALLOWED_PREFIXES: &[&str] = &["to", "as", "into", "from", "try_into", "try_from"];
55const DEFAULT_ALLOWED_TRAITS_WITH_RENAMED_PARAMS: &[&str] =
56    &["core::convert::From", "core::convert::TryFrom", "core::str::FromStr"];
57const DEFAULT_MODULE_ITEM_ORDERING_GROUPS: &[(&str, &[SourceItemOrderingModuleItemKind])] = {
58    #[allow(clippy::enum_glob_use)] // Very local glob use for legibility.
59    use SourceItemOrderingModuleItemKind::*;
60    &[
61        ("modules", &[ExternCrate, Mod, ForeignMod]),
62        ("use", &[Use]),
63        ("macros", &[Macro]),
64        ("global_asm", &[GlobalAsm]),
65        ("UPPER_SNAKE_CASE", &[Static, Const]),
66        ("PascalCase", &[TyAlias, Enum, Struct, Union, Trait, TraitAlias, Impl]),
67        ("lower_snake_case", &[Fn]),
68    ]
69};
70const DEFAULT_TRAIT_ASSOC_ITEM_KINDS_ORDER: &[SourceItemOrderingTraitAssocItemKind] = {
71    #[allow(clippy::enum_glob_use)] // Very local glob use for legibility.
72    use SourceItemOrderingTraitAssocItemKind::*;
73    &[Const, Type, Fn]
74};
75const DEFAULT_SOURCE_ITEM_ORDERING: &[SourceItemOrderingCategory] = {
76    #[allow(clippy::enum_glob_use)] // Very local glob use for legibility.
77    use SourceItemOrderingCategory::*;
78    &[Enum, Impl, Module, Struct, Trait]
79};
80
81/// Conf with parse errors
82#[derive(Default)]
83struct TryConf {
84    conf: Conf,
85    value_spans: HashMap<String, Range<usize>>,
86    errors: Vec<ConfError>,
87    warnings: Vec<ConfError>,
88}
89
90impl TryConf {
91    fn from_toml_error(file: &SourceFile, error: &toml::de::Error) -> Self {
92        Self {
93            conf: Conf::default(),
94            value_spans: HashMap::default(),
95            errors: vec![ConfError::from_toml(file, error)],
96            warnings: vec![],
97        }
98    }
99}
100
101#[derive(Debug)]
102struct ConfError {
103    message: String,
104    suggestion: Option<Suggestion>,
105    span: Span,
106}
107
108impl ConfError {
109    fn from_toml(file: &SourceFile, error: &toml::de::Error) -> Self {
110        let span = error.span().unwrap_or(0..file.source_len.0 as usize);
111        Self::spanned(file, error.message(), None, span)
112    }
113
114    fn spanned(
115        file: &SourceFile,
116        message: impl Into<String>,
117        suggestion: Option<Suggestion>,
118        span: Range<usize>,
119    ) -> Self {
120        Self {
121            message: message.into(),
122            suggestion,
123            span: span_from_toml_range(file, span),
124        }
125    }
126}
127
128// Remove code tags and code behind '# 's, as they are not needed for the lint docs and --explain
129pub fn sanitize_explanation(raw_docs: &str) -> String {
130    // Remove tags and hidden code:
131    let mut explanation = String::with_capacity(128);
132    let mut in_code = false;
133    for line in raw_docs.lines() {
134        let line = line.strip_prefix(' ').unwrap_or(line);
135
136        if let Some(lang) = line.strip_prefix("```") {
137            let tag = lang.split_once(',').map_or(lang, |(left, _)| left);
138            if !in_code && matches!(tag, "" | "rust" | "ignore" | "should_panic" | "no_run" | "compile_fail") {
139                explanation += "```rust\n";
140            } else {
141                explanation += line;
142                explanation.push('\n');
143            }
144            in_code = !in_code;
145        } else if !(in_code && line.starts_with("# ")) {
146            explanation += line;
147            explanation.push('\n');
148        }
149    }
150
151    explanation
152}
153
154macro_rules! wrap_option {
155    () => {
156        None
157    };
158    ($x:literal) => {
159        Some($x)
160    };
161}
162
163macro_rules! default_text {
164    ($value:expr) => {{
165        let mut text = String::new();
166        $value.serialize(toml::ser::ValueSerializer::new(&mut text)).unwrap();
167        text
168    }};
169    ($value:expr, $override:expr) => {
170        $override.to_string()
171    };
172}
173
174macro_rules! deserialize {
175    ($map:expr, $ty:ty, $errors:expr, $file:expr) => {{
176        let raw_value = $map.next_value::<toml::Spanned<toml::Value>>()?;
177        let value_span = raw_value.span();
178        let value = match <$ty>::deserialize(raw_value.into_inner()) {
179            Err(e) => {
180                $errors.push(ConfError::spanned(
181                    $file,
182                    e.to_string().replace('\n', " ").trim(),
183                    None,
184                    value_span,
185                ));
186                continue;
187            },
188            Ok(value) => value,
189        };
190        (value, value_span)
191    }};
192
193    ($map:expr, $ty:ty, $errors:expr, $file:expr, $replacements_allowed:expr) => {{
194        let array = $map.next_value::<Vec<toml::Spanned<toml::Value>>>()?;
195        let mut disallowed_paths_span = Range {
196            start: usize::MAX,
197            end: usize::MIN,
198        };
199        let mut disallowed_paths = Vec::new();
200        for raw_value in array {
201            let value_span = raw_value.span();
202            let mut disallowed_path = match DisallowedPath::<$replacements_allowed>::deserialize(raw_value.into_inner())
203            {
204                Err(e) => {
205                    $errors.push(ConfError::spanned(
206                        $file,
207                        e.to_string().replace('\n', " ").trim(),
208                        None,
209                        value_span,
210                    ));
211                    continue;
212                },
213                Ok(disallowed_path) => disallowed_path,
214            };
215            disallowed_paths_span = union(&disallowed_paths_span, &value_span);
216            disallowed_path.set_span(span_from_toml_range($file, value_span));
217            disallowed_paths.push(disallowed_path);
218        }
219        (disallowed_paths, disallowed_paths_span)
220    }};
221}
222
223macro_rules! define_Conf {
224    ($(
225        $(#[doc = $doc:literal])+
226        $(#[conf_deprecated($dep:literal, $new_conf:ident)])?
227        $(#[default_text = $default_text:expr])?
228        $(#[disallowed_paths_allow_replacements = $replacements_allowed:expr])?
229        $(#[lints($($for_lints:ident),* $(,)?)])?
230        $name:ident: $ty:ty = $default:expr,
231    )*) => {
232        /// Clippy lint configuration
233        pub struct Conf {
234            $($(#[cfg_attr(doc, doc = $doc)])+ pub $name: $ty,)*
235        }
236
237        mod defaults {
238            use super::*;
239            $(pub fn $name() -> $ty { $default })*
240        }
241
242        impl Default for Conf {
243            fn default() -> Self {
244                Self { $($name: defaults::$name(),)* }
245            }
246        }
247
248        #[derive(Deserialize)]
249        #[serde(field_identifier, rename_all = "kebab-case")]
250        #[allow(non_camel_case_types)]
251        enum Field { $($name,)* third_party, }
252
253        struct ConfVisitor<'a>(&'a SourceFile);
254
255        impl<'de> Visitor<'de> for ConfVisitor<'_> {
256            type Value = TryConf;
257
258            fn expecting(&self, formatter: &mut fmt::Formatter<'_>) -> fmt::Result {
259                formatter.write_str("Conf")
260            }
261
262            fn visit_map<V>(self, mut map: V) -> Result<Self::Value, V::Error> where V: MapAccess<'de> {
263                let mut value_spans = HashMap::new();
264                let mut errors = Vec::new();
265                let mut warnings = Vec::new();
266
267                // Declare a local variable for each field available to a configuration file.
268                $(let mut $name = None;)*
269
270                // could get `Field` here directly, but get `String` first for diagnostics
271                while let Some(name) = map.next_key::<toml::Spanned<String>>()? {
272                    let field = match Field::deserialize(name.get_ref().as_str().into_deserializer()) {
273                        Err(e) => {
274                            let e: FieldError = e;
275                            errors.push(ConfError::spanned(self.0, e.error, e.suggestion, name.span()));
276                            continue;
277                        }
278                        Ok(field) => field
279                    };
280
281                    match field {
282                        $(Field::$name => {
283                            // Is this a deprecated field, i.e., is `$dep` set? If so, push a warning.
284                            $(warnings.push(ConfError::spanned(self.0, format!("deprecated field `{}`. {}", name.get_ref(), $dep), None, name.span()));)?
285                            let (value, value_span) =
286                                deserialize!(map, $ty, errors, self.0 $(, $replacements_allowed)?);
287                            // Was this field set previously?
288                            if $name.is_some() {
289                                errors.push(ConfError::spanned(self.0, format!("duplicate field `{}`", name.get_ref()), None, name.span()));
290                                continue;
291                            }
292                            $name = Some(value);
293                            value_spans.insert(name.get_ref().as_str().to_string(), value_span);
294                            // If this is a deprecated field, was the new field (`$new_conf`) set previously?
295                            // Note that `$new_conf` is one of the defined `$name`s.
296                            $(match $new_conf {
297                                Some(_) => errors.push(ConfError::spanned(self.0, concat!(
298                                    "duplicate field `", stringify!($new_conf),
299                                    "` (provided as `", stringify!($name), "`)"
300                                ), None, name.span())),
301                                None => $new_conf = $name.clone(),
302                            })?
303                        })*
304                        // ignore contents of the third_party key
305                        Field::third_party => drop(map.next_value::<IgnoredAny>())
306                    }
307                }
308                let conf = Conf { $($name: $name.unwrap_or_else(defaults::$name),)* };
309                Ok(TryConf { conf, value_spans, errors, warnings })
310            }
311        }
312
313        pub fn get_configuration_metadata() -> Vec<ClippyConfiguration> {
314            vec![$(
315                ClippyConfiguration {
316                    name: stringify!($name).replace('_', "-"),
317                    default: default_text!(defaults::$name() $(, $default_text)?),
318                    lints: &[$($(stringify!($for_lints)),*)?],
319                    doc: concat!($($doc, '\n',)*),
320                    deprecation_reason: wrap_option!($($dep)?)
321                },
322            )*]
323        }
324    };
325}
326
327fn union(x: &Range<usize>, y: &Range<usize>) -> Range<usize> {
328    Range {
329        start: cmp::min(x.start, y.start),
330        end: cmp::max(x.end, y.end),
331    }
332}
333
334fn span_from_toml_range(file: &SourceFile, span: Range<usize>) -> Span {
335    Span::new(
336        file.start_pos + BytePos::from_usize(span.start),
337        file.start_pos + BytePos::from_usize(span.end),
338        SyntaxContext::root(),
339        None,
340    )
341}
342
343define_Conf! {
344    /// Which crates to allow absolute paths from
345    #[lints(absolute_paths)]
346    absolute_paths_allowed_crates: Vec<String> = Vec::new(),
347    /// The maximum number of segments a path can have before being linted, anything above this will
348    /// be linted.
349    #[lints(absolute_paths)]
350    absolute_paths_max_segments: u64 = 2,
351    /// Whether to accept a safety comment to be placed above the attributes for the `unsafe` block
352    #[lints(undocumented_unsafe_blocks)]
353    accept_comment_above_attributes: bool = true,
354    /// Whether to accept a safety comment to be placed above the statement containing the `unsafe` block
355    #[lints(undocumented_unsafe_blocks)]
356    accept_comment_above_statement: bool = true,
357    /// Don't lint when comparing the result of a modulo operation to zero.
358    #[lints(modulo_arithmetic)]
359    allow_comparison_to_zero: bool = true,
360    /// Whether `dbg!` should be allowed in test functions or `#[cfg(test)]`
361    #[lints(dbg_macro)]
362    allow_dbg_in_tests: bool = false,
363    /// Whether `expect` should be allowed in code always evaluated at compile time
364    #[lints(expect_used)]
365    allow_expect_in_consts: bool = true,
366    /// Whether `expect` should be allowed in test functions or `#[cfg(test)]`
367    #[lints(expect_used)]
368    allow_expect_in_tests: bool = false,
369    /// Whether `indexing_slicing` should be allowed in test functions or `#[cfg(test)]`
370    #[lints(indexing_slicing)]
371    allow_indexing_slicing_in_tests: bool = false,
372    /// Whether to allow mixed uninlined format args, e.g. `format!("{} {}", a, foo.bar)`
373    #[lints(uninlined_format_args)]
374    allow_mixed_uninlined_format_args: bool = true,
375    /// Whether to allow `r#""#` when `r""` can be used
376    #[lints(needless_raw_string_hashes)]
377    allow_one_hash_in_raw_strings: bool = false,
378    /// Whether `panic` should be allowed in test functions or `#[cfg(test)]`
379    #[lints(panic)]
380    allow_panic_in_tests: bool = false,
381    /// Whether print macros (ex. `println!`) should be allowed in test functions or `#[cfg(test)]`
382    #[lints(print_stderr, print_stdout)]
383    allow_print_in_tests: bool = false,
384    /// Whether to allow module inception if it's not public.
385    #[lints(module_inception)]
386    allow_private_module_inception: bool = false,
387    /// List of trait paths to ignore when checking renamed function parameters.
388    ///
389    /// #### Example
390    ///
391    /// ```toml
392    /// allow-renamed-params-for = [ "std::convert::From" ]
393    /// ```
394    ///
395    /// #### Noteworthy
396    ///
397    /// - By default, the following traits are ignored: `From`, `TryFrom`, `FromStr`
398    /// - `".."` can be used as part of the list to indicate that the configured values should be appended to the
399    /// default configuration of Clippy. By default, any configuration will replace the default value.
400    #[lints(renamed_function_params)]
401    allow_renamed_params_for: Vec<String> =
402        DEFAULT_ALLOWED_TRAITS_WITH_RENAMED_PARAMS.iter().map(ToString::to_string).collect(),
403    /// Whether `unwrap` should be allowed in code always evaluated at compile time
404    #[lints(unwrap_used)]
405    allow_unwrap_in_consts: bool = true,
406    /// Whether `unwrap` should be allowed in test functions or `#[cfg(test)]`
407    #[lints(unwrap_used)]
408    allow_unwrap_in_tests: bool = false,
409    /// Whether `useless_vec` should ignore test functions or `#[cfg(test)]`
410    #[lints(useless_vec)]
411    allow_useless_vec_in_tests: bool = false,
412    /// Additional dotfiles (files or directories starting with a dot) to allow
413    #[lints(path_ends_with_ext)]
414    allowed_dotfiles: Vec<String> = Vec::default(),
415    /// A list of crate names to allow duplicates of
416    #[lints(multiple_crate_versions)]
417    allowed_duplicate_crates: Vec<String> = Vec::new(),
418    /// Allowed names below the minimum allowed characters. The value `".."` can be used as part of
419    /// the list to indicate, that the configured values should be appended to the default
420    /// configuration of Clippy. By default, any configuration will replace the default value.
421    #[lints(min_ident_chars)]
422    allowed_idents_below_min_chars: Vec<String> =
423        DEFAULT_ALLOWED_IDENTS_BELOW_MIN_CHARS.iter().map(ToString::to_string).collect(),
424    /// List of prefixes to allow when determining whether an item's name ends with the module's name.
425    /// If the rest of an item's name is an allowed prefix (e.g. item `ToFoo` or `to_foo` in module `foo`),
426    /// then don't emit a warning.
427    ///
428    /// #### Example
429    ///
430    /// ```toml
431    /// allowed-prefixes = [ "to", "from" ]
432    /// ```
433    ///
434    /// #### Noteworthy
435    ///
436    /// - By default, the following prefixes are allowed: `to`, `as`, `into`, `from`, `try_into` and `try_from`
437    /// - PascalCase variant is included automatically for each snake_case variant (e.g. if `try_into` is included,
438    ///   `TryInto` will also be included)
439    /// - Use `".."` as part of the list to indicate that the configured values should be appended to the
440    /// default configuration of Clippy. By default, any configuration will replace the default value
441    #[lints(module_name_repetitions)]
442    allowed_prefixes: Vec<String> = DEFAULT_ALLOWED_PREFIXES.iter().map(ToString::to_string).collect(),
443    /// The list of unicode scripts allowed to be used in the scope.
444    #[lints(disallowed_script_idents)]
445    allowed_scripts: Vec<String> = vec!["Latin".to_string()],
446    /// List of path segments allowed to have wildcard imports.
447    ///
448    /// #### Example
449    ///
450    /// ```toml
451    /// allowed-wildcard-imports = [ "utils", "common" ]
452    /// ```
453    ///
454    /// #### Noteworthy
455    ///
456    /// 1. This configuration has no effects if used with `warn_on_all_wildcard_imports = true`.
457    /// 2. Paths with any segment that containing the word 'prelude'
458    /// are already allowed by default.
459    #[lints(wildcard_imports)]
460    allowed_wildcard_imports: Vec<String> = Vec::new(),
461    /// Suppress checking of the passed type names in all types of operations.
462    ///
463    /// If a specific operation is desired, consider using `arithmetic_side_effects_allowed_binary` or `arithmetic_side_effects_allowed_unary` instead.
464    ///
465    /// #### Example
466    ///
467    /// ```toml
468    /// arithmetic-side-effects-allowed = ["SomeType", "AnotherType"]
469    /// ```
470    ///
471    /// #### Noteworthy
472    ///
473    /// A type, say `SomeType`, listed in this configuration has the same behavior of
474    /// `["SomeType" , "*"], ["*", "SomeType"]` in `arithmetic_side_effects_allowed_binary`.
475    #[lints(arithmetic_side_effects)]
476    arithmetic_side_effects_allowed: Vec<String> = <_>::default(),
477    /// Suppress checking of the passed type pair names in binary operations like addition or
478    /// multiplication.
479    ///
480    /// Supports the "*" wildcard to indicate that a certain type won't trigger the lint regardless
481    /// of the involved counterpart. For example, `["SomeType", "*"]` or `["*", "AnotherType"]`.
482    ///
483    /// Pairs are asymmetric, which means that `["SomeType", "AnotherType"]` is not the same as
484    /// `["AnotherType", "SomeType"]`.
485    ///
486    /// #### Example
487    ///
488    /// ```toml
489    /// arithmetic-side-effects-allowed-binary = [["SomeType" , "f32"], ["AnotherType", "*"]]
490    /// ```
491    #[lints(arithmetic_side_effects)]
492    arithmetic_side_effects_allowed_binary: Vec<(String, String)> = <_>::default(),
493    /// Suppress checking of the passed type names in unary operations like "negation" (`-`).
494    ///
495    /// #### Example
496    ///
497    /// ```toml
498    /// arithmetic-side-effects-allowed-unary = ["SomeType", "AnotherType"]
499    /// ```
500    #[lints(arithmetic_side_effects)]
501    arithmetic_side_effects_allowed_unary: Vec<String> = <_>::default(),
502    /// The maximum allowed size for arrays on the stack
503    #[lints(large_const_arrays, large_stack_arrays)]
504    array_size_threshold: u64 = 16 * 1024,
505    /// Suppress lints whenever the suggested change would cause breakage for other crates.
506    #[lints(
507        box_collection,
508        enum_variant_names,
509        large_types_passed_by_value,
510        linkedlist,
511        needless_pass_by_ref_mut,
512        option_option,
513        owned_cow,
514        rc_buffer,
515        rc_mutex,
516        redundant_allocation,
517        ref_option,
518        single_call_fn,
519        trivially_copy_pass_by_ref,
520        unnecessary_box_returns,
521        unnecessary_wraps,
522        unused_self,
523        upper_case_acronyms,
524        vec_box,
525        wrong_self_convention,
526    )]
527    avoid_breaking_exported_api: bool = true,
528    /// The list of types which may not be held across an await point.
529    #[disallowed_paths_allow_replacements = false]
530    #[lints(await_holding_invalid_type)]
531    await_holding_invalid_types: Vec<DisallowedPathWithoutReplacement> = Vec::new(),
532    /// DEPRECATED LINT: BLACKLISTED_NAME.
533    ///
534    /// Use the Disallowed Names lint instead
535    #[conf_deprecated("Please use `disallowed-names` instead", disallowed_names)]
536    blacklisted_names: Vec<String> = Vec::new(),
537    /// For internal testing only, ignores the current `publish` settings in the Cargo manifest.
538    #[lints(cargo_common_metadata)]
539    cargo_ignore_publish: bool = false,
540    /// Whether to check MSRV compatibility in `#[test]` and `#[cfg(test)]` code.
541    #[lints(incompatible_msrv)]
542    check_incompatible_msrv_in_tests: bool = false,
543    /// Whether to suggest reordering constructor fields when initializers are present.
544    ///
545    /// Warnings produced by this configuration aren't necessarily fixed by just reordering the fields. Even if the
546    /// suggested code would compile, it can change semantics if the initializer expressions have side effects. The
547    /// following example [from rust-clippy#11846] shows how the suggestion can run into borrow check errors:
548    ///
549    /// ```rust
550    /// struct MyStruct {
551    ///     vector: Vec<u32>,
552    ///     length: usize
553    /// }
554    /// fn main() {
555    ///     let vector = vec![1,2,3];
556    ///     MyStruct { length: vector.len(), vector};
557    /// }
558    /// ```
559    ///
560    /// [from rust-clippy#11846]: https://github.com/rust-lang/rust-clippy/issues/11846#issuecomment-1820747924
561    #[lints(inconsistent_struct_constructor)]
562    check_inconsistent_struct_field_initializers: bool = false,
563    /// Whether to also run the listed lints on private items.
564    #[lints(missing_errors_doc, missing_panics_doc, missing_safety_doc, unnecessary_safety_doc)]
565    check_private_items: bool = false,
566    /// The maximum cognitive complexity a function can have
567    #[lints(cognitive_complexity)]
568    cognitive_complexity_threshold: u64 = 25,
569    /// DEPRECATED LINT: CYCLOMATIC_COMPLEXITY.
570    ///
571    /// Use the Cognitive Complexity lint instead.
572    #[conf_deprecated("Please use `cognitive-complexity-threshold` instead", cognitive_complexity_threshold)]
573    cyclomatic_complexity_threshold: u64 = 25,
574    /// The list of disallowed macros, written as fully qualified paths.
575    #[disallowed_paths_allow_replacements = true]
576    #[lints(disallowed_macros)]
577    disallowed_macros: Vec<DisallowedPath> = Vec::new(),
578    /// The list of disallowed methods, written as fully qualified paths.
579    #[disallowed_paths_allow_replacements = true]
580    #[lints(disallowed_methods)]
581    disallowed_methods: Vec<DisallowedPath> = Vec::new(),
582    /// The list of disallowed names to lint about. NB: `bar` is not here since it has legitimate uses. The value
583    /// `".."` can be used as part of the list to indicate that the configured values should be appended to the
584    /// default configuration of Clippy. By default, any configuration will replace the default value.
585    #[lints(disallowed_names)]
586    disallowed_names: Vec<String> = DEFAULT_DISALLOWED_NAMES.iter().map(ToString::to_string).collect(),
587    /// The list of disallowed types, written as fully qualified paths.
588    #[disallowed_paths_allow_replacements = true]
589    #[lints(disallowed_types)]
590    disallowed_types: Vec<DisallowedPath> = Vec::new(),
591    /// The list of words this lint should not consider as identifiers needing ticks. The value
592    /// `".."` can be used as part of the list to indicate, that the configured values should be appended to the
593    /// default configuration of Clippy. By default, any configuration will replace the default value. For example:
594    /// * `doc-valid-idents = ["ClipPy"]` would replace the default list with `["ClipPy"]`.
595    /// * `doc-valid-idents = ["ClipPy", ".."]` would append `ClipPy` to the default list.
596    #[lints(doc_markdown)]
597    doc_valid_idents: Vec<String> = DEFAULT_DOC_VALID_IDENTS.iter().map(ToString::to_string).collect(),
598    /// Whether to apply the raw pointer heuristic to determine if a type is `Send`.
599    #[lints(non_send_fields_in_send_ty)]
600    enable_raw_pointer_heuristic_for_send: bool = true,
601    /// Whether to recommend using implicit into iter for reborrowed values.
602    ///
603    /// #### Example
604    /// ```no_run
605    /// let mut vec = vec![1, 2, 3];
606    /// let rmvec = &mut vec;
607    /// for _ in rmvec.iter() {}
608    /// for _ in rmvec.iter_mut() {}
609    /// ```
610    ///
611    /// Use instead:
612    /// ```no_run
613    /// let mut vec = vec![1, 2, 3];
614    /// let rmvec = &mut vec;
615    /// for _ in &*rmvec {}
616    /// for _ in &mut *rmvec {}
617    /// ```
618    #[lints(explicit_iter_loop)]
619    enforce_iter_loop_reborrow: bool = false,
620    /// The list of imports to always rename, a fully qualified path followed by the rename.
621    #[lints(missing_enforced_import_renames)]
622    enforced_import_renames: Vec<Rename> = Vec::new(),
623    /// The minimum number of enum variants for the lints about variant names to trigger
624    #[lints(enum_variant_names)]
625    enum_variant_name_threshold: u64 = 3,
626    /// The maximum size of an enum's variant to avoid box suggestion
627    #[lints(large_enum_variant)]
628    enum_variant_size_threshold: u64 = 200,
629    /// The maximum amount of nesting a block can reside in
630    #[lints(excessive_nesting)]
631    excessive_nesting_threshold: u64 = 0,
632    /// The maximum byte size a `Future` can have, before it triggers the `clippy::large_futures` lint
633    #[lints(large_futures)]
634    future_size_threshold: u64 = 16 * 1024,
635    /// A list of paths to types that should be treated as if they do not contain interior mutability
636    #[lints(borrow_interior_mutable_const, declare_interior_mutable_const, ifs_same_cond, mutable_key_type)]
637    ignore_interior_mutability: Vec<String> = Vec::from(["bytes::Bytes".into()]),
638    /// The maximum size of the `Err`-variant in a `Result` returned from a function
639    #[lints(result_large_err)]
640    large_error_threshold: u64 = 128,
641    /// Whether collapsible `if` chains are linted if they contain comments inside the parts
642    /// that would be collapsed.
643    #[lints(collapsible_if)]
644    lint_commented_code: bool = false,
645    /// Whether to suggest reordering constructor fields when initializers are present.
646    /// DEPRECATED CONFIGURATION: lint-inconsistent-struct-field-initializers
647    ///
648    /// Use the `check-inconsistent-struct-field-initializers` configuration instead.
649    #[conf_deprecated("Please use `check-inconsistent-struct-field-initializers` instead", check_inconsistent_struct_field_initializers)]
650    lint_inconsistent_struct_field_initializers: bool = false,
651    /// The lower bound for linting decimal literals
652    #[lints(decimal_literal_representation)]
653    literal_representation_threshold: u64 = 16384,
654    /// Whether the matches should be considered by the lint, and whether there should
655    /// be filtering for common types.
656    #[lints(manual_let_else)]
657    matches_for_let_else: MatchLintBehaviour = MatchLintBehaviour::WellKnownTypes,
658    /// The maximum number of bool parameters a function can have
659    #[lints(fn_params_excessive_bools)]
660    max_fn_params_bools: u64 = 3,
661    /// The maximum size of a file included via `include_bytes!()` or `include_str!()`, in bytes
662    #[lints(large_include_file)]
663    max_include_file_size: u64 = 1_000_000,
664    /// The maximum number of bool fields a struct can have
665    #[lints(struct_excessive_bools)]
666    max_struct_bools: u64 = 3,
667    /// When Clippy suggests using a slice pattern, this is the maximum number of elements allowed in
668    /// the slice pattern that is suggested. If more elements are necessary, the lint is suppressed.
669    /// For example, `[_, _, _, e, ..]` is a slice pattern with 4 elements.
670    #[lints(index_refutable_slice)]
671    max_suggested_slice_pattern_length: u64 = 3,
672    /// The maximum number of bounds a trait can have to be linted
673    #[lints(type_repetition_in_bounds)]
674    max_trait_bounds: u64 = 3,
675    /// Minimum chars an ident can have, anything below or equal to this will be linted.
676    #[lints(min_ident_chars)]
677    min_ident_chars_threshold: u64 = 1,
678    /// Whether to **only** check for missing documentation in items visible within the current
679    /// crate. For example, `pub(crate)` items.
680    #[lints(missing_docs_in_private_items)]
681    missing_docs_in_crate_items: bool = false,
682    /// The named groupings of different source item kinds within modules.
683    #[lints(arbitrary_source_item_ordering)]
684    module_item_order_groupings: SourceItemOrderingModuleItemGroupings = DEFAULT_MODULE_ITEM_ORDERING_GROUPS.into(),
685    /// Whether the items within module groups should be ordered alphabetically or not.
686    ///
687    /// This option can be configured to "all", "none", or a list of specific grouping names that should be checked
688    /// (e.g. only "enums").
689    #[lints(arbitrary_source_item_ordering)]
690    module_items_ordered_within_groupings: SourceItemOrderingWithinModuleItemGroupings =
691        SourceItemOrderingWithinModuleItemGroupings::None,
692    /// The minimum rust version that the project supports. Defaults to the `rust-version` field in `Cargo.toml`
693    #[default_text = "current version"]
694    #[lints(
695        allow_attributes,
696        allow_attributes_without_reason,
697        almost_complete_range,
698        approx_constant,
699        assigning_clones,
700        borrow_as_ptr,
701        cast_abs_to_unsigned,
702        checked_conversions,
703        cloned_instead_of_copied,
704        collapsible_match,
705        collapsible_str_replace,
706        deprecated_cfg_attr,
707        derivable_impls,
708        err_expect,
709        filter_map_next,
710        from_over_into,
711        if_then_some_else_none,
712        index_refutable_slice,
713        io_other_error,
714        iter_kv_map,
715        legacy_numeric_constants,
716        lines_filter_map_ok,
717        manual_abs_diff,
718        manual_bits,
719        manual_c_str_literals,
720        manual_clamp,
721        manual_div_ceil,
722        manual_flatten,
723        manual_hash_one,
724        manual_is_ascii_check,
725        manual_is_power_of_two,
726        manual_let_else,
727        manual_midpoint,
728        manual_non_exhaustive,
729        manual_option_as_slice,
730        manual_pattern_char_comparison,
731        manual_range_contains,
732        manual_rem_euclid,
733        manual_repeat_n,
734        manual_retain,
735        manual_slice_fill,
736        manual_split_once,
737        manual_str_repeat,
738        manual_strip,
739        manual_try_fold,
740        map_clone,
741        map_unwrap_or,
742        map_with_unused_argument_over_ranges,
743        match_like_matches_macro,
744        mem_replace_option_with_some,
745        mem_replace_with_default,
746        missing_const_for_fn,
747        needless_borrow,
748        non_std_lazy_statics,
749        option_as_ref_deref,
750        option_map_unwrap_or,
751        ptr_as_ptr,
752        question_mark,
753        redundant_field_names,
754        redundant_static_lifetimes,
755        repeat_vec_with_capacity,
756        same_item_push,
757        seek_from_current,
758        seek_rewind,
759        transmute_ptr_to_ref,
760        tuple_array_conversions,
761        type_repetition_in_bounds,
762        unchecked_duration_subtraction,
763        uninlined_format_args,
764        unnecessary_lazy_evaluations,
765        unnested_or_patterns,
766        unused_trait_names,
767        use_self,
768    )]
769    msrv: Msrv = Msrv::default(),
770    /// The minimum size (in bytes) to consider a type for passing by reference instead of by value.
771    #[lints(large_types_passed_by_value)]
772    pass_by_value_size_limit: u64 = 256,
773    /// Lint "public" fields in a struct that are prefixed with an underscore based on their
774    /// exported visibility, or whether they are marked as "pub".
775    #[lints(pub_underscore_fields)]
776    pub_underscore_fields_behavior: PubUnderscoreFieldsBehaviour = PubUnderscoreFieldsBehaviour::PubliclyExported,
777    /// Whether to lint only if it's multiline.
778    #[lints(semicolon_inside_block)]
779    semicolon_inside_block_ignore_singleline: bool = false,
780    /// Whether to lint only if it's singleline.
781    #[lints(semicolon_outside_block)]
782    semicolon_outside_block_ignore_multiline: bool = false,
783    /// The maximum number of single char bindings a scope may have
784    #[lints(many_single_char_names)]
785    single_char_binding_names_threshold: u64 = 4,
786    /// Which kind of elements should be ordered internally, possible values being `enum`, `impl`, `module`, `struct`, `trait`.
787    #[lints(arbitrary_source_item_ordering)]
788    source_item_ordering: SourceItemOrdering = DEFAULT_SOURCE_ITEM_ORDERING.into(),
789    /// The maximum allowed stack size for functions in bytes
790    #[lints(large_stack_frames)]
791    stack_size_threshold: u64 = 512_000,
792    /// Enforce the named macros always use the braces specified.
793    ///
794    /// A `MacroMatcher` can be added like so `{ name = "macro_name", brace = "(" }`. If the macro
795    /// could be used with a full path two `MacroMatcher`s have to be added one with the full path
796    /// `crate_name::macro_name` and one with just the macro name.
797    #[lints(nonstandard_macro_braces)]
798    standard_macro_braces: Vec<MacroMatcher> = Vec::new(),
799    /// The minimum number of struct fields for the lints about field names to trigger
800    #[lints(struct_field_names)]
801    struct_field_name_threshold: u64 = 3,
802    /// Whether to suppress a restriction lint in constant code. In same
803    /// cases the restructured operation might not be unavoidable, as the
804    /// suggested counterparts are unavailable in constant code. This
805    /// configuration will cause restriction lints to trigger even
806    /// if no suggestion can be made.
807    #[lints(indexing_slicing)]
808    suppress_restriction_lint_in_const: bool = false,
809    /// The maximum size of objects (in bytes) that will be linted. Larger objects are ok on the heap
810    #[lints(boxed_local, useless_vec)]
811    too_large_for_stack: u64 = 200,
812    /// The maximum number of argument a function or method can have
813    #[lints(too_many_arguments)]
814    too_many_arguments_threshold: u64 = 7,
815    /// The maximum number of lines a function or method can have
816    #[lints(too_many_lines)]
817    too_many_lines_threshold: u64 = 100,
818    /// The order of associated items in traits.
819    #[lints(arbitrary_source_item_ordering)]
820    trait_assoc_item_kinds_order: SourceItemOrderingTraitAssocItemKinds = DEFAULT_TRAIT_ASSOC_ITEM_KINDS_ORDER.into(),
821    /// The maximum size (in bytes) to consider a `Copy` type for passing by value instead of by
822    /// reference.
823    #[default_text = "target_pointer_width * 2"]
824    #[lints(trivially_copy_pass_by_ref)]
825    trivial_copy_size_limit: Option<u64> = None,
826    /// The maximum complexity a type can have
827    #[lints(type_complexity)]
828    type_complexity_threshold: u64 = 250,
829    /// The byte size a `T` in `Box<T>` can have, below which it triggers the `clippy::unnecessary_box` lint
830    #[lints(unnecessary_box_returns)]
831    unnecessary_box_size: u64 = 128,
832    /// Should the fraction of a decimal be linted to include separators.
833    #[lints(unreadable_literal)]
834    unreadable_literal_lint_fractions: bool = true,
835    /// Enables verbose mode. Triggers if there is more than one uppercase char next to each other
836    #[lints(upper_case_acronyms)]
837    upper_case_acronyms_aggressive: bool = false,
838    /// The size of the boxed type in bytes, where boxing in a `Vec` is allowed
839    #[lints(vec_box)]
840    vec_box_size_threshold: u64 = 4096,
841    /// The maximum allowed size of a bit mask before suggesting to use 'trailing_zeros'
842    #[lints(verbose_bit_mask)]
843    verbose_bit_mask_threshold: u64 = 1,
844    /// Whether to emit warnings on all wildcard imports, including those from `prelude`, from `super` in tests,
845    /// or for `pub use` reexports.
846    #[lints(wildcard_imports)]
847    warn_on_all_wildcard_imports: bool = false,
848    /// Whether to also emit warnings for unsafe blocks with metavariable expansions in **private** macros.
849    #[lints(macro_metavars_in_unsafe)]
850    warn_unsafe_macro_metavars_in_private_macros: bool = false,
851}
852
853/// Search for the configuration file.
854///
855/// # Errors
856///
857/// Returns any unexpected filesystem error encountered when searching for the config file
858pub fn lookup_conf_file() -> io::Result<(Option<PathBuf>, Vec<String>)> {
859    /// Possible filename to search for.
860    const CONFIG_FILE_NAMES: [&str; 2] = [".clippy.toml", "clippy.toml"];
861
862    // Start looking for a config file in CLIPPY_CONF_DIR, or failing that, CARGO_MANIFEST_DIR.
863    // If neither of those exist, use ".". (Update documentation if this priority changes)
864    let mut current = env::var_os("CLIPPY_CONF_DIR")
865        .or_else(|| env::var_os("CARGO_MANIFEST_DIR"))
866        .map_or_else(|| PathBuf::from("."), PathBuf::from)
867        .canonicalize()?;
868
869    let mut found_config: Option<PathBuf> = None;
870    let mut warnings = vec![];
871
872    loop {
873        for config_file_name in &CONFIG_FILE_NAMES {
874            if let Ok(config_file) = current.join(config_file_name).canonicalize() {
875                match fs::metadata(&config_file) {
876                    Err(e) if e.kind() == io::ErrorKind::NotFound => {},
877                    Err(e) => return Err(e),
878                    Ok(md) if md.is_dir() => {},
879                    Ok(_) => {
880                        // warn if we happen to find two config files #8323
881                        if let Some(ref found_config) = found_config {
882                            warnings.push(format!(
883                                "using config file `{}`, `{}` will be ignored",
884                                found_config.display(),
885                                config_file.display()
886                            ));
887                        } else {
888                            found_config = Some(config_file);
889                        }
890                    },
891                }
892            }
893        }
894
895        if found_config.is_some() {
896            return Ok((found_config, warnings));
897        }
898
899        // If the current directory has no parent, we're done searching.
900        if !current.pop() {
901            return Ok((None, warnings));
902        }
903    }
904}
905
906fn deserialize(file: &SourceFile) -> TryConf {
907    match toml::de::Deserializer::new(file.src.as_ref().unwrap()).deserialize_map(ConfVisitor(file)) {
908        Ok(mut conf) => {
909            extend_vec_if_indicator_present(&mut conf.conf.disallowed_names, DEFAULT_DISALLOWED_NAMES);
910            extend_vec_if_indicator_present(&mut conf.conf.allowed_prefixes, DEFAULT_ALLOWED_PREFIXES);
911            extend_vec_if_indicator_present(
912                &mut conf.conf.allow_renamed_params_for,
913                DEFAULT_ALLOWED_TRAITS_WITH_RENAMED_PARAMS,
914            );
915
916            // Confirms that the user has not accidentally configured ordering requirements for groups that
917            // aren't configured.
918            if let SourceItemOrderingWithinModuleItemGroupings::Custom(groupings) =
919                &conf.conf.module_items_ordered_within_groupings
920            {
921                for grouping in groupings {
922                    if !conf.conf.module_item_order_groupings.is_grouping(grouping) {
923                        // Since this isn't fixable by rustfix, don't emit a `Suggestion`. This just adds some useful
924                        // info for the user instead.
925
926                        let names = conf.conf.module_item_order_groupings.grouping_names();
927                        let suggestion = suggest_candidate(grouping, names.iter().map(String::as_str))
928                            .map(|s| format!(" perhaps you meant `{s}`?"))
929                            .unwrap_or_default();
930                        let names = names.iter().map(|s| format!("`{s}`")).join(", ");
931                        let message = format!(
932                            "unknown ordering group: `{grouping}` was not specified in `module-items-ordered-within-groupings`,{suggestion} expected one of: {names}"
933                        );
934
935                        let span = conf
936                            .value_spans
937                            .get("module_item_order_groupings")
938                            .cloned()
939                            .unwrap_or_default();
940                        conf.errors.push(ConfError::spanned(file, message, None, span));
941                    }
942                }
943            }
944
945            // TODO: THIS SHOULD BE TESTED, this comment will be gone soon
946            if conf.conf.allowed_idents_below_min_chars.iter().any(|e| e == "..") {
947                conf.conf
948                    .allowed_idents_below_min_chars
949                    .extend(DEFAULT_ALLOWED_IDENTS_BELOW_MIN_CHARS.iter().map(ToString::to_string));
950            }
951            if conf.conf.doc_valid_idents.iter().any(|e| e == "..") {
952                conf.conf
953                    .doc_valid_idents
954                    .extend(DEFAULT_DOC_VALID_IDENTS.iter().map(ToString::to_string));
955            }
956
957            conf
958        },
959        Err(e) => TryConf::from_toml_error(file, &e),
960    }
961}
962
963fn extend_vec_if_indicator_present(vec: &mut Vec<String>, default: &[&str]) {
964    if vec.contains(&"..".to_string()) {
965        vec.extend(default.iter().map(ToString::to_string));
966    }
967}
968
969impl Conf {
970    pub fn read(sess: &Session, path: &io::Result<(Option<PathBuf>, Vec<String>)>) -> &'static Conf {
971        static CONF: OnceLock<Conf> = OnceLock::new();
972        CONF.get_or_init(|| Conf::read_inner(sess, path))
973    }
974
975    fn read_inner(sess: &Session, path: &io::Result<(Option<PathBuf>, Vec<String>)>) -> Conf {
976        match path {
977            Ok((_, warnings)) => {
978                for warning in warnings {
979                    sess.dcx().warn(warning.clone());
980                }
981            },
982            Err(error) => {
983                sess.dcx()
984                    .err(format!("error finding Clippy's configuration file: {error}"));
985            },
986        }
987
988        let TryConf {
989            mut conf,
990            value_spans: _,
991            errors,
992            warnings,
993        } = match path {
994            Ok((Some(path), _)) => match sess.source_map().load_file(path) {
995                Ok(file) => deserialize(&file),
996                Err(error) => {
997                    sess.dcx().err(format!("failed to read `{}`: {error}", path.display()));
998                    TryConf::default()
999                },
1000            },
1001            _ => TryConf::default(),
1002        };
1003
1004        conf.msrv.read_cargo(sess);
1005
1006        // all conf errors are non-fatal, we just use the default conf in case of error
1007        for error in errors {
1008            let mut diag = sess.dcx().struct_span_err(
1009                error.span,
1010                format!("error reading Clippy's configuration file: {}", error.message),
1011            );
1012
1013            if let Some(sugg) = error.suggestion {
1014                diag.span_suggestion(error.span, sugg.message, sugg.suggestion, Applicability::MaybeIncorrect);
1015            }
1016
1017            diag.emit();
1018        }
1019
1020        for warning in warnings {
1021            sess.dcx().span_warn(
1022                warning.span,
1023                format!("error reading Clippy's configuration file: {}", warning.message),
1024            );
1025        }
1026
1027        conf
1028    }
1029}
1030
1031const SEPARATOR_WIDTH: usize = 4;
1032
1033#[derive(Debug)]
1034struct FieldError {
1035    error: String,
1036    suggestion: Option<Suggestion>,
1037}
1038
1039#[derive(Debug)]
1040struct Suggestion {
1041    message: &'static str,
1042    suggestion: &'static str,
1043}
1044
1045impl std::error::Error for FieldError {}
1046
1047impl Display for FieldError {
1048    fn fmt(&self, f: &mut Formatter<'_>) -> fmt::Result {
1049        f.pad(&self.error)
1050    }
1051}
1052
1053impl serde::de::Error for FieldError {
1054    fn custom<T: Display>(msg: T) -> Self {
1055        Self {
1056            error: msg.to_string(),
1057            suggestion: None,
1058        }
1059    }
1060
1061    fn unknown_field(field: &str, expected: &'static [&'static str]) -> Self {
1062        // List the available fields sorted and at least one per line, more if `CLIPPY_TERMINAL_WIDTH` is
1063        // set and allows it.
1064        use fmt::Write;
1065
1066        let metadata = get_configuration_metadata();
1067        let deprecated = metadata
1068            .iter()
1069            .filter_map(|conf| {
1070                if conf.deprecation_reason.is_some() {
1071                    Some(conf.name.as_str())
1072                } else {
1073                    None
1074                }
1075            })
1076            .collect::<Vec<_>>();
1077
1078        let mut expected = expected
1079            .iter()
1080            .copied()
1081            .filter(|name| !deprecated.contains(name))
1082            .collect::<Vec<_>>();
1083        expected.sort_unstable();
1084
1085        let (rows, column_widths) = calculate_dimensions(&expected);
1086
1087        let mut msg = format!("unknown field `{field}`, expected one of");
1088        for row in 0..rows {
1089            writeln!(msg).unwrap();
1090            for (column, column_width) in column_widths.iter().copied().enumerate() {
1091                let index = column * rows + row;
1092                let field = expected.get(index).copied().unwrap_or_default();
1093                write!(msg, "{:SEPARATOR_WIDTH$}{field:column_width$}", " ").unwrap();
1094            }
1095        }
1096
1097        let suggestion = suggest_candidate(field, expected).map(|suggestion| Suggestion {
1098            message: "perhaps you meant",
1099            suggestion,
1100        });
1101
1102        Self { error: msg, suggestion }
1103    }
1104}
1105
1106fn calculate_dimensions(fields: &[&str]) -> (usize, Vec<usize>) {
1107    let columns = env::var("CLIPPY_TERMINAL_WIDTH")
1108        .ok()
1109        .and_then(|s| <usize as FromStr>::from_str(&s).ok())
1110        .map_or(1, |terminal_width| {
1111            let max_field_width = fields.iter().map(|field| field.len()).max().unwrap();
1112            cmp::max(1, terminal_width / (SEPARATOR_WIDTH + max_field_width))
1113        });
1114
1115    let rows = fields.len().div_ceil(columns);
1116
1117    let column_widths = (0..columns)
1118        .map(|column| {
1119            if column < columns - 1 {
1120                (0..rows)
1121                    .map(|row| {
1122                        let index = column * rows + row;
1123                        let field = fields.get(index).copied().unwrap_or_default();
1124                        field.len()
1125                    })
1126                    .max()
1127                    .unwrap()
1128            } else {
1129                // Avoid adding extra space to the last column.
1130                0
1131            }
1132        })
1133        .collect::<Vec<_>>();
1134
1135    (rows, column_widths)
1136}
1137
1138/// Given a user-provided value that couldn't be matched to a known option, finds the most likely
1139/// candidate among candidates that the user might have meant.
1140fn suggest_candidate<'a, I>(value: &str, candidates: I) -> Option<&'a str>
1141where
1142    I: IntoIterator<Item = &'a str>,
1143{
1144    candidates
1145        .into_iter()
1146        .filter_map(|expected| {
1147            let dist = edit_distance(value, expected, 4)?;
1148            Some((dist, expected))
1149        })
1150        .min_by_key(|&(dist, _)| dist)
1151        .map(|(_, suggestion)| suggestion)
1152}
1153
1154#[cfg(test)]
1155mod tests {
1156    use serde::de::IgnoredAny;
1157    use std::collections::{HashMap, HashSet};
1158    use std::fs;
1159    use walkdir::WalkDir;
1160
1161    #[test]
1162    fn configs_are_tested() {
1163        let mut names: HashSet<String> = crate::get_configuration_metadata()
1164            .into_iter()
1165            .filter_map(|meta| {
1166                if meta.deprecation_reason.is_none() {
1167                    Some(meta.name.replace('_', "-"))
1168                } else {
1169                    None
1170                }
1171            })
1172            .collect();
1173
1174        let toml_files = WalkDir::new("../tests")
1175            .into_iter()
1176            .map(Result::unwrap)
1177            .filter(|entry| entry.file_name() == "clippy.toml");
1178
1179        for entry in toml_files {
1180            let file = fs::read_to_string(entry.path()).unwrap();
1181            #[allow(clippy::zero_sized_map_values)]
1182            if let Ok(map) = toml::from_str::<HashMap<String, IgnoredAny>>(&file) {
1183                for name in map.keys() {
1184                    names.remove(name.as_str());
1185                }
1186            }
1187        }
1188
1189        assert!(
1190            names.is_empty(),
1191            "Configuration variable lacks test: {names:?}\nAdd a test to `tests/ui-toml`"
1192        );
1193    }
1194}