From 2b61b1d22d78694bdc83d2b6cb0b3b749a0e668a Mon Sep 17 00:00:00 2001
From: yjh <yjh465402634@gmail.com>
Date: Tue, 11 Apr 2023 18:42:49 +0800
Subject: [PATCH] chore: remove duplicated arc (#13871)

---
 client/executor/benches/bench.rs             |  6 +++---
 client/executor/src/executor.rs              |  2 +-
 client/executor/src/integration_tests/mod.rs |  2 +-
 client/executor/src/wasm_runtime.rs          | 20 ++++++++++----------
 4 files changed, 15 insertions(+), 15 deletions(-)

diff --git a/client/executor/benches/bench.rs b/client/executor/benches/bench.rs
index 6cce05f6c75f4..772898b8c76b3 100644
--- a/client/executor/benches/bench.rs
+++ b/client/executor/benches/bench.rs
@@ -48,7 +48,7 @@ fn initialize(
 	_tmpdir: &mut Option<tempfile::TempDir>,
 	runtime: &[u8],
 	method: Method,
-) -> Arc<dyn WasmModule> {
+) -> Box<dyn WasmModule> {
 	let blob = RuntimeBlob::uncompress_if_needed(runtime).unwrap();
 	let host_functions = sp_io::SubstrateHostFunctions::host_functions();
 	let allow_missing_func_imports = true;
@@ -60,7 +60,7 @@ fn initialize(
 			host_functions,
 			allow_missing_func_imports,
 		)
-		.map(|runtime| -> Arc<dyn WasmModule> { Arc::new(runtime) }),
+		.map(|runtime| -> Box<dyn WasmModule> { Box::new(runtime) }),
 		Method::Compiled { instantiation_strategy, precompile } => {
 			let config = sc_executor_wasmtime::Config {
 				allow_missing_func_imports,
@@ -98,7 +98,7 @@ fn initialize(
 			} else {
 				sc_executor_wasmtime::create_runtime::<sp_io::SubstrateHostFunctions>(blob, config)
 			}
-			.map(|runtime| -> Arc<dyn WasmModule> { Arc::new(runtime) })
+			.map(|runtime| -> Box<dyn WasmModule> { Box::new(runtime) })
 		},
 	}
 	.unwrap()
diff --git a/client/executor/src/executor.rs b/client/executor/src/executor.rs
index 79250ee0d621c..a3717f4d29002 100644
--- a/client/executor/src/executor.rs
+++ b/client/executor/src/executor.rs
@@ -320,7 +320,7 @@ where
 	) -> Result<R>
 	where
 		F: FnOnce(
-			AssertUnwindSafe<&Arc<dyn WasmModule>>,
+			AssertUnwindSafe<&dyn WasmModule>,
 			AssertUnwindSafe<&mut dyn WasmInstance>,
 			Option<&RuntimeVersion>,
 			AssertUnwindSafe<&mut dyn Externalities>,
diff --git a/client/executor/src/integration_tests/mod.rs b/client/executor/src/integration_tests/mod.rs
index 63a642f990642..b65aeb8d01109 100644
--- a/client/executor/src/integration_tests/mod.rs
+++ b/client/executor/src/integration_tests/mod.rs
@@ -483,7 +483,7 @@ fn should_trap_when_heap_exhausted(wasm_method: WasmExecutionMethod) {
 fn mk_test_runtime(
 	wasm_method: WasmExecutionMethod,
 	pages: HeapAllocStrategy,
-) -> Arc<dyn WasmModule> {
+) -> Box<dyn WasmModule> {
 	let blob = RuntimeBlob::uncompress_if_needed(wasm_binary_unwrap())
 		.expect("failed to create a runtime blob out of test runtime");
 
diff --git a/client/executor/src/wasm_runtime.rs b/client/executor/src/wasm_runtime.rs
index 351b6f377b8af..41a095081f8d1 100644
--- a/client/executor/src/wasm_runtime.rs
+++ b/client/executor/src/wasm_runtime.rs
@@ -71,14 +71,14 @@ struct VersionedRuntimeId {
 /// A Wasm runtime object along with its cached runtime version.
 struct VersionedRuntime {
 	/// Shared runtime that can spawn instances.
-	module: Arc<dyn WasmModule>,
+	module: Box<dyn WasmModule>,
 	/// Runtime version according to `Core_version` if any.
 	version: Option<RuntimeVersion>,
 
 	// TODO: Remove this once the legacy instance reuse instantiation strategy
 	//       for `wasmtime` is gone, as this only makes sense with that particular strategy.
 	/// Cached instance pool.
-	instances: Arc<Vec<Mutex<Option<Box<dyn WasmInstance>>>>>,
+	instances: Vec<Mutex<Option<Box<dyn WasmInstance>>>>,
 }
 
 impl VersionedRuntime {
@@ -86,7 +86,7 @@ impl VersionedRuntime {
 	fn with_instance<R, F>(&self, ext: &mut dyn Externalities, f: F) -> Result<R, Error>
 	where
 		F: FnOnce(
-			&Arc<dyn WasmModule>,
+			&dyn WasmModule,
 			&mut dyn WasmInstance,
 			Option<&RuntimeVersion>,
 			&mut dyn Externalities,
@@ -106,7 +106,7 @@ impl VersionedRuntime {
 					.map(|r| Ok((r, false)))
 					.unwrap_or_else(|| self.module.new_instance().map(|i| (i, true)))?;
 
-				let result = f(&self.module, &mut *instance, self.version.as_ref(), ext);
+				let result = f(&*self.module, &mut *instance, self.version.as_ref(), ext);
 				if let Err(e) = &result {
 					if new_inst {
 						tracing::warn!(
@@ -142,7 +142,7 @@ impl VersionedRuntime {
 				// Allocate a new instance
 				let mut instance = self.module.new_instance()?;
 
-				f(&self.module, &mut *instance, self.version.as_ref(), ext)
+				f(&*self.module, &mut *instance, self.version.as_ref(), ext)
 			},
 		}
 	}
@@ -228,7 +228,7 @@ impl RuntimeCache {
 	where
 		H: HostFunctions,
 		F: FnOnce(
-			&Arc<dyn WasmModule>,
+			&dyn WasmModule,
 			&mut dyn WasmInstance,
 			Option<&RuntimeVersion>,
 			&mut dyn Externalities,
@@ -294,7 +294,7 @@ pub fn create_wasm_runtime_with_code<H>(
 	blob: RuntimeBlob,
 	allow_missing_func_imports: bool,
 	cache_path: Option<&Path>,
-) -> Result<Arc<dyn WasmModule>, WasmError>
+) -> Result<Box<dyn WasmModule>, WasmError>
 where
 	H: HostFunctions,
 {
@@ -312,7 +312,7 @@ where
 				H::host_functions(),
 				allow_missing_func_imports,
 			)
-			.map(|runtime| -> Arc<dyn WasmModule> { Arc::new(runtime) })
+			.map(|runtime| -> Box<dyn WasmModule> { Box::new(runtime) })
 		},
 		WasmExecutionMethod::Compiled { instantiation_strategy } =>
 			sc_executor_wasmtime::create_runtime::<H>(
@@ -333,7 +333,7 @@ where
 					},
 				},
 			)
-			.map(|runtime| -> Arc<dyn WasmModule> { Arc::new(runtime) }),
+			.map(|runtime| -> Box<dyn WasmModule> { Box::new(runtime) }),
 	}
 }
 
@@ -448,7 +448,7 @@ where
 	let mut instances = Vec::with_capacity(max_instances);
 	instances.resize_with(max_instances, || Mutex::new(None));
 
-	Ok(VersionedRuntime { module: runtime, version, instances: Arc::new(instances) })
+	Ok(VersionedRuntime { module: runtime, version, instances })
 }
 
 #[cfg(test)]