You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spamassassin.apache.org by jm...@apache.org on 2005/07/23 20:03:28 UTC

svn commit: r224490 - /spamassassin/trunk/masses/mboxget

Author: jm
Date: Sat Jul 23 11:03:27 2005
New Revision: 224490

URL: http://svn.apache.org/viewcvs?rev=224490&view=rev
Log:
minor tweak: mboxget should use proper args parsing with Getopt::Long

Modified:
    spamassassin/trunk/masses/mboxget

Modified: spamassassin/trunk/masses/mboxget
URL: http://svn.apache.org/viewcvs/spamassassin/trunk/masses/mboxget?rev=224490&r1=224489&r2=224490&view=diff
==============================================================================
--- spamassassin/trunk/masses/mboxget (original)
+++ spamassassin/trunk/masses/mboxget Sat Jul 23 11:03:27 2005
@@ -31,29 +31,25 @@
 
 sub mywarn;
 
-my $annotate = 1;
-while ($#ARGV >= 0) {
-  $_ = $ARGV[0]; shift;
-  if ($_ eq '-noannotate') { $annotate = 0; }
-  else { unshift @ARGV, $_; last; }
-}
+use Getopt::Long;
+use vars qw($opt_noannotate);
+GetOptions("noannotate");
 
-my @inputs;
-push @inputs, @ARGV;
+my $annotate = ($opt_noannotate ? 0 : 1);
 
-if (!@inputs) {
-  while (<STDIN>) {
-    s/^[^\s:]+://;  # filenames, from "grep foo *"
+my @inputs = @ARGV;
 
-    if (/^[Y\.]\s+-?\d+\s+(\S+)\s+\S+/) {
-      # mass-check format
-      handle_input($1);
-    }
-    else {
-      next if /^#/;
-      chomp;
-      handle_input($_);
-    }
+while (<>) {
+  s/^[^\s:]+://;  # filenames, from "grep foo *"
+
+  if (/^[Y\.]\s+-?\d+\s+(\S+)\s+\S+/) {
+    # mass-check format
+    handle_input($1);
+  }
+  else {
+    next if /^#/;
+    chomp;
+    handle_input($_);
   }
 }
 exit;