On May 25, 2010, at 3:32 PM, Paul Johnson wrote:



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?


Since the 'time' column has apparently never been documented, we could assert that it's never been part of the formal interface or promised results, and that no one should ever have relied on it for anything. In which case, we would be justified in removing it.

Alternative approaches to this might be: (a) uploading a new version to CPAN which deprecates this feature on a fixed time schedule; or (b) uploading a new version in which the time column comes only upon request via command-line option.

But, in either case, I don't think it should be part of the 'default' results unless someone is willing to document and maintain it. And I, for one, would rather have Paul (and others) focus on other aspects of Devel::Cover than this one.

Thank you very much.
Jim Keenan

Reply via email to