From 762639e7fd968905d97599f9396568554c304f61 Mon Sep 17 00:00:00 2001 From: topecongiro Date: Sat, 10 Jun 2017 14:16:12 +0900 Subject: [PATCH] Reset the error count of parser from the previous file --- src/lib.rs | 21 +++++++++++++++------ 1 file changed, 15 insertions(+), 6 deletions(-) diff --git a/src/lib.rs b/src/lib.rs index b7edc183618..6488c83e983 100644 --- a/src/lib.rs +++ b/src/lib.rs @@ -427,9 +427,10 @@ fn fmt(&self, fmt: &mut fmt::Formatter) -> Result<(), fmt::Error> { // Formatting which depends on the AST. fn format_ast(krate: &ast::Crate, - parse_session: &ParseSess, + mut parse_session: &mut ParseSess, main_file: &Path, config: &Config, + codemap: &Rc, mut after_file: F) -> Result<(FileMap, bool), io::Error> where F: FnMut(&str, &mut StringBuffer) -> Result @@ -449,12 +450,19 @@ fn format_ast(krate: &ast::Crate, if config.verbose() { println!("Formatting {}", path); } - let mut visitor = FmtVisitor::from_codemap(parse_session, config); - visitor.format_separate_mod(module); + { + let mut visitor = FmtVisitor::from_codemap(parse_session, config); + visitor.format_separate_mod(module); - has_diff |= after_file(path, &mut visitor.buffer)?; + has_diff |= after_file(path, &mut visitor.buffer)?; - result.push((path.to_owned(), visitor.buffer)); + result.push((path.to_owned(), visitor.buffer)); + } + // Reset the error count. + if parse_session.span_diagnostic.has_errors() { + parse_session.span_diagnostic = + Handler::with_tty_emitter(ColorConfig::Auto, true, false, Some(codemap.clone())); + } } Ok((result, has_diff)) @@ -612,9 +620,10 @@ pub fn format_input(input: Input, match format_ast( &krate, - &parse_session, + &mut parse_session, &main_file, config, + &codemap, |file_name, file| { // For some reason, the codemap does not include terminating // newlines so we must add one on for each file. This is sad.