I've been helping a little with the tapx project, and was unable to get
any decent coverage.
Most modules simply reported 'n/a' as their coverage for lines, branches
etc, and
Devel::Cover's html_minimal was warning of an attempt to use an un
initialised value.
The tapx team stated that they all had the same problem, but hadn't had
time to address it.
Ovid also had the same issue with coverage reporting on App::Pgrep.
The issue was isolated(by Eric Wilhelm) to a change between 0.55 and
0.56 of D::C,
so I dove in and diff'ed the two releases to see if I could resolve
the issue. Below is a patch that seems to restore coverage reporting in
0.61
--- Devel-Cover-0.61-bad/lib/Devel/Cover/DB.pm 2007-01-11
06:42:48.000000000 +1100
+++ Devel-Cover-0.61-good/lib/Devel/Cover/DB.pm 2007-08-12
01:03:18.000000000 +1000
@@ -708,7 +708,7 @@
while (my ($criterion, $fc) = each %$f)
{
my $get = "get_$criterion";
- my $sc = $st->$get($digests{$digest});
+ my $sc = $st->$get($file);
# print STDERR "$criterion: ", Dumper $sc, $fc;
next unless $sc; # TODO - why?
my $cc = $cf->{$criterion} ||= {};
Note that I havent tried to work out what is wrong with the digests per
se,
I stopped when I got reports again - D::C is a mighty work of kung fu,
and
I dont think I could fiddle the digests with any knowledgable poking
anyway.
If your not having a problem with D::C, I dont think you need worry -
D::C's
test harness passes without issue with and without this patch, so its a
subtle
bug - but if your getting 'n/a''s across your coverage reports, maybe
this'll
help.
Should I also send this patch to RT for D::C ?
L
**********************************************************************
IMPORTANT
The contents of this e-mail and its attachments are confidential and intended
solely for the use of the individual or entity to whom they are addressed. If
you received this e-mail in error, please notify the HPA Postmaster, [EMAIL
PROTECTED],
then delete the e-mail.
This footnote also confirms that this e-mail message has been swept for the
presence of computer viruses by Ironport. Before opening or using any
attachments, check them for viruses and defects.
Our liability is limited to resupplying any affected attachments.
HPA collects personal information to provide and market our services. For more
information about use, disclosure and access see our Privacy Policy at
www.hpa.com.au
**********************************************************************
