develooper Front page | perl.perl5.porters | Postings from September 2012

Re: [perl #113684] Bleadperl v5.17.0-326-g6a31dbf breaksPJCJ/Devel-Cover-0.89.tar.gz

Thread Previous | Thread Next
From:
Paul Johnson
Date:
September 17, 2012 17:26
Subject:
Re: [perl #113684] Bleadperl v5.17.0-326-g6a31dbf breaksPJCJ/Devel-Cover-0.89.tar.gz
Message ID:
20120918002642.GM16381@pjcj.net
On Thu, Aug 02, 2012 at 09:57:45PM +0200, Paul Johnson wrote:
> On Thu, Aug 02, 2012 at 12:46:33PM -0700, Father Chrysostomos via RT wrote:
> > On Thu Aug 02 10:54:27 2012, paul@pjcj.net wrote:
> > > On Fri, Jul 27, 2012 at 04:29:54PM -0700, Father Chrysostomos via RT
> > wrote:
> > > 
> > > > In commit 1f039d60d3, I have fixed loop exits to accept computed labels,
> > > > documented it in 8a7e748e29.
> > > > 
> > > > In commit 1eb0b7be2f, I have made B::Deparse deparse them with the
> > > > correct precedence.
> > > 
> > > Thanks for doing this.
> > > 
> > > So now we have:
> > > 
> > > $ perl -MO=Deparse -e '$a = $b || next'
> > > $a = $b || (next);
> > > 
> > > Those parentheses are new (from 5.16.0) and superfluous.  Are they there
> > > by design?
> > > 
> > > For reference:
> > > 
> > > $ perl -MO=Deparse -e '$a = $b || return'
> > > $a = $b || (return);
> > > 
> > > Those parentheses first appeared in 5.14.0.
> > 
> > Please see my explanation at
> > <https://rt.perl.org/rt3/Ticket/Display.html?id=106892#txn-1043534>. 
> > The same applies here.  I don’t want to rewrite B::Deparse completely
> > just yet. :-)
> 
> Thanks for the explanation (again).  As soon as 5.17.3 comes out I'll
> fix up Devel::Cover to take this into account.
> 
> Thanks again,

Just to close this out, this is fixed in Devel::Cover 0.94 with 5687c8d.

(I *knew* I shouldn't have written "As soon as".)

-- 
Paul Johnson - paul@pjcj.net
http://www.pjcj.net

Thread Previous | Thread Next


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