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                               RawPointerDerive,
 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         self.register_renamed("raw_pointer_deriving", "raw_pointer_derive");
 251
 252     }
 253
 254     #[allow(unused_variables)]
 255     fn find_lint(&self, lint_name: &str, sess: &Session, span: Option<Span>)
 256                  -> Option<LintId>
 257     {
 258         match self.by_name.get(lint_name) {
 259             Some(&Id(lint_id)) => Some(lint_id),
 260             Some(&Renamed(ref new_name, lint_id)) => {
 261                 let warning = format!("lint {} has been renamed to {}",
 262                                       lint_name, new_name);
 263                 match span {
 264                     Some(span) => sess.span_warn(span, warning[]),
 265                     None => sess.warn(warning[]),
 266                 };
 267                 Some(lint_id)
 268             }
 269             None => None
 270         }
 271     }
 272
 273     pub fn process_command_line(&mut self, sess: &Session) {
 274         for &(ref lint_name, level) in sess.opts.lint_opts.iter() {
 275             match self.find_lint(lint_name[], sess, None) {
 276                 Some(lint_id) => self.set_level(lint_id, (level, CommandLine)),
 277                 None => {
 278                     match self.lint_groups.iter().map(|(&x, pair)| (x, pair.0.clone()))
 279                                                  .collect::<FnvHashMap<&'static str,
 280                                                                        Vec<LintId>>>()
 281                                                  .get(lint_name[]) {
 282                         Some(v) => {
 283                             v.iter()
 284                              .map(|lint_id: &LintId|
 285                                      self.set_level(*lint_id, (level, CommandLine)))
 286                              .collect::<Vec<()>>();
 287                         }
 288                         None => sess.err(format!("unknown {} flag: {}",
 289                                                  level.as_str(), lint_name)[]),
 290                     }
 291                 }
 292             }
 293         }
 294     }
 295 }
 296
 297 /// Context for lint checking.
 298 pub struct Context<'a, 'tcx: 'a> {
 299     /// Type context we're checking in.
 300     pub tcx: &'a ty::ctxt<'tcx>,
 301
 302     /// The crate being checked.
 303     pub krate: &'a ast::Crate,
 304
 305     /// Items exported from the crate being checked.
 306     pub exported_items: &'a ExportedItems,
 307
 308     /// The store of registered lints.
 309     lints: LintStore,
 310
 311     /// When recursing into an attributed node of the ast which modifies lint
 312     /// levels, this stack keeps track of the previous lint levels of whatever
 313     /// was modified.
 314     level_stack: Vec<(LintId, LevelSource)>,
 315
 316     /// Level of lints for certain NodeIds, stored here because the body of
 317     /// the lint needs to run in trans.
 318     node_levels: RefCell<FnvHashMap<(ast::NodeId, LintId), LevelSource>>,
 319 }
 320
 321 /// Convenience macro for calling a `LintPass` method on every pass in the context.
 322 macro_rules! run_lints { ($cx:expr, $f:ident, $($args:expr),*) => ({
 323     // Move the vector of passes out of `$cx` so that we can
 324     // iterate over it mutably while passing `$cx` to the methods.
 325     let mut passes = $cx.lints.passes.take().unwrap();
 326     for obj in passes.iter_mut() {
 327         obj.$f($cx, $($args),*);
 328     }
 329     $cx.lints.passes = Some(passes);
 330 }) }
 331
 332 /// Parse the lint attributes into a vector, with `Err`s for malformed lint
 333 /// attributes. Writing this as an iterator is an enormous mess.
 334 pub fn gather_attrs(attrs: &[ast::Attribute])
 335                     -> Vec<Result<(InternedString, Level, Span), Span>> {
 336     let mut out = vec!();
 337     for attr in attrs.iter() {
 338         let level = match Level::from_str(attr.name().get()) {
 339             None => continue,
 340             Some(lvl) => lvl,
 341         };
 342
 343         attr::mark_used(attr);
 344
 345         let meta = &attr.node.value;
 346         let metas = match meta.node {
 347             ast::MetaList(_, ref metas) => metas,
 348             _ => {
 349                 out.push(Err(meta.span));
 350                 continue;
 351             }
 352         };
 353
 354         for meta in metas.iter() {
 355             out.push(match meta.node {
 356                 ast::MetaWord(ref lint_name) => Ok((lint_name.clone(), level, meta.span)),
 357                 _ => Err(meta.span),
 358             });
 359         }
 360     }
 361     out
 362 }
 363
 364 /// Emit a lint as a warning or an error (or not at all)
 365 /// according to `level`.
 366 ///
 367 /// This lives outside of `Context` so it can be used by checks
 368 /// in trans that run after the main lint pass is finished. Most
 369 /// lints elsewhere in the compiler should call
 370 /// `Session::add_lint()` instead.
 371 pub fn raw_emit_lint(sess: &Session, lint: &'static Lint,
 372                      lvlsrc: LevelSource, span: Option<Span>, msg: &str) {
 373     let (mut level, source) = lvlsrc;
 374     if level == Allow { return }
 375
 376     let name = lint.name_lower();
 377     let mut note = None;
 378     let msg = match source {
 379         Default => {
 380             format!("{}, #[{}({})] on by default", msg,
 381                     level.as_str(), name)
 382         },
 383         CommandLine => {
 384             format!("{} [-{} {}]", msg,
 385                     match level {
 386                         Warn => 'W', Deny => 'D', Forbid => 'F',
 387                         Allow => panic!()
 388                     }, name.replace("_", "-"))
 389         },
 390         Node(src) => {
 391             note = Some(src);
 392             msg.to_string()
 393         }
 394     };
 395
 396     // For purposes of printing, we can treat forbid as deny.
 397     if level == Forbid { level = Deny; }
 398
 399     match (level, span) {
 400         (Warn, Some(sp)) => sess.span_warn(sp, msg[]),
 401         (Warn, None)     => sess.warn(msg[]),
 402         (Deny, Some(sp)) => sess.span_err(sp, msg[]),
 403         (Deny, None)     => sess.err(msg[]),
 404         _ => sess.bug("impossible level in raw_emit_lint"),
 405     }
 406
 407     for span in note.into_iter() {
 408         sess.span_note(span, "lint level defined here");
 409     }
 410 }
 411
 412 impl<'a, 'tcx> Context<'a, 'tcx> {
 413     fn new(tcx: &'a ty::ctxt<'tcx>,
 414            krate: &'a ast::Crate,
 415            exported_items: &'a ExportedItems) -> Context<'a, 'tcx> {
 416         // We want to own the lint store, so move it out of the session.
 417         let lint_store = mem::replace(&mut *tcx.sess.lint_store.borrow_mut(),
 418                                       LintStore::new());
 419
 420         Context {
 421             tcx: tcx,
 422             krate: krate,
 423             exported_items: exported_items,
 424             lints: lint_store,
 425             level_stack: vec![],
 426             node_levels: RefCell::new(FnvHashMap::new()),
 427         }
 428     }
 429
 430     /// Get the overall compiler `Session` object.
 431     pub fn sess(&'a self) -> &'a Session {
 432         &self.tcx.sess
 433     }
 434
 435     /// Get the level of `lint` at the current position of the lint
 436     /// traversal.
 437     pub fn current_level(&self, lint: &'static Lint) -> Level {
 438         self.lints.levels.get(&LintId::of(lint)).map_or(Allow, |&(lvl, _)| lvl)
 439     }
 440
 441     fn lookup_and_emit(&self, lint: &'static Lint, span: Option<Span>, msg: &str) {
 442         let (level, src) = match self.lints.levels.get(&LintId::of(lint)) {
 443             None => return,
 444             Some(&(Warn, src)) => {
 445                 let lint_id = LintId::of(builtin::WARNINGS);
 446                 (self.lints.get_level_source(lint_id).0, src)
 447             }
 448             Some(&pair) => pair,
 449         };
 450
 451         raw_emit_lint(&self.tcx.sess, lint, (level, src), span, msg);
 452     }
 453
 454     /// Emit a lint at the appropriate level, with no associated span.
 455     pub fn lint(&self, lint: &'static Lint, msg: &str) {
 456         self.lookup_and_emit(lint, None, msg);
 457     }
 458
 459     /// Emit a lint at the appropriate level, for a particular span.
 460     pub fn span_lint(&self, lint: &'static Lint, span: Span, msg: &str) {
 461         self.lookup_and_emit(lint, Some(span), msg);
 462     }
 463
 464     /// Merge the lints specified by any lint attributes into the
 465     /// current lint context, call the provided function, then reset the
 466     /// lints in effect to their previous state.
 467     fn with_lint_attrs<F>(&mut self,
 468                           attrs: &[ast::Attribute],
 469                           f: F) where
 470         F: FnOnce(&mut Context),
 471     {
 472         // Parse all of the lint attributes, and then add them all to the
 473         // current dictionary of lint information. Along the way, keep a history
 474         // of what we changed so we can roll everything back after invoking the
 475         // specified closure
 476         let mut pushed = 0u;
 477
 478         for result in gather_attrs(attrs).into_iter() {
 479             let v = match result {
 480                 Err(span) => {
 481                     self.tcx.sess.span_err(span, "malformed lint attribute");
 482                     continue;
 483                 }
 484                 Ok((lint_name, level, span)) => {
 485                     match self.lints.find_lint(lint_name.get(), &self.tcx.sess, Some(span)) {
 486                         Some(lint_id) => vec![(lint_id, level, span)],
 487                         None => {
 488                             match self.lints.lint_groups.get(lint_name.get()) {
 489                                 Some(&(ref v, _)) => v.iter()
 490                                                       .map(|lint_id: &LintId|
 491                                                            (*lint_id, level, span))
 492                                                       .collect(),
 493                                 None => {
 494                                     self.span_lint(builtin::UNKNOWN_LINTS, span,
 495                                                format!("unknown `{}` attribute: `{}`",
 496                                                        level.as_str(), lint_name)[]);
 497                                     continue;
 498                                 }
 499                             }
 500                         }
 501                     }
 502                 }
 503             };
 504
 505             for (lint_id, level, span) in v.into_iter() {
 506                 let now = self.lints.get_level_source(lint_id).0;
 507                 if now == Forbid && level != Forbid {
 508                     let lint_name = lint_id.as_str();
 509                     self.tcx.sess.span_err(span,
 510                                            format!("{}({}) overruled by outer forbid({})",
 511                                                    level.as_str(), lint_name,
 512                                                    lint_name)[]);
 513                 } else if now != level {
 514                     let src = self.lints.get_level_source(lint_id).1;
 515                     self.level_stack.push((lint_id, (now, src)));
 516                     pushed += 1;
 517                     self.lints.set_level(lint_id, (level, Node(span)));
 518                 }
 519             }
 520         }
 521
 522         run_lints!(self, enter_lint_attrs, attrs);
 523         f(self);
 524         run_lints!(self, exit_lint_attrs, attrs);
 525
 526         // rollback
 527         for _ in range(0, pushed) {
 528             let (lint, lvlsrc) = self.level_stack.pop().unwrap();
 529             self.lints.set_level(lint, lvlsrc);
 530         }
 531     }
 532
 533     fn visit_ids<F>(&mut self, f: F) where
 534         F: FnOnce(&mut ast_util::IdVisitor<Context>)
 535     {
 536         let mut v = ast_util::IdVisitor {
 537             operation: self,
 538             pass_through_items: false,
 539             visited_outermost: false,
 540         };
 541         f(&mut v);
 542     }
 543 }
 544
 545 impl<'a, 'tcx, 'v> Visitor<'v> for Context<'a, 'tcx> {
 546     fn visit_item(&mut self, it: &ast::Item) {
 547         self.with_lint_attrs(it.attrs[], |cx| {
 548             run_lints!(cx, check_item, it);
 549             cx.visit_ids(|v| v.visit_item(it));
 550             visit::walk_item(cx, it);
 551         })
 552     }
 553
 554     fn visit_foreign_item(&mut self, it: &ast::ForeignItem) {
 555         self.with_lint_attrs(it.attrs[], |cx| {
 556             run_lints!(cx, check_foreign_item, it);
 557             visit::walk_foreign_item(cx, it);
 558         })
 559     }
 560
 561     fn visit_view_item(&mut self, i: &ast::ViewItem) {
 562         self.with_lint_attrs(i.attrs[], |cx| {
 563             run_lints!(cx, check_view_item, i);
 564             cx.visit_ids(|v| v.visit_view_item(i));
 565             visit::walk_view_item(cx, i);
 566         })
 567     }
 568
 569     fn visit_pat(&mut self, p: &ast::Pat) {
 570         run_lints!(self, check_pat, p);
 571         visit::walk_pat(self, p);
 572     }
 573
 574     fn visit_expr(&mut self, e: &ast::Expr) {
 575         run_lints!(self, check_expr, e);
 576         visit::walk_expr(self, e);
 577     }
 578
 579     fn visit_stmt(&mut self, s: &ast::Stmt) {
 580         run_lints!(self, check_stmt, s);
 581         visit::walk_stmt(self, s);
 582     }
 583
 584     fn visit_fn(&mut self, fk: FnKind<'v>, decl: &'v ast::FnDecl,
 585                 body: &'v ast::Block, span: Span, id: ast::NodeId) {
 586         match fk {
 587             visit::FkMethod(_, _, m) => {
 588                 self.with_lint_attrs(m.attrs[], |cx| {
 589                     run_lints!(cx, check_fn, fk, decl, body, span, id);
 590                     cx.visit_ids(|v| {
 591                         v.visit_fn(fk, decl, body, span, id);
 592                     });
 593                     visit::walk_fn(cx, fk, decl, body, span);
 594                 })
 595             },
 596             _ => {
 597                 run_lints!(self, check_fn, fk, decl, body, span, id);
 598                 visit::walk_fn(self, fk, decl, body, span);
 599             }
 600         }
 601     }
 602
 603     fn visit_ty_method(&mut self, t: &ast::TypeMethod) {
 604         self.with_lint_attrs(t.attrs[], |cx| {
 605             run_lints!(cx, check_ty_method, t);
 606             visit::walk_ty_method(cx, t);
 607         })
 608     }
 609
 610     fn visit_struct_def(&mut self,
 611                         s: &ast::StructDef,
 612                         ident: ast::Ident,
 613                         g: &ast::Generics,
 614                         id: ast::NodeId) {
 615         run_lints!(self, check_struct_def, s, ident, g, id);
 616         visit::walk_struct_def(self, s);
 617         run_lints!(self, check_struct_def_post, s, ident, g, id);
 618     }
 619
 620     fn visit_struct_field(&mut self, s: &ast::StructField) {
 621         self.with_lint_attrs(s.node.attrs[], |cx| {
 622             run_lints!(cx, check_struct_field, s);
 623             visit::walk_struct_field(cx, s);
 624         })
 625     }
 626
 627     fn visit_variant(&mut self, v: &ast::Variant, g: &ast::Generics) {
 628         self.with_lint_attrs(v.node.attrs[], |cx| {
 629             run_lints!(cx, check_variant, v, g);
 630             visit::walk_variant(cx, v, g);
 631             run_lints!(cx, check_variant_post, v, g);
 632         })
 633     }
 634
 635     // FIXME(#10894) should continue recursing
 636     fn visit_ty(&mut self, t: &ast::Ty) {
 637         run_lints!(self, check_ty, t);
 638     }
 639
 640     fn visit_ident(&mut self, sp: Span, id: ast::Ident) {
 641         run_lints!(self, check_ident, sp, id);
 642     }
 643
 644     fn visit_mod(&mut self, m: &ast::Mod, s: Span, n: ast::NodeId) {
 645         run_lints!(self, check_mod, m, s, n);
 646         visit::walk_mod(self, m);
 647     }
 648
 649     fn visit_local(&mut self, l: &ast::Local) {
 650         run_lints!(self, check_local, l);
 651         visit::walk_local(self, l);
 652     }
 653
 654     fn visit_block(&mut self, b: &ast::Block) {
 655         run_lints!(self, check_block, b);
 656         visit::walk_block(self, b);
 657     }
 658
 659     fn visit_arm(&mut self, a: &ast::Arm) {
 660         run_lints!(self, check_arm, a);
 661         visit::walk_arm(self, a);
 662     }
 663
 664     fn visit_decl(&mut self, d: &ast::Decl) {
 665         run_lints!(self, check_decl, d);
 666         visit::walk_decl(self, d);
 667     }
 668
 669     fn visit_expr_post(&mut self, e: &ast::Expr) {
 670         run_lints!(self, check_expr_post, e);
 671     }
 672
 673     fn visit_generics(&mut self, g: &ast::Generics) {
 674         run_lints!(self, check_generics, g);
 675         visit::walk_generics(self, g);
 676     }
 677
 678     fn visit_trait_item(&mut self, m: &ast::TraitItem) {
 679         run_lints!(self, check_trait_method, m);
 680         visit::walk_trait_item(self, m);
 681     }
 682
 683     fn visit_opt_lifetime_ref(&mut self, sp: Span, lt: &Option<ast::Lifetime>) {
 684         run_lints!(self, check_opt_lifetime_ref, sp, lt);
 685     }
 686
 687     fn visit_lifetime_ref(&mut self, lt: &ast::Lifetime) {
 688         run_lints!(self, check_lifetime_ref, lt);
 689     }
 690
 691     fn visit_lifetime_def(&mut self, lt: &ast::LifetimeDef) {
 692         run_lints!(self, check_lifetime_def, lt);
 693     }
 694
 695     fn visit_explicit_self(&mut self, es: &ast::ExplicitSelf) {
 696         run_lints!(self, check_explicit_self, es);
 697         visit::walk_explicit_self(self, es);
 698     }
 699
 700     fn visit_mac(&mut self, mac: &ast::Mac) {
 701         run_lints!(self, check_mac, mac);
 702         visit::walk_mac(self, mac);
 703     }
 704
 705     fn visit_path(&mut self, p: &ast::Path, id: ast::NodeId) {
 706         run_lints!(self, check_path, p, id);
 707         visit::walk_path(self, p);
 708     }
 709
 710     fn visit_attribute(&mut self, attr: &ast::Attribute) {
 711         run_lints!(self, check_attribute, attr);
 712     }
 713 }
 714
 715 // Output any lints that were previously added to the session.
 716 impl<'a, 'tcx> IdVisitingOperation for Context<'a, 'tcx> {
 717     fn visit_id(&mut self, id: ast::NodeId) {
 718         match self.tcx.sess.lints.borrow_mut().remove(&id) {
 719             None => {}
 720             Some(lints) => {
 721                 for (lint_id, span, msg) in lints.into_iter() {
 722                     self.span_lint(lint_id.lint, span, msg[])
 723                 }
 724             }
 725         }
 726     }
 727 }
 728
 729 // This lint pass is defined here because it touches parts of the `Context`
 730 // that we don't want to expose. It records the lint level at certain AST
 731 // nodes, so that the variant size difference check in trans can call
 732 // `raw_emit_lint`.
 733
 734 struct GatherNodeLevels;
 735
 736 impl LintPass for GatherNodeLevels {
 737     fn get_lints(&self) -> LintArray {
 738         lint_array!()
 739     }
 740
 741     fn check_item(&mut self, cx: &Context, it: &ast::Item) {
 742         match it.node {
 743             ast::ItemEnum(..) => {
 744                 let lint_id = LintId::of(builtin::VARIANT_SIZE_DIFFERENCES);
 745                 let lvlsrc = cx.lints.get_level_source(lint_id);
 746                 match lvlsrc {
 747                     (lvl, _) if lvl != Allow => {
 748                         cx.node_levels.borrow_mut()
 749                             .insert((it.id, lint_id), lvlsrc);
 750                     },
 751                     _ => { }
 752                 }
 753             },
 754             _ => { }
 755         }
 756     }
 757 }
 758
 759 /// Perform lint checking on a crate.
 760 ///
 761 /// Consumes the `lint_store` field of the `Session`.
 762 pub fn check_crate(tcx: &ty::ctxt,
 763                    exported_items: &ExportedItems) {
 764     let krate = tcx.map.krate();
 765     let mut cx = Context::new(tcx, krate, exported_items);
 766
 767     // Visit the whole crate.
 768     cx.with_lint_attrs(krate.attrs[], |cx| {
 769         cx.visit_id(ast::CRATE_NODE_ID);
 770         cx.visit_ids(|v| {
 771             v.visited_outermost = true;
 772             visit::walk_crate(v, krate);
 773         });
 774
 775         // since the root module isn't visited as an item (because it isn't an
 776         // item), warn for it here.
 777         run_lints!(cx, check_crate, krate);
 778
 779         visit::walk_crate(cx, krate);
 780     });
 781
 782     // If we missed any lints added to the session, then there's a bug somewhere
 783     // in the iteration code.
 784     for (id, v) in tcx.sess.lints.borrow().iter() {
 785         for &(lint, span, ref msg) in v.iter() {
 786             tcx.sess.span_bug(span,
 787                               format!("unprocessed lint {} at {}: {}",
 788                                       lint.as_str(), tcx.map.node_to_string(*id), *msg)[])
 789         }
 790     }
 791
 792     tcx.sess.abort_if_errors();
 793     *tcx.node_lint_levels.borrow_mut() = cx.node_levels.into_inner();
 794 }