https://issues.apache.org/SpamAssassin/show_bug.cgi?id=5934





--- Comment #3 from Jorge Valdes <[email protected]>  2008-12-18 08:10:24 PST ---
Following Karsten's lead, here is a modification to the patch from Daniel,
instead of comparing $cc several times,

    $crosscountries +=1 if (($lcc ne '') and ($lcc ne $cc) and ($cc !~
/XX|\*\*/));
    $countriesvisited{$cc}++ if $cc !~ /XX|\*\*/;
    $lcc = $cc if $cc !~ /XX|\*\*/;

why not just doit once?

    if ($cc !~ /XX|\*\*/) {
      $crosscountries += 1 if (($lcc ne '') and ($lcc ne $cc));
      $countriesvisited{$cc}++;
      $lcc = $cc;
    }


-- 
Configure bugmail: 
https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

Reply via email to