You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spamassassin.apache.org by jm...@apache.org on 2007/09/20 22:30:22 UTC
svn commit: r577906 -
/spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/OneLineBodyRuleType.pm
Author: jm
Date: Thu Sep 20 13:30:22 2007
New Revision: 577906
URL: http://svn.apache.org/viewvc?rev=577906&view=rev
Log:
remove unused loopid var
Modified:
spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/OneLineBodyRuleType.pm
Modified: spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/OneLineBodyRuleType.pm
URL: http://svn.apache.org/viewvc/spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/OneLineBodyRuleType.pm?rev=577906&r1=577905&r2=577906&view=diff
==============================================================================
--- spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/OneLineBodyRuleType.pm (original)
+++ spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/OneLineBodyRuleType.pm Thu Sep 20 13:30:22 2007
@@ -76,7 +76,6 @@
sub do_one_line_body_tests {
my ($self, $pms, $priority) = @_;
- my $loopid = 0;
# TODO: should have a consttype for plugin-defined "alien" rule types,
# probably something like TYPE_ALIEN_TESTS. it's only used as a key
@@ -90,12 +89,11 @@
loop_body => sub
{
my ($self, $pms, $conf, $rulename, $pat, %opts) = @_;
- my $sub;
$pat = untaint_var($pat);
+ my $sub;
if (($conf->{tflags}->{$rulename}||'') =~ /\bmultiple\b/)
{
- $loopid++; # support multiple matches
$sub = '
pos $_[1] = 0;
'.$self->hash_line_for_rule($pms, $rulename).'