Don't assume color=always when explicitally specified

Fixes #31546
rust-lang · Feb 11, 2016 · 03ef55b · 03ef55b
1 parent 5d771cd
commit 03ef55b
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/src/librustc/session/config.rs b/src/librustc/session/config.rs
@@ -934,17 +934,17 @@ pub fn build_session_options(matches: &getopts::Matches) -> Options {
             Some("human")   => ErrorOutputType::HumanReadable(color),
             Some("json") => ErrorOutputType::Json,
 
-            None => ErrorOutputType::default(),
+            None => ErrorOutputType::HumanReadable(color),
 
             Some(arg) => {
-                early_error(ErrorOutputType::default(), &format!("argument for --error-format must \
-                                                                  be human or json (instead was \
-                                                                  `{}`)",
-                                                                 arg))
+                early_error(ErrorOutputType::HumanReadable(color),
+                            &format!("argument for --error-format must be human or json (instead \
+                                      was `{}`)",
+                                     arg))
             }
         }
     } else {
-        ErrorOutputType::default()
+        ErrorOutputType::HumanReadable(color)
     };
 
     let unparsed_crate_types = matches.opt_strs("crate-type");