Use if-let rather than map() in parse_logging_spec.

rust-lang · Oct 14, 2015 · b38b9e1 · b38b9e1
1 parent 294ef5b
commit b38b9e1
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/liblog/directive.rs b/src/liblog/directive.rs
@@ -46,7 +46,7 @@ pub fn parse_logging_spec(spec: &str) -> (Vec<LogDirective>, Option<String>) {
                  spec);
         return (dirs, None);
     }
-    mods.map(|m| {
+    if let Some(m) = mods {
         for s in m.split(',') {
             if s.is_empty() {
                 continue
@@ -83,7 +83,7 @@ pub fn parse_logging_spec(spec: &str) -> (Vec<LogDirective>, Option<String>) {
                 level: log_level,
             });
         }
-    });
+    }
 
     (dirs, filter.map(str::to_owned))
 }