chore: remove redundant

src/eval/mod.rs

@@ -183,7 +183,7 @@ impl Evaluator {
                 alternative,
             } => {
                 let cond = self.eval_expression(condition)?;
-                self.eval_if_expression(cond, consequence, alternative)
+                self.eval_if_expression(&cond, consequence, alternative)
             }
             ast::Expression::For {
                 condition,
@@ -260,7 +260,7 @@ impl Evaluator {
 
     fn eval_if_expression(
         &mut self,
-        cond: Object,
+        cond: &Object,
         consequence: &ast::BlockStatement,
         alternative: &Option<ast::BlockStatement>,
     ) -> Option<Object> {
@@ -284,7 +284,7 @@ impl Evaluator {
 
         let mut cond = self.eval_expression(condition)?;
 
-        while cond.clone().into() {
+        while (&cond).into() {
             rv = self.eval_block_statements(consequence);
             match rv {
                 Some(Object::Return(_)) => return rv,

src/eval/object.rs

@@ -102,16 +102,6 @@ pub struct CompiledFunction {
     pub(crate) num_params: usize,
 }
 
-impl From<Object> for bool {
-    fn from(value: Object) -> Self {
-        match value {
-            Object::Bool(b) => b,
-            Object::Null => false,
-            _ => true,
-        }
-    }
-}
-
 impl From<&Object> for bool {
     fn from(value: &Object) -> Self {
         match value {
@@ -198,6 +188,15 @@ fn cstring_should_work() {
     assert_eq!(CString("aabb".into()), s1 + s2);
 }
 
+#[test]
+fn object_bool_should_work() {
+    let b1 = &Object::Bool(true);
+    let b2 = &Object::Bool(false);
+
+    assert_eq!(true, b1.into());
+    assert_eq!(false, b2.into());
+}
+
 #[test]
 fn object_display_should_work() {
     assert_eq!("123", Object::Int(123).to_string());