From 5f9e15165578d81465e308486a17f4face143c6a Mon Sep 17 00:00:00 2001
From: Jonathan Turner <jturner@mozilla.com>
Date: Thu, 7 Apr 2016 13:31:33 -0700
Subject: [PATCH] Tidy fixes

---
 src/libsyntax/errors/emitter.rs     | 27 ++++++++++++++++-----------
 src/libsyntax/errors/snippet/mod.rs |  5 +++--
 src/libsyntax/ext/build.rs          |  2 +-
 3 files changed, 20 insertions(+), 14 deletions(-)

diff --git a/src/libsyntax/errors/emitter.rs b/src/libsyntax/errors/emitter.rs
index abf6a91cf2662..04947da0a925c 100644
--- a/src/libsyntax/errors/emitter.rs
+++ b/src/libsyntax/errors/emitter.rs
@@ -107,13 +107,17 @@ impl Emitter for EmitterWriter {
     }
 
     fn emit_struct(&mut self, db: &DiagnosticBuilder) {
-        self.emit_multispan(db.span.as_ref(), &db.message, db.code.as_ref().map(|s| &**s), db.level, true);
+        self.emit_multispan(db.span.as_ref(), &db.message,
+            db.code.as_ref().map(|s| &**s), db.level, true);
+
         for child in &db.children {
             match child.render_span {
                 Some(ref sp) =>
-                    self.emit_renderspan(sp, &child.message, child.level),
+                    self.emit_renderspan(sp, &child.message,
+                        child.level),
                 None =>
-                    self.emit_multispan(child.span.as_ref(), &child.message, None, child.level, false),
+                    self.emit_multispan(child.span.as_ref(),
+                        &child.message, None, child.level, false),
             }
         }
     }
@@ -142,7 +146,8 @@ impl EmitterWriter {
             let dst = Destination::from_stderr();
             EmitterWriter { dst: dst, registry: registry, cm: code_map, first: true }
         } else {
-            EmitterWriter { dst: Raw(Box::new(io::stderr())), registry: registry, cm: code_map, first: true }
+            EmitterWriter { dst: Raw(Box::new(io::stderr())),
+                registry: registry, cm: code_map, first: true }
         }
     }
 
@@ -179,8 +184,8 @@ impl EmitterWriter {
         }
     }
 
-    fn emit_multispan(&mut self, span: Option<&MultiSpan>, msg: &str, code: Option<&str>, lvl: Level,
-        is_header: bool) {
+    fn emit_multispan(&mut self, span: Option<&MultiSpan>, msg: &str,
+        code: Option<&str>, lvl: Level, is_header: bool) {
 
         if is_header {
             self.emit_header(span);
@@ -209,13 +214,13 @@ impl EmitterWriter {
              rsp: &RenderSpan,
              msg: &str,
              code: Option<&str>,
-             lvl: Level, 
+             lvl: Level,
              is_header: bool)
              -> io::Result<()> {
         let msp = rsp.span();
         let bounds = msp.primary_span();
 
-        let ss = 
+        let ss =
             if is_header {
                 String::new()
             }
@@ -225,7 +230,7 @@ impl EmitterWriter {
 
         match code {
             Some(code) if self.registry.as_ref()
-                          .and_then(|registry| registry.find_description(code)).is_some() => 
+                          .and_then(|registry| registry.find_description(code)).is_some() =>
             {
                 let code_with_explain = String::from("--explain ") + code;
                 print_diagnostic(&mut self.dst, &ss, lvl, msg, Some(&code_with_explain))?
@@ -363,10 +368,10 @@ fn line_num_max_digits(line: &codemap::LineInfo) -> usize {
 fn print_diagnostic(dst: &mut Destination,
                     topic: &str,
                     lvl: Level,
-                    msg: &str, 
+                    msg: &str,
                     code: Option<&str>)
                     -> io::Result<()> {
-                        
+
     if !topic.is_empty() {
         dst.start_attr(term::Attr::ForegroundColor(lvl.color()))?;
         write!(dst, "{}: ", topic)?;
diff --git a/src/libsyntax/errors/snippet/mod.rs b/src/libsyntax/errors/snippet/mod.rs
index 67f6ea09dc365..75c3ac3aa3c7e 100644
--- a/src/libsyntax/errors/snippet/mod.rs
+++ b/src/libsyntax/errors/snippet/mod.rs
@@ -260,7 +260,8 @@ impl SnippetData {
             line.text.insert(0, prefix);
             match line.kind {
                 RenderedLineKind::Elision => {}
-                _ => line.text.insert(1, StyledString {text: String::from("|> "), style: FileNameLine})
+                _ => line.text.insert(1, StyledString {text: String::from("|> "),
+                    style: FileNameLine})
             }
         }
     }
@@ -582,7 +583,7 @@ impl RenderedLineKind {
                     text: format!("{}", line_index + 1),
                     style: FileNameLine,
                 },
-            Elision => 
+            Elision =>
                 StyledString {
                     text: String::from("..."),
                     style: FileNameLine,
diff --git a/src/libsyntax/ext/build.rs b/src/libsyntax/ext/build.rs
index b673fefedf281..8ebf4863af5fa 100644
--- a/src/libsyntax/ext/build.rs
+++ b/src/libsyntax/ext/build.rs
@@ -919,7 +919,7 @@ impl<'a> AstBuilder for ExtCtxt<'a> {
             ids.iter().map(|id| self.arg(span, *id, self.ty_infer(span))).collect(),
             self.ty_infer(span));
 
-        // TODO -- We are using `span` as the span of the `|...|`
+        // FIXME -- We are using `span` as the span of the `|...|`
         // part of the lambda, but it probably (maybe?) corresponds to
         // the entire lambda body. Probably we should extend the API
         // here, but that's not entirely clear.