From 10f744649ae0bd2ba4107e5004876eb8cc48db7a Mon Sep 17 00:00:00 2001
From: fmccl <xfinnbar@gmail.com>
Date: Sat, 10 Aug 2024 15:20:23 +0100
Subject: [PATCH] Revert "fmt & clippy"

---
 molang_proc_macro/src/lib.rs |  7 +++++--
 src/data.rs                  |  2 +-
 src/interpreter.rs           | 16 +++++++++-------
 src/parser.rs                | 10 +++++-----
 src/state.rs                 | 11 ++---------
 src/tokeniser.rs             | 12 ++++++++----
 6 files changed, 30 insertions(+), 28 deletions(-)

diff --git a/molang_proc_macro/src/lib.rs b/molang_proc_macro/src/lib.rs
index be0785d..87e8100 100644
--- a/molang_proc_macro/src/lib.rs
+++ b/molang_proc_macro/src/lib.rs
@@ -22,7 +22,10 @@ fn molang_struct(input: TokenStream) -> TokenStream {
         .map(|field| field.ident.clone().unwrap())
         .collect();
 
-    let field_types: Vec<Type> = fields.iter().map(|field| field.ty.clone()).collect();
+    let field_types: Vec<Type> = fields
+        .iter()
+        .map(|field| field.ty.clone())
+        .collect();
 
     quote! {
         impl molang::ToMolangValue for #name {
@@ -62,4 +65,4 @@ fn my_test() {
         })
     );
     panic!("abc");
-}
+}
\ No newline at end of file
diff --git a/src/data.rs b/src/data.rs
index ee86fb9..0160d01 100644
--- a/src/data.rs
+++ b/src/data.rs
@@ -23,4 +23,4 @@ impl Operator {
             Self::Equality => 8,
         }
     }
-}
+}
\ No newline at end of file
diff --git a/src/interpreter.rs b/src/interpreter.rs
index 96a353d..bd7e290 100644
--- a/src/interpreter.rs
+++ b/src/interpreter.rs
@@ -147,14 +147,16 @@ pub fn run(
                                         if let Some(some_current) = variables.get_mut(name) {
                                             current = some_current;
                                             break;
-                                        } else if constants.contains_key(name) {
-                                            return Err(MolangError::NotAssignable(format!(
-                                                "Constant {name}"
-                                            )));
                                         } else {
-                                            return Err(MolangError::VariableNotFound(
-                                                name.to_string(),
-                                            ));
+                                            if constants.contains_key(name) {
+                                                return Err(MolangError::NotAssignable(format!(
+                                                    "Constant {name}"
+                                                )));
+                                            } else {
+                                                return Err(MolangError::VariableNotFound(
+                                                    format!("{name}"),
+                                                ));
+                                            }
                                         }
                                     }
                                 } else if let Value::Struct(struc) =
diff --git a/src/parser.rs b/src/parser.rs
index 3b4b038..aec57cd 100644
--- a/src/parser.rs
+++ b/src/parser.rs
@@ -62,7 +62,7 @@ pub fn treeify(mut tokens: &[Token]) -> Result<Expr, CompileError> {
         let left = &tokens[..i];
         let right = &tokens[i + 1..];
 
-        Ok(Expr::Derived(Box::new(match op {
+        return Ok(Expr::Derived(Box::new(match op {
             Operator::Not => {
                 if !left.is_empty() {
                     return Err(CompileError::TokensBeforePrefixOperator);
@@ -80,10 +80,10 @@ pub fn treeify(mut tokens: &[Token]) -> Result<Expr, CompileError> {
             Operator::NullishCoalescing => {
                 Instruction::NullishCoalescing(treeify(left)?, treeify(right)?)
             }
-        })))
+        })));
     } else {
         match tokens {
-            [Token::Number(n)] => Ok(Expr::Literal(Value::Number(*n))),
+            [Token::Number(n)] => return Ok(Expr::Literal(Value::Number(*n))),
             [Token::Access(accesses)] => {
                 let mut access_exprs = Vec::new();
                 for access in accesses {
@@ -109,7 +109,7 @@ pub fn treeify(mut tokens: &[Token]) -> Result<Expr, CompileError> {
     }
 }
 
-fn comma_split(tokens: &[Token]) -> Vec<&[Token]> {
+fn comma_split<'a>(tokens: &'a Vec<Token>) -> Vec<&'a [Token]> {
     let mut result = Vec::new();
     let mut start = 0;
 
@@ -123,7 +123,7 @@ fn comma_split(tokens: &[Token]) -> Vec<&[Token]> {
     result.push(&tokens[start..]);
 
     if let Some(last) = result.pop() {
-        if !last.is_empty() {
+        if last.len() != 0 {
             result.push(last);
         }
     }
diff --git a/src/state.rs b/src/state.rs
index 362e254..ebcf358 100644
--- a/src/state.rs
+++ b/src/state.rs
@@ -2,18 +2,11 @@ pub trait State<In, Out, Error> {
     fn handle(
         &mut self,
         c: Option<In>,
-    ) -> Result<
-        (
-            Option<Out>,
-            Option<Box<dyn State<In, Out, Error>>>,
-            SequenceAction,
-        ),
-        Error,
-    >;
+    ) -> Result<(Option<Out>, Option<Box<dyn State<In, Out, Error>>>, SequenceAction), Error>;
 }
 
 pub enum SequenceAction {
     Advance,
     Done,
     Hold,
-}
+}
\ No newline at end of file
diff --git a/src/tokeniser.rs b/src/tokeniser.rs
index 6718df7..9cd2e1c 100644
--- a/src/tokeniser.rs
+++ b/src/tokeniser.rs
@@ -201,12 +201,16 @@ impl State<char, Token, TokeniseError> for AccessTokenState {
     > {
         let (access, new_state, action) = self.state.handle(c)?;
 
-        if let Some(access) = access {
-            self.accesses.push(access)
+        match access {
+            Some(access) => self.accesses.push(access),
+            None => {}
         }
 
-        if let Some(new_state) = new_state {
-            self.state = new_state;
+        match new_state {
+            Some(new_state) => {
+                self.state = new_state;
+            }
+            None => {}
         }
 
         match action {