context.rs 34.7 KB
Newer Older
1
// Copyright 2012-2015 The Rust Project Developers. See the COPYRIGHT
2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25
// file at the top-level directory of this distribution and at
// http://rust-lang.org/COPYRIGHT.
//
// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
// option. This file may not be copied, modified, or distributed
// except according to those terms.

//! Implementation of lint checking.
//!
//! The lint checking is mostly consolidated into one pass which runs just
//! before translation to LLVM bytecode. Throughout compilation, lint warnings
//! can be added via the `add_lint` method on the Session structure. This
//! requires a span and an id of the node that the lint is being added to. The
//! lint isn't actually emitted at that time because it is unknown what the
//! actual lint level at that location is.
//!
//! To actually emit lint warnings/errors, a separate pass is used just before
//! translation. A context keeps track of the current state of all lint levels.
//! Upon entering a node of the ast which can modify the lint settings, the
//! previous lint state is pushed onto a stack and the ast is then recursed
//! upon.  As the ast is traversed, this keeps track of the current lint level
//! for all lint attributes.
S
Steven Fackler 已提交
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::cmp;
38
use std::mem;
39
use syntax::ast_util::{self, IdVisitingOperation};
40
use syntax::attr::{self, AttrMetaMethods};
41
use syntax::codemap::Span;
42
use syntax::parse::token::InternedString;
M
Manish Goregaokar 已提交
43 44
use syntax::ast;
use rustc_front::hir;
45
use rustc_front::util;
46 47
use rustc_front::visit as hir_visit;
use syntax::visit as ast_visit;
48
use syntax::diagnostic;
49 50 51 52 53 54 55 56 57 58 59

/// Information about the registered lints.
///
/// This is basically the subset of `Context` that we can
/// build early in the compile pipeline.
pub struct LintStore {
    /// Registered lints. The bool is true if the lint was
    /// added by a plugin.
    lints: Vec<(&'static Lint, bool)>,

    /// Trait objects for each lint pass.
60 61 62
    /// This is only `None` while iterating over the objects. See the definition
    /// of run_lints.
    passes: Option<Vec<LintPassObject>>,
63 64

    /// Lints indexed by name.
65
    by_name: FnvHashMap<String, TargetLint>,
66 67

    /// Current levels of each lint, and where they were set.
68
    levels: FnvHashMap<LintId, LevelSource>,
69 70 71

    /// Map of registered lint groups to what lints they expand to. The bool
    /// is true if the lint group was added by a plugin.
72
    lint_groups: FnvHashMap<&'static str, (Vec<LintId>, bool)>,
73 74 75

    /// Maximum level a lint can be
    lint_cap: Option<Level>,
76 77
}

78 79 80 81 82 83 84
/// The targed of the `by_name` map, which accounts for renaming/deprecation.
enum TargetLint {
    /// A direct lint target
    Id(LintId),

    /// Temporary renaming, used for easing migration pain; see #16545
    Renamed(String, LintId),
85 86 87 88 89 90 91 92 93

    /// Lint with this name existed previously, but has been removed/deprecated.
    /// The string argument is the reason for removal.
    Removed(String),
}

enum FindLintError {
    NotFound,
    Removed
94 95
}

96 97
impl LintStore {
    fn get_level_source(&self, lint: LintId) -> LevelSource {
98
        match self.levels.get(&lint) {
99 100 101 102 103
            Some(&s) => s,
            None => (Allow, Default),
        }
    }

104 105 106 107
    fn set_level(&mut self, lint: LintId, mut lvlsrc: LevelSource) {
        if let Some(cap) = self.lint_cap {
            lvlsrc.0 = cmp::min(lvlsrc.0, cap);
        }
J
Jorge Aparicio 已提交
108
        if lvlsrc.0 == Allow {
109 110 111 112 113 114 115 116 117
            self.levels.remove(&lint);
        } else {
            self.levels.insert(lint, lvlsrc);
        }
    }

    pub fn new() -> LintStore {
        LintStore {
            lints: vec!(),
118
            passes: Some(vec!()),
119 120 121
            by_name: FnvHashMap(),
            levels: FnvHashMap(),
            lint_groups: FnvHashMap(),
122
            lint_cap: None,
123 124 125 126
        }
    }

    pub fn get_lints<'t>(&'t self) -> &'t [(&'static Lint, bool)] {
127
        &self.lints
128 129
    }

130
    pub fn get_lint_groups<'t>(&'t self) -> Vec<(&'static str, Vec<LintId>, bool)> {
131
        self.lint_groups.iter().map(|(k, v)| (*k,
J
Jorge Aparicio 已提交
132 133
                                              v.0.clone(),
                                              v.1)).collect()
134 135
    }

136 137
    pub fn register_pass(&mut self, sess: Option<&Session>,
                         from_plugin: bool, pass: LintPassObject) {
138
        for &lint in pass.get_lints() {
139
            self.lints.push((*lint, from_plugin));
140

141
            let id = LintId::of(*lint);
142
            if self.by_name.insert(lint.name_lower(), Id(id)).is_some() {
143
                let msg = format!("duplicate specification of lint {}", lint.name_lower());
144 145 146
                match (sess, from_plugin) {
                    // We load builtin lints first, so a duplicate is a compiler bug.
                    // Use early_error when handling -W help with no crate.
147
                    (None, _) => early_error(diagnostic::Auto, &msg[..]),
148
                    (Some(sess), false) => sess.bug(&msg[..]),
149 150

                    // A duplicate name from a plugin is a user error.
151
                    (Some(sess), true)  => sess.err(&msg[..]),
152 153 154 155 156 157 158
                }
            }

            if lint.default_level != Allow {
                self.levels.insert(id, (lint.default_level, Default));
            }
        }
159
        self.passes.as_mut().unwrap().push(pass);
160 161
    }

