Skip to content

Commit 8527e6c

Browse files
committed
ln: simplify remove_target closure to return nothing
1 parent d92e684 commit 8527e6c

1 file changed

Lines changed: 3 additions & 4 deletions

File tree

src/uu/ln/src/ln.rs

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -295,7 +295,7 @@ fn link_files_in_dir(files: &[PathBuf], target_dir: &Path, settings: &Settings)
295295
let mut all_successful = true;
296296
for srcpath in files {
297297
let targetpath = if settings.no_dereference && target_dir.is_symlink() {
298-
let remove_target = || -> UResult<()> {
298+
let remove_target = || {
299299
// In that case, we don't want to do link resolution
300300
// We need to clean the target
301301
if target_dir.is_file() {
@@ -318,7 +318,6 @@ fn link_files_in_dir(files: &[PathBuf], target_dir: &Path, settings: &Settings)
318318
);
319319
}
320320
}
321-
Ok(())
322321
};
323322
match settings.overwrite {
324323
OverwriteMode::NoClobber => {}
@@ -327,11 +326,11 @@ fn link_files_in_dir(files: &[PathBuf], target_dir: &Path, settings: &Settings)
327326
"{}",
328327
translate!("ln-prompt-replace", "file" => target_dir.quote())
329328
) {
330-
remove_target()?;
329+
remove_target();
331330
}
332331
}
333332
OverwriteMode::Force => {
334-
remove_target()?;
333+
remove_target();
335334
}
336335
}
337336
target_dir.to_path_buf()

0 commit comments

Comments
 (0)