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 2007/09/19 14:09:27 UTC

svn commit: r577267 - /spamassassin/trunk/masses/mass-check

Author: jm
Date: Wed Sep 19 05:09:26 2007
New Revision: 577267

URL: http://svn.apache.org/viewvc?rev=577267&view=rev
Log:
revert last change, Daryl says it won't work and I believe him

Modified:
    spamassassin/trunk/masses/mass-check

Modified: spamassassin/trunk/masses/mass-check
URL: http://svn.apache.org/viewvc/spamassassin/trunk/masses/mass-check?rev=577267&r1=577266&r2=577267&view=diff
==============================================================================
--- spamassassin/trunk/masses/mass-check (original)
+++ spamassassin/trunk/masses/mass-check Wed Sep 19 05:09:26 2007
@@ -1167,23 +1167,17 @@
 
   # Generate an archive in the temp file
   foreach my $num (@tosend) {
-    my $data = $msgsout->{$num}->{'data'};
-    my $msg;
-    if (!$paths_only) {
-      $msg = ($iter->_run_message($data))[4];
-      if (!$msg) {
-        warn "mass-check: _run_message returned undef! skipping";
-        next;
-      }
-    }
-
     # Archive format, gzip compressed file w/ 3 parts per message:
     # 1- server message number in text format
     # 2- server index string, binary packed format
     # 3- message content -- unless paths_only
     send_line($gzfd, $num) || die "mass-check: error when writing to gz temp file\n";
+
+    my $data = $msgsout->{$num}->{'data'};
     send_line($gzfd, $data) || die "mass-check: error when writing to gz temp file\n";
+
     if (!$paths_only) {
+      my $msg = ($iter->_run_message($data))[4];
       send_line($gzfd, join('', @{$msg})) ||
         die "mass-check: error when writing to gz temp file\n";
     }