1 #![feature(array_windows)]
2 #![feature(binary_heap_into_iter_sorted)]
3 #![feature(box_patterns)]
4 #![feature(control_flow_enum)]
5 #![feature(drain_filter)]
6 #![feature(iter_intersperse)]
7 #![feature(let_chains)]
8 #![feature(lint_reasons)]
9 #![feature(never_type)]
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 // Disable this rustc lint for now, as it was also done in rustc
22 #![allow(rustc::potential_query_instability)]
24 // FIXME: switch to something more ergonomic here, once available.
25 // (Currently there is no way to opt into sysroot crates without `extern crate`.)
26 extern crate rustc_arena;
27 extern crate rustc_ast;
28 extern crate rustc_ast_pretty;
29 extern crate rustc_data_structures;
30 extern crate rustc_driver;
31 extern crate rustc_errors;
32 extern crate rustc_hir;
33 extern crate rustc_hir_analysis;
34 extern crate rustc_hir_pretty;
35 extern crate rustc_hir_typeck;
36 extern crate rustc_index;
37 extern crate rustc_infer;
38 extern crate rustc_lexer;
39 extern crate rustc_lint;
40 extern crate rustc_middle;
41 extern crate rustc_parse;
42 extern crate rustc_session;
43 extern crate rustc_span;
44 extern crate rustc_target;
45 extern crate rustc_trait_selection;
48 extern crate clippy_utils;
50 extern crate declare_clippy_lint;
53 use std::path::PathBuf;
55 use clippy_utils::msrvs::Msrv;
56 use rustc_data_structures::fx::FxHashSet;
57 use rustc_lint::{Lint, LintId};
58 use rustc_session::Session;
60 #[cfg(feature = "internal")]
61 pub mod deprecated_lints;
62 #[cfg_attr(feature = "internal", allow(clippy::missing_clippy_version_attribute))]
68 // begin lints modules, do not remove this comment, it’s used in `update_lints`
69 mod almost_complete_range;
73 mod assertions_on_constants;
74 mod assertions_on_result_states;
75 mod async_yields_async;
77 mod await_holding_invalid;
78 mod blocks_in_if_conditions;
79 mod bool_assert_comparison;
80 mod bool_to_int_with_if;
86 mod checked_conversions;
87 mod cognitive_complexity;
92 mod crate_in_macro_def;
96 mod default_instead_of_iter_empty;
97 mod default_numeric_fallback;
98 mod default_union_representation;
102 mod disallowed_macros;
103 mod disallowed_methods;
104 mod disallowed_names;
105 mod disallowed_script_idents;
106 mod disallowed_types;
111 mod else_if_without_else;
114 mod empty_structs_with_brackets;
118 mod equatable_if_let;
122 mod exhaustive_items;
125 mod fallible_impl_from;
127 mod floating_point_arithmetic;
132 mod format_push_string;
135 mod from_raw_with_void_ptr;
136 mod from_str_radix_10;
141 mod if_then_some_else_none;
144 mod implicit_saturating_add;
145 mod implicit_saturating_sub;
146 mod inconsistent_struct_constructor;
147 mod index_refutable_slice;
148 mod indexing_slicing;
151 mod inherent_to_string;
152 mod init_numbered_fields;
153 mod inline_fn_without_body;
154 mod instant_subtraction;
156 mod invalid_upcast_comparisons;
157 mod invalid_utf8_in_unchecked;
158 mod items_after_statements;
159 mod iter_not_returning_iterator;
160 mod large_const_arrays;
161 mod large_enum_variant;
162 mod large_include_file;
163 mod large_stack_arrays;
168 mod literal_representation;
176 mod manual_is_ascii_check;
178 mod manual_non_exhaustive;
179 mod manual_rem_euclid;
181 mod manual_string_new;
192 mod mismatching_type_param_order;
193 mod missing_const_for_fn;
195 mod missing_enforced_import_rename;
197 mod missing_trait_methods;
198 mod mixed_read_write_in_expression;
200 mod multi_assignments;
204 mod mutable_debug_assertion;
206 mod needless_arbitrary_self_type;
208 mod needless_borrowed_ref;
209 mod needless_continue;
210 mod needless_for_each;
211 mod needless_late_init;
212 mod needless_parens_on_range_literals;
213 mod needless_pass_by_value;
214 mod needless_question_mark;
216 mod neg_cmp_op_on_partial_ord;
218 mod new_without_default;
221 mod non_expressive_names;
222 mod non_octal_unix_permissions;
223 mod non_send_fields_in_send_ty;
224 mod nonstandard_macro_braces;
226 mod only_used_in_recursion;
228 mod option_env_unwrap;
229 mod option_if_let_else;
230 mod overflow_check_conditional;
231 mod panic_in_result_fn;
232 mod panic_unimplemented;
233 mod partial_pub_fields;
234 mod partialeq_ne_impl;
235 mod partialeq_to_none;
236 mod pass_by_ref_or_value;
237 mod pattern_type_mismatch;
238 mod permissions_set_readonly_false;
241 mod ptr_offset_with_cast;
245 mod rc_clone_in_vec_init;
246 mod read_zero_byte_vec;
248 mod redundant_closure_call;
250 mod redundant_field_names;
251 mod redundant_pub_crate;
252 mod redundant_slicing;
253 mod redundant_static_lifetimes;
257 mod return_self_not_must_use;
259 mod same_name_method;
260 mod self_named_constructors;
262 mod semicolon_if_nothing_returned;
265 mod single_char_lifetime_names;
266 mod single_component_path_imports;
267 mod size_of_in_element_count;
269 mod slow_vector_initialization;
270 mod std_instead_of_core;
272 mod strlen_on_c_strings;
273 mod suspicious_operation_groupings;
274 mod suspicious_trait_impl;
275 mod suspicious_xor_used_as_pow;
278 mod tabs_in_doc_comments;
279 mod temporary_assignment;
280 mod to_digit_is_some;
281 mod trailing_empty_array;
285 mod undocumented_unsafe_blocks;
288 mod unit_return_expecting_ord;
291 mod unnecessary_owned_empty_strings;
292 mod unnecessary_self_imports;
293 mod unnecessary_wraps;
294 mod unnested_or_patterns;
295 mod unsafe_removed_from_name;
297 mod unused_io_amount;
303 mod unwrap_in_result;
304 mod upper_case_acronyms;
306 mod useless_conversion;
308 mod vec_init_then_push;
309 mod wildcard_imports;
312 mod zero_sized_map_values;
313 // end lints modules, do not remove this comment, it’s used in `update_lints`
315 use crate::utils::conf::{format_error, TryConf};
316 pub use crate::utils::conf::{lookup_conf_file, Conf};
318 /// Register all pre expansion lints
320 /// Pre-expansion lints run before any macro expansion has happened.
322 /// Note that due to the architecture of the compiler, currently `cfg_attr` attributes on crate
323 /// level (i.e `#![cfg_attr(...)]`) will still be expanded even when using a pre-expansion pass.
325 /// Used in `./src/driver.rs`.
326 pub fn register_pre_expansion_lints(store: &mut rustc_lint::LintStore, sess: &Session, conf: &Conf) {
327 // NOTE: Do not add any more pre-expansion passes. These should be removed eventually.
328 let msrv = Msrv::read(&conf.msrv, sess);
329 let msrv = move || msrv.clone();
331 store.register_pre_expansion_pass(move || Box::new(attrs::EarlyAttributes { msrv: msrv() }));
335 pub fn read_conf(sess: &Session, path: &io::Result<Option<PathBuf>>) -> Conf {
336 let file_name = match path {
337 Ok(Some(path)) => path,
338 Ok(None) => return Conf::default(),
340 sess.struct_err(format!("error finding Clippy's configuration file: {error}"))
342 return Conf::default();
346 let TryConf { conf, errors, warnings } = utils::conf::read(file_name);
347 // all conf errors are non-fatal, we just use the default conf in case of error
348 for error in errors {
350 "error reading Clippy's configuration file `{}`: {}",
356 for warning in warnings {
357 sess.struct_warn(format!(
358 "error reading Clippy's configuration file `{}`: {}",
360 format_error(warning)
369 struct RegistrationGroups {
372 complexity: Vec<LintId>,
373 correctness: Vec<LintId>,
374 nursery: Vec<LintId>,
375 pedantic: Vec<LintId>,
377 restriction: Vec<LintId>,
379 suspicious: Vec<LintId>,
380 #[cfg(feature = "internal")]
381 internal: Vec<LintId>,
384 impl RegistrationGroups {
386 fn register(self, store: &mut rustc_lint::LintStore) {
387 store.register_group(true, "clippy::all", Some("clippy_all"), self.all);
388 store.register_group(true, "clippy::cargo", Some("clippy_cargo"), self.cargo);
389 store.register_group(true, "clippy::complexity", Some("clippy_complexity"), self.complexity);
390 store.register_group(true, "clippy::correctness", Some("clippy_correctness"), self.correctness);
391 store.register_group(true, "clippy::nursery", Some("clippy_nursery"), self.nursery);
392 store.register_group(true, "clippy::pedantic", Some("clippy_pedantic"), self.pedantic);
393 store.register_group(true, "clippy::perf", Some("clippy_perf"), self.perf);
394 store.register_group(true, "clippy::restriction", Some("clippy_restriction"), self.restriction);
395 store.register_group(true, "clippy::style", Some("clippy_style"), self.style);
396 store.register_group(true, "clippy::suspicious", Some("clippy_suspicious"), self.suspicious);
397 #[cfg(feature = "internal")]
398 store.register_group(true, "clippy::internal", Some("clippy_internal"), self.internal);
402 #[derive(Copy, Clone)]
403 pub(crate) enum LintCategory {
413 #[cfg(feature = "internal")]
416 #[allow(clippy::enum_glob_use)]
420 fn is_all(self) -> bool {
421 matches!(self, Correctness | Suspicious | Style | Complexity | Perf)
424 fn group(self, groups: &mut RegistrationGroups) -> &mut Vec<LintId> {
426 Cargo => &mut groups.cargo,
427 Complexity => &mut groups.complexity,
428 Correctness => &mut groups.correctness,
429 Nursery => &mut groups.nursery,
430 Pedantic => &mut groups.pedantic,
431 Perf => &mut groups.perf,
432 Restriction => &mut groups.restriction,
433 Style => &mut groups.style,
434 Suspicious => &mut groups.suspicious,
435 #[cfg(feature = "internal")]
436 Internal => &mut groups.internal,
441 pub(crate) struct LintInfo {
442 /// Double reference to maintain pointer equality
443 lint: &'static &'static Lint,
444 category: LintCategory,
445 explanation: &'static str,
448 pub fn explain(name: &str) {
449 let target = format!("clippy::{}", name.to_ascii_uppercase());
450 match declared_lints::LINTS.iter().find(|info| info.lint.name == target) {
451 Some(info) => print!("{}", info.explanation),
452 None => println!("unknown lint: {name}"),
456 fn register_categories(store: &mut rustc_lint::LintStore) {
457 let mut groups = RegistrationGroups::default();
459 for LintInfo { lint, category, .. } in declared_lints::LINTS {
460 if category.is_all() {
461 groups.all.push(LintId::of(lint));
464 category.group(&mut groups).push(LintId::of(lint));
467 let lints: Vec<&'static Lint> = declared_lints::LINTS.iter().map(|info| *info.lint).collect();
469 store.register_lints(&lints);
470 groups.register(store);
473 /// Register all lints and lint groups with the rustc plugin registry
475 /// Used in `./src/driver.rs`.
476 #[expect(clippy::too_many_lines)]
477 pub fn register_plugins(store: &mut rustc_lint::LintStore, sess: &Session, conf: &Conf) {
478 register_removed_non_tool_lints(store);
479 register_categories(store);
481 include!("lib.deprecated.rs");
483 #[cfg(feature = "internal")]
485 if std::env::var("ENABLE_METADATA_COLLECTION").eq(&Ok("1".to_string())) {
486 store.register_late_pass(|_| Box::new(utils::internal_lints::metadata_collector::MetadataCollector::new()));
491 // all the internal lints
492 #[cfg(feature = "internal")]
494 store.register_early_pass(|| Box::new(utils::internal_lints::clippy_lints_internal::ClippyLintsInternal));
495 store.register_early_pass(|| Box::new(utils::internal_lints::produce_ice::ProduceIce));
496 store.register_late_pass(|_| Box::new(utils::internal_lints::collapsible_calls::CollapsibleCalls));
497 store.register_late_pass(|_| {
498 Box::new(utils::internal_lints::compiler_lint_functions::CompilerLintFunctions::new())
500 store.register_late_pass(|_| Box::new(utils::internal_lints::if_chain_style::IfChainStyle));
501 store.register_late_pass(|_| Box::new(utils::internal_lints::invalid_paths::InvalidPaths));
502 store.register_late_pass(|_| {
503 Box::<utils::internal_lints::interning_defined_symbol::InterningDefinedSymbol>::default()
505 store.register_late_pass(|_| {
506 Box::<utils::internal_lints::lint_without_lint_pass::LintWithoutLintPass>::default()
508 store.register_late_pass(|_| Box::<utils::internal_lints::unnecessary_def_path::UnnecessaryDefPath>::default());
509 store.register_late_pass(|_| Box::new(utils::internal_lints::outer_expn_data_pass::OuterExpnDataPass));
510 store.register_late_pass(|_| Box::new(utils::internal_lints::msrv_attr_impl::MsrvAttrImpl));
513 let arithmetic_side_effects_allowed = conf.arithmetic_side_effects_allowed.clone();
514 let arithmetic_side_effects_allowed_binary = conf.arithmetic_side_effects_allowed_binary.clone();
515 let arithmetic_side_effects_allowed_unary = conf.arithmetic_side_effects_allowed_unary.clone();
516 store.register_late_pass(move |_| {
517 Box::new(operators::arithmetic_side_effects::ArithmeticSideEffects::new(
518 arithmetic_side_effects_allowed
520 .flat_map(|el| [[el.clone(), "*".to_string()], ["*".to_string(), el.clone()]])
521 .chain(arithmetic_side_effects_allowed_binary.clone())
523 arithmetic_side_effects_allowed
525 .chain(arithmetic_side_effects_allowed_unary.iter())
530 store.register_late_pass(|_| Box::new(utils::dump_hir::DumpHir));
531 store.register_late_pass(|_| Box::new(utils::author::Author));
532 let await_holding_invalid_types = conf.await_holding_invalid_types.clone();
533 store.register_late_pass(move |_| {
534 Box::new(await_holding_invalid::AwaitHolding::new(
535 await_holding_invalid_types.clone(),
538 store.register_late_pass(|_| Box::new(serde_api::SerdeApi));
539 let vec_box_size_threshold = conf.vec_box_size_threshold;
540 let type_complexity_threshold = conf.type_complexity_threshold;
541 let avoid_breaking_exported_api = conf.avoid_breaking_exported_api;
542 store.register_late_pass(move |_| {
543 Box::new(types::Types::new(
544 vec_box_size_threshold,
545 type_complexity_threshold,
546 avoid_breaking_exported_api,
549 store.register_late_pass(|_| Box::new(booleans::NonminimalBool));
550 store.register_late_pass(|_| Box::new(enum_clike::UnportableVariant));
551 store.register_late_pass(|_| Box::new(float_literal::FloatLiteral));
552 store.register_late_pass(|_| Box::new(ptr::Ptr));
553 store.register_late_pass(|_| Box::new(needless_bool::NeedlessBool));
554 store.register_late_pass(|_| Box::new(needless_bool::BoolComparison));
555 store.register_late_pass(|_| Box::new(needless_for_each::NeedlessForEach));
556 store.register_late_pass(|_| Box::<misc::LintPass>::default());
557 store.register_late_pass(|_| Box::new(eta_reduction::EtaReduction));
558 store.register_late_pass(|_| Box::new(mut_mut::MutMut));
559 store.register_late_pass(|_| Box::new(mut_reference::UnnecessaryMutPassed));
560 store.register_late_pass(|_| Box::new(len_zero::LenZero));
561 store.register_late_pass(|_| Box::new(attrs::Attributes));
562 store.register_late_pass(|_| Box::new(blocks_in_if_conditions::BlocksInIfConditions));
563 store.register_late_pass(|_| Box::new(unicode::Unicode));
564 store.register_late_pass(|_| Box::new(uninit_vec::UninitVec));
565 store.register_late_pass(|_| Box::new(unit_return_expecting_ord::UnitReturnExpectingOrd));
566 store.register_late_pass(|_| Box::new(strings::StringAdd));
567 store.register_late_pass(|_| Box::new(implicit_return::ImplicitReturn));
568 store.register_late_pass(|_| Box::new(implicit_saturating_sub::ImplicitSaturatingSub));
569 store.register_late_pass(|_| Box::new(default_numeric_fallback::DefaultNumericFallback));
570 store.register_late_pass(|_| Box::new(inconsistent_struct_constructor::InconsistentStructConstructor));
571 store.register_late_pass(|_| Box::new(non_octal_unix_permissions::NonOctalUnixPermissions));
572 store.register_early_pass(|| Box::new(unnecessary_self_imports::UnnecessarySelfImports));
574 let msrv = Msrv::read(&conf.msrv, sess);
575 let msrv = move || msrv.clone();
576 let avoid_breaking_exported_api = conf.avoid_breaking_exported_api;
577 let allow_expect_in_tests = conf.allow_expect_in_tests;
578 let allow_unwrap_in_tests = conf.allow_unwrap_in_tests;
579 let suppress_restriction_lint_in_const = conf.suppress_restriction_lint_in_const;
580 store.register_late_pass(move |_| Box::new(approx_const::ApproxConstant::new(msrv())));
581 store.register_late_pass(move |_| {
582 Box::new(methods::Methods::new(
583 avoid_breaking_exported_api,
585 allow_expect_in_tests,
586 allow_unwrap_in_tests,
589 store.register_late_pass(move |_| Box::new(matches::Matches::new(msrv())));
590 let matches_for_let_else = conf.matches_for_let_else;
591 store.register_late_pass(move |_| Box::new(manual_let_else::ManualLetElse::new(msrv(), matches_for_let_else)));
592 store.register_early_pass(move || Box::new(manual_non_exhaustive::ManualNonExhaustiveStruct::new(msrv())));
593 store.register_late_pass(move |_| Box::new(manual_non_exhaustive::ManualNonExhaustiveEnum::new(msrv())));
594 store.register_late_pass(move |_| Box::new(manual_strip::ManualStrip::new(msrv())));
595 store.register_early_pass(move || Box::new(redundant_static_lifetimes::RedundantStaticLifetimes::new(msrv())));
596 store.register_early_pass(move || Box::new(redundant_field_names::RedundantFieldNames::new(msrv())));
597 store.register_late_pass(move |_| Box::new(checked_conversions::CheckedConversions::new(msrv())));
598 store.register_late_pass(move |_| Box::new(mem_replace::MemReplace::new(msrv())));
599 store.register_late_pass(move |_| Box::new(ranges::Ranges::new(msrv())));
600 store.register_late_pass(move |_| Box::new(from_over_into::FromOverInto::new(msrv())));
601 store.register_late_pass(move |_| Box::new(use_self::UseSelf::new(msrv())));
602 store.register_late_pass(move |_| Box::new(missing_const_for_fn::MissingConstForFn::new(msrv())));
603 store.register_late_pass(move |_| Box::new(needless_question_mark::NeedlessQuestionMark));
604 store.register_late_pass(move |_| Box::new(casts::Casts::new(msrv())));
605 store.register_early_pass(move || Box::new(unnested_or_patterns::UnnestedOrPatterns::new(msrv())));
606 store.register_late_pass(|_| Box::new(size_of_in_element_count::SizeOfInElementCount));
607 store.register_late_pass(|_| Box::new(same_name_method::SameNameMethod));
608 let max_suggested_slice_pattern_length = conf.max_suggested_slice_pattern_length;
609 store.register_late_pass(move |_| {
610 Box::new(index_refutable_slice::IndexRefutableSlice::new(
611 max_suggested_slice_pattern_length,
615 store.register_late_pass(|_| Box::<shadow::Shadow>::default());
616 store.register_late_pass(|_| Box::new(unit_types::UnitTypes));
617 store.register_late_pass(|_| Box::new(loops::Loops));
618 store.register_late_pass(|_| Box::<main_recursion::MainRecursion>::default());
619 store.register_late_pass(|_| Box::new(lifetimes::Lifetimes));
620 store.register_late_pass(|_| Box::new(entry::HashMapPass));
621 store.register_late_pass(|_| Box::new(minmax::MinMaxPass));
622 store.register_late_pass(|_| Box::new(zero_div_zero::ZeroDiv));
623 store.register_late_pass(|_| Box::new(mutex_atomic::Mutex));
624 store.register_late_pass(|_| Box::new(needless_update::NeedlessUpdate));
625 store.register_late_pass(|_| Box::new(needless_borrowed_ref::NeedlessBorrowedRef));
626 store.register_late_pass(|_| Box::new(borrow_deref_ref::BorrowDerefRef));
627 store.register_late_pass(|_| Box::new(no_effect::NoEffect));
628 store.register_late_pass(|_| Box::new(temporary_assignment::TemporaryAssignment));
629 store.register_late_pass(move |_| Box::new(transmute::Transmute::new(msrv())));
630 let cognitive_complexity_threshold = conf.cognitive_complexity_threshold;
631 store.register_late_pass(move |_| {
632 Box::new(cognitive_complexity::CognitiveComplexity::new(
633 cognitive_complexity_threshold,
636 let too_large_for_stack = conf.too_large_for_stack;
637 store.register_late_pass(move |_| Box::new(escape::BoxedLocal { too_large_for_stack }));
638 store.register_late_pass(move |_| Box::new(vec::UselessVec { too_large_for_stack }));
639 store.register_late_pass(|_| Box::new(panic_unimplemented::PanicUnimplemented));
640 store.register_late_pass(|_| Box::new(strings::StringLitAsBytes));
641 store.register_late_pass(|_| Box::new(derive::Derive));
642 store.register_late_pass(|_| Box::new(derivable_impls::DerivableImpls));
643 store.register_late_pass(|_| Box::new(drop_forget_ref::DropForgetRef));
644 store.register_late_pass(|_| Box::new(empty_enum::EmptyEnum));
645 store.register_late_pass(|_| Box::new(invalid_upcast_comparisons::InvalidUpcastComparisons));
646 store.register_late_pass(|_| Box::new(regex::Regex));
647 store.register_late_pass(|_| Box::new(copies::CopyAndPaste));
648 store.register_late_pass(|_| Box::new(copy_iterator::CopyIterator));
649 store.register_late_pass(|_| Box::new(format::UselessFormat));
650 store.register_late_pass(|_| Box::new(swap::Swap));
651 store.register_late_pass(|_| Box::new(overflow_check_conditional::OverflowCheckConditional));
652 store.register_late_pass(|_| Box::<new_without_default::NewWithoutDefault>::default());
653 let disallowed_names = conf.disallowed_names.iter().cloned().collect::<FxHashSet<_>>();
654 store.register_late_pass(move |_| Box::new(disallowed_names::DisallowedNames::new(disallowed_names.clone())));
655 let too_many_arguments_threshold = conf.too_many_arguments_threshold;
656 let too_many_lines_threshold = conf.too_many_lines_threshold;
657 let large_error_threshold = conf.large_error_threshold;
658 store.register_late_pass(move |_| {
659 Box::new(functions::Functions::new(
660 too_many_arguments_threshold,
661 too_many_lines_threshold,
662 large_error_threshold,
665 let doc_valid_idents = conf.doc_valid_idents.iter().cloned().collect::<FxHashSet<_>>();
666 store.register_late_pass(move |_| Box::new(doc::DocMarkdown::new(doc_valid_idents.clone())));
667 store.register_late_pass(|_| Box::new(neg_multiply::NegMultiply));
668 store.register_late_pass(|_| Box::new(mem_forget::MemForget));
669 store.register_late_pass(|_| Box::new(let_if_seq::LetIfSeq));
670 store.register_late_pass(|_| Box::new(mixed_read_write_in_expression::EvalOrderDependence));
671 store.register_late_pass(|_| Box::new(missing_doc::MissingDoc::new()));
672 store.register_late_pass(|_| Box::new(missing_inline::MissingInline));
673 store.register_late_pass(move |_| Box::new(exhaustive_items::ExhaustiveItems));
674 store.register_late_pass(|_| Box::new(match_result_ok::MatchResultOk));
675 store.register_late_pass(|_| Box::new(partialeq_ne_impl::PartialEqNeImpl));
676 store.register_late_pass(|_| Box::new(unused_io_amount::UnusedIoAmount));
677 let enum_variant_size_threshold = conf.enum_variant_size_threshold;
678 store.register_late_pass(move |_| Box::new(large_enum_variant::LargeEnumVariant::new(enum_variant_size_threshold)));
679 store.register_late_pass(|_| Box::new(explicit_write::ExplicitWrite));
680 store.register_late_pass(|_| Box::new(needless_pass_by_value::NeedlessPassByValue));
681 let pass_by_ref_or_value = pass_by_ref_or_value::PassByRefOrValue::new(
682 conf.trivial_copy_size_limit,
683 conf.pass_by_value_size_limit,
684 conf.avoid_breaking_exported_api,
687 store.register_late_pass(move |_| Box::new(pass_by_ref_or_value));
688 store.register_late_pass(|_| Box::new(ref_option_ref::RefOptionRef));
689 store.register_late_pass(|_| Box::new(infinite_iter::InfiniteIter));
690 store.register_late_pass(|_| Box::new(inline_fn_without_body::InlineFnWithoutBody));
691 store.register_late_pass(|_| Box::<useless_conversion::UselessConversion>::default());
692 store.register_late_pass(|_| Box::new(implicit_hasher::ImplicitHasher));
693 store.register_late_pass(|_| Box::new(fallible_impl_from::FallibleImplFrom));
694 store.register_late_pass(|_| Box::new(question_mark::QuestionMark));
695 store.register_early_pass(|| Box::new(suspicious_operation_groupings::SuspiciousOperationGroupings));
696 store.register_late_pass(|_| Box::new(suspicious_trait_impl::SuspiciousImpl));
697 store.register_late_pass(|_| Box::new(map_unit_fn::MapUnit));
698 store.register_late_pass(|_| Box::new(inherent_impl::MultipleInherentImpl));
699 store.register_late_pass(|_| Box::new(neg_cmp_op_on_partial_ord::NoNegCompOpForPartialOrd));
700 store.register_late_pass(|_| Box::new(unwrap::Unwrap));
701 store.register_late_pass(move |_| {
702 Box::new(indexing_slicing::IndexingSlicing::new(
703 suppress_restriction_lint_in_const,
706 store.register_late_pass(|_| Box::new(non_copy_const::NonCopyConst));
707 store.register_late_pass(|_| Box::new(ptr_offset_with_cast::PtrOffsetWithCast));
708 store.register_late_pass(|_| Box::new(redundant_clone::RedundantClone));
709 store.register_late_pass(|_| Box::new(slow_vector_initialization::SlowVectorInit));
710 store.register_late_pass(move |_| Box::new(unnecessary_wraps::UnnecessaryWraps::new(avoid_breaking_exported_api)));
711 store.register_late_pass(|_| Box::new(assertions_on_constants::AssertionsOnConstants));
712 store.register_late_pass(|_| Box::new(assertions_on_result_states::AssertionsOnResultStates));
713 store.register_late_pass(|_| Box::new(inherent_to_string::InherentToString));
714 let max_trait_bounds = conf.max_trait_bounds;
715 store.register_late_pass(move |_| Box::new(trait_bounds::TraitBounds::new(max_trait_bounds)));
716 store.register_late_pass(|_| Box::new(comparison_chain::ComparisonChain));
717 let ignore_interior_mutability = conf.ignore_interior_mutability.clone();
718 store.register_late_pass(move |_| Box::new(mut_key::MutableKeyType::new(ignore_interior_mutability.clone())));
719 store.register_early_pass(|| Box::new(reference::DerefAddrOf));
720 store.register_early_pass(|| Box::new(double_parens::DoubleParens));
721 store.register_late_pass(|_| Box::new(format_impl::FormatImpl::new()));
722 store.register_early_pass(|| Box::new(unsafe_removed_from_name::UnsafeNameRemoval));
723 store.register_early_pass(|| Box::new(else_if_without_else::ElseIfWithoutElse));
724 store.register_early_pass(|| Box::new(int_plus_one::IntPlusOne));
725 store.register_early_pass(|| Box::new(formatting::Formatting));
726 store.register_early_pass(|| Box::new(misc_early::MiscEarlyLints));
727 store.register_early_pass(|| Box::new(redundant_closure_call::RedundantClosureCall));
728 store.register_late_pass(|_| Box::new(redundant_closure_call::RedundantClosureCall));
729 store.register_early_pass(|| Box::new(unused_unit::UnusedUnit));
730 store.register_late_pass(|_| Box::new(returns::Return));
731 store.register_early_pass(|| Box::new(collapsible_if::CollapsibleIf));
732 store.register_early_pass(|| Box::new(items_after_statements::ItemsAfterStatements));
733 store.register_early_pass(|| Box::new(precedence::Precedence));
734 store.register_late_pass(|_| Box::new(needless_parens_on_range_literals::NeedlessParensOnRangeLiterals));
735 store.register_early_pass(|| Box::new(needless_continue::NeedlessContinue));
736 store.register_early_pass(|| Box::new(redundant_else::RedundantElse));
737 store.register_late_pass(|_| Box::new(create_dir::CreateDir));
738 store.register_early_pass(|| Box::new(needless_arbitrary_self_type::NeedlessArbitrarySelfType));
739 let literal_representation_lint_fraction_readability = conf.unreadable_literal_lint_fractions;
740 store.register_early_pass(move || {
741 Box::new(literal_representation::LiteralDigitGrouping::new(
742 literal_representation_lint_fraction_readability,
745 let literal_representation_threshold = conf.literal_representation_threshold;
746 store.register_early_pass(move || {
747 Box::new(literal_representation::DecimalLiteralRepresentation::new(
748 literal_representation_threshold,
751 let enum_variant_name_threshold = conf.enum_variant_name_threshold;
752 store.register_late_pass(move |_| {
753 Box::new(enum_variants::EnumVariantNames::new(
754 enum_variant_name_threshold,
755 avoid_breaking_exported_api,
758 store.register_early_pass(|| Box::new(tabs_in_doc_comments::TabsInDocComments));
759 let upper_case_acronyms_aggressive = conf.upper_case_acronyms_aggressive;
760 store.register_late_pass(move |_| {
761 Box::new(upper_case_acronyms::UpperCaseAcronyms::new(
762 avoid_breaking_exported_api,
763 upper_case_acronyms_aggressive,
766 store.register_late_pass(|_| Box::<default::Default>::default());
767 store.register_late_pass(move |_| Box::new(unused_self::UnusedSelf::new(avoid_breaking_exported_api)));
768 store.register_late_pass(|_| Box::new(mutable_debug_assertion::DebugAssertWithMutCall));
769 store.register_late_pass(|_| Box::new(exit::Exit));
770 store.register_late_pass(|_| Box::new(to_digit_is_some::ToDigitIsSome));
771 let array_size_threshold = conf.array_size_threshold;
772 store.register_late_pass(move |_| Box::new(large_stack_arrays::LargeStackArrays::new(array_size_threshold)));
773 store.register_late_pass(move |_| Box::new(large_const_arrays::LargeConstArrays::new(array_size_threshold)));
774 store.register_late_pass(|_| Box::new(floating_point_arithmetic::FloatingPointArithmetic));
775 store.register_early_pass(|| Box::new(as_conversions::AsConversions));
776 store.register_late_pass(|_| Box::new(let_underscore::LetUnderscore));
777 store.register_early_pass(|| Box::<single_component_path_imports::SingleComponentPathImports>::default());
778 let max_fn_params_bools = conf.max_fn_params_bools;
779 let max_struct_bools = conf.max_struct_bools;
780 store.register_late_pass(move |_| {
781 Box::new(excessive_bools::ExcessiveBools::new(
786 store.register_early_pass(|| Box::new(option_env_unwrap::OptionEnvUnwrap));
787 let warn_on_all_wildcard_imports = conf.warn_on_all_wildcard_imports;
788 store.register_late_pass(move |_| Box::new(wildcard_imports::WildcardImports::new(warn_on_all_wildcard_imports)));
789 store.register_late_pass(|_| Box::<redundant_pub_crate::RedundantPubCrate>::default());
790 store.register_late_pass(|_| Box::new(unnamed_address::UnnamedAddress));
791 store.register_late_pass(move |_| Box::new(dereference::Dereferencing::new(msrv())));
792 store.register_late_pass(|_| Box::new(option_if_let_else::OptionIfLetElse));
793 store.register_late_pass(|_| Box::new(future_not_send::FutureNotSend));
794 store.register_late_pass(|_| Box::new(if_let_mutex::IfLetMutex));
795 store.register_late_pass(|_| Box::new(if_not_else::IfNotElse));
796 store.register_late_pass(|_| Box::new(equatable_if_let::PatternEquality));
797 store.register_late_pass(|_| Box::new(manual_async_fn::ManualAsyncFn));
798 store.register_late_pass(|_| Box::new(panic_in_result_fn::PanicInResultFn));
799 let single_char_binding_names_threshold = conf.single_char_binding_names_threshold;
800 store.register_early_pass(move || {
801 Box::new(non_expressive_names::NonExpressiveNames {
802 single_char_binding_names_threshold,
805 let macro_matcher = conf.standard_macro_braces.iter().cloned().collect::<FxHashSet<_>>();
806 store.register_early_pass(move || Box::new(nonstandard_macro_braces::MacroBraces::new(¯o_matcher)));
807 store.register_late_pass(|_| Box::<macro_use::MacroUseImports>::default());
808 store.register_late_pass(|_| Box::new(pattern_type_mismatch::PatternTypeMismatch));
809 store.register_late_pass(|_| Box::new(unwrap_in_result::UnwrapInResult));
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_macros = conf.disallowed_macros.clone();
813 store.register_late_pass(move |_| Box::new(disallowed_macros::DisallowedMacros::new(disallowed_macros.clone())));
814 let disallowed_methods = conf.disallowed_methods.clone();
815 store.register_late_pass(move |_| Box::new(disallowed_methods::DisallowedMethods::new(disallowed_methods.clone())));
816 store.register_early_pass(|| Box::new(asm_syntax::InlineAsmX86AttSyntax));
817 store.register_early_pass(|| Box::new(asm_syntax::InlineAsmX86IntelSyntax));
818 store.register_late_pass(|_| Box::new(empty_drop::EmptyDrop));
819 store.register_late_pass(|_| Box::new(strings::StrToString));
820 store.register_late_pass(|_| Box::new(strings::StringToString));
821 store.register_late_pass(|_| Box::new(zero_sized_map_values::ZeroSizedMapValues));
822 store.register_late_pass(|_| Box::<vec_init_then_push::VecInitThenPush>::default());
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(move |_| Box::new(if_then_some_else_none::IfThenSomeElseNone::new(msrv())));
826 store.register_late_pass(|_| Box::new(bool_assert_comparison::BoolAssertComparison));
827 store.register_early_pass(move || Box::new(module_style::ModStyle));
828 store.register_late_pass(|_| Box::new(unused_async::UnusedAsync));
829 let disallowed_types = conf.disallowed_types.clone();
830 store.register_late_pass(move |_| Box::new(disallowed_types::DisallowedTypes::new(disallowed_types.clone())));
831 let import_renames = conf.enforced_import_renames.clone();
832 store.register_late_pass(move |_| {
833 Box::new(missing_enforced_import_rename::ImportRename::new(
834 import_renames.clone(),
837 let scripts = conf.allowed_scripts.clone();
838 store.register_early_pass(move || Box::new(disallowed_script_idents::DisallowedScriptIdents::new(&scripts)));
839 store.register_late_pass(|_| Box::new(strlen_on_c_strings::StrlenOnCStrings));
840 store.register_late_pass(move |_| Box::new(self_named_constructors::SelfNamedConstructors));
841 store.register_late_pass(move |_| Box::new(iter_not_returning_iterator::IterNotReturningIterator));
842 store.register_late_pass(move |_| Box::new(manual_assert::ManualAssert));
843 let enable_raw_pointer_heuristic_for_send = conf.enable_raw_pointer_heuristic_for_send;
844 store.register_late_pass(move |_| {
845 Box::new(non_send_fields_in_send_ty::NonSendFieldInSendTy::new(
846 enable_raw_pointer_heuristic_for_send,
849 store.register_late_pass(move |_| Box::new(undocumented_unsafe_blocks::UndocumentedUnsafeBlocks));
850 let allow_mixed_uninlined = conf.allow_mixed_uninlined_format_args;
851 store.register_late_pass(move |_| Box::new(format_args::FormatArgs::new(msrv(), allow_mixed_uninlined)));
852 store.register_late_pass(|_| Box::new(trailing_empty_array::TrailingEmptyArray));
853 store.register_early_pass(|| Box::new(octal_escapes::OctalEscapes));
854 store.register_late_pass(|_| Box::new(needless_late_init::NeedlessLateInit));
855 store.register_late_pass(|_| Box::new(return_self_not_must_use::ReturnSelfNotMustUse));
856 store.register_late_pass(|_| Box::new(init_numbered_fields::NumberedFields));
857 store.register_early_pass(|| Box::new(single_char_lifetime_names::SingleCharLifetimeNames));
858 store.register_late_pass(move |_| Box::new(manual_bits::ManualBits::new(msrv())));
859 store.register_late_pass(|_| Box::new(default_union_representation::DefaultUnionRepresentation));
860 store.register_late_pass(|_| Box::<only_used_in_recursion::OnlyUsedInRecursion>::default());
861 let allow_dbg_in_tests = conf.allow_dbg_in_tests;
862 store.register_late_pass(move |_| Box::new(dbg_macro::DbgMacro::new(allow_dbg_in_tests)));
863 let allow_print_in_tests = conf.allow_print_in_tests;
864 store.register_late_pass(move |_| Box::new(write::Write::new(allow_print_in_tests)));
865 let cargo_ignore_publish = conf.cargo_ignore_publish;
866 store.register_late_pass(move |_| {
867 Box::new(cargo::Cargo {
868 ignore_publish: cargo_ignore_publish,
871 store.register_early_pass(|| Box::new(crate_in_macro_def::CrateInMacroDef));
872 store.register_early_pass(|| Box::new(empty_structs_with_brackets::EmptyStructsWithBrackets));
873 store.register_late_pass(|_| Box::new(unnecessary_owned_empty_strings::UnnecessaryOwnedEmptyStrings));
874 store.register_early_pass(|| Box::new(pub_use::PubUse));
875 store.register_late_pass(|_| Box::new(format_push_string::FormatPushString));
876 let max_include_file_size = conf.max_include_file_size;
877 store.register_late_pass(move |_| Box::new(large_include_file::LargeIncludeFile::new(max_include_file_size)));
878 store.register_late_pass(|_| Box::new(strings::TrimSplitWhitespace));
879 store.register_late_pass(|_| Box::new(rc_clone_in_vec_init::RcCloneInVecInit));
880 store.register_early_pass(|| Box::<duplicate_mod::DuplicateMod>::default());
881 store.register_early_pass(|| Box::new(unused_rounding::UnusedRounding));
882 store.register_early_pass(move || Box::new(almost_complete_range::AlmostCompleteRange::new(msrv())));
883 store.register_late_pass(|_| Box::new(swap_ptr_to_ref::SwapPtrToRef));
884 store.register_late_pass(|_| Box::new(mismatching_type_param_order::TypeParamMismatch));
885 store.register_late_pass(|_| Box::new(read_zero_byte_vec::ReadZeroByteVec));
886 store.register_late_pass(|_| Box::new(default_instead_of_iter_empty::DefaultIterEmpty));
887 store.register_late_pass(move |_| Box::new(manual_rem_euclid::ManualRemEuclid::new(msrv())));
888 store.register_late_pass(move |_| Box::new(manual_retain::ManualRetain::new(msrv())));
889 let verbose_bit_mask_threshold = conf.verbose_bit_mask_threshold;
890 store.register_late_pass(move |_| Box::new(operators::Operators::new(verbose_bit_mask_threshold)));
891 store.register_late_pass(|_| Box::new(invalid_utf8_in_unchecked::InvalidUtf8InUnchecked));
892 store.register_late_pass(|_| Box::<std_instead_of_core::StdReexports>::default());
893 store.register_late_pass(move |_| Box::new(instant_subtraction::InstantSubtraction::new(msrv())));
894 store.register_late_pass(|_| Box::new(partialeq_to_none::PartialeqToNone));
895 store.register_late_pass(move |_| Box::new(manual_clamp::ManualClamp::new(msrv())));
896 store.register_late_pass(|_| Box::new(manual_string_new::ManualStringNew));
897 store.register_late_pass(|_| Box::new(unused_peekable::UnusedPeekable));
898 store.register_early_pass(|| Box::new(multi_assignments::MultiAssignments));
899 store.register_late_pass(|_| Box::new(bool_to_int_with_if::BoolToIntWithIf));
900 store.register_late_pass(|_| Box::new(box_default::BoxDefault));
901 store.register_late_pass(|_| Box::new(implicit_saturating_add::ImplicitSaturatingAdd));
902 store.register_early_pass(|| Box::new(partial_pub_fields::PartialPubFields));
903 store.register_late_pass(|_| Box::new(missing_trait_methods::MissingTraitMethods));
904 store.register_late_pass(|_| Box::new(from_raw_with_void_ptr::FromRawWithVoidPtr));
905 store.register_late_pass(|_| Box::new(suspicious_xor_used_as_pow::ConfusingXorAndPow));
906 store.register_late_pass(move |_| Box::new(manual_is_ascii_check::ManualIsAsciiCheck::new(msrv())));
907 store.register_late_pass(|_| Box::new(semicolon_block::SemicolonBlock));
908 store.register_late_pass(|_| Box::new(fn_null_check::FnNullCheck));
909 store.register_late_pass(|_| Box::new(permissions_set_readonly_false::PermissionsSetReadonlyFalse));
910 store.register_late_pass(|_| Box::new(size_of_ref::SizeOfRef));
911 // add lints here, do not remove this comment, it's used in `new_lint`
915 fn register_removed_non_tool_lints(store: &mut rustc_lint::LintStore) {
916 store.register_removed(
918 "`assert!()` will be more flexible with RFC 2011",
920 store.register_removed(
922 "`.extend_from_slice(_)` is a faster way to extend a Vec by a slice",
924 store.register_removed(
925 "range_step_by_zero",
926 "`iterator.step_by(0)` panics nowadays",
928 store.register_removed(
930 "`Vec::as_slice` has been stabilized in 1.7",
932 store.register_removed(
933 "unstable_as_mut_slice",
934 "`Vec::as_mut_slice` has been stabilized in 1.7",
936 store.register_removed(
937 "misaligned_transmute",
938 "this lint has been split into cast_ptr_alignment and transmute_ptr_to_ptr",
940 store.register_removed(
942 "using compound assignment operators (e.g., `+=`) is harmless",
944 store.register_removed(
945 "if_let_redundant_pattern_matching",
946 "this lint has been changed to redundant_pattern_matching",
948 store.register_removed(
949 "unsafe_vector_initialization",
950 "the replacement suggested by this lint had substantially different behavior",
952 store.register_removed(
953 "reverse_range_loop",
954 "this lint is now included in reversed_empty_ranges",
958 /// Register renamed lints.
960 /// Used in `./src/driver.rs`.
961 pub fn register_renamed(ls: &mut rustc_lint::LintStore) {
962 for (old_name, new_name) in renamed_lints::RENAMED_LINTS {
963 ls.register_renamed(old_name, new_name);
967 // only exists to let the dogfood integration test works.
968 // Don't run clippy as an executable directly
971 panic!("Please use the cargo-clippy executable");