You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spamassassin.apache.org by fe...@apache.org on 2004/10/10 00:07:35 UTC
svn commit: rev 54185 - spamassassin/trunk/lib/Mail/SpamAssassin/BayesStore
Author: felicity
Date: Sat Oct 9 15:07:35 2004
New Revision: 54185
Modified:
spamassassin/trunk/lib/Mail/SpamAssassin/BayesStore/DBM.pm
Log:
bug 3872: when syncing the bayes journal, we really shouldn't send seen data back into a new journal file.
Modified: spamassassin/trunk/lib/Mail/SpamAssassin/BayesStore/DBM.pm
==============================================================================
--- spamassassin/trunk/lib/Mail/SpamAssassin/BayesStore/DBM.pm (original)
+++ spamassassin/trunk/lib/Mail/SpamAssassin/BayesStore/DBM.pm Sat Oct 9 15:07:35 2004
@@ -774,9 +774,13 @@
$self->defer_update ("m $seen $msgid");
}
else {
- $self->{db_seen}->{$msgid} = $seen;
+ $self->seen_put_direct($msgid, $seen);
}
}
+sub seen_put_direct {
+ my ($self, $msgid, $seen) = @_;
+ $self->{db_seen}->{$msgid} = $seen;
+}
sub seen_delete {
my ($self, $msgid) = @_;
@@ -785,9 +789,13 @@
$self->defer_update ("m f $msgid");
}
else {
- delete $self->{db_seen}->{$msgid};
+ $self->seen_delete_direct($msgid);
}
}
+sub seen_delete_direct {
+ my ($self, $msgid) = @_;
+ delete $self->{db_seen}->{$msgid};
+}
###########################################################################
# db reading APIs
@@ -1191,10 +1199,10 @@
$count++;
} elsif (/^m ([hsf]) (.+)$/) { # update msgid seen database
if ( $1 eq "f" ) {
- $self->seen_delete($2);
+ $self->seen_delete_direct($2);
}
else {
- $self->seen_put($2,$1);
+ $self->seen_put_direct($2,$1);
}
$count++;
} else {