MailScanner --lint errors

[Dimitri Yioulos]

On Monday 07 April 2008 1:18 pm, Julian Field wrote:
> 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
>

Thanks, Jules.  I figured as much.  So that's what "+=" means.  Can these 
errors be ignored as being tivial?

Dimitri

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