Bug 21846: Fix pod2usage return values
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Tue, 26 Feb 2019 19:32:44 +0000 (16:32 -0300)
committerNick Clemens <nick@bywatersolutions.com>
Fri, 15 Mar 2019 12:27:44 +0000 (12:27 +0000)
According to existing pattern.
However I am not sure it makes sense to return an errno (and print to STDERR)
if --help is passed.

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>

misc/maintenance/fix_tags_weight.pl

index 549d170..7dc3801 100755 (executable)
@@ -65,15 +65,12 @@ binmode( STDOUT, ":encoding(UTF-8)" );
 my $help;
 my $verbose;
 
-my $result = GetOptions(
+GetOptions(
     'help|h'    => \$help,
     'verbose|v' => \$verbose
-);
+) or pod2usage(2);
 
-if ( not $result or $help ) {
-    pod2usage();
-    exit 0;
-}
+pod2usage(1) if $help;
 
 fix_tags_approval($verbose);
 fix_tags_index( $verbose );