Skip Menu |
 

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Devel-Cover CPAN distribution.

Report information
The Basics
Id: 16379
Status: resolved
Priority: 0/
Queue: Devel-Cover

People
Owner: Nobody in particular
Requestors: tylerm [...] activestate.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: 0.55
Fixed in: (no value)



Subject: Html_minimal gets confused by refactoring
Download (untitled) / with headers
text/plain 1.2k
Hi, I just removed a some branches from my code and re-ran devel::cover on my unit tests. The HTML output showed a lower percentage of coverage than what actually happened, and I got the following output when I ran "cover": Use of uninitialized value in concatenation (.) or string at /usr/local/lib/perl/5.8.7/Devel/Cover/Report/Html_minimal.pm line 530, <$in> line 145. Use of uninitialized value in concatenation (.) or string at /usr/local/lib/perl/5.8.7/Devel/Cover/Report/Html_minimal.pm line 530, <$in> line 146. Use of uninitialized value in concatenation (.) or string at /usr/local/lib/perl/5.8.7/Devel/Cover/Report/Html_minimal.pm line 530, <$in> line 146. Use of uninitialized value in concatenation (.) or string at /usr/local/lib/perl/5.8.7/Devel/Cover/Report/Html_minimal.pm line 530, <$in> line 146. Use of uninitialized value in concatenation (.) or string at /usr/local/lib/perl/5.8.7/Devel/Cover/Report/Html_minimal.pm line 530, <$in> line 146. Use of uninitialized value in concatenation (.) or string at /usr/local/lib/perl/5.8.7/Devel/Cover/Report/Html_minimal.pm line 530, <$in> line 147. After removing the 'cover_db' directory and running again, I got the expected results. Cheers, Tyler
I know it's a litle late, but do you have any code I could use to recreate this problem?
Subject: cover gets confused by TAP::Parser
RT-Send-CC: tapx-dev [...] hexten.net
Download (untitled) / with headers
text/plain 740b
On Wed Aug 02 07:14:53 2006, PJCJ wrote: Show quoted text
> I know it's a litle late, but do you have any code I could use to > recreate this problem?
I do :-D I've checked 0.50 thru 0.61 against TAP::Parser and found that the 'uninitialized value' trouble starts at 0.56 and continues thereafter. I'm using r212 of TAP::Parser from http://svn.hexten.net/tapx/trunk/, though you should be able to replicate it with the latest tarball off of CPAN. You can replicate this experiment with the bash code in the first link, or simply grab my stderr/stdout outputs from the second link. http://scratchcomputing.com/tmp/dc.binary_search.txt http://scratchcomputing.com/tmp/devel-cover-errs.tar.gz Thanks, Eric
From: LEIF [...] cpan.org
Download (untitled) / with headers
text/plain 814b
OK I have a patch --- 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} ||= {}; Maybe there is something wrong with the digest entries, but I'll leave that to Paul. I made this patch to D::C 0.61 and the test harness still passes - this patch actually reverts this line to back how it was in D::C 0.55.
Download (untitled) / with headers
text/plain 1.1k
On Thu Aug 09 15:16:36 2007, EWILHELM wrote: Show quoted text
> On Wed Aug 02 07:14:53 2006, PJCJ wrote:
> > I know it's a litle late, but do you have any code I could use to > > recreate this problem?
> > I do :-D > > I've checked 0.50 thru 0.61 against TAP::Parser and found that the > 'uninitialized value' trouble starts at 0.56 and continues thereafter. > > I'm using r212 of TAP::Parser from http://svn.hexten.net/tapx/trunk/, > though you should be able to replicate it with the latest tarball off > of CPAN. You can replicate this experiment with the bash code in the > first link, or simply grab my stderr/stdout outputs from the second > link. > > http://scratchcomputing.com/tmp/dc.binary_search.txt > http://scratchcomputing.com/tmp/devel-cover-errs.tar.gz > > Thanks, > Eric
Eric, In advance of this year's Perl QA hackathon, I was reviewing older tickets in Devel::Cover's rt.cpan.org queue (it now uses https://github.com/pjcj/Devel--Cover/issues) and came across this one. The links mentioned above give 404s. Are you still experiencing problems with Devel::Cover which would warrant keeping this ticket open? Thank you very much. Jim Keenan
Download (untitled) / with headers
text/plain 255b
Sorry I let this ticket lie for so long. I believe that the current code has solved this problem, and so I'm closing this ticket. If anyone can reproduce the problem with the current code, please feel free to reopen the ticket. Thanks to all concerned,


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.