diff options
| author | Mark Rousskov <mark.simulacrum@gmail.com> | 2019-10-25 13:41:51 -0400 |
|---|---|---|
| committer | Mark Rousskov <mark.simulacrum@gmail.com> | 2019-11-03 21:52:42 -0500 |
| commit | c68df7c503676a1f63c0fcc2a7f10597fb93c375 (patch) | |
| tree | 170b18bab174d5a853861ef72aff81183965df03 /src/librustc/lint | |
| parent | c0fdddcb6001b6bd0cd7a6397b9e01e019e553ff (diff) | |
| download | rust-c68df7c503676a1f63c0fcc2a7f10597fb93c375.tar.gz rust-c68df7c503676a1f63c0fcc2a7f10597fb93c375.zip | |
Delete lint buffer from Session
Diffstat (limited to 'src/librustc/lint')
| -rw-r--r-- | src/librustc/lint/context.rs | 26 | ||||
| -rw-r--r-- | src/librustc/lint/levels.rs | 16 | ||||
| -rw-r--r-- | src/librustc/lint/mod.rs | 2 |
3 files changed, 26 insertions, 18 deletions
diff --git a/src/librustc/lint/context.rs b/src/librustc/lint/context.rs index 1cb53d754dc..eef1cee8db3 100644 --- a/src/librustc/lint/context.rs +++ b/src/librustc/lint/context.rs @@ -34,7 +34,6 @@ use crate::util::common::time; use errors::DiagnosticBuilder; use std::slice; -use std::default::Default as StdDefault; use rustc_data_structures::sync::{self, ParallelIterator, join, par_iter}; use rustc_serialize::{Decoder, Decodable, Encoder, Encodable}; use syntax::ast; @@ -584,12 +583,13 @@ impl<'a> EarlyContext<'a> { lint_store: &'a LintStore, krate: &'a ast::Crate, buffered: LintBuffer, + warn_about_weird_lints: bool, ) -> EarlyContext<'a> { EarlyContext { sess, krate, lint_store, - builder: LintLevelSets::builder(sess, lint_store), + builder: LintLevelSets::builder(sess, warn_about_weird_lints, lint_store), buffered, } } @@ -1490,9 +1490,10 @@ fn early_lint_crate<T: EarlyLintPass>( krate: &ast::Crate, pass: T, buffered: LintBuffer, + warn_about_weird_lints: bool, ) -> LintBuffer { let mut cx = EarlyContextAndPass { - context: EarlyContext::new(sess, lint_store, krate, buffered), + context: EarlyContext::new(sess, lint_store, krate, buffered, warn_about_weird_lints), pass, }; @@ -1514,22 +1515,19 @@ pub fn check_ast_crate<T: EarlyLintPass>( lint_store: &LintStore, krate: &ast::Crate, pre_expansion: bool, + lint_buffer: Option<LintBuffer>, builtin_lints: T, ) { - let (mut passes, mut buffered): (Vec<_>, _) = if pre_expansion { - ( - lint_store.pre_expansion_passes.iter().map(|p| (p)()).collect(), - LintBuffer::default(), - ) + let mut passes: Vec<_> = if pre_expansion { + lint_store.pre_expansion_passes.iter().map(|p| (p)()).collect() } else { - ( - lint_store.early_passes.iter().map(|p| (p)()).collect(), - sess.buffered_lints.borrow_mut().take().unwrap(), - ) + lint_store.early_passes.iter().map(|p| (p)()).collect() }; + let mut buffered = lint_buffer.unwrap_or_default(); if !sess.opts.debugging_opts.no_interleave_lints { - buffered = early_lint_crate(sess, lint_store, krate, builtin_lints, buffered); + buffered = early_lint_crate(sess, lint_store, krate, builtin_lints, buffered, + pre_expansion); if !passes.is_empty() { buffered = early_lint_crate( @@ -1538,6 +1536,7 @@ pub fn check_ast_crate<T: EarlyLintPass>( krate, EarlyLintPassObjects { lints: &mut passes[..] }, buffered, + pre_expansion, ); } } else { @@ -1549,6 +1548,7 @@ pub fn check_ast_crate<T: EarlyLintPass>( krate, EarlyLintPassObjects { lints: slice::from_mut(pass) }, buffered, + pre_expansion, ) }); } diff --git a/src/librustc/lint/levels.rs b/src/librustc/lint/levels.rs index 4c60492e470..e470dbdf323 100644 --- a/src/librustc/lint/levels.rs +++ b/src/librustc/lint/levels.rs @@ -44,8 +44,12 @@ impl LintLevelSets { return me } - pub fn builder<'a>(sess: &'a Session, store: &LintStore) -> LintLevelsBuilder<'a> { - LintLevelsBuilder::new(sess, LintLevelSets::new(sess, store)) + pub fn builder<'a>( + sess: &'a Session, + warn_about_weird_lints: bool, + store: &LintStore, + ) -> LintLevelsBuilder<'a> { + LintLevelsBuilder::new(sess, warn_about_weird_lints, LintLevelSets::new(sess, store)) } fn process_command_line(&mut self, sess: &Session, store: &LintStore) { @@ -160,14 +164,18 @@ pub struct BuilderPush { } impl<'a> LintLevelsBuilder<'a> { - pub fn new(sess: &'a Session, sets: LintLevelSets) -> LintLevelsBuilder<'a> { + pub fn new( + sess: &'a Session, + warn_about_weird_lints: bool, + sets: LintLevelSets, + ) -> LintLevelsBuilder<'a> { assert_eq!(sets.list.len(), 1); LintLevelsBuilder { sess, sets, cur: 0, id_to_set: Default::default(), - warn_about_weird_lints: sess.buffered_lints.borrow().is_some(), + warn_about_weird_lints, } } diff --git a/src/librustc/lint/mod.rs b/src/librustc/lint/mod.rs index 42f33740b23..11d0d0d90fa 100644 --- a/src/librustc/lint/mod.rs +++ b/src/librustc/lint/mod.rs @@ -795,7 +795,7 @@ fn lint_levels(tcx: TyCtxt<'_>, cnum: CrateNum) -> &LintLevelMap { assert_eq!(cnum, LOCAL_CRATE); let store = &tcx.lint_store; let mut builder = LintLevelMapBuilder { - levels: LintLevelSets::builder(tcx.sess, &store), + levels: LintLevelSets::builder(tcx.sess, false, &store), tcx: tcx, store: store, }; |
