Make "make lint" pass for Rust 1.63

lib/cli/src/commands/create_exe.rs

@@ -9,6 +9,7 @@ use serde::{Deserialize, Serialize};
 #[cfg(feature = "http")]
 use std::collections::BTreeMap;
 use std::env;
+use std::fmt::Write as _;
 use std::fs;
 use std::fs::File;
 use std::io::prelude::*;
@@ -912,7 +913,8 @@ impl CreateExe {
         for atom_name in atom_names.iter() {
             let atom_name = Self::normalize_atom_name(atom_name);
 
-            c_code_to_instantiate.push_str(&format!(
+            write!(
+                c_code_to_instantiate,
                 "
 
             wasm_module_t *atom_{atom_name} = wasmer_object_module_new(store, \"{atom_name}\");
@@ -923,11 +925,16 @@ impl CreateExe {
                 return -1;
             }}
             "
-            ));
+            )
-            deallocate_module.push_str(&format!("wasm_module_delete(atom_{atom_name});"));
+            .unwrap();
+            write!(deallocate_module, "wasm_module_delete(atom_{atom_name});").unwrap();
         }
 
-        c_code_to_instantiate.push_str(&format!("wasm_module_t *module = atom_{atom_to_run};"));
+        write!(
+            c_code_to_instantiate,
+            "wasm_module_t *module = atom_{atom_to_run};"
+        )
+        .unwrap();
 
         WASMER_STATIC_MAIN_C_SOURCE
             .replace("#define WASI", "#define WASI\r\n#define WASI_PIRITA")

lib/cli/src/commands/list.rs

@@ -36,7 +36,7 @@ impl List {
         if empty_table {
             table.add_empty_row();
         }
-        let _ = table.printstd();
+        table.printstd();
 
         Ok(())
     }

lib/compiler-cranelift/src/translator/func_state.rs

@@ -189,7 +189,7 @@ impl ControlStackFrame {
     /// Pop values from the value stack so that it is left at the
     /// input-parameters to an else-block.
     pub fn truncate_value_stack_to_else_params(&self, stack: &mut Vec<Value>) {
-        debug_assert!(matches!(self, &ControlStackFrame::If { .. }));
+        debug_assert!(matches!(self, &Self::If { .. }));
         stack.truncate(self.original_stack_size());
     }
 
@@ -202,7 +202,7 @@ impl ControlStackFrame {
         // block can see the same number of parameters as the consequent block. As a matter of
         // fact, we need to substract an extra number of parameter values for if blocks.
         let num_duplicated_params = match self {
-            &ControlStackFrame::If {
+            &Self::If {
                 num_param_values, ..
             } => {
                 debug_assert!(num_param_values <= self.original_stack_size());

lib/registry/src/config.rs

@@ -12,7 +12,7 @@ pub static GLOBAL_CONFIG_FILE_NAME: &str = if cfg!(target_os = "wasi") {
     "wapm.toml"
 };
 
-#[derive(Deserialize, Default, Serialize, Debug, PartialEq)]
+#[derive(Deserialize, Default, Serialize, Debug, PartialEq, Eq)]
 pub struct PartialWapmConfig {
     /// The number of seconds to wait before checking the registry for a new
     /// version of the package.
@@ -41,12 +41,12 @@ pub const fn wax_default_cooldown() -> i32 {
     5 * 60
 }
 
-#[derive(Deserialize, Serialize, Debug, PartialEq, Default)]
+#[derive(Deserialize, Serialize, Debug, PartialEq, Eq, Default)]
 pub struct Proxy {
     pub url: Option<String>,
 }
 
-#[derive(Deserialize, Serialize, Debug, PartialEq, Default)]
+#[derive(Deserialize, Serialize, Debug, PartialEq, Eq, Default)]
 pub struct UpdateNotifications {
     pub enabled: String,
 }
@@ -57,14 +57,14 @@ pub struct Telemetry {
     pub enabled: String,
 }
 
-#[derive(Deserialize, Serialize, Debug, PartialEq, Clone)]
+#[derive(Deserialize, Serialize, Debug, PartialEq, Eq, Clone)]
 #[serde(untagged)]
 pub enum Registries {
     Single(Registry),
     Multi(MultiRegistry),
 }
 
-#[derive(Deserialize, Serialize, Debug, PartialEq, Clone)]
+#[derive(Deserialize, Serialize, Debug, PartialEq, Eq, Clone)]
 pub struct MultiRegistry {
     /// Currently active registry
     pub current: String,
@@ -82,7 +82,7 @@ impl Default for Registries {
     }
 }
 
-#[derive(Deserialize, Serialize, Debug, PartialEq, Clone)]
+#[derive(Deserialize, Serialize, Debug, PartialEq, Eq, Clone)]
 pub struct Registry {
     pub url: String,
     pub token: Option<String>,
@@ -278,7 +278,7 @@ impl PartialWapmConfig {
     pub fn get_folder(test_name: &str) -> Result<PathBuf, String> {
         let test_dir = std::env::temp_dir().join("test_wasmer").join(test_name);
         let _ = std::fs::create_dir_all(&test_dir);
-        Ok(test_dir.to_path_buf())
+        Ok(test_dir)
     }
 
     #[cfg(not(test))]

lib/registry/src/lib.rs

@@ -309,7 +309,7 @@ pub fn query_command_from_registry(
     })
 }
 
-#[derive(Debug, Clone, PartialEq, PartialOrd)]
+#[derive(Debug, Clone, PartialEq, Eq, PartialOrd)]
 pub enum QueryPackageError {
     ErrorSendingQuery(String),
     NoPackageFound {
@@ -329,7 +329,7 @@ impl fmt::Display for QueryPackageError {
     }
 }
 
-#[derive(Debug, Clone, PartialEq, PartialOrd)]
+#[derive(Debug, Clone, PartialEq, Eq, PartialOrd)]
 pub enum GetIfPackageHasNewVersionResult {
     // if version = Some(...) and the ~/.wasmer/checkouts/.../{version} exists, the package is already installed
     UseLocalAlreadyInstalled {
@@ -860,7 +860,7 @@ pub fn test_if_registry_present(registry: &str) -> Result<bool, String> {
     use graphql_client::GraphQLQuery;
 
     let q = TestIfRegistryPresent::build_query(test_if_registry_present::Variables {});
-    let _ = crate::graphql::execute_query_modifier_inner_check_json(
+    crate::graphql::execute_query_modifier_inner_check_json(
         registry,
         "",
         &q,

lib/vfs/src/webc_fs.rs

@@ -23,7 +23,7 @@ where
     pub memory: Arc<MemFileSystem>,
 }
 
-impl<'a, T> WebcFileSystem<T>
+impl<T> WebcFileSystem<T>
 where
     T: std::fmt::Debug + Send + Sync + 'static,
     T: Deref<Target = WebC<'static>>,
@@ -54,7 +54,7 @@ where
     pub memory: Arc<MemFileSystem>,
 }
 
-impl<'a, T> FileOpener for WebCFileOpener<T>
+impl<T> FileOpener for WebCFileOpener<T>
 where
     T: std::fmt::Debug + Send + Sync + 'static,
     T: Deref<Target = WebC<'static>>,

lib/vm/src/store.rs

@@ -261,8 +261,8 @@ impl<T> MaybeInstanceOwned<T> {
     /// Returns underlying pointer to the VM data.
     pub fn as_ptr(&self) -> NonNull<T> {
         match self {
-            MaybeInstanceOwned::Host(p) => unsafe { NonNull::new_unchecked(p.get()) },
+            Self::Host(p) => unsafe { NonNull::new_unchecked(p.get()) },
-            MaybeInstanceOwned::Instance(p) => *p,
+            Self::Instance(p) => *p,
         }
     }
 }
@@ -273,12 +273,12 @@ where
 {
     fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
         match self {
-            MaybeInstanceOwned::Host(p) => {
+            Self::Host(p) => {
                 write!(f, "host(")?;
                 p.as_ref().fmt(f)?;
                 write!(f, ")")
             }
-            MaybeInstanceOwned::Instance(p) => {
+            Self::Instance(p) => {
                 write!(f, "instance(")?;
                 unsafe { p.as_ref().fmt(f)? };
                 write!(f, ")")

lib/vm/src/trap/traphandlers.rs

@@ -829,14 +829,14 @@ enum UnwindReason {
 impl UnwindReason {
     fn into_trap(self) -> Trap {
         match self {
-            UnwindReason::UserTrap(data) => Trap::User(data),
+            Self::UserTrap(data) => Trap::User(data),
-            UnwindReason::LibTrap(trap) => trap,
+            Self::LibTrap(trap) => trap,
-            UnwindReason::WasmTrap {
+            Self::WasmTrap {
                 backtrace,
                 pc,
                 signal_trap,
             } => Trap::wasm(pc, backtrace, signal_trap),
-            UnwindReason::Panic(panic) => std::panic::resume_unwind(panic),
+            Self::Panic(panic) => std::panic::resume_unwind(panic),
         }
     }
 }

lib/wasi-types/src/lib.rs

@@ -1,3 +1,6 @@
+#![doc(html_favicon_url = "https://wasmer.io/images/icons/favicon-32x32.png")]
+#![doc(html_logo_url = "https://github.com/wasmerio.png?size=200")]
+
 pub mod types;
 pub mod wasi;
 

lib/wasi-types/src/types.rs

@@ -1,6 +1,4 @@
 #![deny(unused_mut)]
-#![doc(html_favicon_url = "https://wasmer.io/images/icons/favicon-32x32.png")]
-#![doc(html_logo_url = "https://github.com/wasmerio.png?size=200")]
 #![allow(non_camel_case_types, clippy::identity_op)]
 
 //! Wasmer's WASI types implementation.

tests/compilers/config.rs

@@ -2,7 +2,7 @@ use std::sync::Arc;
 use wasmer::{CompilerConfig, Features, ModuleMiddleware, Store};
 use wasmer_compiler::Engine;
 
-#[derive(Clone, Debug, PartialEq)]
+#[derive(Clone, Debug, PartialEq, Eq)]
 pub enum Compiler {
     LLVM,
     Cranelift,

tests/compilers/traps.rs

@@ -34,8 +34,7 @@ fn test_trap_return(config: crate::Config) -> Result<()> {
 
     let e = run_func
         .call(&mut store, &[])
-        .err()
+        .expect_err("error calling function");
-        .expect("error calling function");
 
     assert_eq!(e.message(), "test 123");
 
@@ -62,8 +61,7 @@ fn test_trap_trace(config: crate::Config) -> Result<()> {
 
     let e = run_func
         .call(&mut store, &[])
-        .err()
+        .expect_err("error calling function");
-        .expect("error calling function");
 
     let trace = e.trace();
     assert_eq!(trace.len(), 2);
@@ -113,8 +111,7 @@ fn test_trap_trace_cb(config: crate::Config) -> Result<()> {
 
     let e = run_func
         .call(&mut store, &[])
-        .err()
+        .expect_err("error calling function");
-        .expect("error calling function");
 
     let trace = e.trace();
     println!("Trace {:?}", trace);
@@ -148,8 +145,7 @@ fn test_trap_stack_overflow(config: crate::Config) -> Result<()> {
 
     let e = run_func
         .call(&mut store, &[])
-        .err()
+        .expect_err("error calling function");
-        .expect("error calling function");
 
     // We specifically don't check the stack trace here: stack traces after
     // stack overflows are not generally possible due to unreliable unwinding
@@ -181,8 +177,7 @@ fn trap_display_pretty(config: crate::Config) -> Result<()> {
 
     let e = run_func
         .call(&mut store, &[])
-        .err()
+        .expect_err("error calling function");
-        .expect("error calling function");
     assert_eq!(
         e.to_string(),
         "\
@@ -236,8 +231,7 @@ fn trap_display_multi_module(config: crate::Config) -> Result<()> {
 
     let e = bar2
         .call(&mut store, &[])
-        .err()
+        .expect_err("error calling function");
-        .expect("error calling function");
     assert_eq!(
         e.to_string(),
         "\
@@ -433,9 +427,7 @@ fn call_signature_mismatch(config: crate::Config) -> Result<()> {
     "#;
 
     let module = Module::new(&store, &binary)?;
-    let err = Instance::new(&mut store, &module, &imports! {})
+    let err = Instance::new(&mut store, &module, &imports! {}).expect_err("expected error");
-        .err()
-        .expect("expected error");
     assert_eq!(
         format!("{}", err),
         "\
@@ -461,9 +453,7 @@ fn start_trap_pretty(config: crate::Config) -> Result<()> {
     "#;
 
     let module = Module::new(&store, wat)?;
-    let err = Instance::new(&mut store, &module, &imports! {})
+    let err = Instance::new(&mut store, &module, &imports! {}).expect_err("expected error");
-        .err()
-        .expect("expected error");
 
     assert_eq!(
         format!("{}", err),

tests/compilers/typed_functions.rs

@@ -1,3 +1,5 @@
+#![allow(clippy::unnecessary_operation)] // We use x1 multiplies for clarity
+
 use anyhow::Result;
 use std::convert::Infallible;
 use std::sync::{Arc, Mutex};