You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spamassassin.apache.org by qu...@apache.org on 2004/10/26 20:18:28 UTC

svn commit: rev 55629 - spamassassin/trunk/lib/Mail/SpamAssassin

Author: quinlan
Date: Tue Oct 26 11:18:27 2004
New Revision: 55629

Modified:
   spamassassin/trunk/lib/Mail/SpamAssassin/EvalTests.pm
Log:
a few tiny speed-ups - kinda lame, actually


Modified: spamassassin/trunk/lib/Mail/SpamAssassin/EvalTests.pm
==============================================================================
--- spamassassin/trunk/lib/Mail/SpamAssassin/EvalTests.pm	(original)
+++ spamassassin/trunk/lib/Mail/SpamAssassin/EvalTests.pm	Tue Oct 26 11:18:27 2004
@@ -969,10 +969,10 @@
   $addr = lc $addr;
   my $found_forged = 0;
   foreach my $white_addr (keys %{$list}) {
-    my $regexp = $list->{$white_addr}{re};
+    my $regexp = qr/$list->{$white_addr}{re}/i;
     foreach my $domain (@{$list->{$white_addr}{domain}}) {
       
-      if ($addr =~ qr/${regexp}/i) {
+      if ($addr =~ $regexp) {
         foreach my $lastunt (@relays) {
           my $rdns = $lastunt->{lc_rdns};
           if ($rdns =~ /(?:^|\.)\Q${domain}\E$/) { return 1; }
@@ -1149,13 +1149,8 @@
       $count{$token}++;
     }
   }
-  my $unique = 0;
-  my $repeat = 0;
-  for my $count (values %count) {
-    $count == 1 ? $unique++ : $repeat++;
-  }
-  $self->{unique_words_repeat} = $repeat;
-  $self->{unique_words_unique} = $unique;
+  $self->{unique_words_unique} = scalar grep { $_ == 1 } values(%count);
+  $self->{unique_words_repeat} = scalar keys(%count) - $self->{unique_words_unique};
 }
 
 ###########################################################################