162 163 164
    pub fn register_group(&mut self, sess: Option<&Session>,
                          from_plugin: bool, name: &'static str,
                          to: Vec<LintId>) {
165
        let new = self.lint_groups.insert(name, (to, from_plugin)).is_none();
166 167 168 169 170 171

        if !new {
            let msg = format!("duplicate specification of lint group {}", name);
            match (sess, from_plugin) {
                // We load builtin lints first, so a duplicate is a compiler bug.
                // Use early_error when handling -W help with no crate.
172
                (None, _) => early_error(diagnostic::Auto, &msg[..]),
173
                (Some(sess), false) => sess.bug(&msg[..]),
174 175

                // A duplicate name from a plugin is a user error.
176
                (Some(sess), true)  => sess.err(&msg[..]),
177 178 179 180
            }
        }
    }

181
    pub fn register_renamed(&mut self, old_name: &str, new_name: &str) {
A
Aaron Turon 已提交
182
        let target = match self.by_name.get(new_name) {
183
            Some(&Id(lint_id)) => lint_id.clone(),
S
Steve Klabnik 已提交
184
            _ => panic!("invalid lint renaming of {} to {}", old_name, new_name)
185 186 187 188
        };
        self.by_name.insert(old_name.to_string(), Renamed(new_name.to_string(), target));
    }

189 190 191 192
    pub fn register_removed(&mut self, name: &str, reason: &str) {
        self.by_name.insert(name.into(), Removed(reason.into()));
    }

A
Alex Crichton 已提交
193
    #[allow(unused_variables)]
194
    fn find_lint(&self, lint_name: &str, sess: &Session, span: Option<Span>)
195
                 -> Result<LintId, FindLintError>
196
    {
A
Aaron Turon 已提交
197
        match self.by_name.get(lint_name) {
198
            Some(&Id(lint_id)) => Ok(lint_id),
199
            Some(&Renamed(ref new_name, lint_id)) => {
200 201 202
                let warning = format!("lint {} has been renamed to {}",
                                      lint_name, new_name);
                match span {
203 204
                    Some(span) => sess.span_warn(span, &warning[..]),
                    None => sess.warn(&warning[..]),
205
                };
206 207 208 209 210 211 212 213 214 215 216
                Ok(lint_id)
            },
            Some(&Removed(ref reason)) => {
                let warning = format!("lint {} has been removed: {}", lint_name, reason);
                match span {
                    Some(span) => sess.span_warn(span, &warning[..]),
                    None => sess.warn(&warning[..])
                }
                Err(FindLintError::Removed)
            },
            None => Err(FindLintError::NotFound)
217
        }
218 219 220
    }

    pub fn process_command_line(&mut self, sess: &Session) {
221
        for &(ref lint_name, level) in &sess.opts.lint_opts {
222
            match self.find_lint(&lint_name[..], sess, None) {
223 224
                Ok(lint_id) => self.set_level(lint_id, (level, CommandLine)),
                Err(_) => {
J
Jorge Aparicio 已提交
225
                    match self.lint_groups.iter().map(|(&x, pair)| (x, pair.0.clone()))
226 227
                                                 .collect::<FnvHashMap<&'static str,
                                                                       Vec<LintId>>>()
228
                                                 .get(&lint_name[..]) {
229 230 231 232 233 234
                        Some(v) => {
                            v.iter()
                             .map(|lint_id: &LintId|
                                     self.set_level(*lint_id, (level, CommandLine)))
                             .collect::<Vec<()>>();
                        }
J
Jorge Aparicio 已提交
235
                        None => sess.err(&format!("unknown {} flag: {}",
236
                                                 level.as_str(), lint_name)),
237 238
                    }
                }
239 240
            }
        }
241 242 243 244 245 246 247

        self.lint_cap = sess.opts.lint_cap;
        if let Some(cap) = self.lint_cap {
            for level in self.levels.iter_mut().map(|p| &mut (p.1).0) {
                *level = cmp::min(*level, cap);
            }
        }
248 249 250
    }
}

251 252
/// Context for lint checking after type checking.
pub struct LateContext<'a, 'tcx: 'a> {
253
    /// Type context we're checking in.
254
    pub tcx: &'a ty::ctxt<'tcx>,
255

256
    /// The crate being checked.
M
Manish Goregaokar 已提交
257
    pub krate: &'a hir::Crate,
258 259 260 261

    /// Items exported from the crate being checked.
    pub exported_items: &'a ExportedItems,

262 263 264 265 266 267 268 269 270 271
    /// The store of registered lints.
    lints: LintStore,

    /// When recursing into an attributed node of the ast which modifies lint
    /// levels, this stack keeps track of the previous lint levels of whatever
    /// was modified.
    level_stack: Vec<(LintId, LevelSource)>,

    /// Level of lints for certain NodeIds, stored here because the body of
    /// the lint needs to run in trans.
272
    node_levels: RefCell<FnvHashMap<(ast::NodeId, LintId), LevelSource>>,
