From 9b84429a9dc8e4bdde6366ce53c06ae25ac494c2 Mon Sep 17 00:00:00 2001
From: Andrew Morris <voltrevo@gmail.com>
Date: Wed, 24 Jul 2024 15:54:20 +0900
Subject: [PATCH] Clippy fixes

---
 valuescript_compiler/src/constants.rs                   | 4 ++--
 valuescript_compiler/src/function_compiler.rs           | 7 -------
 valuescript_compiler/src/optimization/simplify_jumps.rs | 2 +-
 valuescript_vm/src/operations.rs                        | 2 --
 vstc/src/assemble_command.rs                            | 2 +-
 vstc/src/compile_command.rs                             | 2 +-
 vstc/src/run_command.rs                                 | 2 +-
 7 files changed, 6 insertions(+), 15 deletions(-)

diff --git a/valuescript_compiler/src/constants.rs b/valuescript_compiler/src/constants.rs
index 9e1c9c9..78f8941 100644
--- a/valuescript_compiler/src/constants.rs
+++ b/valuescript_compiler/src/constants.rs
@@ -2,6 +2,6 @@ use crate::asm::{Number, Value};
 
 pub const CONSTANTS: [(&str, Value); 3] = [
   ("undefined", Value::Undefined),
-  ("NaN", Value::Number(Number(std::f64::NAN))),
-  ("Infinity", Value::Number(Number(std::f64::INFINITY))),
+  ("NaN", Value::Number(Number(f64::NAN))),
+  ("Infinity", Value::Number(Number(f64::INFINITY))),
 ];
diff --git a/valuescript_compiler/src/function_compiler.rs b/valuescript_compiler/src/function_compiler.rs
index 2700cd8..9687b37 100644
--- a/valuescript_compiler/src/function_compiler.rs
+++ b/valuescript_compiler/src/function_compiler.rs
@@ -60,13 +60,6 @@ impl Functionish {
   }
 }
 
-#[derive(Clone, Debug)]
-pub struct QueuedFunction {
-  pub definition_pointer: Pointer,
-  pub fn_name: Option<String>,
-  pub functionish: Functionish,
-}
-
 pub struct LoopLabels {
   pub continue_: Option<Label>,
   pub break_: Label,
diff --git a/valuescript_compiler/src/optimization/simplify_jumps.rs b/valuescript_compiler/src/optimization/simplify_jumps.rs
index f968e69..6f45767 100644
--- a/valuescript_compiler/src/optimization/simplify_jumps.rs
+++ b/valuescript_compiler/src/optimization/simplify_jumps.rs
@@ -88,7 +88,7 @@ fn simplify_jumps_fn(fn_: &mut Function) {
   }
 }
 
-fn next_instruction_index(body: &Vec<FnLine>, mut i: usize) -> Option<usize> {
+fn next_instruction_index(body: &[FnLine], mut i: usize) -> Option<usize> {
   i += 1;
 
   while i < body.len() {
diff --git a/valuescript_vm/src/operations.rs b/valuescript_vm/src/operations.rs
index 68fa737..9615f9a 100644
--- a/valuescript_vm/src/operations.rs
+++ b/valuescript_vm/src/operations.rs
@@ -387,9 +387,7 @@ pub fn op_triple_eq_impl(left: &Val, right: &Val) -> Result<bool, Val> {
 
       true
     }
-    #[allow(clippy::vtable_address_comparisons)] // TODO: Is this ok?
     (Val::Static(left), Val::Static(right)) => std::ptr::eq(&**left, &**right),
-    #[allow(clippy::vtable_address_comparisons)] // TODO: Is this ok?
     (Val::Dynamic(left), Val::Dynamic(right)) => std::ptr::eq(&**left, &**right),
     (Val::Static(..) | Val::Dynamic(..) | Val::CopyCounter(..), _)
     | (_, Val::Static(..) | Val::Dynamic(..) | Val::CopyCounter(..)) => {
diff --git a/vstc/src/assemble_command.rs b/vstc/src/assemble_command.rs
index 5125c4e..9823438 100644
--- a/vstc/src/assemble_command.rs
+++ b/vstc/src/assemble_command.rs
@@ -2,7 +2,7 @@ use valuescript_compiler::{assemble, parse_module};
 
 use crate::exit_command_failed::exit_command_failed;
 
-pub fn assemble_command(args: &Vec<String>) {
+pub fn assemble_command(args: &[String]) {
   if args.len() != 3 {
     exit_command_failed(args, None, "vstc assemble --help");
   }
diff --git a/vstc/src/compile_command.rs b/vstc/src/compile_command.rs
index 395d43a..689b9ea 100644
--- a/vstc/src/compile_command.rs
+++ b/vstc/src/compile_command.rs
@@ -8,7 +8,7 @@ use super::handle_diagnostics_cli::handle_diagnostics_cli;
 use valuescript_compiler::asm::Structured;
 use valuescript_compiler::compile;
 
-pub fn compile_command(args: &Vec<String>) {
+pub fn compile_command(args: &[String]) {
   if args.len() != 3 {
     exit_command_failed(args, None, "vstc compile --help");
   }
diff --git a/vstc/src/run_command.rs b/vstc/src/run_command.rs
index 05b5fce..81a1764 100644
--- a/vstc/src/run_command.rs
+++ b/vstc/src/run_command.rs
@@ -7,7 +7,7 @@ use valuescript_vm::{vs_value::Val, DecoderMaker};
 use crate::exit_command_failed::exit_command_failed;
 use crate::to_bytecode::{format_from_path, to_bytecode, RunFormat};
 
-pub fn run_command(args: &Vec<String>) {
+pub fn run_command(args: &[String]) {
   if args.len() < 3 {
     exit_command_failed(args, None, "vstc run --help");
   }