Bug 428 - add "-w" flag to top line of mass-check and other front-end scripts
Summary: add "-w" flag to top line of mass-check and other front-end scripts
Status: RESOLVED FIXED
Alias: None
Product: Spamassassin
Classification: Unclassified
Component: spamassassin (show other bugs)
Version: SVN Trunk (Latest Devel Version)
Hardware: All All
: P2 major
Target Milestone: ---
Assignee: Craig Hughes
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2002-06-11 15:51 UTC by Daniel Quinlan
Modified: 2002-06-14 03:48 UTC (History)
0 users


Attachment Type Modified Status Actions Submitter/CLA Status
patch to add -w to mass-check (others front-ends may need it too, but I don't use them... yet) patch None Daniel Quinlan [HasCLA]
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Quinlan 2002-06-11 15:51:18 UTC 
Malte S. Stretz writes:

> Subject: [Bug 420] raises warning
> [...]
> | "my" variable @lines masks earlier declaration in same scope at
> | ../lib/Mail/SpamAssassin/EvalTests.pm line 1254.
> This one should fix it. Maybe SpamAssassin.pm should 'use warnings'...

(Note: I just fixed the typo.)

'use warnings' is left out of SpamAssassin.pm because it was not
supported in perl 5.005.

I didn't get the warning because I was running "mass-check" instead of
"spamassassin".  Maybe someone should add a "-w" to the top line of
mass-check and all the other front-end scripts.
Comment 1 Daniel Quinlan 2002-06-11 16:10:36 UTC 
Created attachment 165 [details]
patch to add -w to mass-check (others front-ends may need it too, but I don't use them... yet)
Comment 2 Craig Hughes 2002-06-14 11:48:59 UTC 
Thanks.  Committed for 2.30.