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