273 274
}

275 276 277 278 279 280 281 282 283 284 285 286 287 288 289 290 291 292 293 294
pub type Context<'a, 'tcx: 'a> = LateContext<'a, 'tcx>;

/// Context for lint checking of the AST, after expansion, before lowering to
/// HIR.
pub struct EarlyContext<'a> {
    /// Type context we're checking in.
    pub sess: &'a Session,

    /// The crate being checked.
    pub krate: &'a ast::Crate,

    /// The store of registered lints.
    lints: LintStore,

    /// When recursing into an attributed node of the ast which modifies lint
    /// levels, this stack keeps track of the previous lint levels of whatever
    /// was modified.
    level_stack: Vec<(LintId, LevelSource)>,
}

295
/// Convenience macro for calling a `LintPass` method on every pass in the context.
296
macro_rules! run_lints { ($cx:expr, $f:ident, $($args:expr),*) => ({
297 298
    // Move the vector of passes out of `$cx` so that we can
    // iterate over it mutably while passing `$cx` to the methods.
299
    let mut passes = $cx.mut_lints().passes.take().unwrap();
300
    for obj in &mut passes {
301
        obj.$f($cx, $($args),*);
302
    }
303
    $cx.mut_lints().passes = Some(passes);
304
}) }
305

306 307
/// Parse the lint attributes into a vector, with `Err`s for malformed lint
/// attributes. Writing this as an iterator is an enormous mess.
308
// See also the hir version just below.
309
pub fn gather_attrs(attrs: &[ast::Attribute])
310 311
                    -> Vec<Result<(InternedString, Level, Span), Span>> {
    let mut out = vec!();
312
    for attr in attrs {
313
        let level = match Level::from_str(&attr.name()) {
314 315 316 317 318 319
            None => continue,
            Some(lvl) => lvl,
        };

        attr::mark_used(attr);

320
        let meta = &attr.node.value;
321
        let metas = match meta.node {
322
            ast::MetaList(_, ref metas) => metas,
323 324 325 326 327 328 329 330
            _ => {
                out.push(Err(meta.span));
                continue;
            }
        };

        for meta in metas {
            out.push(match meta.node {
331
                ast::MetaWord(ref lint_name) => Ok((lint_name.clone(), level, meta.span)),
332 333 334 335 336 337
                _ => Err(meta.span),
            });
        }
    }
    out
}
338

339 340 341 342 343 344 345 346 347 348 349 350
/// Emit a lint as a warning or an error (or not at all)
/// according to `level`.
///
/// This lives outside of `Context` so it can be used by checks
/// in trans that run after the main lint pass is finished. Most
/// lints elsewhere in the compiler should call
/// `Session::add_lint()` instead.
pub fn raw_emit_lint(sess: &Session, lint: &'static Lint,
                     lvlsrc: LevelSource, span: Option<Span>, msg: &str) {
    let (mut level, source) = lvlsrc;
    if level == Allow { return }

351
    let name = lint.name_lower();
B
Brian Anderson 已提交
352
    let mut def = None;
353 354 355
    let msg = match source {
        Default => {
            format!("{}, #[{}({})] on by default", msg,
356
                    level.as_str(), name)
357 358 359
        },
        CommandLine => {
            format!("{} [-{} {}]", msg,
360 361
                    match level {
                        Warn => 'W', Deny => 'D', Forbid => 'F',
S
Steve Klabnik 已提交
362
                        Allow => panic!()
363
                    }, name.replace("_", "-"))
364 365
        },
        Node(src) => {
B
Brian Anderson 已提交
366 367 368
            def = Some(src);
            msg.to_string()
        }
369 370 371 372 373 374
    };

    // For purposes of printing, we can treat forbid as deny.
    if level == Forbid { level = Deny; }

    match (level, span) {
375 376 377 378
        (Warn, Some(sp)) => sess.span_warn(sp, &msg[..]),
        (Warn, None)     => sess.warn(&msg[..]),
        (Deny, Some(sp)) => sess.span_err(sp, &msg[..]),
        (Deny, None)     => sess.err(&msg[..]),
379 380 381
        _ => sess.bug("impossible level in raw_emit_lint"),
    }

382
    if let Some(span) = def {
383 384 385 386
        sess.span_note(span, "lint level defined here");
    }
}

387 388 389 390 391 392 393
pub trait LintContext: Sized {
    fn sess(&self) -> &Session;
    fn lints(&self) -> &LintStore;
    fn mut_lints(&mut self) -> &mut LintStore;
    fn level_stack(&mut self) -> &mut Vec<(LintId, LevelSource)>;
    fn enter_attrs(&mut self, attrs: &[hir::Attribute]);
    fn exit_attrs(&mut self, attrs: &[hir::Attribute]);
394

395 396
    /// Get the level of `lint` at the current position of the lint
    /// traversal.
397 398
    fn current_level(&self, lint: &'static Lint) -> Level {
        self.lints().levels.get(&LintId::of(lint)).map_or(Allow, |&(lvl, _)| lvl)
399 400
    }

401
    fn lookup_and_emit(&self, lint: &'static Lint, span: Option<Span>, msg: &str) {
402
        let (level, src) = match self.lints().levels.get(&LintId::of(lint)) {
403 404
            None => return,
            Some(&(Warn, src)) => {
405
                let lint_id = LintId::of(builtin::WARNINGS);
406
                (self.lints().get_level_source(lint_id).0, src)
407 408 409 410
            }
            Some(&pair) => pair,
        };

411
        raw_emit_lint(&self.sess(), lint, (level, src), span, msg);
412 413 414
    }

