term2: Swallow unsupported errors out of term::Error

In order that we don't explode horribly on `TERM=dumb` or similar,
we need to ensure that we cleanly swallow errors from the term
crate which signify a less-than-ideal terminal.

Signed-off-by: Daniel Silverstone <dsilvers@digital-scurf.org>

Author: kinnison

src/cli/term2.rs

@@ -246,14 +246,24 @@ impl<T: Instantiable + Isatty + io::Write> io::Write for Terminal<T> {
     }
 }
 
+macro_rules! swallow_unsupported {
+    ( $call:expr ) => {{
+        use term::Error::*;
+        match $call {
+            Ok(()) | Err(ColorOutOfRange) | Err(NotSupported) => Ok(()),
+            Err(e) => Err(e),
+        }
+    }};
+}
+
 impl<T: Instantiable + Isatty + io::Write> Terminal<T> {
     pub fn fg(&mut self, color: color::Color) -> Result<(), term::Error> {
         if !T::isatty() {
             return Ok(());
         }
 
         if let Some(ref mut t) = self.0 {
-            t.fg(color)
+            swallow_unsupported!(t.fg(color))
         } else {
             Ok(())
         }
@@ -268,8 +278,8 @@ impl<T: Instantiable + Isatty + io::Write> Terminal<T> {
             if let Err(e) = t.attr(attr) {
                 // If `attr` is not supported, try to emulate it
                 match attr {
-                    Attr::Bold => t.fg(color::BRIGHT_WHITE),
-                    _ => Err(e),
+                    Attr::Bold => swallow_unsupported!(t.fg(color::BRIGHT_WHITE)),
+                    _ => swallow_unsupported!(Err(e)),
                 }
             } else {
                 Ok(())
@@ -285,7 +295,7 @@ impl<T: Instantiable + Isatty + io::Write> Terminal<T> {
         }
 
         if let Some(ref mut t) = self.0 {
-            t.reset()
+            swallow_unsupported!(t.reset())
         } else {
             Ok(())
         }