]> git.lizzy.rs Git - rust.git/blob - clippy_lints/src/utils/conf.rs
Auto merge of #6197 - ThibsG:ImproveFilterNext, r=ebroto
[rust.git] / clippy_lints / src / utils / conf.rs
1 //! Read configurations files.
2
3 #![deny(clippy::missing_docs_in_private_items)]
4
5 use rustc_ast::ast::{LitKind, MetaItemKind, NestedMetaItem};
6 use rustc_span::source_map;
7 use source_map::Span;
8 use std::lazy::SyncLazy;
9 use std::path::{Path, PathBuf};
10 use std::sync::Mutex;
11 use std::{env, fmt, fs, io};
12
13 /// Gets the configuration file from arguments.
14 pub fn file_from_args(args: &[NestedMetaItem]) -> Result<Option<PathBuf>, (&'static str, Span)> {
15     for arg in args.iter().filter_map(NestedMetaItem::meta_item) {
16         if arg.has_name(sym!(conf_file)) {
17             return match arg.kind {
18                 MetaItemKind::Word | MetaItemKind::List(_) => Err(("`conf_file` must be a named value", arg.span)),
19                 MetaItemKind::NameValue(ref value) => {
20                     if let LitKind::Str(ref file, _) = value.kind {
21                         Ok(Some(file.to_string().into()))
22                     } else {
23                         Err(("`conf_file` value must be a string", value.span))
24                     }
25                 },
26             };
27         }
28     }
29
30     Ok(None)
31 }
32
33 /// Error from reading a configuration file.
34 #[derive(Debug)]
35 pub enum Error {
36     /// An I/O error.
37     Io(io::Error),
38     /// Not valid toml or doesn't fit the expected config format
39     Toml(String),
40 }
41
42 impl fmt::Display for Error {
43     fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
44         match self {
45             Self::Io(err) => err.fmt(f),
46             Self::Toml(err) => err.fmt(f),
47         }
48     }
49 }
50
51 impl From<io::Error> for Error {
52     fn from(e: io::Error) -> Self {
53         Self::Io(e)
54     }
55 }
56
57 /// Vec of errors that might be collected during config toml parsing
58 static ERRORS: SyncLazy<Mutex<Vec<Error>>> = SyncLazy::new(|| Mutex::new(Vec::new()));
59
60 macro_rules! define_Conf {
61     ($(#[$doc:meta] ($config:ident, $config_str:literal: $Ty:ty, $default:expr),)+) => {
62         mod helpers {
63             use serde::Deserialize;
64             /// Type used to store lint configuration.
65             #[derive(Deserialize)]
66             #[serde(rename_all = "kebab-case", deny_unknown_fields)]
67             pub struct Conf {
68                 $(
69                     #[$doc]
70                     #[serde(default = $config_str)]
71                     #[serde(with = $config_str)]
72                     pub $config: $Ty,
73                 )+
74                 #[allow(dead_code)]
75                 #[serde(default)]
76                 third_party: Option<::toml::Value>,
77             }
78
79             $(
80                 mod $config {
81                     use serde::Deserialize;
82                     pub fn deserialize<'de, D: serde::Deserializer<'de>>(deserializer: D) -> Result<$Ty, D::Error> {
83                         use super::super::{ERRORS, Error};
84
85                         Ok(
86                             <$Ty>::deserialize(deserializer).unwrap_or_else(|e| {
87                                 ERRORS
88                                     .lock()
89                                     .expect("no threading here")
90                                     .push(Error::Toml(e.to_string()));
91                                 super::$config()
92                             })
93                         )
94                     }
95                 }
96
97                 #[must_use]
98                 fn $config() -> $Ty {
99                     let x = $default;
100                     x
101                 }
102             )+
103         }
104     };
105 }
106
107 pub use self::helpers::Conf;
108 define_Conf! {
109     /// Lint: BLACKLISTED_NAME. The list of blacklisted names to lint about. NB: `bar` is not here since it has legitimate uses
110     (blacklisted_names, "blacklisted_names": Vec<String>, ["foo", "baz", "quux"].iter().map(ToString::to_string).collect()),
111     /// Lint: COGNITIVE_COMPLEXITY. The maximum cognitive complexity a function can have
112     (cognitive_complexity_threshold, "cognitive_complexity_threshold": u64, 25),
113     /// DEPRECATED LINT: CYCLOMATIC_COMPLEXITY. Use the Cognitive Complexity lint instead.
114     (cyclomatic_complexity_threshold, "cyclomatic_complexity_threshold": Option<u64>, None),
115     /// Lint: DOC_MARKDOWN. The list of words this lint should not consider as identifiers needing ticks
116     (doc_valid_idents, "doc_valid_idents": Vec<String>, [
117         "KiB", "MiB", "GiB", "TiB", "PiB", "EiB",
118         "DirectX",
119         "ECMAScript",
120         "GPLv2", "GPLv3",
121         "GitHub", "GitLab",
122         "IPv4", "IPv6",
123         "ClojureScript", "CoffeeScript", "JavaScript", "PureScript", "TypeScript",
124         "NaN", "NaNs",
125         "OAuth", "GraphQL",
126         "OCaml",
127         "OpenGL", "OpenMP", "OpenSSH", "OpenSSL", "OpenStreetMap",
128         "TensorFlow",
129         "TrueType",
130         "iOS", "macOS",
131         "TeX", "LaTeX", "BibTeX", "BibLaTeX",
132         "MinGW",
133         "CamelCase",
134     ].iter().map(ToString::to_string).collect()),
135     /// Lint: TOO_MANY_ARGUMENTS. The maximum number of argument a function or method can have
136     (too_many_arguments_threshold, "too_many_arguments_threshold": u64, 7),
137     /// Lint: TYPE_COMPLEXITY. The maximum complexity a type can have
138     (type_complexity_threshold, "type_complexity_threshold": u64, 250),
139     /// Lint: MANY_SINGLE_CHAR_NAMES. The maximum number of single char bindings a scope may have
140     (single_char_binding_names_threshold, "single_char_binding_names_threshold": u64, 4),
141     /// Lint: BOXED_LOCAL, USELESS_VEC. The maximum size of objects (in bytes) that will be linted. Larger objects are ok on the heap
142     (too_large_for_stack, "too_large_for_stack": u64, 200),
143     /// Lint: ENUM_VARIANT_NAMES. The minimum number of enum variants for the lints about variant names to trigger
144     (enum_variant_name_threshold, "enum_variant_name_threshold": u64, 3),
145     /// Lint: LARGE_ENUM_VARIANT. The maximum size of a enum's variant to avoid box suggestion
146     (enum_variant_size_threshold, "enum_variant_size_threshold": u64, 200),
147     /// Lint: VERBOSE_BIT_MASK. The maximum allowed size of a bit mask before suggesting to use 'trailing_zeros'
148     (verbose_bit_mask_threshold, "verbose_bit_mask_threshold": u64, 1),
149     /// Lint: DECIMAL_LITERAL_REPRESENTATION. The lower bound for linting decimal literals
150     (literal_representation_threshold, "literal_representation_threshold": u64, 16384),
151     /// Lint: TRIVIALLY_COPY_PASS_BY_REF. The maximum size (in bytes) to consider a `Copy` type for passing by value instead of by reference.
152     (trivial_copy_size_limit, "trivial_copy_size_limit": Option<u64>, None),
153     /// Lint: LARGE_TYPE_PASS_BY_MOVE. The minimum size (in bytes) to consider a type for passing by reference instead of by value.
154     (pass_by_value_size_limit, "pass_by_value_size_limit": u64, 256),
155     /// Lint: TOO_MANY_LINES. The maximum number of lines a function or method can have
156     (too_many_lines_threshold, "too_many_lines_threshold": u64, 100),
157     /// Lint: LARGE_STACK_ARRAYS, LARGE_CONST_ARRAYS. The maximum allowed size for arrays on the stack
158     (array_size_threshold, "array_size_threshold": u64, 512_000),
159     /// Lint: VEC_BOX. The size of the boxed type in bytes, where boxing in a `Vec` is allowed
160     (vec_box_size_threshold, "vec_box_size_threshold": u64, 4096),
161     /// Lint: TYPE_REPETITION_IN_BOUNDS. The maximum number of bounds a trait can have to be linted
162     (max_trait_bounds, "max_trait_bounds": u64, 3),
163     /// Lint: STRUCT_EXCESSIVE_BOOLS. The maximum number of bools a struct can have
164     (max_struct_bools, "max_struct_bools": u64, 3),
165     /// Lint: FN_PARAMS_EXCESSIVE_BOOLS. The maximum number of bools function parameters can have
166     (max_fn_params_bools, "max_fn_params_bools": u64, 3),
167     /// Lint: WILDCARD_IMPORTS. Whether to allow certain wildcard imports (prelude, super in tests).
168     (warn_on_all_wildcard_imports, "warn_on_all_wildcard_imports": bool, false),
169     /// Lint: DISALLOWED_METHOD. The list of blacklisted methods to lint about. NB: `bar` is not here since it has legitimate uses
170     (disallowed_methods, "disallowed_methods": Vec<String>, Vec::<String>::new()),
171 }
172
173 impl Default for Conf {
174     #[must_use]
175     fn default() -> Self {
176         toml::from_str("").expect("we never error on empty config files")
177     }
178 }
179
180 /// Search for the configuration file.
181 pub fn lookup_conf_file() -> io::Result<Option<PathBuf>> {
182     /// Possible filename to search for.
183     const CONFIG_FILE_NAMES: [&str; 2] = [".clippy.toml", "clippy.toml"];
184
185     // Start looking for a config file in CLIPPY_CONF_DIR, or failing that, CARGO_MANIFEST_DIR.
186     // If neither of those exist, use ".".
187     let mut current = env::var_os("CLIPPY_CONF_DIR")
188         .or_else(|| env::var_os("CARGO_MANIFEST_DIR"))
189         .map_or_else(|| PathBuf::from("."), PathBuf::from);
190     loop {
191         for config_file_name in &CONFIG_FILE_NAMES {
192             let config_file = current.join(config_file_name);
193             match fs::metadata(&config_file) {
194                 // Only return if it's a file to handle the unlikely situation of a directory named
195                 // `clippy.toml`.
196                 Ok(ref md) if !md.is_dir() => return Ok(Some(config_file)),
197                 // Return the error if it's something other than `NotFound`; otherwise we didn't
198                 // find the project file yet, and continue searching.
199                 Err(e) if e.kind() != io::ErrorKind::NotFound => return Err(e),
200                 _ => {},
201             }
202         }
203
204         // If the current directory has no parent, we're done searching.
205         if !current.pop() {
206             return Ok(None);
207         }
208     }
209 }
210
211 /// Produces a `Conf` filled with the default values and forwards the errors
212 ///
213 /// Used internally for convenience
214 fn default(errors: Vec<Error>) -> (Conf, Vec<Error>) {
215     (Conf::default(), errors)
216 }
217
218 /// Read the `toml` configuration file.
219 ///
220 /// In case of error, the function tries to continue as much as possible.
221 pub fn read(path: &Path) -> (Conf, Vec<Error>) {
222     let content = match fs::read_to_string(path) {
223         Ok(content) => content,
224         Err(err) => return default(vec![err.into()]),
225     };
226
227     assert!(ERRORS.lock().expect("no threading -> mutex always safe").is_empty());
228     match toml::from_str(&content) {
229         Ok(toml) => {
230             let mut errors = ERRORS.lock().expect("no threading -> mutex always safe").split_off(0);
231
232             let toml_ref: &Conf = &toml;
233
234             let cyc_field: Option<u64> = toml_ref.cyclomatic_complexity_threshold;
235
236             if cyc_field.is_some() {
237                 let cyc_err = "found deprecated field `cyclomatic-complexity-threshold`. Please use `cognitive-complexity-threshold` instead.".to_string();
238                 errors.push(Error::Toml(cyc_err));
239             }
240
241             (toml, errors)
242         },
243         Err(e) => {
244             let mut errors = ERRORS.lock().expect("no threading -> mutex always safe").split_off(0);
245             errors.push(Error::Toml(e.to_string()));
246
247             default(errors)
248         },
249     }
250 }