    /// Emit a lint at the appropriate level, for a particular span.
415
    fn span_lint(&self, lint: &'static Lint, span: Span, msg: &str) {
416 417 418
        self.lookup_and_emit(lint, Some(span), msg);
    }

419 420 421 422 423
    /// Emit a lint at the appropriate level, with no associated span.
    fn lint(&self, lint: &'static Lint, msg: &str) {
        self.lookup_and_emit(lint, None, msg);
    }

S
Steve Klabnik 已提交
424 425 426
    /// Merge the lints specified by any lint attributes into the
    /// current lint context, call the provided function, then reset the
    /// lints in effect to their previous state.
J
Jorge Aparicio 已提交
427
    fn with_lint_attrs<F>(&mut self,
428
                          attrs: &[ast::Attribute],
429 430
                          f: F)
        where F: FnOnce(&mut Self),
J
Jorge Aparicio 已提交
431
    {
432 433 434 435
        // Parse all of the lint attributes, and then add them all to the
        // current dictionary of lint information. Along the way, keep a history
        // of what we changed so we can roll everything back after invoking the
        // specified closure
436
        let mut pushed = 0;
437

438
        for result in gather_attrs(attrs) {
439
            let v = match result {
440
                Err(span) => {
441
                    span_err!(self.sess(), span, E0452,
442
                              "malformed lint attribute");
443 444 445
                    continue;
                }
                Ok((lint_name, level, span)) => {
446
                    match self.lints().find_lint(&lint_name, &self.sess(), Some(span)) {
447 448
                        Ok(lint_id) => vec![(lint_id, level, span)],
                        Err(FindLintError::NotFound) => {
449
                            match self.lints().lint_groups.get(&lint_name[..]) {
450 451 452 453 454
                                Some(&(ref v, _)) => v.iter()
                                                      .map(|lint_id: &LintId|
                                                           (*lint_id, level, span))
                                                      .collect(),
                                None => {
A
Aaron Turon 已提交
455
                                    self.span_lint(builtin::UNKNOWN_LINTS, span,
456 457
                                                   &format!("unknown `{}` attribute: `{}`",
                                                            level.as_str(), lint_name));
458 459 460
                                    continue;
                                }
                            }
461 462
                        },
                        Err(FindLintError::Removed) => { continue; }
463 464 465 466
                    }
                }
            };

467
            for (lint_id, level, span) in v {
468
                let now = self.lints().get_level_source(lint_id).0;
469 470
                if now == Forbid && level != Forbid {
                    let lint_name = lint_id.as_str();
471
                    span_err!(self.sess(), span, E0453,
472 473 474
                              "{}({}) overruled by outer forbid({})",
                              level.as_str(), lint_name,
                              lint_name);
475
                } else if now != level {
476 477
                    let src = self.lints().get_level_source(lint_id).1;
                    self.level_stack().push((lint_id, (now, src)));
478
                    pushed += 1;
479
                    self.mut_lints().set_level(lint_id, (level, Node(span)));
480
                }
481 482 483
            }
        }

484
        self.enter_attrs(attrs);
485
        f(self);
486
        self.exit_attrs(attrs);
487 488

        // rollback
489
        for _ in 0..pushed {
490 491 492 493 494 495 496 497 498 499 500 501 502 503 504 505 506 507
            let (lint, lvlsrc) = self.level_stack().pop().unwrap();
            self.mut_lints().set_level(lint, lvlsrc);
        }
    }

    fn with_ast_lint_attrs<F>(&mut self,
                          attrs: &[ast::Attribute],
                          f: F)
        where F: FnOnce(&mut Self),
    {
        self.with_lint_attrs(&lower_attrs(attrs), f)
    }
}


impl<'a> EarlyContext<'a> {
    fn new(sess: &'a Session,
           krate: &'a ast::Crate) -> EarlyContext<'a> {
N
Nick Cameron 已提交
508 509
        // We want to own the lint store, so move it out of the session. Remember
        // to put it back later...
510 511 512 513 514 515 516
        let lint_store = mem::replace(&mut *sess.lint_store.borrow_mut(),
                                      LintStore::new());
        EarlyContext {
            sess: sess,
            krate: krate,
            lints: lint_store,
            level_stack: vec![],
517 518 519
        }
    }

520 521 522 523 524 525 526 527 528 529 530 531 532 533 534 535 536 537 538 539 540 541 542 543 544 545 546 547 548 549 550 551
    fn visit_ids<F>(&mut self, f: F)
        where F: FnOnce(&mut ast_util::IdVisitor<EarlyContext>)
    {
        let mut v = ast_util::IdVisitor {
            operation: self,
            pass_through_items: false,
            visited_outermost: false,
        };
        f(&mut v);
    }
}

impl<'a, 'tcx> LateContext<'a, 'tcx> {
    fn new(tcx: &'a ty::ctxt<'tcx>,
           krate: &'a hir::Crate,
           exported_items: &'a ExportedItems) -> LateContext<'a, 'tcx> {
        // We want to own the lint store, so move it out of the session.
        let lint_store = mem::replace(&mut *tcx.sess.lint_store.borrow_mut(),
                                      LintStore::new());

        LateContext {
            tcx: tcx,
            krate: krate,
            exported_items: exported_items,
            lints: lint_store,
            level_stack: vec![],
            node_levels: RefCell::new(FnvHashMap()),
        }
    }

