Discussion:
TPF Devel::Cover grant report Week 3
Paul Johnson
2012-06-09 00:40:59 UTC
Permalink
In accordance with the terms of my grant from TPF this is the report for
week 3 of my work on improving Devel::Cover.

This report covers 02.06 to 08.06.

This was another week where I wasn't able to put in a lot of work, but I
was able to close some tickets and put out a new release.

I spent most time this week working on
https://rt.perl.org/rt3//Public/Bug/Display.html?id=113464 which isn't a
bug filed against Devel::Cover, but it is a bug when using Devel::Cover.

The problem is that in 5.16.0 Devel::Cover can throw out warnings. This
bug is related to the interaction between Devel::Cover and B::Deparse,
which Devel::Cover uses just before the program ends to visit all the
ops and collect the coverage information which was gathered as the
program ran.

The problem here is that warnings were turned on whilst parts of
B::Deparse were being called, but new code in B::Deparse was not warning
safe. I tracked the commits down to be6cf5cf and b89b7257 which were
fixes for perl #91416. I supplied a patch to quieten the warnings which
is currently awaiting application or further discussion, but in order to
guard against such problems in the future I have also turned off
warnings whilst the code in B::Deparse is being called.

I applied some patches from RT.

I also spent dome time investigating rt 63090 which I thought I had
fixed last week, but which turns out to be more complicated than I had
thought. I don't have a fix for it yet, but I have stopped the tests
failing in the new release.

Oh yes, I released Devel::Cover 0.88.


Closed RT tickets:
75633 Directory names with ++ not supported
77598 [PATCH] Fix test failures when the build directory contains regexp metacharacters.
77599 [PATCH] POD errors

You can see the commits at https://github.com/pjcj/Devel--Cover/commits/master

Hours worked:

02.06 3:15
08.06 2:20

Total 5:35
--
Paul Johnson - ***@pjcj.net
http://www.pjcj.net
Loading...