Fix clippy warnings

.github/workflows/ci.yml

@@ -63,3 +63,23 @@ jobs:
         with:
           command: fmt
           args: --all -- --check
+
+  clippy:
+    name: clippy
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v3
+
+      - name: Install Rust
+        uses: actions-rs/toolchain@v1
+        with:
+          toolchain: stable
+          override: true
+          profile: minimal
+          components: clippy
+
+      - name: Run Clippy
+        uses: actions-rs/cargo@v1
+        with:
+          command: clippy

src/ast.rs

@@ -49,7 +49,7 @@ mod support {
     }
 
     pub(super) fn token<N: AstNode, T: AstToken>(parent: &N) -> Option<T> {
-        children_tokens(parent).nth(0)
+        children_tokens(parent).next()
     }
 
     /// Token untyped

src/ast/tokens.rs

@@ -50,7 +50,7 @@ impl Comment {
     pub fn text(&self) -> &str {
         let text = self.syntax().text();
         // Handle both "#..." and "/*...*/" comments.
-        match text.strip_prefix("#") {
+        match text.strip_prefix('#') {
             Some(s) => s,
             None => text.strip_prefix(r#"/*"#).unwrap().strip_suffix(r#"*/"#).unwrap(),
         }

src/lib.rs

@@ -71,7 +71,7 @@ impl<T: AstNode> Parse<T> {
 
     /// Return all errors in the tree, if any
     pub fn errors(&self) -> &[ParseError] {
-        &*self.errors
+        &self.errors
     }
 
     /// Either return the first error in the tree, or if there are none return self

src/tokenizer.rs

@@ -136,7 +136,7 @@ impl Tokenizer<'_> {
                         }
                         Some('\\') => {
                             self.next().unwrap();
-                            if let None = self.next() {
+                            if self.next().is_none() {
                                 return TOKEN_ERROR;
                             }
                         }
@@ -336,8 +336,8 @@ impl Tokenizer<'_> {
             ':' => TOKEN_COLON,
             ',' => TOKEN_COMMA,
             '.' => {
-                if self.peek().map_or(false, |x| ('0'..='9').contains(&x)) {
-                    self.consume(|c| ('0'..='9').contains(&c));
+                if self.peek().map_or(false, |x| x.is_ascii_digit()) {
+                    self.consume(|c| c.is_ascii_digit());
                     self.consume_scientific()
                 } else {
                     TOKEN_DOT
@@ -436,10 +436,10 @@ impl Tokenizer<'_> {
                 TOKEN_STRING_START
             }
             '0'..='9' => {
-                self.consume(|c| ('0'..='9').contains(&c));
+                self.consume(|c| c.is_ascii_digit());
                 if self.peek() == Some('.') {
                     self.next().unwrap();
-                    self.consume(|c| ('0'..='9').contains(&c));
+                    self.consume(|c| c.is_ascii_digit());
                     self.consume_scientific()
                 } else {
                     TOKEN_INTEGER
@@ -455,7 +455,7 @@ impl Tokenizer<'_> {
             if self.peek() == Some('-') || self.peek() == Some('+') {
                 self.next().unwrap();
             }
-            if self.consume(|c| ('0'..='9').contains(&c)) == 0 {
+            if self.consume(|c| c.is_ascii_digit()) == 0 {
                 return TOKEN_ERROR;
             }
         }