refactor line checks

casey · Oct 5, 2022 · d27b720 · d27b720
1 parent e445cfb
commit d27b720
Showing 1 changed file with 25 additions and 24 deletions.
diff --git a/src/line.rs b/src/line.rs
@@ -1,5 +1,8 @@
 use super::*;
 
+// TODO
+// - don't evaluate comments
+
 /// A single line in a recipe body, consisting of any number of `Fragment`s.
 #[derive(Debug, Clone, PartialEq, Serialize)]
 #[serde(transparent)]
@@ -13,41 +16,39 @@ impl<'src> Line<'src> {
   }
 
   pub(crate) fn is_comment(&self) -> bool {
-    match self.fragments.first() {
-      Some(Fragment::Text { token }) => token.lexeme().starts_with('#'),
-      _ => false,
-    }
+    matches!(
+      self.fragments.first(),
+      Some(Fragment::Text { token }) if token.lexeme().starts_with('#'),
+    )
   }
 
   pub(crate) fn is_continuation(&self) -> bool {
-    match self.fragments.last() {
-      Some(Fragment::Text { token }) => token.lexeme().ends_with('\\'),
-      _ => false,
-    }
+    matches!(
+      self.fragments.last(),
+      Some(Fragment::Text { token }) if token.lexeme().ends_with('\\'),
+    )
   }
 
   pub(crate) fn is_shebang(&self) -> bool {
-    match self.fragments.first() {
-      Some(Fragment::Text { token }) => token.lexeme().starts_with("#!"),
-      _ => false,
-    }
+    matches!(
+      self.fragments.first(),
+      Some(Fragment::Text { token }) if token.lexeme().starts_with("#!"),
+    )
   }
 
   pub(crate) fn is_quiet(&self) -> bool {
-    match self.fragments.first() {
-      Some(Fragment::Text { token }) => {
-        token.lexeme().starts_with('@') || token.lexeme().starts_with("-@")
-      }
-      _ => false,
-    }
+    matches!(
+      self.fragments.first(),
+      Some(Fragment::Text { token })
+        if token.lexeme().starts_with('@') || token.lexeme().starts_with("-@"),
+    )
   }
 
   pub(crate) fn is_infallible(&self) -> bool {
-    match self.fragments.first() {
-      Some(Fragment::Text { token }) => {
-        token.lexeme().starts_with('-') || token.lexeme().starts_with("@-")
-      }
-      _ => false,
-    }
+    matches!(
+      self.fragments.first(),
+      Some(Fragment::Text { token })
+        if token.lexeme().starts_with('-') || token.lexeme().starts_with("@-"),
+    )
   }
 }