[build] Simplify use of Getopt::Long in padimg.pl
authorMichael Brown <mcb30@etherboot.org>
Thu, 16 Apr 2009 05:13:09 +0000 (06:13 +0100)
committerMichael Brown <mcb30@etherboot.org>
Thu, 16 Apr 2009 05:13:09 +0000 (06:13 +0100)
Sometimes it's just so much fun doing things the complicated way that
you forget to check the man page for the existence of a simpler
syntax.

src/util/padimg.pl

index 71d0eee..4421aaf 100755 (executable)
@@ -9,16 +9,15 @@ my $verbosity = 0;
 my $blksize = 512;
 my $byte = 0;
 
-my $opts = {
+my %opts = (
   'verbose|v+' => sub { $verbosity++; },
   'quiet|q+' => sub { $verbosity--; },
   'blksize|s=o' => sub { $blksize = $_[1]; },
   'byte|b=o' => sub { $byte = $_[1]; },
-};
+);
 
 Getopt::Long::Configure ( 'bundling', 'auto_abbrev' );
-GetOptions ( { map { /^(\w+)/; $1 => $opts->{$_} } keys %$opts }, keys %$opts )
-    or die "Could not parse command-line options\n";
+GetOptions ( %opts ) or die "Could not parse command-line options\n";
 
 while ( my $filename = shift ) {
   die "$filename is not a file\n" unless -f $filename;