Google Groups no longer supports new Usenet posts or subscriptions. Historical content remains viewable.
Dismiss

[perl #39826] [CAGE] convert C<perl -w> to C<use warnings;>

8 views
Skip to first unread message

Jerry Gay

unread,
Jul 13, 2006, 2:12:00 PM7/13/06
to bugs-bi...@rt.perl.org
# New Ticket Created by Jerry Gay
# Please include the string: [perl #39826]
# in the subject line of all future correspondence about this issue.
# <URL: http://rt.perl.org/rt3/Ticket/Display.html?id=39826 >


there are many scripts (ack reports 189, below) that contain 'perl -w'.

D:\usr\local\parrot\trunk>ack --perl "perl -w" -l | perl -nle"}END{print $."
189

our minimum required perl version is 5.6.2. it's time these get be
converted to 'use warnings;', except where the '-w' is appropriate
(eg. one-liners)
~jerry

Jerry Gay

unread,
Jul 24, 2006, 1:22:15 PM7/24/06
to parrotbug...@parrotcode.org
On 7/16/06, Guest via RT <parrotbug...@parrotcode.org> wrote:
> Hi,
>
> I've attached a patch for the same.
>
> Please review and if it looks fine, can you please apply the same to trunk?
>
looks good. non-languages/ files have been patched and applied as
r13505. i'll follow up with the language authors, and apply to those
seperately.

> Thanks!
> Swaroop
>
thank you. keep up the good work!
~jerry

Paul Cochrane via RT

unread,
Nov 26, 2006, 4:11:16 PM11/26/06
to perl6-i...@perl.org
On Thu Jul 13 11:11:59 2006, particle wrote:
> there are many scripts (ack reports 189, below) that contain 'perl
> -w'.
>
> D:\usr\local\parrot\trunk>ack --perl "perl -w" -l | perl
> -nle"}END{print $."
> 189
>
> our minimum required perl version is 5.6.2. it's time these get be
> converted to 'use warnings;', except where the '-w' is appropriate
> (eg. one-liners)
> ~jerry

A new Perl::Critic policy has been added which checks for 'perl -
w' (TestingAndDebugging::ProhibitShebangWarningsArg). The source now
passes this test as of revisions: r15861-8

Paul


0 new messages