refactor: minor updates on pr feedback

Author: calebcartwright

src/formatting.rs

@@ -9,7 +9,7 @@ use std::time::{Duration, Instant};
 
 use syntax::ast;
 use syntax::errors::emitter::{ColorConfig, Emitter, EmitterWriter};
-use syntax::errors::{DiagnosticBuilder, Handler, HandlerFlags};
+use syntax::errors::{DiagnosticBuilder, Handler};
 use syntax::parse::{self, ParseSess};
 use syntax::source_map::{FilePathMapping, SourceMap, Span, DUMMY_SP};
 
@@ -81,7 +81,7 @@ fn format_project<T: FormatHandler>(
     let mut parse_session = make_parse_sess(
         source_map.clone(),
         config,
-        ignore_path_set.clone(),
+        Rc::clone(&ignore_path_set),
         can_reset.clone(),
     );
     let mut report = FormatReport::new();
@@ -705,9 +705,8 @@ struct SilentOnIgnoredFilesEmitter {
     ignore_path_set: Rc<IgnorePathSet>,
     source_map: Rc<SourceMap>,
     emitter: EmitterWriter,
-    can_reset: bool,
     has_non_ignorable_parser_errors: bool,
-    parser_error_resetter: Rc<RefCell<bool>>,
+    can_reset: Rc<RefCell<bool>>,
 }
 
 impl Emitter for SilentOnIgnoredFilesEmitter {
@@ -720,9 +719,8 @@ impl Emitter for SilentOnIgnoredFilesEmitter {
                         .ignore_path_set
                         .is_match(&FileName::Real(path.to_path_buf()))
                     {
-                        if !self.has_non_ignorable_parser_errors && !self.can_reset {
-                            self.can_reset = true;
-                            *self.parser_error_resetter.borrow_mut() = true;
+                        if !self.has_non_ignorable_parser_errors {
+                            *self.can_reset.borrow_mut() = true;
                         }
                         return;
                     }
@@ -732,10 +730,7 @@ impl Emitter for SilentOnIgnoredFilesEmitter {
         }
 
         self.has_non_ignorable_parser_errors = true;
-        if self.can_reset {
-            *self.parser_error_resetter.borrow_mut() = false;
-        }
-        self.can_reset = false;
+        *self.can_reset.borrow_mut() = false;
         self.emitter.emit_diagnostic(db);
     }
 }
@@ -755,7 +750,7 @@ fn make_parse_sess(
     source_map: Rc<SourceMap>,
     config: &Config,
     ignore_path_set: Rc<IgnorePathSet>,
-    parser_error_resetter: Rc<RefCell<bool>>,
+    can_reset: Rc<RefCell<bool>>,
 ) -> ParseSess {
     let tty_handler = if config.hide_parse_errors() {
         let silent_emitter = silent_emitter();
@@ -772,20 +767,12 @@ fn make_parse_sess(
             EmitterWriter::stderr(color_cfg, Some(source_map.clone()), false, false, None);
         let emitter = Box::new(SilentOnIgnoredFilesEmitter {
             has_non_ignorable_parser_errors: false,
-            can_reset: false,
             ignore_path_set: ignore_path_set,
-            source_map: source_map.clone(),
+            source_map: Rc::clone(&source_map),
             emitter: emitter_writer,
-            parser_error_resetter,
+            can_reset,
         });
-        Handler::with_emitter_and_flags(
-            emitter,
-            HandlerFlags {
-                can_emit_warnings: false,
-                treat_err_as_bug: None,
-                ..Default::default()
-            },
-        )
+        Handler::with_emitter(true, None, emitter)
     };
 
     ParseSess::with_span_handler(tty_handler, source_map)