MailScanner --lint errors

[Julian Field]

Dimitri Yioulos wrote:
> On Monday 07 April 2008 10:31 am, Kai Schaetzl wrote:
>   
>> Dimitri Yioulos wrote on Mon, 7 Apr 2008 07:53:56 -0400:
>>     
>>> I didn't get that with mailscanner-4.65.3-1, my last version before
>>> upgrading to the latest.
>>>       
>> sounds rather like a problem with SA. Did you also upgrade SA? You may be
>> missing a now required Perl module. A timing package?
>>
>> Kai
>>
>> --
>> Kai Schätzl, Berlin, Germany
>> Get your web at Conactive Internet Services: http://www.conactive.com
>>     
>
> Kai,
>
> I'm running the latest (RPM) version of SA - 3.2.4.  All of the perl SA 
> modules look to be up-to-date, too.
>
> I took a look at /usr/lib/perl5/vendor_perl/5.8.0/Mail/SpqamAssassin/Dns.pm 
> line 371, which read "$total_waiting_time += $waiting_time".  Just for fun, I 
> deleted the +, ran MS --lint, and the first error was gone.  However, I'm not 
> sure if the "+=" isn't a valid construct, and what the consequences of my 
> change would be (and so may just put back the +).
>   
You can't just remove "+" signs like that, sorry!
That line is a shorthand for
$total_waiting_time = $total_waiting_time + $waiting_time;
> As to the second error, I see the line in Dns.pm, but have no idea what it 
> does.  Googling has turned up little.
>
> Dimitri
>
>   

Jules

-- 
Julian Field MEng CITP CEng
www.MailScanner.info
Buy the MailScanner book at www.MailScanner.info/store

MailScanner customisation, or any advanced system administration help?
Contact me at Jules at Jules.FM

PGP footprint: EE81 D763 3DB0 0BFD E1DC 7222 11F6 5947 1415 B654
PGP public key: http://www.jules.fm/julesfm.asc


-- 
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.