You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spamassassin.apache.org by he...@apache.org on 2021/05/09 16:07:07 UTC
svn commit: r1889706 - in /spamassassin/trunk/lib/Mail/SpamAssassin: Conf.pm
Conf/Parser.pm
Author: hege
Date: Sun May 9 16:07:07 2021
New Revision: 1889706
URL: http://svn.apache.org/viewvc?rev=1889706&view=rev
Log:
Shave another 50ms from parsing
Modified:
spamassassin/trunk/lib/Mail/SpamAssassin/Conf.pm
spamassassin/trunk/lib/Mail/SpamAssassin/Conf/Parser.pm
Modified: spamassassin/trunk/lib/Mail/SpamAssassin/Conf.pm
URL: http://svn.apache.org/viewvc/spamassassin/trunk/lib/Mail/SpamAssassin/Conf.pm?rev=1889706&r1=1889705&r2=1889706&view=diff
==============================================================================
--- spamassassin/trunk/lib/Mail/SpamAssassin/Conf.pm (original)
+++ spamassassin/trunk/lib/Mail/SpamAssassin/Conf.pm Sun May 9 16:07:07 2021
@@ -5177,7 +5177,7 @@ sub maybe_header_only {
my($self,$rulename) = @_;
my $type = $self->{test_types}->{$rulename};
- if ($rulename =~ /^AUTOLEARNTEST/) {
+ if (index($rulename, 'AUTOLEARNTEST') == 0) {
dbg("config: auto-learn: $rulename - Test type is $self->{test_types}->{$rulename}.");
}
@@ -5201,7 +5201,7 @@ sub maybe_body_only {
my($self,$rulename) = @_;
my $type = $self->{test_types}->{$rulename};
- if ($rulename =~ /^AUTOLEARNTEST/) {
+ if (index($rulename, 'AUTOLEARNTEST') == 0) {
dbg("config: auto-learn: $rulename - Test type is $self->{test_types}->{$rulename}.");
}
Modified: spamassassin/trunk/lib/Mail/SpamAssassin/Conf/Parser.pm
URL: http://svn.apache.org/viewvc/spamassassin/trunk/lib/Mail/SpamAssassin/Conf/Parser.pm?rev=1889706&r1=1889705&r2=1889706&view=diff
==============================================================================
--- spamassassin/trunk/lib/Mail/SpamAssassin/Conf/Parser.pm (original)
+++ spamassassin/trunk/lib/Mail/SpamAssassin/Conf/Parser.pm Sun May 9 16:07:07 2021
@@ -261,7 +261,7 @@ sub parse {
}
# bug 6800: let X-Spam-Checker-Version also show what sa-update we are at
- if ($line =~ /^\# UPDATE version (\d+)$/) {
+ if (index($line, '# UPD') == 0 && $line =~ /^\# UPDATE version (\d+)$/) {
for ($self->{currentfile}) { # just aliasing, not a loop
$conf->{update_version}{$_} = $1 if defined $_ && $_ ne '(no file)';
}
@@ -276,7 +276,9 @@ sub parse {
next unless($line); # skip empty lines
# handle i18n
- if ($line =~ s/^lang\s+(\S+)\s+//) { next if ($lang !~ /^$1/i); }
+ if (index($line, 'lang') == 0 && $line =~ s/^lang\s+(\S+)\s+//) {
+ next if $lang !~ /^$1/i;
+ }
my($key, $value) = split(/\s+/, $line, 2);
$key = lc $key;