fix: clippy

lykiadb-lang/src/parser/mod.rs

@@ -628,11 +628,11 @@ impl<'a> Parser<'a> {
                     }));
                 }
 
-                return Ok(self.expect_get_path(expr, sym!(DoubleColon))?);
+                return self.expect_get_path(expr, sym!(DoubleColon));
             }
         }
 
-        Ok(self.expect_get_path(expr, sym!(Dot))?)
+        self.expect_get_path(expr, sym!(Dot))
     }
 
     fn finish_call(&mut self, callee: Box<Expr>) -> ParseResult<Box<Expr>> {
@@ -777,7 +777,7 @@ impl<'a> Parser<'a> {
     }
 
     fn expected(&mut self, expected_tok_type: &TokenType) -> ParseResult<&Token> {
-        if self.cmp_tok(&expected_tok_type) {
+        if self.cmp_tok(expected_tok_type) {
             return Ok(self.advance());
         };
         let prev_token = self.peek_bw(1);

lykiadb-server/src/engine/interpreter.rs

@@ -663,6 +663,12 @@ pub struct Output {
     out: Vec<RV>,
 }
 
+impl Default for Output {
+    fn default() -> Self {
+        Self::new()
+    }
+}
+
 impl Output {
     pub fn new() -> Output {
         Output { out: Vec::new() }

lykiadb-server/src/engine/mod.rs

@@ -60,6 +60,12 @@ pub mod test_helpers {
         runtime: Runtime,
     }
 
+    impl Default for RuntimeTester {
+        fn default() -> Self {
+            Self::new()
+        }
+    }
+
     impl RuntimeTester {
         pub fn new() -> RuntimeTester {
             let out = alloc_shared(Output::new());
@@ -127,7 +133,7 @@ pub mod test_helpers {
                     assert_eq!(errors.iter().map(|x| x.to_string()).collect::<Vec<String>>().join("\n"), part[3..].trim());
                 }
 
-                else if part.starts_with(">") {
+                else if part.starts_with('>') {
                     let result = self.runtime.interpret(part[1..].trim());
 
                     if let Err(err) = result {
@@ -139,7 +145,7 @@ pub mod test_helpers {
                     self.out.write().unwrap().expect(vec![RV::Str(Arc::new(part.to_string()))]);
                 }
                 else {
-                    self.out.write().unwrap().expect_str(part.to_string().split("\n").map(|x| x.to_string()).collect());
+                    self.out.write().unwrap().expect_str(part.to_string().split('\n').map(|x| x.to_string()).collect());
                 }     
             }
         }

lykiadb-server/src/plan/mod.rs

@@ -1,4 +1,4 @@
-use std::{collections::HashMap, fmt::Display};
+use std::fmt::Display;
 
 use lykiadb_lang::{
     ast::{

lykiadb-server/src/plan/planner.rs

@@ -67,7 +67,7 @@ impl<'a> Planner<'a> {
         if core.projection.as_slice() != [SqlProjection::All { collection: None }] {
             for projection in &core.projection {
                 if let SqlProjection::Expr { expr, .. } = projection {
-                    self.build_expr(&expr, false, true)?;
+                    self.build_expr(expr, false, true)?;
                 }
             }
             node = Node::Projection {
@@ -110,7 +110,7 @@ impl<'a> Planner<'a> {
                 None
             }
             Expr::FieldPath { head, tail, .. } => {
-                println!("FieldPath {} {}", head.to_string(), tail.iter().map(|x| x.to_string()).collect::<String>());
+                println!("FieldPath {} {}", head, tail.iter().map(|x| x.to_string()).collect::<String>());
                 None
             }
             Expr::Call { callee, args, .. } => {