---

yaml
---
r: 277947
b: refs/heads/auto
c: 79f61a4
h: refs/heads/master
i:
  277945: 124f280
  277943: 26a5021

Author: Jonathan Turner

[refs]

@@ -8,7 +8,7 @@ refs/tags/release-0.3: b5f0d0f648d9a6153664837026ba1be43d3e2503
 refs/tags/release-0.3.1: 495bae036dfe5ec6ceafd3312b4dca48741e845b
 refs/tags/release-0.4: e828ea2080499553b97dfe33b3f4d472b4562ad7
 refs/tags/release-0.5: 7e3bcfbf21278251ee936ad53e92e9b719702d73
-refs/heads/auto: 84cb56f8ee11ba89914462e478f06e9c1e8e7971
+refs/heads/auto: 79f61a45328e534e52cf705452b33ae4b8ae474d
 refs/tags/release-0.6: b4ebcfa1812664df5e142f0134a5faea3918544c
 refs/tags/0.1: b19db808c2793fe2976759b85a355c3ad8c8b336
 refs/tags/0.2: 1754d02027f2924bed83b0160ee340c7f41d5ea1

branches/auto/src/librustc_trans/back/write.rs

@@ -106,7 +106,8 @@ impl CoreEmitter for SharedEmitter {
                     msg: &str,
                     code: Option<&str>,
                     lvl: Level,
-                    _is_header: bool) {
+                    _is_header: bool,
+                    _show_snippet: bool) {
         self.buffer.lock().unwrap().push(Diagnostic {
             msg: msg.to_string(),
             code: code.map(|s| s.to_string()),

branches/auto/src/libsyntax/errors/emitter.rs

@@ -40,7 +40,8 @@ pub trait CoreEmitter {
                     msg: &str,
                     code: Option<&str>,
                     lvl: Level,
-                    is_header: bool);
+                    is_header: bool,
+                    show_snippet: bool);
 }
 
 impl<T: CoreEmitter> Emitter for T {
@@ -53,25 +54,47 @@ impl<T: CoreEmitter> Emitter for T {
                           msg,
                           code,
                           lvl,
+                          true,
                           true);
     }
 
     fn emit_struct(&mut self, db: &DiagnosticBuilder) {
+        let old_school = match ::std::env::var("RUST_NEW_ERROR_FORMAT") {
+            Ok(_) => false,
+            Err(_) => true,
+        };
+        let db_span = FullSpan(db.span.clone());
         self.emit_message(&FullSpan(db.span.clone()),
                           &db.message,
                           db.code.as_ref().map(|s| &**s),
                           db.level,
+                          true,
                           true);
         for child in &db.children {
             let render_span = child.render_span
                                    .clone()
                                    .unwrap_or_else(
                                        || FullSpan(child.span.clone()));
-            self.emit_message(&render_span,
-                              &child.message,
-                              None,
-                              child.level,
-                              false);
+
+            if !old_school {
+                self.emit_message(&render_span,
+                                    &child.message,
+                                    None,
+                                    child.level,
+                                    false,
+                                    true);
+            } else {
+                let (render_span, show_snippet) = match render_span.span().primary_span() {
+                    None => (db_span.clone(), false),
+                    _ => (render_span, true)
+                };
+                self.emit_message(&render_span,
+                                    &child.message,
+                                    None,
+                                    child.level,
+                                    false,
+                                    show_snippet);
+            }
         }
     }
 }
@@ -108,7 +131,8 @@ impl CoreEmitter for BasicEmitter {
                     msg: &str,
                     code: Option<&str>,
                     lvl: Level,
-                    _is_header: bool) {
+                    _is_header: bool,
+                    _show_snippet: bool) {
         // we ignore the span as we have no access to a codemap at this point
         if let Err(e) = print_diagnostic(&mut self.dst, "", lvl, msg, code) {
             panic!("failed to print diagnostics: {:?}", e);
@@ -146,8 +170,9 @@ impl CoreEmitter for EmitterWriter {
                     msg: &str,
                     code: Option<&str>,
                     lvl: Level,
-                    is_header: bool) {
-        match self.emit_message_(rsp, msg, code, lvl, is_header) {
+                    is_header: bool,
+                    show_snippet: bool) {
+        match self.emit_message_(rsp, msg, code, lvl, is_header, show_snippet) {
             Ok(()) => { }
             Err(e) => panic!("failed to emit error: {}", e)
         }
@@ -213,7 +238,8 @@ impl EmitterWriter {
                      msg: &str,
                      code: Option<&str>,
                      lvl: Level,
-                     is_header: bool)
+                     is_header: bool,
+                     show_snippet: bool)
                      -> io::Result<()> {
         if is_header {
             if self.first {
@@ -243,10 +269,24 @@ impl EmitterWriter {
                 }
             }
             _ => {
-                print_diagnostic(&mut self.dst, "", lvl, msg, code)?
+                if self.old_school {
+                    let loc = match rsp.span().primary_span() {
+                        Some(COMMAND_LINE_SP) | Some(DUMMY_SP) => "".to_string(),
+                        Some(ps) => self.cm.span_to_string(ps),
+                        None => "".to_string()
+                    };
+                    print_diagnostic(&mut self.dst, &loc, lvl, msg, code)?
+                }
+                else {
+                    print_diagnostic(&mut self.dst, "", lvl, msg, code)?
+                }
             }
         }
 
+        if !show_snippet {
+            return Ok(());
+        }
+
         // Watch out for various nasty special spans; don't try to
         // print any filename or anything for those.
         match rsp.span().primary_span() {
@@ -333,8 +373,10 @@ impl EmitterWriter {
                                                 msp.primary_span());
         if self.old_school {
             let mut output_vec = vec![];
+
             for span_label in msp.span_labels() {
                 let mut snippet_data = snippet_data.clone();
+
                 snippet_data.push(span_label.span,
                                   span_label.is_primary,
                                   span_label.label);
@@ -412,7 +454,16 @@ fn print_diagnostic(dst: &mut Destination,
                     code: Option<&str>)
                     -> io::Result<()> {
     if !topic.is_empty() {
-        write!(dst, "{}: ", topic)?;
+        let old_school = match ::std::env::var("RUST_NEW_ERROR_FORMAT") {
+            Ok(_) => false,
+            Err(_) => true,
+        };
+        if !old_school {
+            write!(dst, "{}: ", topic)?;
+        }
+        else {
+            write!(dst, "{} ", topic)?;
+        }
         dst.reset_attrs()?;
     }
     dst.start_attr(term::Attr::Bold)?;