| //! Lints, aka compiler warnings. |
| //! |
| //! A 'lint' check is a kind of miscellaneous constraint that a user _might_ |
| //! want to enforce, but might reasonably want to permit as well, on a |
| //! module-by-module basis. They contrast with static constraints enforced by |
| //! other phases of the compiler, which are generally required to hold in order |
| //! to compile the program at all. |
| //! |
| //! Most lints can be written as `LintPass` instances. These run after |
| //! all other analyses. The `LintPass`es built into rustc are defined |
| //! within `builtin.rs`, which has further comments on how to add such a lint. |
| //! rustc can also load user-defined lint plugins via the plugin mechanism. |
| //! |
| //! Some of rustc's lints are defined elsewhere in the compiler and work by |
| //! calling `add_lint()` on the overall `Session` object. This works when |
| //! it happens before the main lint pass, which emits the lints stored by |
| //! `add_lint()`. To emit lints after the main lint pass (from codegen, for |
| //! example) requires more effort. See `emit_lint` and `GatherNodeLevels` |
| //! in `context.rs`. |
| |
| pub use self::Level::*; |
| pub use self::LintSource::*; |
| |
| use rustc_data_structures::sync; |
| |
| use crate::hir; |
| use crate::lint::builtin::BuiltinLintDiagnostics; |
| use crate::ty::TyCtxt; |
| use errors::{DiagnosticBuilder, DiagnosticId}; |
| use rustc_session::node_id::NodeMap; |
| use rustc_session::{DiagnosticMessageId, Session}; |
| use rustc_span::hygiene::MacroKind; |
| use rustc_span::source_map::{DesugaringKind, ExpnKind, MultiSpan}; |
| use rustc_span::symbol::Symbol; |
| use rustc_span::Span; |
| use syntax::ast; |
| |
| pub use crate::lint::context::{ |
| BufferedEarlyLint, CheckLintNameResult, EarlyContext, LateContext, LintContext, LintStore, |
| }; |
| |
| pub use rustc_session::lint::{FutureIncompatibleInfo, Level, Lint, LintId}; |
| |
| /// Declares a static `LintArray` and return it as an expression. |
| #[macro_export] |
| macro_rules! lint_array { |
| ($( $lint:expr ),* ,) => { lint_array!( $($lint),* ) }; |
| ($( $lint:expr ),*) => {{ |
| vec![$($lint),*] |
| }} |
| } |
| |
| pub type LintArray = Vec<&'static Lint>; |
| |
| pub trait LintPass { |
| fn name(&self) -> &'static str; |
| } |
| |
| /// Implements `LintPass for $name` with the given list of `Lint` statics. |
| #[macro_export] |
| macro_rules! impl_lint_pass { |
| ($name:ident => [$($lint:expr),* $(,)?]) => { |
| impl LintPass for $name { |
| fn name(&self) -> &'static str { stringify!($name) } |
| } |
| impl $name { |
| pub fn get_lints() -> LintArray { $crate::lint_array!($($lint),*) } |
| } |
| }; |
| } |
| |
| /// Declares a type named `$name` which implements `LintPass`. |
| /// To the right of `=>` a comma separated list of `Lint` statics is given. |
| #[macro_export] |
| macro_rules! declare_lint_pass { |
| ($(#[$m:meta])* $name:ident => [$($lint:expr),* $(,)?]) => { |
| $(#[$m])* #[derive(Copy, Clone)] pub struct $name; |
| $crate::impl_lint_pass!($name => [$($lint),*]); |
| }; |
| } |
| |
| #[macro_export] |
| macro_rules! late_lint_methods { |
| ($macro:path, $args:tt, [$hir:tt]) => ( |
| $macro!($args, [$hir], [ |
| fn check_param(a: &$hir hir::Param<$hir>); |
| fn check_body(a: &$hir hir::Body<$hir>); |
| fn check_body_post(a: &$hir hir::Body<$hir>); |
| fn check_name(a: Span, b: ast::Name); |
| fn check_crate(a: &$hir hir::Crate<$hir>); |
| fn check_crate_post(a: &$hir hir::Crate<$hir>); |
| fn check_mod(a: &$hir hir::Mod<$hir>, b: Span, c: hir::HirId); |
| fn check_mod_post(a: &$hir hir::Mod<$hir>, b: Span, c: hir::HirId); |
| fn check_foreign_item(a: &$hir hir::ForeignItem<$hir>); |
| fn check_foreign_item_post(a: &$hir hir::ForeignItem<$hir>); |
| fn check_item(a: &$hir hir::Item<$hir>); |
| fn check_item_post(a: &$hir hir::Item<$hir>); |
| fn check_local(a: &$hir hir::Local<$hir>); |
| fn check_block(a: &$hir hir::Block<$hir>); |
| fn check_block_post(a: &$hir hir::Block<$hir>); |
| fn check_stmt(a: &$hir hir::Stmt<$hir>); |
| fn check_arm(a: &$hir hir::Arm<$hir>); |
| fn check_pat(a: &$hir hir::Pat<$hir>); |
| fn check_expr(a: &$hir hir::Expr<$hir>); |
| fn check_expr_post(a: &$hir hir::Expr<$hir>); |
| fn check_ty(a: &$hir hir::Ty<$hir>); |
| fn check_generic_param(a: &$hir hir::GenericParam<$hir>); |
| fn check_generics(a: &$hir hir::Generics<$hir>); |
| fn check_where_predicate(a: &$hir hir::WherePredicate<$hir>); |
| fn check_poly_trait_ref(a: &$hir hir::PolyTraitRef<$hir>, b: hir::TraitBoundModifier); |
| fn check_fn( |
| a: hir::intravisit::FnKind<$hir>, |
| b: &$hir hir::FnDecl<$hir>, |
| c: &$hir hir::Body<$hir>, |
| d: Span, |
| e: hir::HirId); |
| fn check_fn_post( |
| a: hir::intravisit::FnKind<$hir>, |
| b: &$hir hir::FnDecl<$hir>, |
| c: &$hir hir::Body<$hir>, |
| d: Span, |
| e: hir::HirId |
| ); |
| fn check_trait_item(a: &$hir hir::TraitItem<$hir>); |
| fn check_trait_item_post(a: &$hir hir::TraitItem<$hir>); |
| fn check_impl_item(a: &$hir hir::ImplItem<$hir>); |
| fn check_impl_item_post(a: &$hir hir::ImplItem<$hir>); |
| fn check_struct_def(a: &$hir hir::VariantData<$hir>); |
| fn check_struct_def_post(a: &$hir hir::VariantData<$hir>); |
| fn check_struct_field(a: &$hir hir::StructField<$hir>); |
| fn check_variant(a: &$hir hir::Variant<$hir>); |
| fn check_variant_post(a: &$hir hir::Variant<$hir>); |
| fn check_lifetime(a: &$hir hir::Lifetime); |
| fn check_path(a: &$hir hir::Path<$hir>, b: hir::HirId); |
| fn check_attribute(a: &$hir ast::Attribute); |
| |
| /// Called when entering a syntax node that can have lint attributes such |
| /// as `#[allow(...)]`. Called with *all* the attributes of that node. |
| fn enter_lint_attrs(a: &$hir [ast::Attribute]); |
| |
| /// Counterpart to `enter_lint_attrs`. |
| fn exit_lint_attrs(a: &$hir [ast::Attribute]); |
| ]); |
| ) |
| } |
| |
| /// Trait for types providing lint checks. |
| /// |
| /// Each `check` method checks a single syntax node, and should not |
| /// invoke methods recursively (unlike `Visitor`). By default they |
| /// do nothing. |
| // |
| // FIXME: eliminate the duplication with `Visitor`. But this also |
| // contains a few lint-specific methods with no equivalent in `Visitor`. |
| |
| macro_rules! expand_lint_pass_methods { |
| ($context:ty, [$($(#[$attr:meta])* fn $name:ident($($param:ident: $arg:ty),*);)*]) => ( |
| $(#[inline(always)] fn $name(&mut self, _: $context, $(_: $arg),*) {})* |
| ) |
| } |
| |
| macro_rules! declare_late_lint_pass { |
| ([], [$hir:tt], [$($methods:tt)*]) => ( |
| pub trait LateLintPass<'a, $hir>: LintPass { |
| expand_lint_pass_methods!(&LateContext<'a, $hir>, [$($methods)*]); |
| } |
| ) |
| } |
| |
| late_lint_methods!(declare_late_lint_pass, [], ['tcx]); |
| |
| #[macro_export] |
| macro_rules! expand_combined_late_lint_pass_method { |
| ([$($passes:ident),*], $self: ident, $name: ident, $params:tt) => ({ |
| $($self.$passes.$name $params;)* |
| }) |
| } |
| |
| #[macro_export] |
| macro_rules! expand_combined_late_lint_pass_methods { |
| ($passes:tt, [$($(#[$attr:meta])* fn $name:ident($($param:ident: $arg:ty),*);)*]) => ( |
| $(fn $name(&mut self, context: &LateContext<'a, 'tcx>, $($param: $arg),*) { |
| expand_combined_late_lint_pass_method!($passes, self, $name, (context, $($param),*)); |
| })* |
| ) |
| } |
| |
| #[macro_export] |
| macro_rules! declare_combined_late_lint_pass { |
| ([$v:vis $name:ident, [$($passes:ident: $constructor:expr,)*]], [$hir:tt], $methods:tt) => ( |
| #[allow(non_snake_case)] |
| $v struct $name { |
| $($passes: $passes,)* |
| } |
| |
| impl $name { |
| $v fn new() -> Self { |
| Self { |
| $($passes: $constructor,)* |
| } |
| } |
| |
| $v fn get_lints() -> LintArray { |
| let mut lints = Vec::new(); |
| $(lints.extend_from_slice(&$passes::get_lints());)* |
| lints |
| } |
| } |
| |
| impl<'a, 'tcx> LateLintPass<'a, 'tcx> for $name { |
| expand_combined_late_lint_pass_methods!([$($passes),*], $methods); |
| } |
| |
| impl LintPass for $name { |
| fn name(&self) -> &'static str { |
| panic!() |
| } |
| } |
| ) |
| } |
| |
| #[macro_export] |
| macro_rules! early_lint_methods { |
| ($macro:path, $args:tt) => ( |
| $macro!($args, [ |
| fn check_param(a: &ast::Param); |
| fn check_ident(a: ast::Ident); |
| fn check_crate(a: &ast::Crate); |
| fn check_crate_post(a: &ast::Crate); |
| fn check_mod(a: &ast::Mod, b: Span, c: ast::NodeId); |
| fn check_mod_post(a: &ast::Mod, b: Span, c: ast::NodeId); |
| fn check_foreign_item(a: &ast::ForeignItem); |
| fn check_foreign_item_post(a: &ast::ForeignItem); |
| fn check_item(a: &ast::Item); |
| fn check_item_post(a: &ast::Item); |
| fn check_local(a: &ast::Local); |
| fn check_block(a: &ast::Block); |
| fn check_block_post(a: &ast::Block); |
| fn check_stmt(a: &ast::Stmt); |
| fn check_arm(a: &ast::Arm); |
| fn check_pat(a: &ast::Pat); |
| fn check_pat_post(a: &ast::Pat); |
| fn check_expr(a: &ast::Expr); |
| fn check_expr_post(a: &ast::Expr); |
| fn check_ty(a: &ast::Ty); |
| fn check_generic_param(a: &ast::GenericParam); |
| fn check_generics(a: &ast::Generics); |
| fn check_where_predicate(a: &ast::WherePredicate); |
| fn check_poly_trait_ref(a: &ast::PolyTraitRef, |
| b: &ast::TraitBoundModifier); |
| fn check_fn(a: syntax::visit::FnKind<'_>, b: &ast::FnDecl, c: Span, d_: ast::NodeId); |
| fn check_fn_post( |
| a: syntax::visit::FnKind<'_>, |
| b: &ast::FnDecl, |
| c: Span, |
| d: ast::NodeId |
| ); |
| fn check_trait_item(a: &ast::AssocItem); |
| fn check_trait_item_post(a: &ast::AssocItem); |
| fn check_impl_item(a: &ast::AssocItem); |
| fn check_impl_item_post(a: &ast::AssocItem); |
| fn check_struct_def(a: &ast::VariantData); |
| fn check_struct_def_post(a: &ast::VariantData); |
| fn check_struct_field(a: &ast::StructField); |
| fn check_variant(a: &ast::Variant); |
| fn check_variant_post(a: &ast::Variant); |
| fn check_lifetime(a: &ast::Lifetime); |
| fn check_path(a: &ast::Path, b: ast::NodeId); |
| fn check_attribute(a: &ast::Attribute); |
| fn check_mac_def(a: &ast::MacroDef, b: ast::NodeId); |
| fn check_mac(a: &ast::Mac); |
| |
| /// Called when entering a syntax node that can have lint attributes such |
| /// as `#[allow(...)]`. Called with *all* the attributes of that node. |
| fn enter_lint_attrs(a: &[ast::Attribute]); |
| |
| /// Counterpart to `enter_lint_attrs`. |
| fn exit_lint_attrs(a: &[ast::Attribute]); |
| ]); |
| ) |
| } |
| |
| macro_rules! expand_early_lint_pass_methods { |
| ($context:ty, [$($(#[$attr:meta])* fn $name:ident($($param:ident: $arg:ty),*);)*]) => ( |
| $(#[inline(always)] fn $name(&mut self, _: $context, $(_: $arg),*) {})* |
| ) |
| } |
| |
| macro_rules! declare_early_lint_pass { |
| ([], [$($methods:tt)*]) => ( |
| pub trait EarlyLintPass: LintPass { |
| expand_early_lint_pass_methods!(&EarlyContext<'_>, [$($methods)*]); |
| } |
| ) |
| } |
| |
| early_lint_methods!(declare_early_lint_pass, []); |
| |
| #[macro_export] |
| macro_rules! expand_combined_early_lint_pass_method { |
| ([$($passes:ident),*], $self: ident, $name: ident, $params:tt) => ({ |
| $($self.$passes.$name $params;)* |
| }) |
| } |
| |
| #[macro_export] |
| macro_rules! expand_combined_early_lint_pass_methods { |
| ($passes:tt, [$($(#[$attr:meta])* fn $name:ident($($param:ident: $arg:ty),*);)*]) => ( |
| $(fn $name(&mut self, context: &EarlyContext<'_>, $($param: $arg),*) { |
| expand_combined_early_lint_pass_method!($passes, self, $name, (context, $($param),*)); |
| })* |
| ) |
| } |
| |
| #[macro_export] |
| macro_rules! declare_combined_early_lint_pass { |
| ([$v:vis $name:ident, [$($passes:ident: $constructor:expr,)*]], $methods:tt) => ( |
| #[allow(non_snake_case)] |
| $v struct $name { |
| $($passes: $passes,)* |
| } |
| |
| impl $name { |
| $v fn new() -> Self { |
| Self { |
| $($passes: $constructor,)* |
| } |
| } |
| |
| $v fn get_lints() -> LintArray { |
| let mut lints = Vec::new(); |
| $(lints.extend_from_slice(&$passes::get_lints());)* |
| lints |
| } |
| } |
| |
| impl EarlyLintPass for $name { |
| expand_combined_early_lint_pass_methods!([$($passes),*], $methods); |
| } |
| |
| impl LintPass for $name { |
| fn name(&self) -> &'static str { |
| panic!() |
| } |
| } |
| ) |
| } |
| |
| /// A lint pass boxed up as a trait object. |
| pub type EarlyLintPassObject = Box<dyn EarlyLintPass + sync::Send + sync::Sync + 'static>; |
| pub type LateLintPassObject = |
| Box<dyn for<'a, 'tcx> LateLintPass<'a, 'tcx> + sync::Send + sync::Sync + 'static>; |
| |
| /// How a lint level was set. |
| #[derive(Clone, Copy, PartialEq, Eq, HashStable)] |
| pub enum LintSource { |
| /// Lint is at the default level as declared |
| /// in rustc or a plugin. |
| Default, |
| |
| /// Lint level was set by an attribute. |
| Node(ast::Name, Span, Option<Symbol> /* RFC 2383 reason */), |
| |
| /// Lint level was set by a command-line flag. |
| CommandLine(Symbol), |
| } |
| |
| pub type LevelSource = (Level, LintSource); |
| |
| pub mod builtin; |
| mod context; |
| pub mod internal; |
| mod levels; |
| |
| pub use self::levels::{LintLevelMap, LintLevelSets, LintLevelsBuilder}; |
| |
| #[derive(Default)] |
| pub struct LintBuffer { |
| pub map: NodeMap<Vec<BufferedEarlyLint>>, |
| } |
| |
| impl LintBuffer { |
| pub fn add_lint( |
| &mut self, |
| lint: &'static Lint, |
| id: ast::NodeId, |
| sp: MultiSpan, |
| msg: &str, |
| diagnostic: BuiltinLintDiagnostics, |
| ) { |
| let early_lint = BufferedEarlyLint { |
| lint_id: LintId::of(lint), |
| ast_id: id, |
| span: sp, |
| msg: msg.to_string(), |
| diagnostic, |
| }; |
| let arr = self.map.entry(id).or_default(); |
| if !arr.contains(&early_lint) { |
| arr.push(early_lint); |
| } |
| } |
| |
| pub fn take(&mut self, id: ast::NodeId) -> Vec<BufferedEarlyLint> { |
| self.map.remove(&id).unwrap_or_default() |
| } |
| |
| pub fn buffer_lint<S: Into<MultiSpan>>( |
| &mut self, |
| lint: &'static Lint, |
| id: ast::NodeId, |
| sp: S, |
| msg: &str, |
| ) { |
| self.add_lint(lint, id, sp.into(), msg, BuiltinLintDiagnostics::Normal) |
| } |
| |
| pub fn buffer_lint_with_diagnostic<S: Into<MultiSpan>>( |
| &mut self, |
| lint: &'static Lint, |
| id: ast::NodeId, |
| sp: S, |
| msg: &str, |
| diagnostic: BuiltinLintDiagnostics, |
| ) { |
| self.add_lint(lint, id, sp.into(), msg, diagnostic) |
| } |
| } |
| |
| pub fn struct_lint_level<'a>( |
| sess: &'a Session, |
| lint: &'static Lint, |
| level: Level, |
| src: LintSource, |
| span: Option<MultiSpan>, |
| msg: &str, |
| ) -> DiagnosticBuilder<'a> { |
| let mut err = match (level, span) { |
| (Level::Allow, _) => return sess.diagnostic().struct_dummy(), |
| (Level::Warn, Some(span)) => sess.struct_span_warn(span, msg), |
| (Level::Warn, None) => sess.struct_warn(msg), |
| (Level::Deny, Some(span)) | (Level::Forbid, Some(span)) => sess.struct_span_err(span, msg), |
| (Level::Deny, None) | (Level::Forbid, None) => sess.struct_err(msg), |
| }; |
| |
| // Check for future incompatibility lints and issue a stronger warning. |
| let lint_id = LintId::of(lint); |
| let future_incompatible = lint.future_incompatible; |
| |
| // If this code originates in a foreign macro, aka something that this crate |
| // did not itself author, then it's likely that there's nothing this crate |
| // can do about it. We probably want to skip the lint entirely. |
| if err.span.primary_spans().iter().any(|s| in_external_macro(sess, *s)) { |
| // Any suggestions made here are likely to be incorrect, so anything we |
| // emit shouldn't be automatically fixed by rustfix. |
| err.allow_suggestions(false); |
| |
| // If this is a future incompatible lint it'll become a hard error, so |
| // we have to emit *something*. Also allow lints to whitelist themselves |
| // on a case-by-case basis for emission in a foreign macro. |
| if future_incompatible.is_none() && !lint.report_in_external_macro { |
| err.cancel(); |
| // Don't continue further, since we don't want to have |
| // `diag_span_note_once` called for a diagnostic that isn't emitted. |
| return err; |
| } |
| } |
| |
| let name = lint.name_lower(); |
| match src { |
| LintSource::Default => { |
| sess.diag_note_once( |
| &mut err, |
| DiagnosticMessageId::from(lint), |
| &format!("`#[{}({})]` on by default", level.as_str(), name), |
| ); |
| } |
| LintSource::CommandLine(lint_flag_val) => { |
| let flag = match level { |
| Level::Warn => "-W", |
| Level::Deny => "-D", |
| Level::Forbid => "-F", |
| Level::Allow => panic!(), |
| }; |
| let hyphen_case_lint_name = name.replace("_", "-"); |
| if lint_flag_val.as_str() == name { |
| sess.diag_note_once( |
| &mut err, |
| DiagnosticMessageId::from(lint), |
| &format!( |
| "requested on the command line with `{} {}`", |
| flag, hyphen_case_lint_name |
| ), |
| ); |
| } else { |
| let hyphen_case_flag_val = lint_flag_val.as_str().replace("_", "-"); |
| sess.diag_note_once( |
| &mut err, |
| DiagnosticMessageId::from(lint), |
| &format!( |
| "`{} {}` implied by `{} {}`", |
| flag, hyphen_case_lint_name, flag, hyphen_case_flag_val |
| ), |
| ); |
| } |
| } |
| LintSource::Node(lint_attr_name, src, reason) => { |
| if let Some(rationale) = reason { |
| err.note(&rationale.as_str()); |
| } |
| sess.diag_span_note_once( |
| &mut err, |
| DiagnosticMessageId::from(lint), |
| src, |
| "lint level defined here", |
| ); |
| if lint_attr_name.as_str() != name { |
| let level_str = level.as_str(); |
| sess.diag_note_once( |
| &mut err, |
| DiagnosticMessageId::from(lint), |
| &format!( |
| "`#[{}({})]` implied by `#[{}({})]`", |
| level_str, name, level_str, lint_attr_name |
| ), |
| ); |
| } |
| } |
| } |
| |
| err.code(DiagnosticId::Lint(name)); |
| |
| if let Some(future_incompatible) = future_incompatible { |
| const STANDARD_MESSAGE: &str = "this was previously accepted by the compiler but is being phased out; \ |
| it will become a hard error"; |
| |
| let explanation = if lint_id == LintId::of(builtin::UNSTABLE_NAME_COLLISIONS) { |
| "once this method is added to the standard library, \ |
| the ambiguity may cause an error or change in behavior!" |
| .to_owned() |
| } else if lint_id == LintId::of(builtin::MUTABLE_BORROW_RESERVATION_CONFLICT) { |
| "this borrowing pattern was not meant to be accepted, \ |
| and may become a hard error in the future" |
| .to_owned() |
| } else if let Some(edition) = future_incompatible.edition { |
| format!("{} in the {} edition!", STANDARD_MESSAGE, edition) |
| } else { |
| format!("{} in a future release!", STANDARD_MESSAGE) |
| }; |
| let citation = format!("for more information, see {}", future_incompatible.reference); |
| err.warn(&explanation); |
| err.note(&citation); |
| } |
| |
| return err; |
| } |
| |
| pub fn maybe_lint_level_root(tcx: TyCtxt<'_>, id: hir::HirId) -> bool { |
| let attrs = tcx.hir().attrs(id); |
| attrs.iter().any(|attr| Level::from_symbol(attr.name_or_empty()).is_some()) |
| } |
| |
| /// Returns whether `span` originates in a foreign crate's external macro. |
| /// |
| /// This is used to test whether a lint should not even begin to figure out whether it should |
| /// be reported on the current node. |
| pub fn in_external_macro(sess: &Session, span: Span) -> bool { |
| let expn_data = span.ctxt().outer_expn_data(); |
| match expn_data.kind { |
| ExpnKind::Root | ExpnKind::Desugaring(DesugaringKind::ForLoop) => false, |
| ExpnKind::AstPass(_) | ExpnKind::Desugaring(_) => true, // well, it's "external" |
| ExpnKind::Macro(MacroKind::Bang, _) => { |
| if expn_data.def_site.is_dummy() { |
| // Dummy span for the `def_site` means it's an external macro. |
| return true; |
| } |
| match sess.source_map().span_to_snippet(expn_data.def_site) { |
| Ok(code) => !code.starts_with("macro_rules"), |
| // No snippet means external macro or compiler-builtin expansion. |
| Err(_) => true, |
| } |
| } |
| ExpnKind::Macro(..) => true, // definitely a plugin |
| } |
| } |
| |
| /// Returns `true` if `span` originates in a derive-macro's expansion. |
| pub fn in_derive_expansion(span: Span) -> bool { |
| if let ExpnKind::Macro(MacroKind::Derive, _) = span.ctxt().outer_expn_data().kind { |
| return true; |
| } |
| false |
| } |