[Ltp-coverage] Issue when doing a merge of info files with genhtml

View: New views

2 Messages — Rating Filter: Alert me

[Ltp-coverage] Issue when doing a merge of info files with genhtml

by Arnaud Ponchon :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Some parts of this message have been removed. Learn more about Nabble's security policy.

Hi,

I’m using genhtml to generate an html report with two different info files.

Unfortunately it’s not working as I’m getting this issue: “Can't use an undefined value as a HASH reference at /usr/bin/genhtml line 1506”

Looking quickly through the genhtml perl script code and through my info files I noticed that the issue was due to an SF entry containing no FN: and FNDA: entries as you can see below:

TN:

SF:/usr/include/c++/4.3/iostream

DA:77,2

LF:1

LH:1

end_of_record

I made a quick fix on my machine to be able to generate my html report by adding a check on $funcdata1 to make sure it was defined before merging the funcdata (in combine_info_entries function). As I’m not a perl expert as well as a lcov/genhtml expert I would like to know:

· Is this a known issue?

· Is this the correct way to fix it?

Thanks

Regards,

Arnaud

PS: You have the same issue using lcov –a options as the code is similar.

Arnaud PONCHON

Software Engineer

Open-Plug

2600 Route des Cretes, 06903 Sophia Antipolis Cedex, France

Email: thomas.watt@...

Tel: +33 4 97 24 58 66

__________ Information from ESET NOD32 Antivirus, version of virus signature database 4565 (20091102) __________

The message was checked by ESET NOD32 Antivirus.

http://www.eset.com

------------------------------------------------------------------------------
Come build with us! The BlackBerry(R) Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay
ahead of the curve. Join us from November 9 - 12, 2009. Register now!
http://p.sf.net/sfu/devconference
_______________________________________________
Ltp-coverage mailing list
Ltp-coverage@...
https://lists.sourceforge.net/lists/listinfo/ltp-coverage

Re: [Ltp-coverage] Issue when doing a merge of info files with genhtml

by Peter Oberparleiter-4 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Hello,

Arnaud Ponchon wrote:
> Unfortunately it's not working as I'm getting this issue: "Can't use an
> undefined value as a HASH reference at /usr/bin/genhtml line 1506"
>
> . Is this a known issue?
>
> . Is this the correct way to fix it?
>

This is a known issue with lcov 1.7. It has been fixed in the current
CVS version of lcov but there hasn't been any lcov release since (should
be coming soon). The fix in the CVS repository looks like this:

--- utils/analysis/lcov/bin/genhtml 1.24
+++ utils/analysis/lcov/bin/genhtml 1.25
@@ -1503,7 +1503,9 @@ sub merge_func_data($$$)
my %result;
my $func;

- %result = %{$funcdata1};
+ if (defined($funcdata1)) {
+ %result = %{$funcdata1};
+ }

foreach $func (keys(%{$funcdata2})) {
my $line1 = $result{$func};
@@ -1535,7 +1537,9 @@ sub add_fnccount($$)
my $fn_hit;
my $function;

- %result = %{$fnccount1};
+ if (defined($fnccount1)) {
+ %result = %{$fnccount1};
+ }
foreach $function (keys(%{$fnccount2})) {
$result{$function} += $fnccount2->{$function};
}

You can also see http://ltp.sf.net/coverage/lcov.php for instructions on
how to obtain the CVS version.

Regards,
Peter Oberparleiter

------------------------------------------------------------------------------
Come build with us! The BlackBerry(R) Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay
ahead of the curve. Join us from November 9 - 12, 2009. Register now!
http://p.sf.net/sfu/devconference
_______________________________________________
Ltp-coverage mailing list
Ltp-coverage@...
https://lists.sourceforge.net/lists/listinfo/ltp-coverage