]> git.lizzy.rs Git - rust.git/blob - src/tools/clippy/clippy_lints/src/utils/conf.rs
Merge commit 'bf1c6f9871f430e284b17aa44059e0d0395e28a6' into clippyup
[rust.git] / src / tools / clippy / 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: TOO_MANY_LINES. The maximum number of lines a function or method can have
154     (too_many_lines_threshold, "too_many_lines_threshold": u64, 100),
155     /// Lint: LARGE_STACK_ARRAYS, LARGE_CONST_ARRAYS. The maximum allowed size for arrays on the stack
156     (array_size_threshold, "array_size_threshold": u64, 512_000),
157     /// Lint: VEC_BOX. The size of the boxed type in bytes, where boxing in a `Vec` is allowed
158     (vec_box_size_threshold, "vec_box_size_threshold": u64, 4096),
159     /// Lint: TYPE_REPETITION_IN_BOUNDS. The maximum number of bounds a trait can have to be linted
160     (max_trait_bounds, "max_trait_bounds": u64, 3),
161     /// Lint: STRUCT_EXCESSIVE_BOOLS. The maximum number of bools a struct can have
162     (max_struct_bools, "max_struct_bools": u64, 3),
163     /// Lint: FN_PARAMS_EXCESSIVE_BOOLS. The maximum number of bools function parameters can have
164     (max_fn_params_bools, "max_fn_params_bools": u64, 3),
165     /// Lint: WILDCARD_IMPORTS. Whether to allow certain wildcard imports (prelude, super in tests).
166     (warn_on_all_wildcard_imports, "warn_on_all_wildcard_imports": bool, false),
167     /// Lint: DISALLOWED_METHOD. The list of blacklisted methods to lint about. NB: `bar` is not here since it has legitimate uses
168     (disallowed_methods, "disallowed_methods": Vec<String>, Vec::<String>::new()),
169 }
170
171 impl Default for Conf {
172     #[must_use]
173     fn default() -> Self {
174         toml::from_str("").expect("we never error on empty config files")
175     }
176 }
177
178 /// Search for the configuration file.
179 pub fn lookup_conf_file() -> io::Result<Option<PathBuf>> {
180     /// Possible filename to search for.
181     const CONFIG_FILE_NAMES: [&str; 2] = [".clippy.toml", "clippy.toml"];
182
183     // Start looking for a config file in CLIPPY_CONF_DIR, or failing that, CARGO_MANIFEST_DIR.
184     // If neither of those exist, use ".".
185     let mut current = env::var_os("CLIPPY_CONF_DIR")
186         .or_else(|| env::var_os("CARGO_MANIFEST_DIR"))
187         .map_or_else(|| PathBuf::from("."), PathBuf::from);
188     loop {
189         for config_file_name in &CONFIG_FILE_NAMES {
190             let config_file = current.join(config_file_name);
191             match fs::metadata(&config_file) {
192                 // Only return if it's a file to handle the unlikely situation of a directory named
193                 // `clippy.toml`.
194                 Ok(ref md) if !md.is_dir() => return Ok(Some(config_file)),
195                 // Return the error if it's something other than `NotFound`; otherwise we didn't
196                 // find the project file yet, and continue searching.
197                 Err(e) if e.kind() != io::ErrorKind::NotFound => return Err(e),
198                 _ => {},
199             }
200         }
201
202         // If the current directory has no parent, we're done searching.
203         if !current.pop() {
204             return Ok(None);
205         }
206     }
207 }
208
209 /// Produces a `Conf` filled with the default values and forwards the errors
210 ///
211 /// Used internally for convenience
212 fn default(errors: Vec<Error>) -> (Conf, Vec<Error>) {
213     (Conf::default(), errors)
214 }
215
216 /// Read the `toml` configuration file.
217 ///
218 /// In case of error, the function tries to continue as much as possible.
219 pub fn read(path: &Path) -> (Conf, Vec<Error>) {
220     let content = match fs::read_to_string(path) {
221         Ok(content) => content,
222         Err(err) => return default(vec![err.into()]),
223     };
224
225     assert!(ERRORS.lock().expect("no threading -> mutex always safe").is_empty());
226     match toml::from_str(&content) {
227         Ok(toml) => {
228             let mut errors = ERRORS.lock().expect("no threading -> mutex always safe").split_off(0);
229
230             let toml_ref: &Conf = &toml;
231
232             let cyc_field: Option<u64> = toml_ref.cyclomatic_complexity_threshold;
233
234             if cyc_field.is_some() {
235                 let cyc_err = "found deprecated field `cyclomatic-complexity-threshold`. Please use `cognitive-complexity-threshold` instead.".to_string();
236                 errors.push(Error::Toml(cyc_err));
237             }
238
239             (toml, errors)
240         },
241         Err(e) => {
242             let mut errors = ERRORS.lock().expect("no threading -> mutex always safe").split_off(0);
243             errors.push(Error::Toml(e.to_string()));
244
245             default(errors)
246         },
247     }
248 }