Clippy fixes

src/main.rs

@@ -207,7 +207,7 @@ fn init_rayon(threads: Option<usize>) {
     // build_global only seems to return an error
     // if called twice, so this should be fine.
     let _ = rayon::ThreadPoolBuilder::new()
-        .num_threads(threads as usize)
+        .num_threads(threads)
         .build_global();
 }
 

src/reporter/html.rs

@@ -191,7 +191,7 @@ impl<'a> HTMLReporter<'a> {
 
     /// Create the output directory
     fn create_output_directory(&self) -> Result<()> {
-        std::fs::create_dir_all(&self.output_directory)?;
+        std::fs::create_dir_all(self.output_directory)?;
         Ok(())
     }
 
@@ -254,8 +254,8 @@ impl<'a> HTMLReporter<'a> {
         let data = BTreeMap::from([
             ("source_files", handlebars::to_json(source_files)),
             ("file", handlebars::to_json::<Option<String>>(None)),
-            ("report_info", handlebars::to_json(&report_info)),
-            ("stats", handlebars::to_json(&stats)),
+            ("report_info", handlebars::to_json(report_info)),
+            ("stats", handlebars::to_json(stats)),
         ]);
         let writer = BufWriter::new(File::create(self.output_directory.join("index.html"))?);
         template_engine

src/runtime/wasmer.rs

@@ -145,7 +145,7 @@ impl WasmerRuntime {
             .exports
             .get_function("_start")
             .context("Failed to resolve _start function")?
-            .typed::<(), ()>(&mut self.store)
+            .typed::<(), ()>(&self.store)
             .context("Failed to get native _start function")?;
 
         let result = func.call(&mut self.store).map(|_| 0);
@@ -258,8 +258,8 @@ fn create_store(compiler: Compiler) -> Store {
     let metering = Arc::new(Metering::new(u64::MAX, cost_function));
 
     let mut compiler_config: Box<dyn CompilerConfig> = match compiler {
-        Compiler::Singlepass => Box::new(Singlepass::default()),
-        Compiler::Cranelift => Box::new(Cranelift::default()),
+        Compiler::Singlepass => Box::<Singlepass>::default(),
+        Compiler::Cranelift => Box::<Cranelift>::default(),
     };
 
     compiler_config.push_middleware(metering);
@@ -270,7 +270,7 @@ fn create_store(compiler: Compiler) -> Store {
 
 fn create_module(module: &WasmModule, store: &Store) -> Result<Module> {
     let bytecode: Vec<u8> = module.to_bytes()?;
-    let module = Module::new(store, &bytecode).context("Failed to create wasmer module")?;
+    let module = Module::new(store, bytecode).context("Failed to create wasmer module")?;
 
     Ok(module)
 }

src/wasmmodule.rs

@@ -72,7 +72,7 @@ impl<'a> WasmModule<'a> {
             .code_section()
             .context("Module has no code section")?;
 
-        let bytes = std::fs::read(&self.path.as_ref())
+        let bytes = std::fs::read(self.path.as_ref())
             .with_context(|| format!("Could not read bytecode from {}", self.path))?;
 
         Ok(code_section