    fn visit_ids<F>(&mut self, f: F)
        where F: FnOnce(&mut util::IdVisitor<LateContext>)
J
Jorge Aparicio 已提交
552
    {
M
Manish Goregaokar 已提交
553
        let mut v = util::IdVisitor {
554 555 556 557 558 559 560 561
            operation: self,
            pass_through_items: false,
            visited_outermost: false,
        };
        f(&mut v);
    }
}

562 563 564 565 566 567 568 569 570 571 572 573 574 575 576 577 578 579 580 581 582 583 584 585 586 587 588 589 590 591 592 593 594 595 596 597 598 599 600 601 602 603 604 605 606 607 608 609 610 611 612 613 614 615 616
impl<'a, 'tcx> LintContext for LateContext<'a, 'tcx> {
    /// Get the overall compiler `Session` object.
    fn sess(&self) -> &Session {
        &self.tcx.sess
    }

    fn lints(&self) -> &LintStore {
        &self.lints
    }

    fn mut_lints(&mut self) -> &mut LintStore {
        &mut self.lints
    }

    fn level_stack(&mut self) -> &mut Vec<(LintId, LevelSource)> {
        &mut self.level_stack
    }

    fn enter_attrs(&mut self, attrs: &[hir::Attribute]) {
        run_lints!(self, enter_lint_attrs, attrs);
    }

    fn exit_attrs(&mut self, attrs: &[hir::Attribute]) {
        run_lints!(self, exit_lint_attrs, attrs);
    }
}

impl<'a> LintContext for EarlyContext<'a> {
    /// Get the overall compiler `Session` object.
    fn sess(&self) -> &Session {
        &self.sess
    }

    fn lints(&self) -> &LintStore {
        &self.lints
    }

    fn mut_lints(&mut self) -> &mut LintStore {
        &mut self.lints
    }

    fn level_stack(&mut self) -> &mut Vec<(LintId, LevelSource)> {
        &mut self.level_stack
    }

    fn enter_attrs(&mut self, attrs: &[hir::Attribute]) {
        run_lints!(self, ast_enter_lint_attrs, attrs);
    }

    fn exit_attrs(&mut self, attrs: &[hir::Attribute]) {
        run_lints!(self, ast_exit_lint_attrs, attrs);
    }
}

impl<'a, 'tcx, 'v> hir_visit::Visitor<'v> for LateContext<'a, 'tcx> {
M
Manish Goregaokar 已提交
617
    fn visit_item(&mut self, it: &hir::Item) {
618
        self.with_lint_attrs(&it.attrs, |cx| {
619
            run_lints!(cx, check_item, it);
620
            cx.visit_ids(|v| v.visit_item(it));
621
            hir_visit::walk_item(cx, it);
622 623 624
        })
    }

M
Manish Goregaokar 已提交
625
    fn visit_foreign_item(&mut self, it: &hir::ForeignItem) {
626
        self.with_lint_attrs(&it.attrs, |cx| {
627
            run_lints!(cx, check_foreign_item, it);
628
            hir_visit::walk_foreign_item(cx, it);
629 630 631
        })
    }

M
Manish Goregaokar 已提交
632
    fn visit_pat(&mut self, p: &hir::Pat) {
633
        run_lints!(self, check_pat, p);
634
        hir_visit::walk_pat(self, p);
635 636
    }

M
Manish Goregaokar 已提交
637
    fn visit_expr(&mut self, e: &hir::Expr) {
638
        run_lints!(self, check_expr, e);
639
        hir_visit::walk_expr(self, e);
640 641
    }

M
Manish Goregaokar 已提交
642
    fn visit_stmt(&mut self, s: &hir::Stmt) {
643
        run_lints!(self, check_stmt, s);
644
        hir_visit::walk_stmt(self, s);
645 646
    }

647
    fn visit_fn(&mut self, fk: hir_visit::FnKind<'v>, decl: &'v hir::FnDecl,
M
Manish Goregaokar 已提交
648
                body: &'v hir::Block, span: Span, id: ast::NodeId) {
649
        run_lints!(self, check_fn, fk, decl, body, span, id);
650
        hir_visit::walk_fn(self, fk, decl, body, span);
651 652 653
    }

