]> git.lizzy.rs Git - rust.git/blob - clippy_lints/src/lib.rs
Auto merge of #8228 - Jarcho:iter_not_returning_iterator_8225, r=giraffate
[rust.git] / clippy_lints / src / lib.rs
1 // error-pattern:cargo-clippy
2
3 #![feature(binary_heap_into_iter_sorted)]
4 #![feature(box_patterns)]
5 #![feature(control_flow_enum)]
6 #![feature(drain_filter)]
7 #![feature(in_band_lifetimes)]
8 #![feature(iter_intersperse)]
9 #![feature(let_else)]
10 #![feature(once_cell)]
11 #![feature(rustc_private)]
12 #![feature(stmt_expr_attributes)]
13 #![recursion_limit = "512"]
14 #![cfg_attr(feature = "deny-warnings", deny(warnings))]
15 #![allow(clippy::missing_docs_in_private_items, clippy::must_use_candidate)]
16 #![warn(trivial_casts, trivial_numeric_casts)]
17 // warn on lints, that are included in `rust-lang/rust`s bootstrap
18 #![warn(rust_2018_idioms, unused_lifetimes)]
19 // warn on rustc internal lints
20 #![warn(rustc::internal)]
21
22 // FIXME: switch to something more ergonomic here, once available.
23 // (Currently there is no way to opt into sysroot crates without `extern crate`.)
24 extern crate rustc_ast;
25 extern crate rustc_ast_pretty;
26 extern crate rustc_data_structures;
27 extern crate rustc_driver;
28 extern crate rustc_errors;
29 extern crate rustc_hir;
30 extern crate rustc_hir_pretty;
31 extern crate rustc_index;
32 extern crate rustc_infer;
33 extern crate rustc_lexer;
34 extern crate rustc_lint;
35 extern crate rustc_middle;
36 extern crate rustc_mir_dataflow;
37 extern crate rustc_parse;
38 extern crate rustc_parse_format;
39 extern crate rustc_session;
40 extern crate rustc_span;
41 extern crate rustc_target;
42 extern crate rustc_trait_selection;
43 extern crate rustc_typeck;
44
45 #[macro_use]
46 extern crate clippy_utils;
47
48 use clippy_utils::parse_msrv;
49 use rustc_data_structures::fx::FxHashSet;
50 use rustc_lint::LintId;
51 use rustc_session::Session;
52
53 /// Macro used to declare a Clippy lint.
54 ///
55 /// Every lint declaration consists of 4 parts:
56 ///
57 /// 1. The documentation, which is used for the website
58 /// 2. The `LINT_NAME`. See [lint naming][lint_naming] on lint naming conventions.
59 /// 3. The `lint_level`, which is a mapping from *one* of our lint groups to `Allow`, `Warn` or
60 ///    `Deny`. The lint level here has nothing to do with what lint groups the lint is a part of.
61 /// 4. The `description` that contains a short explanation on what's wrong with code where the
62 ///    lint is triggered.
63 ///
64 /// Currently the categories `style`, `correctness`, `suspicious`, `complexity` and `perf` are
65 /// enabled by default. As said in the README.md of this repository, if the lint level mapping
66 /// changes, please update README.md.
67 ///
68 /// # Example
69 ///
70 /// ```
71 /// #![feature(rustc_private)]
72 /// extern crate rustc_session;
73 /// use rustc_session::declare_tool_lint;
74 /// use clippy_lints::declare_clippy_lint;
75 ///
76 /// declare_clippy_lint! {
77 ///     /// ### What it does
78 ///     /// Checks for ... (describe what the lint matches).
79 ///     ///
80 ///     /// ### Why is this bad?
81 ///     /// Supply the reason for linting the code.
82 ///     ///
83 ///     /// ### Example
84 ///     /// ```rust
85 ///     /// // Bad
86 ///     /// Insert a short example of code that triggers the lint
87 ///     ///
88 ///     /// // Good
89 ///     /// Insert a short example of improved code that doesn't trigger the lint
90 ///     /// ```
91 ///     pub LINT_NAME,
92 ///     pedantic,
93 ///     "description"
94 /// }
95 /// ```
96 /// [lint_naming]: https://rust-lang.github.io/rfcs/0344-conventions-galore.html#lints
97 #[macro_export]
98 macro_rules! declare_clippy_lint {
99     { $(#[$attr:meta])* pub $name:tt, style, $description:tt } => {
100         declare_tool_lint! {
101             $(#[$attr])* pub clippy::$name, Warn, $description, report_in_external_macro: true
102         }
103     };
104     { $(#[$attr:meta])* pub $name:tt, correctness, $description:tt } => {
105         declare_tool_lint! {
106             $(#[$attr])* pub clippy::$name, Deny, $description, report_in_external_macro: true
107         }
108     };
109     { $(#[$attr:meta])* pub $name:tt, suspicious, $description:tt } => {
110         declare_tool_lint! {
111             $(#[$attr])* pub clippy::$name, Warn, $description, report_in_external_macro: true
112         }
113     };
114     { $(#[$attr:meta])* pub $name:tt, complexity, $description:tt } => {
115         declare_tool_lint! {
116             $(#[$attr])* pub clippy::$name, Warn, $description, report_in_external_macro: true
117         }
118     };
119     { $(#[$attr:meta])* pub $name:tt, perf, $description:tt } => {
120         declare_tool_lint! {
121             $(#[$attr])* pub clippy::$name, Warn, $description, report_in_external_macro: true
122         }
123     };
124     { $(#[$attr:meta])* pub $name:tt, pedantic, $description:tt } => {
125         declare_tool_lint! {
126             $(#[$attr])* pub clippy::$name, Allow, $description, report_in_external_macro: true
127         }
128     };
129     { $(#[$attr:meta])* pub $name:tt, restriction, $description:tt } => {
130         declare_tool_lint! {
131             $(#[$attr])* pub clippy::$name, Allow, $description, report_in_external_macro: true
132         }
133     };
134     { $(#[$attr:meta])* pub $name:tt, cargo, $description:tt } => {
135         declare_tool_lint! {
136             $(#[$attr])* pub clippy::$name, Allow, $description, report_in_external_macro: true
137         }
138     };
139     { $(#[$attr:meta])* pub $name:tt, nursery, $description:tt } => {
140         declare_tool_lint! {
141             $(#[$attr])* pub clippy::$name, Allow, $description, report_in_external_macro: true
142         }
143     };
144     { $(#[$attr:meta])* pub $name:tt, internal, $description:tt } => {
145         declare_tool_lint! {
146             $(#[$attr])* pub clippy::$name, Allow, $description, report_in_external_macro: true
147         }
148     };
149     { $(#[$attr:meta])* pub $name:tt, internal_warn, $description:tt } => {
150         declare_tool_lint! {
151             $(#[$attr])* pub clippy::$name, Warn, $description, report_in_external_macro: true
152         }
153     };
154 }
155
156 #[cfg(feature = "internal")]
157 mod deprecated_lints;
158 #[cfg_attr(feature = "internal", allow(clippy::missing_clippy_version_attribute))]
159 mod utils;
160
161 // begin lints modules, do not remove this comment, it’s used in `update_lints`
162 mod absurd_extreme_comparisons;
163 mod approx_const;
164 mod arithmetic;
165 mod as_conversions;
166 mod asm_syntax;
167 mod assertions_on_constants;
168 mod assign_ops;
169 mod async_yields_async;
170 mod attrs;
171 mod await_holding_invalid;
172 mod bit_mask;
173 mod blacklisted_name;
174 mod blocks_in_if_conditions;
175 mod bool_assert_comparison;
176 mod booleans;
177 mod bytecount;
178 mod cargo_common_metadata;
179 mod case_sensitive_file_extension_comparisons;
180 mod casts;
181 mod checked_conversions;
182 mod cognitive_complexity;
183 mod collapsible_if;
184 mod collapsible_match;
185 mod comparison_chain;
186 mod copies;
187 mod copy_iterator;
188 mod create_dir;
189 mod dbg_macro;
190 mod default;
191 mod default_numeric_fallback;
192 mod dereference;
193 mod derivable_impls;
194 mod derive;
195 mod disallowed_methods;
196 mod disallowed_script_idents;
197 mod disallowed_types;
198 mod doc;
199 mod double_comparison;
200 mod double_parens;
201 mod drop_forget_ref;
202 mod duration_subsec;
203 mod else_if_without_else;
204 mod empty_enum;
205 mod entry;
206 mod enum_clike;
207 mod enum_variants;
208 mod eq_op;
209 mod equatable_if_let;
210 mod erasing_op;
211 mod escape;
212 mod eta_reduction;
213 mod eval_order_dependence;
214 mod excessive_bools;
215 mod exhaustive_items;
216 mod exit;
217 mod explicit_write;
218 mod fallible_impl_from;
219 mod feature_name;
220 mod float_equality_without_abs;
221 mod float_literal;
222 mod floating_point_arithmetic;
223 mod format;
224 mod format_args;
225 mod formatting;
226 mod from_over_into;
227 mod from_str_radix_10;
228 mod functions;
229 mod future_not_send;
230 mod get_last_with_len;
231 mod identity_op;
232 mod if_let_mutex;
233 mod if_not_else;
234 mod if_then_some_else_none;
235 mod implicit_hasher;
236 mod implicit_return;
237 mod implicit_saturating_sub;
238 mod inconsistent_struct_constructor;
239 mod index_refutable_slice;
240 mod indexing_slicing;
241 mod infinite_iter;
242 mod inherent_impl;
243 mod inherent_to_string;
244 mod init_numbered_fields;
245 mod inline_fn_without_body;
246 mod int_plus_one;
247 mod integer_division;
248 mod invalid_upcast_comparisons;
249 mod items_after_statements;
250 mod iter_not_returning_iterator;
251 mod large_const_arrays;
252 mod large_enum_variant;
253 mod large_stack_arrays;
254 mod len_zero;
255 mod let_if_seq;
256 mod let_underscore;
257 mod lifetimes;
258 mod literal_representation;
259 mod loops;
260 mod macro_use;
261 mod main_recursion;
262 mod manual_assert;
263 mod manual_async_fn;
264 mod manual_map;
265 mod manual_non_exhaustive;
266 mod manual_ok_or;
267 mod manual_strip;
268 mod manual_unwrap_or;
269 mod map_clone;
270 mod map_err_ignore;
271 mod map_unit_fn;
272 mod match_on_vec_items;
273 mod match_result_ok;
274 mod match_str_case_mismatch;
275 mod matches;
276 mod mem_forget;
277 mod mem_replace;
278 mod methods;
279 mod minmax;
280 mod misc;
281 mod misc_early;
282 mod missing_const_for_fn;
283 mod missing_doc;
284 mod missing_enforced_import_rename;
285 mod missing_inline;
286 mod module_style;
287 mod modulo_arithmetic;
288 mod multiple_crate_versions;
289 mod mut_key;
290 mod mut_mut;
291 mod mut_mutex_lock;
292 mod mut_reference;
293 mod mutable_debug_assertion;
294 mod mutex_atomic;
295 mod needless_arbitrary_self_type;
296 mod needless_bitwise_bool;
297 mod needless_bool;
298 mod needless_borrowed_ref;
299 mod needless_continue;
300 mod needless_for_each;
301 mod needless_late_init;
302 mod needless_option_as_deref;
303 mod needless_pass_by_value;
304 mod needless_question_mark;
305 mod needless_update;
306 mod neg_cmp_op_on_partial_ord;
307 mod neg_multiply;
308 mod new_without_default;
309 mod no_effect;
310 mod non_copy_const;
311 mod non_expressive_names;
312 mod non_octal_unix_permissions;
313 mod non_send_fields_in_send_ty;
314 mod nonstandard_macro_braces;
315 mod octal_escapes;
316 mod open_options;
317 mod option_env_unwrap;
318 mod option_if_let_else;
319 mod overflow_check_conditional;
320 mod panic_in_result_fn;
321 mod panic_unimplemented;
322 mod partialeq_ne_impl;
323 mod pass_by_ref_or_value;
324 mod path_buf_push_overwrite;
325 mod pattern_type_mismatch;
326 mod precedence;
327 mod ptr;
328 mod ptr_eq;
329 mod ptr_offset_with_cast;
330 mod question_mark;
331 mod ranges;
332 mod redundant_clone;
333 mod redundant_closure_call;
334 mod redundant_else;
335 mod redundant_field_names;
336 mod redundant_pub_crate;
337 mod redundant_slicing;
338 mod redundant_static_lifetimes;
339 mod ref_option_ref;
340 mod reference;
341 mod regex;
342 mod repeat_once;
343 mod return_self_not_must_use;
344 mod returns;
345 mod same_name_method;
346 mod self_assignment;
347 mod self_named_constructors;
348 mod semicolon_if_nothing_returned;
349 mod serde_api;
350 mod shadow;
351 mod single_char_lifetime_names;
352 mod single_component_path_imports;
353 mod size_of_in_element_count;
354 mod slow_vector_initialization;
355 mod stable_sort_primitive;
356 mod strings;
357 mod strlen_on_c_strings;
358 mod suspicious_operation_groupings;
359 mod suspicious_trait_impl;
360 mod swap;
361 mod tabs_in_doc_comments;
362 mod temporary_assignment;
363 mod to_digit_is_some;
364 mod to_string_in_display;
365 mod trailing_empty_array;
366 mod trait_bounds;
367 mod transmute;
368 mod transmuting_null;
369 mod try_err;
370 mod types;
371 mod undocumented_unsafe_blocks;
372 mod undropped_manually_drops;
373 mod unicode;
374 mod uninit_vec;
375 mod unit_hash;
376 mod unit_return_expecting_ord;
377 mod unit_types;
378 mod unnamed_address;
379 mod unnecessary_self_imports;
380 mod unnecessary_sort_by;
381 mod unnecessary_wraps;
382 mod unnested_or_patterns;
383 mod unsafe_removed_from_name;
384 mod unused_async;
385 mod unused_io_amount;
386 mod unused_self;
387 mod unused_unit;
388 mod unwrap;
389 mod unwrap_in_result;
390 mod upper_case_acronyms;
391 mod use_self;
392 mod useless_conversion;
393 mod vec;
394 mod vec_init_then_push;
395 mod vec_resize_to_zero;
396 mod verbose_file_reads;
397 mod wildcard_dependencies;
398 mod wildcard_imports;
399 mod write;
400 mod zero_div_zero;
401 mod zero_sized_map_values;
402 // end lints modules, do not remove this comment, it’s used in `update_lints`
403
404 pub use crate::utils::conf::Conf;
405 use crate::utils::conf::TryConf;
406
407 /// Register all pre expansion lints
408 ///
409 /// Pre-expansion lints run before any macro expansion has happened.
410 ///
411 /// Note that due to the architecture of the compiler, currently `cfg_attr` attributes on crate
412 /// level (i.e `#![cfg_attr(...)]`) will still be expanded even when using a pre-expansion pass.
413 ///
414 /// Used in `./src/driver.rs`.
415 pub fn register_pre_expansion_lints(store: &mut rustc_lint::LintStore, sess: &Session, conf: &Conf) {
416     // NOTE: Do not add any more pre-expansion passes. These should be removed eventually.
417
418     let msrv = conf.msrv.as_ref().and_then(|s| {
419         parse_msrv(s, None, None).or_else(|| {
420             sess.err(&format!(
421                 "error reading Clippy's configuration file. `{}` is not a valid Rust version",
422                 s
423             ));
424             None
425         })
426     });
427
428     store.register_pre_expansion_pass(|| Box::new(write::Write::default()));
429     store.register_pre_expansion_pass(move || Box::new(attrs::EarlyAttributes { msrv }));
430     store.register_pre_expansion_pass(|| Box::new(dbg_macro::DbgMacro));
431 }
432
433 #[doc(hidden)]
434 pub fn read_conf(sess: &Session) -> Conf {
435     let file_name = match utils::conf::lookup_conf_file() {
436         Ok(Some(path)) => path,
437         Ok(None) => return Conf::default(),
438         Err(error) => {
439             sess.struct_err(&format!("error finding Clippy's configuration file: {}", error))
440                 .emit();
441             return Conf::default();
442         },
443     };
444
445     let TryConf { conf, errors } = utils::conf::read(&file_name);
446     // all conf errors are non-fatal, we just use the default conf in case of error
447     for error in errors {
448         sess.struct_err(&format!(
449             "error reading Clippy's configuration file `{}`: {}",
450             file_name.display(),
451             error
452         ))
453         .emit();
454     }
455
456     conf
457 }
458
459 /// Register all lints and lint groups with the rustc plugin registry
460 ///
461 /// Used in `./src/driver.rs`.
462 #[allow(clippy::too_many_lines)]
463 pub fn register_plugins(store: &mut rustc_lint::LintStore, sess: &Session, conf: &Conf) {
464     register_removed_non_tool_lints(store);
465
466     include!("lib.deprecated.rs");
467
468     include!("lib.register_lints.rs");
469     include!("lib.register_restriction.rs");
470     include!("lib.register_pedantic.rs");
471
472     #[cfg(feature = "internal")]
473     include!("lib.register_internal.rs");
474
475     include!("lib.register_all.rs");
476     include!("lib.register_style.rs");
477     include!("lib.register_complexity.rs");
478     include!("lib.register_correctness.rs");
479     include!("lib.register_suspicious.rs");
480     include!("lib.register_perf.rs");
481     include!("lib.register_cargo.rs");
482     include!("lib.register_nursery.rs");
483
484     #[cfg(feature = "internal")]
485     {
486         if std::env::var("ENABLE_METADATA_COLLECTION").eq(&Ok("1".to_string())) {
487             store.register_late_pass(|| Box::new(utils::internal_lints::metadata_collector::MetadataCollector::new()));
488             return;
489         }
490     }
491
492     // all the internal lints
493     #[cfg(feature = "internal")]
494     {
495         store.register_early_pass(|| Box::new(utils::internal_lints::ClippyLintsInternal));
496         store.register_early_pass(|| Box::new(utils::internal_lints::ProduceIce));
497         store.register_late_pass(|| Box::new(utils::inspector::DeepCodeInspector));
498         store.register_late_pass(|| Box::new(utils::internal_lints::CollapsibleCalls));
499         store.register_late_pass(|| Box::new(utils::internal_lints::CompilerLintFunctions::new()));
500         store.register_late_pass(|| Box::new(utils::internal_lints::IfChainStyle));
501         store.register_late_pass(|| Box::new(utils::internal_lints::InvalidPaths));
502         store.register_late_pass(|| Box::new(utils::internal_lints::InterningDefinedSymbol::default()));
503         store.register_late_pass(|| Box::new(utils::internal_lints::LintWithoutLintPass::default()));
504         store.register_late_pass(|| Box::new(utils::internal_lints::MatchTypeOnDiagItem));
505         store.register_late_pass(|| Box::new(utils::internal_lints::OuterExpnDataPass));
506     }
507
508     store.register_late_pass(|| Box::new(utils::author::Author));
509     store.register_late_pass(|| Box::new(await_holding_invalid::AwaitHolding));
510     store.register_late_pass(|| Box::new(serde_api::SerdeApi));
511     let vec_box_size_threshold = conf.vec_box_size_threshold;
512     let type_complexity_threshold = conf.type_complexity_threshold;
513     let avoid_breaking_exported_api = conf.avoid_breaking_exported_api;
514     store.register_late_pass(move || {
515         Box::new(types::Types::new(
516             vec_box_size_threshold,
517             type_complexity_threshold,
518             avoid_breaking_exported_api,
519         ))
520     });
521     store.register_late_pass(|| Box::new(booleans::NonminimalBool));
522     store.register_late_pass(|| Box::new(needless_bitwise_bool::NeedlessBitwiseBool));
523     store.register_late_pass(|| Box::new(eq_op::EqOp));
524     store.register_late_pass(|| Box::new(enum_clike::UnportableVariant));
525     store.register_late_pass(|| Box::new(float_literal::FloatLiteral));
526     let verbose_bit_mask_threshold = conf.verbose_bit_mask_threshold;
527     store.register_late_pass(move || Box::new(bit_mask::BitMask::new(verbose_bit_mask_threshold)));
528     store.register_late_pass(|| Box::new(ptr::Ptr));
529     store.register_late_pass(|| Box::new(ptr_eq::PtrEq));
530     store.register_late_pass(|| Box::new(needless_bool::NeedlessBool));
531     store.register_late_pass(|| Box::new(needless_option_as_deref::OptionNeedlessDeref));
532     store.register_late_pass(|| Box::new(needless_bool::BoolComparison));
533     store.register_late_pass(|| Box::new(needless_for_each::NeedlessForEach));
534     store.register_late_pass(|| Box::new(misc::MiscLints));
535     store.register_late_pass(|| Box::new(eta_reduction::EtaReduction));
536     store.register_late_pass(|| Box::new(identity_op::IdentityOp));
537     store.register_late_pass(|| Box::new(erasing_op::ErasingOp));
538     store.register_late_pass(|| Box::new(mut_mut::MutMut));
539     store.register_late_pass(|| Box::new(mut_reference::UnnecessaryMutPassed));
540     store.register_late_pass(|| Box::new(len_zero::LenZero));
541     store.register_late_pass(|| Box::new(attrs::Attributes));
542     store.register_late_pass(|| Box::new(blocks_in_if_conditions::BlocksInIfConditions));
543     store.register_late_pass(|| Box::new(collapsible_match::CollapsibleMatch));
544     store.register_late_pass(|| Box::new(unicode::Unicode));
545     store.register_late_pass(|| Box::new(uninit_vec::UninitVec));
546     store.register_late_pass(|| Box::new(unit_hash::UnitHash));
547     store.register_late_pass(|| Box::new(unit_return_expecting_ord::UnitReturnExpectingOrd));
548     store.register_late_pass(|| Box::new(strings::StringAdd));
549     store.register_late_pass(|| Box::new(implicit_return::ImplicitReturn));
550     store.register_late_pass(|| Box::new(implicit_saturating_sub::ImplicitSaturatingSub));
551     store.register_late_pass(|| Box::new(default_numeric_fallback::DefaultNumericFallback));
552     store.register_late_pass(|| Box::new(inconsistent_struct_constructor::InconsistentStructConstructor));
553     store.register_late_pass(|| Box::new(non_octal_unix_permissions::NonOctalUnixPermissions));
554     store.register_early_pass(|| Box::new(unnecessary_self_imports::UnnecessarySelfImports));
555
556     let msrv = conf.msrv.as_ref().and_then(|s| {
557         parse_msrv(s, None, None).or_else(|| {
558             sess.err(&format!(
559                 "error reading Clippy's configuration file. `{}` is not a valid Rust version",
560                 s
561             ));
562             None
563         })
564     });
565
566     let avoid_breaking_exported_api = conf.avoid_breaking_exported_api;
567     store.register_late_pass(move || Box::new(approx_const::ApproxConstant::new(msrv)));
568     store.register_late_pass(move || Box::new(methods::Methods::new(avoid_breaking_exported_api, msrv)));
569     store.register_late_pass(move || Box::new(matches::Matches::new(msrv)));
570     store.register_early_pass(move || Box::new(manual_non_exhaustive::ManualNonExhaustive::new(msrv)));
571     store.register_late_pass(move || Box::new(manual_strip::ManualStrip::new(msrv)));
572     store.register_early_pass(move || Box::new(redundant_static_lifetimes::RedundantStaticLifetimes::new(msrv)));
573     store.register_early_pass(move || Box::new(redundant_field_names::RedundantFieldNames::new(msrv)));
574     store.register_late_pass(move || Box::new(checked_conversions::CheckedConversions::new(msrv)));
575     store.register_late_pass(move || Box::new(mem_replace::MemReplace::new(msrv)));
576     store.register_late_pass(move || Box::new(ranges::Ranges::new(msrv)));
577     store.register_late_pass(move || Box::new(from_over_into::FromOverInto::new(msrv)));
578     store.register_late_pass(move || Box::new(use_self::UseSelf::new(msrv)));
579     store.register_late_pass(move || Box::new(missing_const_for_fn::MissingConstForFn::new(msrv)));
580     store.register_late_pass(move || Box::new(needless_question_mark::NeedlessQuestionMark));
581     store.register_late_pass(move || Box::new(casts::Casts::new(msrv)));
582     store.register_early_pass(move || Box::new(unnested_or_patterns::UnnestedOrPatterns::new(msrv)));
583
584     store.register_late_pass(|| Box::new(size_of_in_element_count::SizeOfInElementCount));
585     store.register_late_pass(|| Box::new(same_name_method::SameNameMethod));
586     let max_suggested_slice_pattern_length = conf.max_suggested_slice_pattern_length;
587     store.register_late_pass(move || {
588         Box::new(index_refutable_slice::IndexRefutableSlice::new(
589             max_suggested_slice_pattern_length,
590             msrv,
591         ))
592     });
593     store.register_late_pass(|| Box::new(map_clone::MapClone));
594     store.register_late_pass(|| Box::new(map_err_ignore::MapErrIgnore));
595     store.register_late_pass(|| Box::new(shadow::Shadow::default()));
596     store.register_late_pass(|| Box::new(unit_types::UnitTypes));
597     store.register_late_pass(|| Box::new(loops::Loops));
598     store.register_late_pass(|| Box::new(main_recursion::MainRecursion::default()));
599     store.register_late_pass(|| Box::new(lifetimes::Lifetimes));
600     store.register_late_pass(|| Box::new(entry::HashMapPass));
601     store.register_late_pass(|| Box::new(minmax::MinMaxPass));
602     store.register_late_pass(|| Box::new(open_options::OpenOptions));
603     store.register_late_pass(|| Box::new(zero_div_zero::ZeroDiv));
604     store.register_late_pass(|| Box::new(mutex_atomic::Mutex));
605     store.register_late_pass(|| Box::new(needless_update::NeedlessUpdate));
606     store.register_late_pass(|| Box::new(needless_borrowed_ref::NeedlessBorrowedRef));
607     store.register_late_pass(|| Box::new(no_effect::NoEffect));
608     store.register_late_pass(|| Box::new(temporary_assignment::TemporaryAssignment));
609     store.register_late_pass(|| Box::new(transmute::Transmute));
610     let cognitive_complexity_threshold = conf.cognitive_complexity_threshold;
611     store.register_late_pass(move || {
612         Box::new(cognitive_complexity::CognitiveComplexity::new(
613             cognitive_complexity_threshold,
614         ))
615     });
616     let too_large_for_stack = conf.too_large_for_stack;
617     store.register_late_pass(move || Box::new(escape::BoxedLocal { too_large_for_stack }));
618     store.register_late_pass(move || Box::new(vec::UselessVec { too_large_for_stack }));
619     store.register_late_pass(|| Box::new(panic_unimplemented::PanicUnimplemented));
620     store.register_late_pass(|| Box::new(strings::StringLitAsBytes));
621     store.register_late_pass(|| Box::new(derive::Derive));
622     store.register_late_pass(|| Box::new(derivable_impls::DerivableImpls));
623     store.register_late_pass(|| Box::new(get_last_with_len::GetLastWithLen));
624     store.register_late_pass(|| Box::new(drop_forget_ref::DropForgetRef));
625     store.register_late_pass(|| Box::new(empty_enum::EmptyEnum));
626     store.register_late_pass(|| Box::new(absurd_extreme_comparisons::AbsurdExtremeComparisons));
627     store.register_late_pass(|| Box::new(invalid_upcast_comparisons::InvalidUpcastComparisons));
628     store.register_late_pass(|| Box::new(regex::Regex));
629     store.register_late_pass(|| Box::new(copies::CopyAndPaste));
630     store.register_late_pass(|| Box::new(copy_iterator::CopyIterator));
631     store.register_late_pass(|| Box::new(format::UselessFormat));
632     store.register_late_pass(|| Box::new(swap::Swap));
633     store.register_late_pass(|| Box::new(overflow_check_conditional::OverflowCheckConditional));
634     store.register_late_pass(|| Box::new(new_without_default::NewWithoutDefault::default()));
635     let blacklisted_names = conf.blacklisted_names.iter().cloned().collect::<FxHashSet<_>>();
636     store.register_late_pass(move || Box::new(blacklisted_name::BlacklistedName::new(blacklisted_names.clone())));
637     let too_many_arguments_threshold = conf.too_many_arguments_threshold;
638     let too_many_lines_threshold = conf.too_many_lines_threshold;
639     store.register_late_pass(move || {
640         Box::new(functions::Functions::new(
641             too_many_arguments_threshold,
642             too_many_lines_threshold,
643         ))
644     });
645     let doc_valid_idents = conf.doc_valid_idents.iter().cloned().collect::<FxHashSet<_>>();
646     store.register_late_pass(move || Box::new(doc::DocMarkdown::new(doc_valid_idents.clone())));
647     store.register_late_pass(|| Box::new(neg_multiply::NegMultiply));
648     store.register_late_pass(|| Box::new(mem_forget::MemForget));
649     store.register_late_pass(|| Box::new(arithmetic::Arithmetic::default()));
650     store.register_late_pass(|| Box::new(assign_ops::AssignOps));
651     store.register_late_pass(|| Box::new(let_if_seq::LetIfSeq));
652     store.register_late_pass(|| Box::new(eval_order_dependence::EvalOrderDependence));
653     store.register_late_pass(|| Box::new(missing_doc::MissingDoc::new()));
654     store.register_late_pass(|| Box::new(missing_inline::MissingInline));
655     store.register_late_pass(move || Box::new(exhaustive_items::ExhaustiveItems));
656     store.register_late_pass(|| Box::new(match_result_ok::MatchResultOk));
657     store.register_late_pass(|| Box::new(partialeq_ne_impl::PartialEqNeImpl));
658     store.register_late_pass(|| Box::new(unused_io_amount::UnusedIoAmount));
659     let enum_variant_size_threshold = conf.enum_variant_size_threshold;
660     store.register_late_pass(move || Box::new(large_enum_variant::LargeEnumVariant::new(enum_variant_size_threshold)));
661     store.register_late_pass(|| Box::new(explicit_write::ExplicitWrite));
662     store.register_late_pass(|| Box::new(needless_pass_by_value::NeedlessPassByValue));
663     let pass_by_ref_or_value = pass_by_ref_or_value::PassByRefOrValue::new(
664         conf.trivial_copy_size_limit,
665         conf.pass_by_value_size_limit,
666         conf.avoid_breaking_exported_api,
667         &sess.target,
668     );
669     store.register_late_pass(move || Box::new(pass_by_ref_or_value));
670     store.register_late_pass(|| Box::new(ref_option_ref::RefOptionRef));
671     store.register_late_pass(|| Box::new(try_err::TryErr));
672     store.register_late_pass(|| Box::new(bytecount::ByteCount));
673     store.register_late_pass(|| Box::new(infinite_iter::InfiniteIter));
674     store.register_late_pass(|| Box::new(inline_fn_without_body::InlineFnWithoutBody));
675     store.register_late_pass(|| Box::new(useless_conversion::UselessConversion::default()));
676     store.register_late_pass(|| Box::new(implicit_hasher::ImplicitHasher));
677     store.register_late_pass(|| Box::new(fallible_impl_from::FallibleImplFrom));
678     store.register_late_pass(|| Box::new(double_comparison::DoubleComparisons));
679     store.register_late_pass(|| Box::new(question_mark::QuestionMark));
680     store.register_early_pass(|| Box::new(suspicious_operation_groupings::SuspiciousOperationGroupings));
681     store.register_late_pass(|| Box::new(suspicious_trait_impl::SuspiciousImpl));
682     store.register_late_pass(|| Box::new(map_unit_fn::MapUnit));
683     store.register_late_pass(|| Box::new(inherent_impl::MultipleInherentImpl));
684     store.register_late_pass(|| Box::new(neg_cmp_op_on_partial_ord::NoNegCompOpForPartialOrd));
685     store.register_late_pass(|| Box::new(unwrap::Unwrap));
686     store.register_late_pass(|| Box::new(duration_subsec::DurationSubsec));
687     store.register_late_pass(|| Box::new(indexing_slicing::IndexingSlicing));
688     store.register_late_pass(|| Box::new(non_copy_const::NonCopyConst));
689     store.register_late_pass(|| Box::new(ptr_offset_with_cast::PtrOffsetWithCast));
690     store.register_late_pass(|| Box::new(redundant_clone::RedundantClone));
691     store.register_late_pass(|| Box::new(slow_vector_initialization::SlowVectorInit));
692     store.register_late_pass(|| Box::new(unnecessary_sort_by::UnnecessarySortBy));
693     store.register_late_pass(move || Box::new(unnecessary_wraps::UnnecessaryWraps::new(avoid_breaking_exported_api)));
694     store.register_late_pass(|| Box::new(assertions_on_constants::AssertionsOnConstants));
695     store.register_late_pass(|| Box::new(transmuting_null::TransmutingNull));
696     store.register_late_pass(|| Box::new(path_buf_push_overwrite::PathBufPushOverwrite));
697     store.register_late_pass(|| Box::new(integer_division::IntegerDivision));
698     store.register_late_pass(|| Box::new(inherent_to_string::InherentToString));
699     let max_trait_bounds = conf.max_trait_bounds;
700     store.register_late_pass(move || Box::new(trait_bounds::TraitBounds::new(max_trait_bounds)));
701     store.register_late_pass(|| Box::new(comparison_chain::ComparisonChain));
702     store.register_late_pass(|| Box::new(mut_key::MutableKeyType));
703     store.register_late_pass(|| Box::new(modulo_arithmetic::ModuloArithmetic));
704     store.register_early_pass(|| Box::new(reference::DerefAddrOf));
705     store.register_early_pass(|| Box::new(reference::RefInDeref));
706     store.register_early_pass(|| Box::new(double_parens::DoubleParens));
707     store.register_late_pass(|| Box::new(to_string_in_display::ToStringInDisplay::new()));
708     store.register_early_pass(|| Box::new(unsafe_removed_from_name::UnsafeNameRemoval));
709     store.register_early_pass(|| Box::new(else_if_without_else::ElseIfWithoutElse));
710     store.register_early_pass(|| Box::new(int_plus_one::IntPlusOne));
711     store.register_early_pass(|| Box::new(formatting::Formatting));
712     store.register_early_pass(|| Box::new(misc_early::MiscEarlyLints));
713     store.register_early_pass(|| Box::new(redundant_closure_call::RedundantClosureCall));
714     store.register_late_pass(|| Box::new(redundant_closure_call::RedundantClosureCall));
715     store.register_early_pass(|| Box::new(unused_unit::UnusedUnit));
716     store.register_late_pass(|| Box::new(returns::Return));
717     store.register_early_pass(|| Box::new(collapsible_if::CollapsibleIf));
718     store.register_early_pass(|| Box::new(items_after_statements::ItemsAfterStatements));
719     store.register_early_pass(|| Box::new(precedence::Precedence));
720     store.register_early_pass(|| Box::new(needless_continue::NeedlessContinue));
721     store.register_early_pass(|| Box::new(redundant_else::RedundantElse));
722     store.register_late_pass(|| Box::new(create_dir::CreateDir));
723     store.register_early_pass(|| Box::new(needless_arbitrary_self_type::NeedlessArbitrarySelfType));
724     let cargo_ignore_publish = conf.cargo_ignore_publish;
725     store.register_late_pass(move || Box::new(cargo_common_metadata::CargoCommonMetadata::new(cargo_ignore_publish)));
726     store.register_late_pass(|| Box::new(multiple_crate_versions::MultipleCrateVersions));
727     store.register_late_pass(|| Box::new(wildcard_dependencies::WildcardDependencies));
728     let literal_representation_lint_fraction_readability = conf.unreadable_literal_lint_fractions;
729     store.register_early_pass(move || {
730         Box::new(literal_representation::LiteralDigitGrouping::new(
731             literal_representation_lint_fraction_readability,
732         ))
733     });
734     let literal_representation_threshold = conf.literal_representation_threshold;
735     store.register_early_pass(move || {
736         Box::new(literal_representation::DecimalLiteralRepresentation::new(
737             literal_representation_threshold,
738         ))
739     });
740     let enum_variant_name_threshold = conf.enum_variant_name_threshold;
741     store.register_late_pass(move || {
742         Box::new(enum_variants::EnumVariantNames::new(
743             enum_variant_name_threshold,
744             avoid_breaking_exported_api,
745         ))
746     });
747     store.register_early_pass(|| Box::new(tabs_in_doc_comments::TabsInDocComments));
748     let upper_case_acronyms_aggressive = conf.upper_case_acronyms_aggressive;
749     store.register_late_pass(move || {
750         Box::new(upper_case_acronyms::UpperCaseAcronyms::new(
751             avoid_breaking_exported_api,
752             upper_case_acronyms_aggressive,
753         ))
754     });
755     store.register_late_pass(|| Box::new(default::Default::default()));
756     store.register_late_pass(|| Box::new(unused_self::UnusedSelf));
757     store.register_late_pass(|| Box::new(mutable_debug_assertion::DebugAssertWithMutCall));
758     store.register_late_pass(|| Box::new(exit::Exit));
759     store.register_late_pass(|| Box::new(to_digit_is_some::ToDigitIsSome));
760     let array_size_threshold = conf.array_size_threshold;
761     store.register_late_pass(move || Box::new(large_stack_arrays::LargeStackArrays::new(array_size_threshold)));
762     store.register_late_pass(move || Box::new(large_const_arrays::LargeConstArrays::new(array_size_threshold)));
763     store.register_late_pass(|| Box::new(floating_point_arithmetic::FloatingPointArithmetic));
764     store.register_early_pass(|| Box::new(as_conversions::AsConversions));
765     store.register_late_pass(|| Box::new(let_underscore::LetUnderscore));
766     store.register_early_pass(|| Box::new(single_component_path_imports::SingleComponentPathImports));
767     let max_fn_params_bools = conf.max_fn_params_bools;
768     let max_struct_bools = conf.max_struct_bools;
769     store.register_early_pass(move || {
770         Box::new(excessive_bools::ExcessiveBools::new(
771             max_struct_bools,
772             max_fn_params_bools,
773         ))
774     });
775     store.register_early_pass(|| Box::new(option_env_unwrap::OptionEnvUnwrap));
776     let warn_on_all_wildcard_imports = conf.warn_on_all_wildcard_imports;
777     store.register_late_pass(move || Box::new(wildcard_imports::WildcardImports::new(warn_on_all_wildcard_imports)));
778     store.register_late_pass(|| Box::new(verbose_file_reads::VerboseFileReads));
779     store.register_late_pass(|| Box::new(redundant_pub_crate::RedundantPubCrate::default()));
780     store.register_late_pass(|| Box::new(unnamed_address::UnnamedAddress));
781     store.register_late_pass(|| Box::new(dereference::Dereferencing::default()));
782     store.register_late_pass(|| Box::new(option_if_let_else::OptionIfLetElse));
783     store.register_late_pass(|| Box::new(future_not_send::FutureNotSend));
784     store.register_late_pass(|| Box::new(if_let_mutex::IfLetMutex));
785     store.register_late_pass(|| Box::new(if_not_else::IfNotElse));
786     store.register_late_pass(|| Box::new(equatable_if_let::PatternEquality));
787     store.register_late_pass(|| Box::new(mut_mutex_lock::MutMutexLock));
788     store.register_late_pass(|| Box::new(match_on_vec_items::MatchOnVecItems));
789     store.register_late_pass(|| Box::new(manual_async_fn::ManualAsyncFn));
790     store.register_late_pass(|| Box::new(vec_resize_to_zero::VecResizeToZero));
791     store.register_late_pass(|| Box::new(panic_in_result_fn::PanicInResultFn));
792     let single_char_binding_names_threshold = conf.single_char_binding_names_threshold;
793     store.register_early_pass(move || {
794         Box::new(non_expressive_names::NonExpressiveNames {
795             single_char_binding_names_threshold,
796         })
797     });
798     let macro_matcher = conf.standard_macro_braces.iter().cloned().collect::<FxHashSet<_>>();
799     store.register_early_pass(move || Box::new(nonstandard_macro_braces::MacroBraces::new(&macro_matcher)));
800     store.register_late_pass(|| Box::new(macro_use::MacroUseImports::default()));
801     store.register_late_pass(|| Box::new(pattern_type_mismatch::PatternTypeMismatch));
802     store.register_late_pass(|| Box::new(stable_sort_primitive::StableSortPrimitive));
803     store.register_late_pass(|| Box::new(repeat_once::RepeatOnce));
804     store.register_late_pass(|| Box::new(unwrap_in_result::UnwrapInResult));
805     store.register_late_pass(|| Box::new(self_assignment::SelfAssignment));
806     store.register_late_pass(|| Box::new(manual_unwrap_or::ManualUnwrapOr));
807     store.register_late_pass(|| Box::new(manual_ok_or::ManualOkOr));
808     store.register_late_pass(|| Box::new(float_equality_without_abs::FloatEqualityWithoutAbs));
809     store.register_late_pass(|| Box::new(semicolon_if_nothing_returned::SemicolonIfNothingReturned));
810     store.register_late_pass(|| Box::new(async_yields_async::AsyncYieldsAsync));
811     let disallowed_methods = conf.disallowed_methods.clone();
812     store.register_late_pass(move || Box::new(disallowed_methods::DisallowedMethods::new(disallowed_methods.clone())));
813     store.register_early_pass(|| Box::new(asm_syntax::InlineAsmX86AttSyntax));
814     store.register_early_pass(|| Box::new(asm_syntax::InlineAsmX86IntelSyntax));
815     store.register_late_pass(|| Box::new(undropped_manually_drops::UndroppedManuallyDrops));
816     store.register_late_pass(|| Box::new(strings::StrToString));
817     store.register_late_pass(|| Box::new(strings::StringToString));
818     store.register_late_pass(|| Box::new(zero_sized_map_values::ZeroSizedMapValues));
819     store.register_late_pass(|| Box::new(vec_init_then_push::VecInitThenPush::default()));
820     store.register_late_pass(|| {
821         Box::new(case_sensitive_file_extension_comparisons::CaseSensitiveFileExtensionComparisons)
822     });
823     store.register_late_pass(|| Box::new(redundant_slicing::RedundantSlicing));
824     store.register_late_pass(|| Box::new(from_str_radix_10::FromStrRadix10));
825     store.register_late_pass(|| Box::new(manual_map::ManualMap));
826     store.register_late_pass(move || Box::new(if_then_some_else_none::IfThenSomeElseNone::new(msrv)));
827     store.register_late_pass(|| Box::new(bool_assert_comparison::BoolAssertComparison));
828     store.register_early_pass(move || Box::new(module_style::ModStyle));
829     store.register_late_pass(|| Box::new(unused_async::UnusedAsync));
830     let disallowed_types = conf.disallowed_types.clone();
831     store.register_late_pass(move || Box::new(disallowed_types::DisallowedTypes::new(disallowed_types.clone())));
832     let import_renames = conf.enforced_import_renames.clone();
833     store.register_late_pass(move || {
834         Box::new(missing_enforced_import_rename::ImportRename::new(
835             import_renames.clone(),
836         ))
837     });
838     let scripts = conf.allowed_scripts.clone();
839     store.register_early_pass(move || Box::new(disallowed_script_idents::DisallowedScriptIdents::new(&scripts)));
840     store.register_late_pass(|| Box::new(strlen_on_c_strings::StrlenOnCStrings));
841     store.register_late_pass(move || Box::new(self_named_constructors::SelfNamedConstructors));
842     store.register_late_pass(move || Box::new(feature_name::FeatureName));
843     store.register_late_pass(move || Box::new(iter_not_returning_iterator::IterNotReturningIterator));
844     store.register_late_pass(move || Box::new(manual_assert::ManualAssert));
845     let enable_raw_pointer_heuristic_for_send = conf.enable_raw_pointer_heuristic_for_send;
846     store.register_late_pass(move || {
847         Box::new(non_send_fields_in_send_ty::NonSendFieldInSendTy::new(
848             enable_raw_pointer_heuristic_for_send,
849         ))
850     });
851     store.register_late_pass(move || Box::new(undocumented_unsafe_blocks::UndocumentedUnsafeBlocks::default()));
852     store.register_late_pass(|| Box::new(match_str_case_mismatch::MatchStrCaseMismatch));
853     store.register_late_pass(move || Box::new(format_args::FormatArgs));
854     store.register_late_pass(|| Box::new(trailing_empty_array::TrailingEmptyArray));
855     store.register_early_pass(|| Box::new(octal_escapes::OctalEscapes));
856     store.register_late_pass(|| Box::new(needless_late_init::NeedlessLateInit));
857     store.register_late_pass(|| Box::new(return_self_not_must_use::ReturnSelfNotMustUse));
858     store.register_late_pass(|| Box::new(init_numbered_fields::NumberedFields));
859     store.register_early_pass(|| Box::new(single_char_lifetime_names::SingleCharLifetimeNames));
860     // add lints here, do not remove this comment, it's used in `new_lint`
861 }
862
863 #[rustfmt::skip]
864 fn register_removed_non_tool_lints(store: &mut rustc_lint::LintStore) {
865     store.register_removed(
866         "should_assert_eq",
867         "`assert!()` will be more flexible with RFC 2011",
868     );
869     store.register_removed(
870         "extend_from_slice",
871         "`.extend_from_slice(_)` is a faster way to extend a Vec by a slice",
872     );
873     store.register_removed(
874         "range_step_by_zero",
875         "`iterator.step_by(0)` panics nowadays",
876     );
877     store.register_removed(
878         "unstable_as_slice",
879         "`Vec::as_slice` has been stabilized in 1.7",
880     );
881     store.register_removed(
882         "unstable_as_mut_slice",
883         "`Vec::as_mut_slice` has been stabilized in 1.7",
884     );
885     store.register_removed(
886         "misaligned_transmute",
887         "this lint has been split into cast_ptr_alignment and transmute_ptr_to_ptr",
888     );
889     store.register_removed(
890         "assign_ops",
891         "using compound assignment operators (e.g., `+=`) is harmless",
892     );
893     store.register_removed(
894         "if_let_redundant_pattern_matching",
895         "this lint has been changed to redundant_pattern_matching",
896     );
897     store.register_removed(
898         "unsafe_vector_initialization",
899         "the replacement suggested by this lint had substantially different behavior",
900     );
901     store.register_removed(
902         "reverse_range_loop",
903         "this lint is now included in reversed_empty_ranges",
904     );
905 }
906
907 /// Register renamed lints.
908 ///
909 /// Used in `./src/driver.rs`.
910 pub fn register_renamed(ls: &mut rustc_lint::LintStore) {
911     // NOTE: when renaming a lint, add a corresponding test to tests/ui/rename.rs
912     ls.register_renamed("clippy::stutter", "clippy::module_name_repetitions");
913     ls.register_renamed("clippy::new_without_default_derive", "clippy::new_without_default");
914     ls.register_renamed("clippy::cyclomatic_complexity", "clippy::cognitive_complexity");
915     ls.register_renamed("clippy::const_static_lifetime", "clippy::redundant_static_lifetimes");
916     ls.register_renamed("clippy::option_and_then_some", "clippy::bind_instead_of_map");
917     ls.register_renamed("clippy::box_vec", "clippy::box_collection");
918     ls.register_renamed("clippy::block_in_if_condition_expr", "clippy::blocks_in_if_conditions");
919     ls.register_renamed("clippy::block_in_if_condition_stmt", "clippy::blocks_in_if_conditions");
920     ls.register_renamed("clippy::option_map_unwrap_or", "clippy::map_unwrap_or");
921     ls.register_renamed("clippy::option_map_unwrap_or_else", "clippy::map_unwrap_or");
922     ls.register_renamed("clippy::result_map_unwrap_or_else", "clippy::map_unwrap_or");
923     ls.register_renamed("clippy::option_unwrap_used", "clippy::unwrap_used");
924     ls.register_renamed("clippy::result_unwrap_used", "clippy::unwrap_used");
925     ls.register_renamed("clippy::option_expect_used", "clippy::expect_used");
926     ls.register_renamed("clippy::result_expect_used", "clippy::expect_used");
927     ls.register_renamed("clippy::for_loop_over_option", "clippy::for_loops_over_fallibles");
928     ls.register_renamed("clippy::for_loop_over_result", "clippy::for_loops_over_fallibles");
929     ls.register_renamed("clippy::identity_conversion", "clippy::useless_conversion");
930     ls.register_renamed("clippy::zero_width_space", "clippy::invisible_characters");
931     ls.register_renamed("clippy::single_char_push_str", "clippy::single_char_add_str");
932     ls.register_renamed("clippy::if_let_some_result", "clippy::match_result_ok");
933     ls.register_renamed("clippy::disallowed_type", "clippy::disallowed_types");
934     ls.register_renamed("clippy::disallowed_method", "clippy::disallowed_methods");
935
936     // uplifted lints
937     ls.register_renamed("clippy::invalid_ref", "invalid_value");
938     ls.register_renamed("clippy::into_iter_on_array", "array_into_iter");
939     ls.register_renamed("clippy::unused_label", "unused_labels");
940     ls.register_renamed("clippy::drop_bounds", "drop_bounds");
941     ls.register_renamed("clippy::temporary_cstring_as_ptr", "temporary_cstring_as_ptr");
942     ls.register_renamed("clippy::panic_params", "non_fmt_panics");
943     ls.register_renamed("clippy::unknown_clippy_lints", "unknown_lints");
944     ls.register_renamed("clippy::invalid_atomic_ordering", "invalid_atomic_ordering");
945     ls.register_renamed("clippy::mem_discriminant_non_enum", "enum_intrinsics_non_enums");
946 }
947
948 // only exists to let the dogfood integration test works.
949 // Don't run clippy as an executable directly
950 #[allow(dead_code)]
951 fn main() {
952     panic!("Please use the cargo-clippy executable");
953 }