src/librustc/lint/context.rs

   1 // Copyright 2012-2014 The Rust Project Developers. See the COPYRIGHT
   2 // file at the top-level directory of this distribution and at
   3 // http://rust-lang.org/COPYRIGHT.
   4 //
   5 // Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
   6 // http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
   7 // <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
   8 // option. This file may not be copied, modified, or distributed
   9 // except according to those terms.
  10
  11 //! Implementation of lint checking.
  12 //!
  13 //! The lint checking is mostly consolidated into one pass which runs just
  14 //! before translation to LLVM bytecode. Throughout compilation, lint warnings
  15 //! can be added via the `add_lint` method on the Session structure. This
  16 //! requires a span and an id of the node that the lint is being added to. The
  17 //! lint isn't actually emitted at that time because it is unknown what the
  18 //! actual lint level at that location is.
  19 //!
  20 //! To actually emit lint warnings/errors, a separate pass is used just before
  21 //! translation. A context keeps track of the current state of all lint levels.
  22 //! Upon entering a node of the ast which can modify the lint settings, the
  23 //! previous lint state is pushed onto a stack and the ast is then recursed
  24 //! upon.  As the ast is traversed, this keeps track of the current lint level
  25 //! for all lint attributes.
  26 use self::TargetLint::*;
  27
  28 use middle::privacy::ExportedItems;
  29 use middle::ty::{self, Ty};
  30 use session::{early_error, Session};
  31 use lint::{Level, LevelSource, Lint, LintId, LintArray, LintPass, LintPassObject};
  32 use lint::{Default, CommandLine, Node, Allow, Warn, Deny, Forbid};
  33 use lint::builtin;
  34 use util::nodemap::FnvHashMap;
  35
  36 use std::cell::RefCell;
  37 use std::mem;
  38 use syntax::ast_util::IdVisitingOperation;
  39 use syntax::attr::AttrMetaMethods;
  40 use syntax::attr;
  41 use syntax::codemap::Span;
  42 use syntax::visit::{Visitor, FnKind};
  43 use syntax::parse::token::InternedString;
  44 use syntax::{ast, ast_util, visit};
  45
  46 /// Information about the registered lints.
  47 ///
  48 /// This is basically the subset of `Context` that we can
  49 /// build early in the compile pipeline.
  50 pub struct LintStore {
  51     /// Registered lints. The bool is true if the lint was
  52     /// added by a plugin.
  53     lints: Vec<(&'static Lint, bool)>,
  54
  55     /// Trait objects for each lint pass.
  56     /// This is only `None` while iterating over the objects. See the definition
  57     /// of run_lints.
  58     passes: Option<Vec<LintPassObject>>,
  59
  60     /// Lints indexed by name.
  61     by_name: FnvHashMap<String, TargetLint>,
  62
  63     /// Current levels of each lint, and where they were set.
  64     levels: FnvHashMap<LintId, LevelSource>,
  65
  66     /// Map of registered lint groups to what lints they expand to. The bool
  67     /// is true if the lint group was added by a plugin.
  68     lint_groups: FnvHashMap<&'static str, (Vec<LintId>, bool)>,
  69 }
  70
  71 /// The targed of the `by_name` map, which accounts for renaming/deprecation.
  72 enum TargetLint {
  73     /// A direct lint target
  74     Id(LintId),
  75
  76     /// Temporary renaming, used for easing migration pain; see #16545
  77     Renamed(String, LintId),
  78 }
  79
  80 impl LintStore {
  81     fn get_level_source(&self, lint: LintId) -> LevelSource {
  82         match self.levels.get(&lint) {
  83             Some(&s) => s,
  84             None => (Allow, Default),
  85         }
  86     }
  87
  88     fn set_level(&mut self, lint: LintId, lvlsrc: LevelSource) {
  89         if lvlsrc.0 == Allow {
  90             self.levels.remove(&lint);
  91         } else {
  92             self.levels.insert(lint, lvlsrc);
  93         }
  94     }
  95
  96     pub fn new() -> LintStore {
  97         LintStore {
  98             lints: vec!(),
  99             passes: Some(vec!()),
 100             by_name: FnvHashMap::new(),
 101             levels: FnvHashMap::new(),
 102             lint_groups: FnvHashMap::new(),
 103         }
 104     }
 105
 106     pub fn get_lints<'t>(&'t self) -> &'t [(&'static Lint, bool)] {
 107         self.lints[]
 108     }
 109
 110     pub fn get_lint_groups<'t>(&'t self) -> Vec<(&'static str, Vec<LintId>, bool)> {
 111         self.lint_groups.iter().map(|(k, v)| (*k,
 112                                               v.0.clone(),
 113                                               v.1)).collect()
 114     }
 115
 116     pub fn register_pass(&mut self, sess: Option<&Session>,
 117                          from_plugin: bool, pass: LintPassObject) {
 118         for &lint in pass.get_lints().iter() {
 119             self.lints.push((*lint, from_plugin));
 120
 121             let id = LintId::of(*lint);
 122             if self.by_name.insert(lint.name_lower(), Id(id)).is_some() {
 123                 let msg = format!("duplicate specification of lint {}", lint.name_lower());
 124                 match (sess, from_plugin) {
 125                     // We load builtin lints first, so a duplicate is a compiler bug.
 126                     // Use early_error when handling -W help with no crate.
 127                     (None, _) => early_error(msg[]),
 128                     (Some(sess), false) => sess.bug(msg[]),
 129
 130                     // A duplicate name from a plugin is a user error.
 131                     (Some(sess), true)  => sess.err(msg[]),
 132                 }
 133             }
 134
 135             if lint.default_level != Allow {
 136                 self.levels.insert(id, (lint.default_level, Default));
 137             }
 138         }
 139         self.passes.as_mut().unwrap().push(pass);
 140     }
 141
 142     pub fn register_group(&mut self, sess: Option<&Session>,
 143                           from_plugin: bool, name: &'static str,
 144                           to: Vec<LintId>) {
 145         let new = self.lint_groups.insert(name, (to, from_plugin)).is_none();
 146
 147         if !new {
 148             let msg = format!("duplicate specification of lint group {}", name);
 149             match (sess, from_plugin) {
 150                 // We load builtin lints first, so a duplicate is a compiler bug.
 151                 // Use early_error when handling -W help with no crate.
 152                 (None, _) => early_error(msg[]),
 153                 (Some(sess), false) => sess.bug(msg[]),
 154
 155                 // A duplicate name from a plugin is a user error.
 156                 (Some(sess), true)  => sess.err(msg[]),
 157             }
 158         }
 159     }
 160
 161     fn register_renamed(&mut self, old_name: &str, new_name: &str) {
 162         let target = match self.by_name.get(new_name) {
 163             Some(&Id(lint_id)) => lint_id.clone(),
 164             _ => panic!("invalid lint renaming of {} to {}", old_name, new_name)
 165         };
 166         self.by_name.insert(old_name.to_string(), Renamed(new_name.to_string(), target));
 167     }
 168
 169     pub fn register_builtin(&mut self, sess: Option<&Session>) {
 170         macro_rules! add_builtin ( ( $sess:ident, $($name:ident),*, ) => (
 171             {$(
 172                 self.register_pass($sess, false, box builtin::$name as LintPassObject);
 173             )*}
 174         ));
 175
 176         macro_rules! add_builtin_with_new ( ( $sess:ident, $($name:ident),*, ) => (
 177             {$(
 178                 self.register_pass($sess, false, box builtin::$name::new() as LintPassObject);
 179             )*}
 180         ));
 181
 182         macro_rules! add_lint_group ( ( $sess:ident, $name:expr, $($lint:ident),* ) => (
 183             self.register_group($sess, false, $name, vec![$(LintId::of(builtin::$lint)),*]);
 184         ));
 185
 186         add_builtin!(sess,
 187                      HardwiredLints,
 188                      WhileTrue,
 189                      UnusedCasts,
 190                      ImproperCTypes,
 191                      BoxPointers,
 192                      UnusedAttributes,
 193                      PathStatements,
 194                      UnusedResults,
 195                      NonCamelCaseTypes,
 196                      NonSnakeCase,
 197                      NonUpperCaseGlobals,
 198                      UnusedParens,
 199                      UnusedImportBraces,
 200                      NonShorthandFieldPatterns,
 201                      UnusedUnsafe,
 202                      UnsafeBlocks,
 203                      UnusedMut,
 204                      UnusedAllocation,
 205                      Stability,
 206                      MissingCopyImplementations,
 207         );
 208
 209         add_builtin_with_new!(sess,
 210                               TypeLimits,
 211                               RawPointerDeriving,
 212                               MissingDoc,
 213         );
 214
 215         add_lint_group!(sess, "bad_style",
 216                         NON_CAMEL_CASE_TYPES, NON_SNAKE_CASE, NON_UPPER_CASE_GLOBALS);
 217
 218         add_lint_group!(sess, "unused",
 219                         UNUSED_IMPORTS, UNUSED_VARIABLES, UNUSED_ASSIGNMENTS, DEAD_CODE,
 220                         UNUSED_MUT, UNREACHABLE_CODE, UNUSED_MUST_USE,
 221                         UNUSED_UNSAFE, PATH_STATEMENTS);
 222
 223         // We have one lint pass defined in this module.
 224         self.register_pass(sess, false, box GatherNodeLevels as LintPassObject);
 225
 226         // Insert temporary renamings for a one-time deprecation (#16545)
 227         self.register_renamed("unnecessary_typecast", "unused_typecasts");
 228         self.register_renamed("unsigned_negate", "unsigned_negation");
 229         self.register_renamed("type_limits", "unused_comparisons");
 230         self.register_renamed("type_overflow", "overflowing_literals");
 231         self.register_renamed("ctypes", "improper_ctypes");
 232         self.register_renamed("owned_heap_memory", "box_pointers");
 233         self.register_renamed("unused_attribute", "unused_attributes");
 234         self.register_renamed("path_statement", "path_statements");
 235         self.register_renamed("unused_result", "unused_results");
 236         self.register_renamed("non_uppercase_statics", "non_upper_case_globals");
 237         self.register_renamed("unnecessary_parens", "unused_parens");
 238         self.register_renamed("unnecessary_import_braces", "unused_import_braces");
 239         self.register_renamed("unsafe_block", "unsafe_blocks");
 240         self.register_renamed("unnecessary_allocation", "unused_allocation");
 241         self.register_renamed("missing_doc", "missing_docs");
 242         self.register_renamed("unused_extern_crate", "unused_extern_crates");
 243         self.register_renamed("unnecessary_qualification", "unused_qualifications");
 244         self.register_renamed("unrecognized_lint", "unknown_lints");
 245         self.register_renamed("unused_variable", "unused_variables");
 246         self.register_renamed("dead_assignment", "unused_assignments");
 247         self.register_renamed("unknown_crate_type", "unknown_crate_types");
 248         self.register_renamed("variant_size_difference", "variant_size_differences");
 249         self.register_renamed("transmute_fat_ptr", "fat_ptr_transmutes");
 250
 251     }
 252
 253     #[allow(unused_variables)]
 254     fn find_lint(&self, lint_name: &str, sess: &Session, span: Option<Span>)
 255                  -> Option<LintId>
 256     {
 257         match self.by_name.get(lint_name) {
 258             Some(&Id(lint_id)) => Some(lint_id),
 259             Some(&Renamed(ref new_name, lint_id)) => {
 260                 let warning = format!("lint {} has been renamed to {}",
 261                                       lint_name, new_name);
 262                 match span {
 263                     Some(span) => sess.span_warn(span, warning[]),
 264                     None => sess.warn(warning[]),
 265                 };
 266                 Some(lint_id)
 267             }
 268             None => None
 269         }
 270     }
 271
 272     pub fn process_command_line(&mut self, sess: &Session) {
 273         for &(ref lint_name, level) in sess.opts.lint_opts.iter() {
 274             match self.find_lint(lint_name[], sess, None) {
 275                 Some(lint_id) => self.set_level(lint_id, (level, CommandLine)),
 276                 None => {
 277                     match self.lint_groups.iter().map(|(&x, pair)| (x, pair.0.clone()))
 278                                                  .collect::<FnvHashMap<&'static str,
 279                                                                        Vec<LintId>>>()
 280                                                  .get(lint_name[]) {
 281                         Some(v) => {
 282                             v.iter()
 283                              .map(|lint_id: &LintId|
 284                                      self.set_level(*lint_id, (level, CommandLine)))
 285                              .collect::<Vec<()>>();
 286                         }
 287                         None => sess.err(format!("unknown {} flag: {}",
 288                                                  level.as_str(), lint_name)[]),
 289                     }
 290                 }
 291             }
 292         }
 293     }
 294 }
 295
 296 /// Context for lint checking.
 297 pub struct Context<'a, 'tcx: 'a> {
 298     /// Type context we're checking in.
 299     pub tcx: &'a ty::ctxt<'tcx>,
 300
 301     /// The crate being checked.
 302     pub krate: &'a ast::Crate,
 303
 304     /// Items exported from the crate being checked.
 305     pub exported_items: &'a ExportedItems,
 306
 307     /// The store of registered lints.
 308     lints: LintStore,
 309
 310     /// When recursing into an attributed node of the ast which modifies lint
 311     /// levels, this stack keeps track of the previous lint levels of whatever
 312     /// was modified.
 313     level_stack: Vec<(LintId, LevelSource)>,
 314
 315     /// Level of lints for certain NodeIds, stored here because the body of
 316     /// the lint needs to run in trans.
 317     node_levels: RefCell<FnvHashMap<(ast::NodeId, LintId), LevelSource>>,
 318 }
 319
 320 /// Convenience macro for calling a `LintPass` method on every pass in the context.
 321 macro_rules! run_lints { ($cx:expr, $f:ident, $($args:expr),*) => ({
 322     // Move the vector of passes out of `$cx` so that we can
 323     // iterate over it mutably while passing `$cx` to the methods.
 324     let mut passes = $cx.lints.passes.take().unwrap();
 325     for obj in passes.iter_mut() {
 326         obj.$f($cx, $($args),*);
 327     }
 328     $cx.lints.passes = Some(passes);
 329 }) }
 330
 331 /// Parse the lint attributes into a vector, with `Err`s for malformed lint
 332 /// attributes. Writing this as an iterator is an enormous mess.
 333 pub fn gather_attrs(attrs: &[ast::Attribute])
 334                     -> Vec<Result<(InternedString, Level, Span), Span>> {
 335     let mut out = vec!();
 336     for attr in attrs.iter() {
 337         let level = match Level::from_str(attr.name().get()) {
 338             None => continue,
 339             Some(lvl) => lvl,
 340         };
 341
 342         attr::mark_used(attr);
 343
 344         let meta = &attr.node.value;
 345         let metas = match meta.node {
 346             ast::MetaList(_, ref metas) => metas,
 347             _ => {
 348                 out.push(Err(meta.span));
 349                 continue;
 350             }
 351         };
 352
 353         for meta in metas.iter() {
 354             out.push(match meta.node {
 355                 ast::MetaWord(ref lint_name) => Ok((lint_name.clone(), level, meta.span)),
 356                 _ => Err(meta.span),
 357             });
 358         }
 359     }
 360     out
 361 }
 362
 363 /// Emit a lint as a warning or an error (or not at all)
 364 /// according to `level`.
 365 ///
 366 /// This lives outside of `Context` so it can be used by checks
 367 /// in trans that run after the main lint pass is finished. Most
 368 /// lints elsewhere in the compiler should call
 369 /// `Session::add_lint()` instead.
 370 pub fn raw_emit_lint(sess: &Session, lint: &'static Lint,
 371                      lvlsrc: LevelSource, span: Option<Span>, msg: &str) {
 372     let (mut level, source) = lvlsrc;
 373     if level == Allow { return }
 374
 375     let name = lint.name_lower();
 376     let mut note = None;
 377     let msg = match source {
 378         Default => {
 379             format!("{}, #[{}({})] on by default", msg,
 380                     level.as_str(), name)
 381         },
 382         CommandLine => {
 383             format!("{} [-{} {}]", msg,
 384                     match level {
 385                         Warn => 'W', Deny => 'D', Forbid => 'F',
 386                         Allow => panic!()
 387                     }, name.replace("_", "-"))
 388         },
 389         Node(src) => {
 390             note = Some(src);
 391             msg.to_string()
 392         }
 393     };
 394
 395     // For purposes of printing, we can treat forbid as deny.
 396     if level == Forbid { level = Deny; }
 397
 398     match (level, span) {
 399         (Warn, Some(sp)) => sess.span_warn(sp, msg[]),
 400         (Warn, None)     => sess.warn(msg[]),
 401         (Deny, Some(sp)) => sess.span_err(sp, msg[]),
 402         (Deny, None)     => sess.err(msg[]),
 403         _ => sess.bug("impossible level in raw_emit_lint"),
 404     }
 405
 406     for span in note.into_iter() {
 407         sess.span_note(span, "lint level defined here");
 408     }
 409 }
 410
 411 impl<'a, 'tcx> Context<'a, 'tcx> {
 412     fn new(tcx: &'a ty::ctxt<'tcx>,
 413            krate: &'a ast::Crate,
 414            exported_items: &'a ExportedItems) -> Context<'a, 'tcx> {
 415         // We want to own the lint store, so move it out of the session.
 416         let lint_store = mem::replace(&mut *tcx.sess.lint_store.borrow_mut(),
 417                                       LintStore::new());
 418
 419         Context {
 420             tcx: tcx,
 421             krate: krate,
 422             exported_items: exported_items,
 423             lints: lint_store,
 424             level_stack: vec![],
 425             node_levels: RefCell::new(FnvHashMap::new()),
 426         }
 427     }
 428
 429     /// Get the overall compiler `Session` object.
 430     pub fn sess(&'a self) -> &'a Session {
 431         &self.tcx.sess
 432     }
 433
 434     /// Get the level of `lint` at the current position of the lint
 435     /// traversal.
 436     pub fn current_level(&self, lint: &'static Lint) -> Level {
 437         self.lints.levels.get(&LintId::of(lint)).map_or(Allow, |&(lvl, _)| lvl)
 438     }
 439
 440     fn lookup_and_emit(&self, lint: &'static Lint, span: Option<Span>, msg: &str) {
 441         let (level, src) = match self.lints.levels.get(&LintId::of(lint)) {
 442             None => return,
 443             Some(&(Warn, src)) => {
 444                 let lint_id = LintId::of(builtin::WARNINGS);
 445                 (self.lints.get_level_source(lint_id).0, src)
 446             }
 447             Some(&pair) => pair,
 448         };
 449
 450         raw_emit_lint(&self.tcx.sess, lint, (level, src), span, msg);
 451     }
 452
 453     /// Emit a lint at the appropriate level, with no associated span.
 454     pub fn lint(&self, lint: &'static Lint, msg: &str) {
 455         self.lookup_and_emit(lint, None, msg);
 456     }
 457
 458     /// Emit a lint at the appropriate level, for a particular span.
 459     pub fn span_lint(&self, lint: &'static Lint, span: Span, msg: &str) {
 460         self.lookup_and_emit(lint, Some(span), msg);
 461     }
 462
 463     /// Merge the lints specified by any lint attributes into the
 464     /// current lint context, call the provided function, then reset the
 465     /// lints in effect to their previous state.
 466     fn with_lint_attrs<F>(&mut self,
 467                           attrs: &[ast::Attribute],
 468                           f: F) where
 469         F: FnOnce(&mut Context),
 470     {
 471         // Parse all of the lint attributes, and then add them all to the
 472         // current dictionary of lint information. Along the way, keep a history
 473         // of what we changed so we can roll everything back after invoking the
 474         // specified closure
 475         let mut pushed = 0u;
 476
 477         for result in gather_attrs(attrs).into_iter() {
 478             let v = match result {
 479                 Err(span) => {
 480                     self.tcx.sess.span_err(span, "malformed lint attribute");
 481                     continue;
 482                 }
 483                 Ok((lint_name, level, span)) => {
 484                     match self.lints.find_lint(lint_name.get(), &self.tcx.sess, Some(span)) {
 485                         Some(lint_id) => vec![(lint_id, level, span)],
 486                         None => {
 487                             match self.lints.lint_groups.get(lint_name.get()) {
 488                                 Some(&(ref v, _)) => v.iter()
 489                                                       .map(|lint_id: &LintId|
 490                                                            (*lint_id, level, span))
 491                                                       .collect(),
 492                                 None => {
 493                                     self.span_lint(builtin::UNKNOWN_LINTS, span,
 494                                                format!("unknown `{}` attribute: `{}`",
 495                                                        level.as_str(), lint_name)[]);
 496                                     continue;
 497                                 }
 498                             }
 499                         }
 500                     }
 501                 }
 502             };
 503
 504             for (lint_id, level, span) in v.into_iter() {
 505                 let now = self.lints.get_level_source(lint_id).0;
 506                 if now == Forbid && level != Forbid {
 507                     let lint_name = lint_id.as_str();
 508                     self.tcx.sess.span_err(span,
 509                                            format!("{}({}) overruled by outer forbid({})",
 510                                                    level.as_str(), lint_name,
 511                                                    lint_name)[]);
 512                 } else if now != level {
 513                     let src = self.lints.get_level_source(lint_id).1;
 514                     self.level_stack.push((lint_id, (now, src)));
 515                     pushed += 1;
 516                     self.lints.set_level(lint_id, (level, Node(span)));
 517                 }
 518             }
 519         }
 520
 521         run_lints!(self, enter_lint_attrs, attrs);
 522         f(self);
 523         run_lints!(self, exit_lint_attrs, attrs);
 524
 525         // rollback
 526         for _ in range(0, pushed) {
 527             let (lint, lvlsrc) = self.level_stack.pop().unwrap();
 528             self.lints.set_level(lint, lvlsrc);
 529         }
 530     }
 531
 532     fn visit_ids<F>(&mut self, f: F) where
 533         F: FnOnce(&mut ast_util::IdVisitor<Context>)
 534     {
 535         let mut v = ast_util::IdVisitor {
 536             operation: self,
 537             pass_through_items: false,
 538             visited_outermost: false,
 539         };
 540         f(&mut v);
 541     }
 542 }
 543
 544 impl<'a, 'tcx, 'v> Visitor<'v> for Context<'a, 'tcx> {
 545     fn visit_item(&mut self, it: &ast::Item) {
 546         self.with_lint_attrs(it.attrs[], |cx| {
 547             run_lints!(cx, check_item, it);
 548             cx.visit_ids(|v| v.visit_item(it));
 549             visit::walk_item(cx, it);
 550         })
 551     }
 552
 553     fn visit_foreign_item(&mut self, it: &ast::ForeignItem) {
 554         self.with_lint_attrs(it.attrs[], |cx| {
 555             run_lints!(cx, check_foreign_item, it);
 556             visit::walk_foreign_item(cx, it);
 557         })
 558     }
 559
 560     fn visit_view_item(&mut self, i: &ast::ViewItem) {
 561         self.with_lint_attrs(i.attrs[], |cx| {
 562             run_lints!(cx, check_view_item, i);
 563             cx.visit_ids(|v| v.visit_view_item(i));
 564             visit::walk_view_item(cx, i);
 565         })
 566     }
 567
 568     fn visit_pat(&mut self, p: &ast::Pat) {
 569         run_lints!(self, check_pat, p);
 570         visit::walk_pat(self, p);
 571     }
 572
 573     fn visit_expr(&mut self, e: &ast::Expr) {
 574         run_lints!(self, check_expr, e);
 575         visit::walk_expr(self, e);
 576     }
 577
 578     fn visit_stmt(&mut self, s: &ast::Stmt) {
 579         run_lints!(self, check_stmt, s);
 580         visit::walk_stmt(self, s);
 581     }
 582
 583     fn visit_fn(&mut self, fk: FnKind<'v>, decl: &'v ast::FnDecl,
 584                 body: &'v ast::Block, span: Span, id: ast::NodeId) {
 585         match fk {
 586             visit::FkMethod(_, _, m) => {
 587                 self.with_lint_attrs(m.attrs[], |cx| {
 588                     run_lints!(cx, check_fn, fk, decl, body, span, id);
 589                     cx.visit_ids(|v| {
 590                         v.visit_fn(fk, decl, body, span, id);
 591                     });
 592                     visit::walk_fn(cx, fk, decl, body, span);
 593                 })
 594             },
 595             _ => {
 596                 run_lints!(self, check_fn, fk, decl, body, span, id);
 597                 visit::walk_fn(self, fk, decl, body, span);
 598             }
 599         }
 600     }
 601
 602     fn visit_ty_method(&mut self, t: &ast::TypeMethod) {
 603         self.with_lint_attrs(t.attrs[], |cx| {
 604             run_lints!(cx, check_ty_method, t);
 605             visit::walk_ty_method(cx, t);
 606         })
 607     }
 608
 609     fn visit_struct_def(&mut self,
 610                         s: &ast::StructDef,
 611                         ident: ast::Ident,
 612                         g: &ast::Generics,
 613                         id: ast::NodeId) {
 614         run_lints!(self, check_struct_def, s, ident, g, id);
 615         visit::walk_struct_def(self, s);
 616         run_lints!(self, check_struct_def_post, s, ident, g, id);
 617     }
 618
 619     fn visit_struct_field(&mut self, s: &ast::StructField) {
 620         self.with_lint_attrs(s.node.attrs[], |cx| {
 621             run_lints!(cx, check_struct_field, s);
 622             visit::walk_struct_field(cx, s);
 623         })
 624     }
 625
 626     fn visit_variant(&mut self, v: &ast::Variant, g: &ast::Generics) {
 627         self.with_lint_attrs(v.node.attrs[], |cx| {
 628             run_lints!(cx, check_variant, v, g);
 629             visit::walk_variant(cx, v, g);
 630             run_lints!(cx, check_variant_post, v, g);
 631         })
 632     }
 633
 634     // FIXME(#10894) should continue recursing
 635     fn visit_ty(&mut self, t: &ast::Ty) {
 636         run_lints!(self, check_ty, t);
 637     }
 638
 639     fn visit_ident(&mut self, sp: Span, id: ast::Ident) {
 640         run_lints!(self, check_ident, sp, id);
 641     }
 642
 643     fn visit_mod(&mut self, m: &ast::Mod, s: Span, n: ast::NodeId) {
 644         run_lints!(self, check_mod, m, s, n);
 645         visit::walk_mod(self, m);
 646     }
 647
 648     fn visit_local(&mut self, l: &ast::Local) {
 649         run_lints!(self, check_local, l);
 650         visit::walk_local(self, l);
 651     }
 652
 653     fn visit_block(&mut self, b: &ast::Block) {
 654         run_lints!(self, check_block, b);
 655         visit::walk_block(self, b);
 656     }
 657
 658     fn visit_arm(&mut self, a: &ast::Arm) {
 659         run_lints!(self, check_arm, a);
 660         visit::walk_arm(self, a);
 661     }
 662
 663     fn visit_decl(&mut self, d: &ast::Decl) {
 664         run_lints!(self, check_decl, d);
 665         visit::walk_decl(self, d);
 666     }
 667
 668     fn visit_expr_post(&mut self, e: &ast::Expr) {
 669         run_lints!(self, check_expr_post, e);
 670     }
 671
 672     fn visit_generics(&mut self, g: &ast::Generics) {
 673         run_lints!(self, check_generics, g);
 674         visit::walk_generics(self, g);
 675     }
 676
 677     fn visit_trait_item(&mut self, m: &ast::TraitItem) {
 678         run_lints!(self, check_trait_method, m);
 679         visit::walk_trait_item(self, m);
 680     }
 681
 682     fn visit_opt_lifetime_ref(&mut self, sp: Span, lt: &Option<ast::Lifetime>) {
 683         run_lints!(self, check_opt_lifetime_ref, sp, lt);
 684     }
 685
 686     fn visit_lifetime_ref(&mut self, lt: &ast::Lifetime) {
 687         run_lints!(self, check_lifetime_ref, lt);
 688     }
 689
 690     fn visit_lifetime_def(&mut self, lt: &ast::LifetimeDef) {
 691         run_lints!(self, check_lifetime_def, lt);
 692     }
 693
 694     fn visit_explicit_self(&mut self, es: &ast::ExplicitSelf) {
 695         run_lints!(self, check_explicit_self, es);
 696         visit::walk_explicit_self(self, es);
 697     }
 698
 699     fn visit_mac(&mut self, mac: &ast::Mac) {
 700         run_lints!(self, check_mac, mac);
 701         visit::walk_mac(self, mac);
 702     }
 703
 704     fn visit_path(&mut self, p: &ast::Path, id: ast::NodeId) {
 705         run_lints!(self, check_path, p, id);
 706         visit::walk_path(self, p);
 707     }
 708
 709     fn visit_attribute(&mut self, attr: &ast::Attribute) {
 710         run_lints!(self, check_attribute, attr);
 711     }
 712 }
 713
 714 // Output any lints that were previously added to the session.
 715 impl<'a, 'tcx> IdVisitingOperation for Context<'a, 'tcx> {
 716     fn visit_id(&mut self, id: ast::NodeId) {
 717         match self.tcx.sess.lints.borrow_mut().remove(&id) {
 718             None => {}
 719             Some(lints) => {
 720                 for (lint_id, span, msg) in lints.into_iter() {
 721                     self.span_lint(lint_id.lint, span, msg[])
 722                 }
 723             }
 724         }
 725     }
 726 }
 727
 728 // This lint pass is defined here because it touches parts of the `Context`
 729 // that we don't want to expose. It records the lint level at certain AST
 730 // nodes, so that the variant size difference check in trans can call
 731 // `raw_emit_lint`.
 732
 733 struct GatherNodeLevels;
 734
 735 impl LintPass for GatherNodeLevels {
 736     fn get_lints(&self) -> LintArray {
 737         lint_array!()
 738     }
 739
 740     fn check_item(&mut self, cx: &Context, it: &ast::Item) {
 741         match it.node {
 742             ast::ItemEnum(..) => {
 743                 let lint_id = LintId::of(builtin::VARIANT_SIZE_DIFFERENCES);
 744                 let lvlsrc = cx.lints.get_level_source(lint_id);
 745                 match lvlsrc {
 746                     (lvl, _) if lvl != Allow => {
 747                         cx.node_levels.borrow_mut()
 748                             .insert((it.id, lint_id), lvlsrc);
 749                     },
 750                     _ => { }
 751                 }
 752             },
 753             _ => { }
 754         }
 755     }
 756 }
 757
 758 /// Perform lint checking on a crate.
 759 ///
 760 /// Consumes the `lint_store` field of the `Session`.
 761 pub fn check_crate(tcx: &ty::ctxt,
 762                    exported_items: &ExportedItems) {
 763     let krate = tcx.map.krate();
 764     let mut cx = Context::new(tcx, krate, exported_items);
 765
 766     // Visit the whole crate.
 767     cx.with_lint_attrs(krate.attrs[], |cx| {
 768         cx.visit_id(ast::CRATE_NODE_ID);
 769         cx.visit_ids(|v| {
 770             v.visited_outermost = true;
 771             visit::walk_crate(v, krate);
 772         });
 773
 774         // since the root module isn't visited as an item (because it isn't an
 775         // item), warn for it here.
 776         run_lints!(cx, check_crate, krate);
 777
 778         visit::walk_crate(cx, krate);
 779     });
 780
 781     // If we missed any lints added to the session, then there's a bug somewhere
 782     // in the iteration code.
 783     for (id, v) in tcx.sess.lints.borrow().iter() {
 784         for &(lint, span, ref msg) in v.iter() {
 785             tcx.sess.span_bug(span,
 786                               format!("unprocessed lint {} at {}: {}",
 787                                       lint.as_str(), tcx.map.node_to_string(*id), *msg)[])
 788         }
 789     }
 790
 791     tcx.sess.abort_if_errors();
 792     *tcx.node_lint_levels.borrow_mut() = cx.node_levels.into_inner();
 793 }