Merge pull request #5755 from sylvestre/syntax

rm: improve the code a tiny bit
uutils · Dec 31, 2023 · ed11132 · ed11132
2 parents 5cbf6c2 + b07ea95
commit ed11132
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/uu/rm/src/rm.rs b/src/uu/rm/src/rm.rs
@@ -334,7 +334,7 @@ fn handle_dir(path: &Path, options: &Options) -> bool {
  // remove_dir_all failed. maybe it is because of the permissions
  // but if the directory is empty, remove_dir might work.
  // So, let's try that before failing for real
- if let Err(_e) = fs::remove_dir(path) {
+ if fs::remove_dir(path).is_err() {
  had_err = true;
  if e.kind() == std::io::ErrorKind::PermissionDenied {
  // GNU compatibility (rm/fail-eacces.sh)