Skip to content

Commit

Permalink
rm: simplify process_options()
Browse files Browse the repository at this point in the history
* is_recursive() is already smart enough to check both -R and -r, so manually setting opt{"R"} is not required
* As a regression test I removed some subdirs, first using -r, then -R
  • Loading branch information
mknos authored Jan 7, 2025
1 parent 09cf50b commit 93537f8
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions bin/rm
Original file line number Diff line number Diff line change
Expand Up @@ -196,8 +196,6 @@ sub process_options {
);
usage() unless $ret;

$opts{'R'} = 1 if $opts{'r'};

$self->{options} = { map { defined $_ ? $_ : 0 } %opts };
$self->{files} = $self->{args};

Expand Down

0 comments on commit 93537f8

Please sign in to comment.