Address remarks made

quattor · Aug 13, 2015 · bf24658 · bf24658
1 parent c641b1b
commit bf24658
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/src/main/conf/quattor-ccm b/src/main/conf/quattor-ccm
@@ -12,7 +12,7 @@ _quattor_find_bin()
 {
     local dir bin fn
     fn="$1"
-    # use the "slow" sed here
+
     for dir in `echo $PATH | sed 's/:/ /g'`; do
         bin="$dir/$fn"
         if [ -x "$bin" ]; then

diff --git a/src/main/perl/Options.pm b/src/main/perl/Options.pm
@@ -281,8 +281,7 @@ sub add_actions
     my ($self, $newactions) = @_;
 
     while (my ($action, $help) = each %$newactions) {
-        my $method = $self->can("action_$action");
-        if($method) {
+        if($self->can("action_$action")) {
             $_actions->{$action} = $help;
         } else {
             $self->warn("Not adding non-existing action $action");