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};
}
###########################################################################