cleaned up hardlink_dir_all

ranger-ross · ranger-ross · commit 3c0d6412a076 · 2025-11-01T20:54:08.000+09:00
diff --git a/crates/cargo-util/src/paths.rs b/crates/cargo-util/src/paths.rs
@@ -881,11 +881,9 @@ pub fn hardlink_dir_all(src: impl AsRef<Path>, dst: impl AsRef<Path>) -> io::Res
             hardlink_dir_all(entry.path(), dst.as_ref().join(entry.file_name()))?;
         } else {
             if let Err(err) = std::fs::hard_link(entry.path(), dst.as_ref().join(entry.file_name()))
+                && !matches!(err.kind(), io::ErrorKind::AlreadyExists)
             {
-                match err.kind() {
-                    io::ErrorKind::AlreadyExists => continue,
-                    _ => return Err(err),
-                }
+                return Err(err);
             }
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -881,11 +881,9 @@ pub fn hardlink_dir_all(src: impl AsRef<Path>, dst: impl AsRef<Path>) -> io::Res`
`881`	`881`	`hardlink_dir_all(entry.path(), dst.as_ref().join(entry.file_name()))?;`
`882`	`882`	`} else {`
`883`	`883`	`if let Err(err) = std::fs::hard_link(entry.path(), dst.as_ref().join(entry.file_name()))`
	`884`	`+ && !matches!(err.kind(), io::ErrorKind::AlreadyExists)`
`884`	`885`	`{`
`885`		`- match err.kind() {`
`886`		`- io::ErrorKind::AlreadyExists => continue,`
`887`		`- _ => return Err(err),`
`888`		`- }`
	`886`	`+ return Err(err);`
`889`	`887`	`}`
`890`	`888`	`}`
`891`	`889`	`}`