refactor: use explicit .clone() where possible, instead of "to owne…

…d"-like methods
eduardosm · Sep 9, 2024 · 4625014 · 4625014
1 parent 9bd387b
commit 4625014
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/src/git/import/temp_storage.rs b/src/git/import/temp_storage.rs
@@ -27,7 +27,7 @@ impl TempStorage {
             .write(true)
             .open(&path)
             .map_err(|e| ImportError::CreateFileError {
-                path: path.to_path_buf(),
+                path: path.clone(),
                 error: e,
             })?;
 

diff --git a/src/main.rs b/src/main.rs
@@ -173,7 +173,7 @@ fn main_inner() -> Result<(), RunError> {
                 })?;
                 conv_params_path_parent.join(user_map_path)
             } else {
-                user_map_path.to_path_buf()
+                user_map_path.clone()
             };
 
             let user_map_file = std::fs::OpenOptions::new()

diff --git a/src/user_map.rs b/src/user_map.rs
@@ -51,7 +51,7 @@ impl UserMap {
 
             match parse_line(&line) {
                 Some(Some((user, entry))) => {
-                    map.entry(user.to_vec()).or_default().push(entry);
+                    map.entry(user.clone()).or_default().push(entry);
                 }
                 Some(None) => {}
                 None => return Err(AuthorMapParseError::BadLine(line_i, line)),