clean up

Author: timfennis

ndc_lib/src/ast/parser.rs

@@ -1112,13 +1112,10 @@ impl Parser {
         let identifier = match self.peek_current_token() {
             Some(Token::LeftParentheses) => None,
             Some(Token::Identifier(_)) => {
-                // TODO: clean this up
-                let ExpressionLocation {
+                let Ok(ExpressionLocation {
                     expression: Expression::Identifier { name, .. },
                     ..
-                } = self
-                    .require_identifier()
-                    .expect("this is guaranteed to produce an identifier by previous match")
+                }) = self.require_identifier()
                 else {
                     unreachable!("guaranteed to to produce identifier")
                 };

ndc_lib/src/interpreter/evaluate/mod.rs

@@ -954,7 +954,7 @@ fn execute_for_iterations(
 
                 if tail.is_empty() {
                     match execute_body(body, &mut scope, out_values) {
-                        Err(FunctionCarrier::Continue) => {},
+                        Err(FunctionCarrier::Continue) => {}
                         Err(error) => return Err(error),
                         Ok(_value) => {}
                     }

ndc_lib/src/interpreter/resolve.rs

@@ -29,6 +29,7 @@ pub fn resolve_pass(
             resolved,
         } => {
             if ident == "None" {
+                // THIS IS VERY UNHINGED
                 return Ok(());
             }
             let binding = lexical_data.get_binding_any(ident).ok_or_else(|| {