[prev in list] [next in list] [prev in thread] [next in thread] 

List:       perl-qa
Subject:    Re: Devel::Cover 'time' html column
From:       Nicholas Clark <nick () ccl4 ! org>
Date:       2010-05-26 16:04:23
Message-ID: 20100526160423.GB2576 () plum ! flirble ! org
[Download RAW message or body]

On Tue, May 25, 2010 at 09:32:44PM +0200, Paul Johnson wrote:
> On Mon, May 24, 2010 at 06:52:15AM -0400, James E Keenan wrote:

> > In any event, you needn't worry too much about that.  It may be  
> > profiling information, but if that is the sort of information you are  
> > looking for you are better off with tools specifically designed for that  
> > purpose such as Devel::Pprof and Devel NYTprof.
> 
> Certainly you should be using NYTprof for anything serious in this area,
> but it didn't exist when I first wrote this code.  Now it does, I'm
> considering removing the time coverage.  Does anyone have any thoughts
> one way or the other about that?

Devel::NYTProf does this so much better. I'd suggest removing it.


Longer term, is Devel::Cover's runtime capture doing anything that
Devel::NYTProf doesn't do these days? ie - is it possible to refactor
Devel::Cover to provide the same coverage reports, but generate them from
an nytprof.out file? If not, what's missing from the nytprof.out file?
Would this reduce the maintenance load on Devel::Cover?

Nicholas Clark
[prev in list] [next in list] [prev in thread] [next in thread]