Skip to content

Commit e6be0af

Browse files
author
Seulgi Kim
committed
Prefer Arc::clone(&a) to a.clone()
It enhances the readability since it shows clone function creates a new reference and does not copy the entire object. https://doc.rust-lang.org/std/sync/struct.Arc.html#cloning-references
1 parent 8fe6a96 commit e6be0af

File tree

1 file changed

+18
-18
lines changed

1 file changed

+18
-18
lines changed

state/src/db/state_db.rs

+18-18
Original file line numberDiff line numberDiff line change
@@ -369,15 +369,15 @@ impl StateDB {
369369
pub fn clone_canon(&self, parent: &H256) -> StateDB {
370370
StateDB {
371371
db: self.db.boxed_clone(),
372-
account_cache: self.account_cache.clone(),
373-
regular_account_cache: self.regular_account_cache.clone(),
374-
metadata_cache: self.metadata_cache.clone(),
375-
shard_cache: self.shard_cache.clone(),
376-
shard_metadata_cache: self.shard_metadata_cache.clone(),
377-
world_cache: self.world_cache.clone(),
378-
asset_scheme_cache: self.asset_scheme_cache.clone(),
379-
asset_cache: self.asset_cache.clone(),
380-
action_data_cache: self.action_data_cache.clone(),
372+
account_cache: Arc::clone(&self.account_cache),
373+
regular_account_cache: Arc::clone(&self.regular_account_cache),
374+
metadata_cache: Arc::clone(&self.metadata_cache),
375+
shard_cache: Arc::clone(&self.shard_cache),
376+
shard_metadata_cache: Arc::clone(&self.shard_metadata_cache),
377+
world_cache: Arc::clone(&self.world_cache),
378+
asset_scheme_cache: Arc::clone(&self.asset_scheme_cache),
379+
asset_cache: Arc::clone(&self.asset_cache),
380+
action_data_cache: Arc::clone(&self.action_data_cache),
381381

382382
local_account_cache: LocalCache::new(),
383383
local_regular_account_cache: LocalCache::new(),
@@ -464,15 +464,15 @@ impl Clone for StateDB {
464464
fn clone(&self) -> Self {
465465
Self {
466466
db: self.db.boxed_clone(),
467-
account_cache: self.account_cache.clone(),
468-
regular_account_cache: self.regular_account_cache.clone(),
469-
metadata_cache: self.metadata_cache.clone(),
470-
shard_cache: self.shard_cache.clone(),
471-
shard_metadata_cache: self.shard_metadata_cache.clone(),
472-
world_cache: self.world_cache.clone(),
473-
asset_scheme_cache: self.asset_scheme_cache.clone(),
474-
asset_cache: self.asset_cache.clone(),
475-
action_data_cache: self.action_data_cache.clone(),
467+
account_cache: Arc::clone(&self.account_cache),
468+
regular_account_cache: Arc::clone(&self.regular_account_cache),
469+
metadata_cache: Arc::clone(&self.metadata_cache),
470+
shard_cache: Arc::clone(&self.shard_cache),
471+
shard_metadata_cache: Arc::clone(&self.shard_metadata_cache),
472+
world_cache: Arc::clone(&self.world_cache),
473+
asset_scheme_cache: Arc::clone(&self.asset_scheme_cache),
474+
asset_cache: Arc::clone(&self.asset_cache),
475+
action_data_cache: Arc::clone(&self.action_data_cache),
476476

477477
local_account_cache: LocalCache::new(),
478478
local_regular_account_cache: LocalCache::new(),

0 commit comments

Comments
 (0)