Fix clippy lints (except unused method I can't explain)

Author: gefjon

crates/core/src/host/module_host.rs

@@ -395,8 +395,6 @@ impl Instance {
     }
 }
 
-pub trait ModuleInstance: Send + 'static {}
-
 /// Creates the table for `table_def` in `stdb`.
 pub fn create_table_from_def(
     stdb: &RelationalDB,
@@ -981,7 +979,6 @@ impl ModuleHost {
             me.call_identity_disconnected_inner(caller_identity, caller_connection_id, inst)
         })
         .await?
-        .map_err(Into::into)
     }
 
     /// Empty the system tables tracking clients without running any lifecycle reducers.

crates/core/src/host/wasm_common/instrumentation.rs

@@ -91,6 +91,12 @@ pub struct CallTimes {
     times: EnumMap<AbiCall, Duration>,
 }
 
+impl Default for CallTimes {
+    fn default() -> Self {
+        Self::new()
+    }
+}
+
 impl CallTimes {
     /// Create a new timing structure, with times for all calls set to zero.
     pub fn new() -> Self {
@@ -115,6 +121,6 @@ impl CallTimes {
     /// will `take`` the CallTimes after running a reducer and report the taken times,
     /// leaving a fresh zeroed CallTimes for the next reducer invocation.
     pub fn take(&mut self) -> CallTimes {
-        std::mem::replace(self, Self::new())
+        std::mem::take(self)
     }
 }

crates/core/src/startup.rs

@@ -443,6 +443,6 @@ impl DatabaseCores {
     /// let database_cores = cores.databases.make_database_runners(rt.handle());
     /// ```
     pub fn make_database_runners(self, global_runtime: &tokio::runtime::Handle) -> JobCores {
-        JobCores::from_pinned_cores(self.0.into_iter(), global_runtime.clone())
+        JobCores::from_pinned_cores(self.0, global_runtime.clone())
     }
 }