diff --git a/Cargo.lock b/Cargo.lock
index f48cb10..fe0b30d 100644
--- a/Cargo.lock
+++ b/Cargo.lock
@@ -648,7 +648,6 @@ checksum = "1ad1d488a557b235fc46dae55512ffbfc429d2482b08b4d9435ab07384ca8aec"
 name = "simplexpr"
 version = "0.1.0"
 dependencies = [
- "codespan-reporting",
  "itertools",
  "lalrpop",
  "lalrpop-util",
diff --git a/src/lib.rs b/src/lib.rs
index ffcee55..30aa4fc 100644
--- a/src/lib.rs
+++ b/src/lib.rs
@@ -19,12 +19,15 @@ use itertools::Itertools;
 
 use lalrpop_util::lalrpop_mod;
 
-lalrpop_mod!(pub parser);
+lalrpop_mod!(
+    #[allow(clippy::all)]
+    pub parser
+);
 
 pub fn parse_string(file_id: usize, s: &str) -> AstResult<Ast> {
     let lexer = lexer::Lexer::new(file_id, s);
     let parser = parser::AstParser::new();
-    Ok(parser.parse(file_id, lexer).map_err(|e| AstError::from_parse_error(file_id, e))?)
+    parser.parse(file_id, lexer).map_err(|e| AstError::from_parse_error(file_id, e))
 }
 
 macro_rules! test_parser {
diff --git a/src/value/coords.rs b/src/value/coords.rs
index da15e70..11728d6 100644
--- a/src/value/coords.rs
+++ b/src/value/coords.rs
@@ -64,7 +64,7 @@ impl FromStr for Coords {
     fn from_str(s: &str) -> Result<Self, Self::Err> {
         let (x, y) = s
             .split_once(|x: char| x.to_ascii_lowercase() == 'x' || x.to_ascii_lowercase() == '*')
-            .ok_or_else(|| Error::MalformedCoords)?;
+            .ok_or(Error::MalformedCoords)?;
         Coords::from_strs(x, y)
     }
 }