Drop unnecessary if branch

Author: darinmorrison

examples/examples/sloggish/sloggish_subscriber.rs

@@ -228,9 +228,7 @@ impl Subscriber for SloggishSubscriber {
         let spans = self.spans.lock().unwrap();
         let data = spans.get(span_id);
         let parent = data.and_then(|span| span.parent.as_ref());
-        if stack.iter().any(|id| id == span_id) {
-            // We are already in this span, do nothing.
-        } else {
+        if !stack.iter().any(|id| id == span_id) {
             let indent = if let Some(idx) = stack
                 .iter()
                 .position(|id| parent.map(|p| id == p).unwrap_or(false))
@@ -248,8 +246,7 @@ impl Subscriber for SloggishSubscriber {
                 self.print_kvs(&mut stderr, data.kvs.iter().map(|(k, v)| (k, v)), "")
                     .unwrap();
             }
-            #[allow(clippy::write_with_newline)]
-            write!(&mut stderr, "\n").unwrap();
+            writeln!(&mut stderr).unwrap();
         }
     }
 
@@ -270,8 +267,7 @@ impl Subscriber for SloggishSubscriber {
             comma: false,
         };
         event.record(&mut visitor);
-        #[allow(clippy::write_with_newline)]
-        write!(&mut visitor.stderr, "\n").unwrap();
+        writeln!(&mut visitor.stderr).unwrap();
     }
 
     #[inline]