    fn visit_struct_def(&mut self,
M
Manish Goregaokar 已提交
654
                        s: &hir::StructDef,
655
                        ident: ast::Ident,
M
Manish Goregaokar 已提交
656
                        g: &hir::Generics,
657
                        id: ast::NodeId) {
658
        run_lints!(self, check_struct_def, s, ident, g, id);
659
        hir_visit::walk_struct_def(self, s);
660 661 662
        run_lints!(self, check_struct_def_post, s, ident, g, id);
    }

M
Manish Goregaokar 已提交
663
    fn visit_struct_field(&mut self, s: &hir::StructField) {
664
        self.with_lint_attrs(&s.node.attrs, |cx| {
665
            run_lints!(cx, check_struct_field, s);
666
            hir_visit::walk_struct_field(cx, s);
667 668 669
        })
    }

M
Manish Goregaokar 已提交
670
    fn visit_variant(&mut self, v: &hir::Variant, g: &hir::Generics) {
671
        self.with_lint_attrs(&v.node.attrs, |cx| {
672
            run_lints!(cx, check_variant, v, g);
673
            hir_visit::walk_variant(cx, v, g);
S
Steven Fackler 已提交
674
            run_lints!(cx, check_variant_post, v, g);
675 676 677
        })
    }

M
Manish Goregaokar 已提交
678
    fn visit_ty(&mut self, t: &hir::Ty) {
679
        run_lints!(self, check_ty, t);
680
        hir_visit::walk_ty(self, t);
681 682
    }

683
    fn visit_ident(&mut self, sp: Span, id: ast::Ident) {
684 685 686
        run_lints!(self, check_ident, sp, id);
    }

M
Manish Goregaokar 已提交
687
    fn visit_mod(&mut self, m: &hir::Mod, s: Span, n: ast::NodeId) {
688
        run_lints!(self, check_mod, m, s, n);
689
        hir_visit::walk_mod(self, m);
690 691
    }

M
Manish Goregaokar 已提交
692
    fn visit_local(&mut self, l: &hir::Local) {
693
        run_lints!(self, check_local, l);
694
        hir_visit::walk_local(self, l);
695 696
    }

M
Manish Goregaokar 已提交
697
    fn visit_block(&mut self, b: &hir::Block) {
698
        run_lints!(self, check_block, b);
699
        hir_visit::walk_block(self, b);
700 701
    }

M
Manish Goregaokar 已提交
702
    fn visit_arm(&mut self, a: &hir::Arm) {
703
        run_lints!(self, check_arm, a);
704
        hir_visit::walk_arm(self, a);
705 706
    }

M
Manish Goregaokar 已提交
707
    fn visit_decl(&mut self, d: &hir::Decl) {
708
        run_lints!(self, check_decl, d);
709
        hir_visit::walk_decl(self, d);
710 711
    }

M
Manish Goregaokar 已提交
712
    fn visit_expr_post(&mut self, e: &hir::Expr) {
713 714 715
        run_lints!(self, check_expr_post, e);
    }

M
Manish Goregaokar 已提交
716
    fn visit_generics(&mut self, g: &hir::Generics) {
717
        run_lints!(self, check_generics, g);
718
        hir_visit::walk_generics(self, g);
719 720
    }

M
Manish Goregaokar 已提交
721
    fn visit_trait_item(&mut self, trait_item: &hir::TraitItem) {
722 723 724
        self.with_lint_attrs(&trait_item.attrs, |cx| {
            run_lints!(cx, check_trait_item, trait_item);
            cx.visit_ids(|v| v.visit_trait_item(trait_item));
725
            hir_visit::walk_trait_item(cx, trait_item);
726 727 728
        });
    }

M
Manish Goregaokar 已提交
729
    fn visit_impl_item(&mut self, impl_item: &hir::ImplItem) {
730 731 732
        self.with_lint_attrs(&impl_item.attrs, |cx| {
            run_lints!(cx, check_impl_item, impl_item);
            cx.visit_ids(|v| v.visit_impl_item(impl_item));
733
            hir_visit::walk_impl_item(cx, impl_item);
734
        });
735 736
    }

M
Manish Goregaokar 已提交
737
    fn visit_opt_lifetime_ref(&mut self, sp: Span, lt: &Option<hir::Lifetime>) {
738 739 740
        run_lints!(self, check_opt_lifetime_ref, sp, lt);
    }

M
Manish Goregaokar 已提交
741
    fn visit_lifetime_ref(&mut self, lt: &hir::Lifetime) {
742 743 744
        run_lints!(self, check_lifetime_ref, lt);
    }

M
Manish Goregaokar 已提交
745
    fn visit_lifetime_def(&mut self, lt: &hir::LifetimeDef) {
746
        run_lints!(self, check_lifetime_def, lt);
747 748
    }

M
Manish Goregaokar 已提交
749
    fn visit_explicit_self(&mut self, es: &hir::ExplicitSelf) {
750
        run_lints!(self, check_explicit_self, es);
751
        hir_visit::walk_explicit_self(self, es);
752 753
    }

M
Manish Goregaokar 已提交
754
    fn visit_path(&mut self, p: &hir::Path, id: ast::NodeId) {
755
        run_lints!(self, check_path, p, id);
756
        hir_visit::walk_path(self, p);
757 758
    }

759
    fn visit_attribute(&mut self, attr: &ast::Attribute) {
760 761 762 763
        run_lints!(self, check_attribute, attr);
    }
}

764 765 766 767 768 769 770 771 772 773 774 775 776 777 778 779 780 781 782 783 784 785 786 787 788 789 790 791 792 793 794 795 796 797 798 799 800 801 802 803 804 805 806 807 808 809 810 811 812 813 814 815 816 817 818 819 820 821 822 823 824 825 826 827 828 829 830 831 832 833 834 835 836 837 838 839 840 841 842 843 844 845 846 847 848 849 850 851 852 853 854 855 856 857 858 859 860 861 862 863 864 865 866 867 868 869 870 871 872 873 874 875 876 877 878 879 880 881 882 883 884 885 886 887 888 889 890 891 892 893 894 895 896 897 898 899 900 901 902 903 904 905 906 907 908 909 910 911
impl<'a, 'v> ast_visit::Visitor<'v> for EarlyContext<'a> {
    fn visit_item(&mut self, it: &ast::Item) {
        self.with_ast_lint_attrs(&it.attrs, |cx| {
            run_lints!(cx, check_ast_item, it);
            cx.visit_ids(|v| v.visit_item(it));
            ast_visit::walk_item(cx, it);
        })
    }

