f Drop unnecessary clone

Author: tnull

lightning-persister/src/fs_store.rs

@@ -417,21 +417,21 @@ fn get_key_from_dir_entry(p: &Path, base_path: &Path) -> Result<String, lightnin
 
 impl MigratableKVStore for FilesystemStore {
 	fn list_all_keys(&self) -> Result<Vec<(String, String, String)>, lightning::io::Error> {
-		let prefixed_dest = self.data_dir.clone();
+		let prefixed_dest = &self.data_dir;
 		if !prefixed_dest.exists() {
 			return Ok(Vec::new());
 		}
 
 		let mut keys = Vec::new();
 
-		'primary_loop: for primary_entry in fs::read_dir(&prefixed_dest)? {
+		'primary_loop: for primary_entry in fs::read_dir(prefixed_dest)? {
 			let primary_entry = primary_entry?;
 			let primary_path = primary_entry.path();
 
 			if dir_entry_is_key(&primary_path)? {
 				let primary_namespace = "".to_string();
 				let secondary_namespace = "".to_string();
-				let key = get_key_from_dir_entry(&primary_path, &prefixed_dest)?;
+				let key = get_key_from_dir_entry(&primary_path, prefixed_dest)?;
 				keys.push((primary_namespace, secondary_namespace, key));
 				continue 'primary_loop;
 			}
@@ -442,7 +442,7 @@ impl MigratableKVStore for FilesystemStore {
 				let secondary_path = secondary_entry.path();
 
 				if dir_entry_is_key(&secondary_path)? {
-					let primary_namespace = get_key_from_dir_entry(&primary_path, &prefixed_dest)?;
+					let primary_namespace = get_key_from_dir_entry(&primary_path, prefixed_dest)?;
 					let secondary_namespace = "".to_string();
 					let key = get_key_from_dir_entry(&secondary_path, &primary_path)?;
 					keys.push((primary_namespace, secondary_namespace, key));
@@ -456,7 +456,7 @@ impl MigratableKVStore for FilesystemStore {
 
 					if dir_entry_is_key(&tertiary_path)? {
 						let primary_namespace =
-							get_key_from_dir_entry(&primary_path, &prefixed_dest)?;
+							get_key_from_dir_entry(&primary_path, prefixed_dest)?;
 						let secondary_namespace =
 							get_key_from_dir_entry(&secondary_path, &primary_path)?;
 						let key = get_key_from_dir_entry(&tertiary_path, &secondary_path)?;