-----BEGIN PGP SIGNED MESSAGE----- Moin,
On Thursday 29 April 2004 21:57, Paul Johnson wrote: > > Exactly! Especially the ability to look at > > > > http://pjcj.sytes.net/cpancover/Math-BigRat-0.12/lib-Math-BigRat-pm.htm > >l > > > > without having the "hassle" of downloading/installing Devel::Cover is a > > big plus! I just saw that BigRat misses tests for as_bin() and as_hex() > > :-) > > > > Thank you! > > You're welcome. But be aware that cpancover is probably best used as a > guide. You'll want to run the tests yourself and generate your own > coverage so you can be sure the tests pass and that you understand the > environment in which they are being run. (Tels, I know you do that > anyway.) Well, actually I might not do the coverage due to extremely limited time ( :-) but I do look at the code and the tests closely and generate some tests and doc in the hope to increase the coverage. Of course, compared to some other stuff, BigInt is already in the top-notch category :) > > Tels > > > > PS: One stupid question: > > > > In Calc.pm (part of BigInt): > > > > 1350 149 50 408 if ($cx->[0] == 0) > > 1351 { > > 1352 0 0 $zero_elements ++; shift @$cx; > > > > If the branch is taken 50% of the time (I think that is what 50 means > > in the second column, then why is the line "$zero_elements++" etc not > > covered at all? Bug in Devel::Cover or did I get something wrong? > > The latter ;-) > > When branch coverage is 50% that means that 50% of the branches have > been taken at least once, and thus 50% of the branches have never been > taken. If you follow the 50 link you will get to the detailed branch > coverage report which tells you that the true branch (the body of the > if statement) was never taken, but the false branch was (the missing > else). So one out of two, or 50% branch coverage. This fits in with > the statement coverage for the statements in the body of the if > statement being zero. Ah yes. (I missed the link to the branch statement page. Cool) The branch coverage is surely confusing to the unknowing :) Cheers, Tels - -- Signed on Thu Apr 29 23:51:51 2004 with key 0x93B84C15. Visit my photo gallery at http://bloodgate.com/photos/ PGP key on http://bloodgate.com/tels.asc or per email. "I can imagine what you're thinking though: this girl keeps her brains in her backside! But actually only the more primitive parts of my brain are in my butt. The more interesting parts are kept in a PC - my spinal cord is actually an RS232 lead!" - Lucy, the OrangUtan Robot Girl http://tinyurl.com/3fv6z -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.2.2-rc1-SuSE (GNU/Linux) Comment: When cryptography is outlawed, bayl bhgynjf jvyy unir cevinpl. iQEVAwUBQJF5KXcLPEOTuEwVAQH2Cgf9F0T8mCVNWltyisvITIkG8wdTCqi9e+rU Ug3Axh0H5YJ//SJFhAFc1f7XpF8NVw1O7Hc97OWMHwrMPzLC1jarpnDZhw6yP2sq C2hmDg2EVaF4TXjhIqwf0TEwiF8S593TVf4s2bTJcBw34Auzb+12XB3VZfBTMo6K 0gXti3hHZhCR61EOxxi3GG6C5CJozubriWSiFlEUwXcHvRjj81qPjXvPY2MRAOu2 sLrjyzzQ6oan8Ok5JFO8r/gqoI9145JxbcgEsotrVShbTNe4+xv3oAeaahtscKCe lWe/5amZA/aIPAoCZ1q8qNC7sbZMwwpfLFND9qF9z7pcCvaGV018LQ== =NDRj -----END PGP SIGNATURE-----
