develooper Front page | perl.perl5.porters | Postings from August 2009

Re: [perl #68458] 5.10.1: regression regarding bareword in readline()

Thread Previous
From:
Rafael Garcia-Suarez
Date:
August 23, 2009 06:32
Subject:
Re: [perl #68458] 5.10.1: regression regarding bareword in readline()
Message ID:
b77c1dce0908230631g4dd04ecbxc9cf2b6d952ddcc7@mail.gmail.com
2009/8/13 Nicholas Clark <nick@ccl4.org>:
> On Wed, Aug 12, 2009 at 07:51:25PM -0400, jesse wrote:
>>
>>
>>
>> On Wed, Aug 12, 2009 at 11:54:58AM -0700, eserte@biokovo-amd64.herceg.de (via RT) wrote:
>> > This is a bug report for perl from eserte@biokovo-amd64.herceg.de,
>> > generated with the help of perlbug 1.39 running under perl 5.10.1.
>> >
>>
>> Failing tests for this issue were added in commit cb4bdfcf.
>
> And as they're not likely to get fixed in blead within the next few weeks,
> let alone the next few days* I marked them as TODO in 972ee0bb
>
> Nicholas Clark
>
> * Go on list subscribers, prove me wrong.

Ok. The following patch should make everyone happy...

commit 9a073a1d1a270eb53990cb82ec7af8ae00d667c4
Author: Rafael Garcia-Suarez <rgarciasuarez@gmail.com>
Date:   Sun Aug 23 15:28:22 2009 +0200

    Fix parsing of readline(FH) [perl #68458]

    This was broken by commit 984f9f66477bc722578262ae8520584e44e881af,
    which was committed to solve bug [perl #53806].

    Basically, to make everyone happy, we need only to enforce strictures
    on barewords that follow the specially parsed keywords "print" and
    "exec" (and similar) -- except when they have been already parsed.

Thread Previous


nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About