    fn visit_foreign_item(&mut self, it: &ast::ForeignItem) {
        self.with_ast_lint_attrs(&it.attrs, |cx| {
            run_lints!(cx, check_ast_foreign_item, it);
            ast_visit::walk_foreign_item(cx, it);
        })
    }

    fn visit_pat(&mut self, p: &ast::Pat) {
        run_lints!(self, check_ast_pat, p);
        ast_visit::walk_pat(self, p);
    }

    fn visit_expr(&mut self, e: &ast::Expr) {
        run_lints!(self, check_ast_expr, e);
        ast_visit::walk_expr(self, e);
    }

    fn visit_stmt(&mut self, s: &ast::Stmt) {
        run_lints!(self, check_ast_stmt, s);
        ast_visit::walk_stmt(self, s);
    }

    fn visit_fn(&mut self, fk: ast_visit::FnKind<'v>, decl: &'v ast::FnDecl,
                body: &'v ast::Block, span: Span, id: ast::NodeId) {
        run_lints!(self, check_ast_fn, fk, decl, body, span, id);
        ast_visit::walk_fn(self, fk, decl, body, span);
    }

    fn visit_struct_def(&mut self,
                        s: &ast::StructDef,
                        ident: ast::Ident,
                        g: &ast::Generics,
                        id: ast::NodeId) {
        run_lints!(self, check_ast_struct_def, s, ident, g, id);
        ast_visit::walk_struct_def(self, s);
        run_lints!(self, check_ast_struct_def_post, s, ident, g, id);
    }

    fn visit_struct_field(&mut self, s: &ast::StructField) {
        self.with_ast_lint_attrs(&s.node.attrs, |cx| {
            run_lints!(cx, check_ast_struct_field, s);
            ast_visit::walk_struct_field(cx, s);
        })
    }

    fn visit_variant(&mut self, v: &ast::Variant, g: &ast::Generics) {
        self.with_ast_lint_attrs(&v.node.attrs, |cx| {
            run_lints!(cx, check_ast_variant, v, g);
            ast_visit::walk_variant(cx, v, g);
            run_lints!(cx, check_ast_variant_post, v, g);
        })
    }

    fn visit_ty(&mut self, t: &ast::Ty) {
        run_lints!(self, check_ast_ty, t);
        ast_visit::walk_ty(self, t);
    }

    fn visit_ident(&mut self, sp: Span, id: ast::Ident) {
        run_lints!(self, check_ast_ident, sp, id);
    }

    fn visit_mod(&mut self, m: &ast::Mod, s: Span, n: ast::NodeId) {
        run_lints!(self, check_ast_mod, m, s, n);
        ast_visit::walk_mod(self, m);
    }

    fn visit_local(&mut self, l: &ast::Local) {
        run_lints!(self, check_ast_local, l);
        ast_visit::walk_local(self, l);
    }

    fn visit_block(&mut self, b: &ast::Block) {
        run_lints!(self, check_ast_block, b);
        ast_visit::walk_block(self, b);
    }

    fn visit_arm(&mut self, a: &ast::Arm) {
        run_lints!(self, check_ast_arm, a);
        ast_visit::walk_arm(self, a);
    }

    fn visit_decl(&mut self, d: &ast::Decl) {
        run_lints!(self, check_ast_decl, d);
        ast_visit::walk_decl(self, d);
    }

    fn visit_expr_post(&mut self, e: &ast::Expr) {
        run_lints!(self, check_ast_expr_post, e);
    }

    fn visit_generics(&mut self, g: &ast::Generics) {
        run_lints!(self, check_ast_generics, g);
        ast_visit::walk_generics(self, g);
    }

    fn visit_trait_item(&mut self, trait_item: &ast::TraitItem) {
        self.with_ast_lint_attrs(&trait_item.attrs, |cx| {
            run_lints!(cx, check_ast_trait_item, trait_item);
            cx.visit_ids(|v| v.visit_trait_item(trait_item));
            ast_visit::walk_trait_item(cx, trait_item);
        });
    }

    fn visit_impl_item(&mut self, impl_item: &ast::ImplItem) {
        self.with_ast_lint_attrs(&impl_item.attrs, |cx| {
            run_lints!(cx, check_ast_impl_item, impl_item);
            cx.visit_ids(|v| v.visit_impl_item(impl_item));
            ast_visit::walk_impl_item(cx, impl_item);
        });
    }

    fn visit_opt_lifetime_ref(&mut self, sp: Span, lt: &Option<ast::Lifetime>) {
        run_lints!(self, check_ast_opt_lifetime_ref, sp, lt);
    }

    fn visit_lifetime_ref(&mut self, lt: &ast::Lifetime) {
        run_lints!(self, check_ast_lifetime_ref, lt);
    }

    fn visit_lifetime_def(&mut self, lt: &ast::LifetimeDef) {
        run_lints!(self, check_ast_lifetime_def, lt);
    }

    fn visit_explicit_self(&mut self, es: &ast::ExplicitSelf) {
        run_lints!(self, check_ast_explicit_self, es);
        ast_visit::walk_explicit_self(self, es);
    }

