Fix clippy warnings (!38)

Author: jeparlefrancais

src/cli/utils/config.rs

@@ -70,7 +70,7 @@ impl Config {
     fn read_default_file() -> Result<Self, CliError> {
         DEFAULT_CONFIG_PATHS
             .iter()
-            .map(|path| Path::new(path))
+            .map(Path::new)
             .filter(|path| path.exists())
             .find_map(|path| Some(Self::read_file(path)))
             .unwrap_or_else(|| Ok(Self::default()))

src/nodes/arguments.rs

@@ -21,7 +21,7 @@ impl TupleArgumentsTokens {
     }
 }
 
-#[derive(Clone, Debug, PartialEq, Eq)]
+#[derive(Clone, Debug, Default, PartialEq, Eq)]
 pub struct TupleArguments {
     values: Vec<Expression>,
     tokens: Option<TupleArgumentsTokens>,
@@ -93,15 +93,6 @@ impl TupleArguments {
     }
 }
 
-impl Default for TupleArguments {
-    fn default() -> Self {
-        Self {
-            values: Vec::new(),
-            tokens: None,
-        }
-    }
-}
-
 impl From<Arguments> for TupleArguments {
     fn from(arguments: Arguments) -> Self {
         match arguments {

src/nodes/expressions/function.rs

@@ -38,7 +38,7 @@ impl FunctionExpressionTokens {
     }
 }
 
-#[derive(Clone, Debug, PartialEq, Eq)]
+#[derive(Clone, Debug, Default, PartialEq, Eq)]
 pub struct FunctionExpression {
     block: Block,
     parameters: Vec<Identifier>,
@@ -152,14 +152,3 @@ impl FunctionExpression {
         }
     }
 }
-
-impl Default for FunctionExpression {
-    fn default() -> Self {
-        Self {
-            block: Block::default(),
-            parameters: Vec::new(),
-            is_variadic: false,
-            tokens: None,
-        }
-    }
-}

src/parser.rs

@@ -75,7 +75,7 @@ struct FunctionBodyTokens {
     pub variable_arguments: Option<Token>,
 }
 
-#[derive(Clone, Debug, PartialEq, Eq)]
+#[derive(Clone, Debug, Default, PartialEq, Eq)]
 pub struct Parser {
     hold_token_data: bool,
 }
@@ -1014,14 +1014,6 @@ impl Parser {
     }
 }
 
-impl Default for Parser {
-    fn default() -> Self {
-        Self {
-            hold_token_data: false,
-        }
-    }
-}
-
 #[derive(Clone, Debug)]
 pub enum ConvertError {
     Statement {

src/process/evaluator/mod.rs

@@ -5,19 +5,11 @@ pub use lua_value::*;
 use crate::nodes::*;
 
 /// A struct to convert an Expression node into a LuaValue object.
-#[derive(Debug, Clone, PartialEq, Eq)]
+#[derive(Debug, Clone, Default, PartialEq, Eq)]
 pub struct Evaluator {
     pure_metamethods: bool,
 }
 
-impl Default for Evaluator {
-    fn default() -> Self {
-        Self {
-            pure_metamethods: false,
-        }
-    }
-}
-
 impl Evaluator {
     /// When evaluating expressions related to tables, this value tells the evaluator if
     /// metamethods can have side effects. For example, indexing a normal table in Lua does not

src/rules/empty_do.rs

@@ -6,6 +6,7 @@ use crate::rules::{
 
 use super::verify_no_rule_properties;
 
+#[derive(Debug, Default)]
 struct EmptyDoFilter {
     mutated: bool,
 }
@@ -16,12 +17,6 @@ impl EmptyDoFilter {
     }
 }
 
-impl Default for EmptyDoFilter {
-    fn default() -> Self {
-        Self { mutated: false }
-    }
-}
-
 impl NodeProcessor for EmptyDoFilter {
     fn process_block(&mut self, block: &mut Block) {
         block.filter_statements(|statement| match statement {