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

[perl #77566] Re: Bleadperl ac56e7d breaks Devel-Cover-0.70

From:
Nicholas Clark
Date:
August 31, 2010 13:35
Subject:
[perl #77566] Re: Bleadperl ac56e7d breaks Devel-Cover-0.70
Message ID:
rt-3.6.HEAD-5116-1283256902-1895.77566-75-0@perl.org
# New Ticket Created by  Nicholas Clark 
# Please include the string:  [perl #77566]
# in the subject line of all future correspondence about this issue. 
# <URL: http://rt.perl.org/rt3/Ticket/Display.html?id=77566 >


On Tue, Aug 31, 2010 at 12:31:16PM +0200, Paul Johnson wrote:
> On Tue, Aug 31, 2010 at 04:36:21AM +0200, Andreas J. Koenig wrote:
> 
> > cpan rt ticket:
> > 
> >   https://rt.cpan.org/Ticket/Display.html?id=60901  
> > 
> > git bisect:
> > 
> >   commit ac56e7de46621c6f2e373d11984c0a0fe4839b0b
> >   Author: Nicholas Clark <nick@ccl4.org>
> >   Date:   Fri Aug 27 21:48:55 2010 +0100
> > 
> >   Peephole optimise adjacent pairs of nextstate ops.
> 
> Thanks very much for flagging this up so quickly.

Yes, thanks for keeping us honest.

> The solution is simple.  I will fix up the tests and put out a new
> release of Devel::Cover.  However, I won't do this until there has been

Thanks for doing this. (Well, thanks for it not being me doing this)

> In the meantime, anyone wanting to use Devel::Cover under bleadperl can
> simply ignore the test failures and note that their coverage has
> probably gone down as a result of this optimisation.
> 
> I read someone's blog yesterday where they had made a git commit hook
> which failed if the coverage with the commit was lower than the previous
> coverage.  They may get a surprise the first time they commit something
> under 5.14.0.

Muhahahahaha

Nicholas Clark




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