Address remarks made

quattor · Aug 16, 2015 · cf126aa · cf126aa
1 parent c641b1b
commit cf126aa
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 5 deletions.
diff --git a/src/main/bin/ccm b/src/main/bin/ccm
@@ -54,7 +54,7 @@ ccm [I<OPTIONS>]
 
 =head1 OPTIONS
 
-For the list op options, check the C<EDG::WP4::CCM::CLI> and
-C<EDG::WP4::CCM::Options> documentation.
+For the list of options, check the L<EDG::WP4::CCM::CLI> and
+L<EDG::WP4::CCM::Options> documentation.
 
 =cut
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");