    fn visit_path(&mut self, p: &ast::Path, id: ast::NodeId) {
        run_lints!(self, check_ast_path, p, id);
        ast_visit::walk_path(self, p);
    }

    fn visit_attribute(&mut self, attr: &ast::Attribute) {
        run_lints!(self, check_ast_attribute, attr);
    }
}

912
// Output any lints that were previously added to the session.
913 914 915 916 917 918 919 920 921 922 923 924 925
impl<'a, 'tcx> IdVisitingOperation for LateContext<'a, 'tcx> {
    fn visit_id(&mut self, id: ast::NodeId) {
        match self.sess().lints.borrow_mut().remove(&id) {
            None => {}
            Some(lints) => {
                for (lint_id, span, msg) in lints {
                    self.span_lint(lint_id.lint, span, &msg[..])
                }
            }
        }
    }
}
impl<'a> IdVisitingOperation for EarlyContext<'a> {
A
Ariel Ben-Yehuda 已提交
926
    fn visit_id(&mut self, id: ast::NodeId) {
927
        match self.sess.lints.borrow_mut().remove(&id) {
928 929
            None => {}
            Some(lints) => {
930
                for (lint_id, span, msg) in lints {
931
                    self.span_lint(lint_id.lint, span, &msg[..])
932 933 934 935 936 937
                }
            }
        }
    }
}

938
// This lint pass is defined here because it touches parts of the `LateContext`
939 940 941 942
// that we don't want to expose. It records the lint level at certain AST
// nodes, so that the variant size difference check in trans can call
// `raw_emit_lint`.

943
pub struct GatherNodeLevels;
944 945 946 947 948 949

impl LintPass for GatherNodeLevels {
    fn get_lints(&self) -> LintArray {
        lint_array!()
    }

950
    fn check_item(&mut self, cx: &LateContext, it: &hir::Item) {
951
        match it.node {
M
Manish Goregaokar 已提交
952
            hir::ItemEnum(..) => {
A
Aaron Turon 已提交
953
                let lint_id = LintId::of(builtin::VARIANT_SIZE_DIFFERENCES);
954 955 956
                let lvlsrc = cx.lints.get_level_source(lint_id);
                match lvlsrc {
                    (lvl, _) if lvl != Allow => {
957 958 959 960 961 962 963 964 965 966 967 968 969 970 971
                        cx.node_levels.borrow_mut()
                            .insert((it.id, lint_id), lvlsrc);
                    },
                    _ => { }
                }
            },
            _ => { }
        }
    }
}

/// Perform lint checking on a crate.
///
/// Consumes the `lint_store` field of the `Session`.
pub fn check_crate(tcx: &ty::ctxt,
M
Manish Goregaokar 已提交
972
                   krate: &hir::Crate,
973
                   exported_items: &ExportedItems) {
B
Brian Anderson 已提交
974

975
    let mut cx = LateContext::new(tcx, krate, exported_items);
976 977

    // Visit the whole crate.
978
    cx.with_lint_attrs(&krate.attrs, |cx| {
979 980 981
        cx.visit_id(ast::CRATE_NODE_ID);
        cx.visit_ids(|v| {
            v.visited_outermost = true;
982
            hir_visit::walk_crate(v, krate);
983 984 985 986
        });

        // since the root module isn't visited as an item (because it isn't an
        // item), warn for it here.
987
        run_lints!(cx, check_crate, krate);
988

989
        hir_visit::walk_crate(cx, krate);
990 991 992 993
    });

    // If we missed any lints added to the session, then there's a bug somewhere
    // in the iteration code.
994
    for (id, v) in tcx.sess.lints.borrow().iter() {
995
        for &(lint, span, ref msg) in v {
996
            tcx.sess.span_bug(span,
997 998
                              &format!("unprocessed lint {} at {}: {}",
                                       lint.as_str(), tcx.map.node_to_string(*id), *msg))
999 1000 1001
        }
    }

1002
    *tcx.node_lint_levels.borrow_mut() = cx.node_levels.into_inner();
1003
}
1004 1005 1006 1007 1008 1009 1010 1011 1012 1013 1014 1015 1016 1017 1018 1019 1020 1021 1022 1023 1024 1025 1026 1027 1028 1029 1030 1031 1032 1033 1034 1035

pub fn check_ast_crate(sess: &Session, krate: &ast::Crate) {
    let mut cx = EarlyContext::new(sess, krate);

    // Visit the whole crate.
    cx.with_ast_lint_attrs(&krate.attrs, |cx| {
        cx.visit_id(ast::CRATE_NODE_ID);
        cx.visit_ids(|v| {
            v.visited_outermost = true;
            ast_visit::walk_crate(v, krate);
        });

        // since the root module isn't visited as an item (because it isn't an
        // item), warn for it here.
        run_lints!(cx, check_ast_crate, krate);

        ast_visit::walk_crate(cx, krate);
    });

    // Put the lint store back in the session.
    mem::replace(&mut *sess.lint_store.borrow_mut(), cx.lints);

    // If we missed any lints added to the session, then there's a bug somewhere
    // in the iteration code.
    for (_, v) in sess.lints.borrow().iter() {
        for &(lint, span, ref msg) in v {
            sess.span_bug(span,
                          &format!("unprocessed lint {}: {}",
                                   lint.as_str(), *msg))
        }
    }
}