[PATCH 2/2] Perl coverage support using Devel::Cover.

2009-10-18 Thread Ralf Wildenhues
With this, you can run make -k check-coverage and on the next day, you might have useful coverage data. :-) Well, if something failed, you might still need to follow up with make check-coverage-report Pausing for some coffee is still worthwhile at this point. For a reasonably sane

[PATCH 0/5] Perl module coverage

2009-10-18 Thread Ralf Wildenhues
I'm applying this patch series to the perl-coverage branch and merging it into master. It improves coverage for Version.pm, Wrap.pm, Condition.pm, and DisjConditions.pm, where now, only statements aren't counted as covered which consist of code used with ithreads only. The last patch is a very

[PATCH 1/5] Coverage for Version.pm.

2009-10-18 Thread Ralf Wildenhues
* lib/Automake/tests/Version.pl (test_version_compare): Also try Automake::Version::check for the version pairs, taking into account the special-case naming of code forks. (@tests): Add more test cases. (test_bad_versions, @bad_versions): New function, new test cases, to ensure bad version strings

[PATCH 2/5] Coverage for Wrap.pm.

2009-10-18 Thread Ralf Wildenhues
* lib/Automake/tests/Wrap.pl (@tests): Add test for word with trailing space. (test_makefile_wrap, @makefile_tests): New function, new list of tests, to test makefile_wrap. Signed-off-by: Ralf Wildenhues ralf.wildenh...@gmx.de --- ChangeLog |6 ++

[PATCH 3/5] Coverage and fixes for Condition.pm.

2009-10-18 Thread Ralf Wildenhues
* lib/Automake/Condition.pm (new): Catch common programming errors better by checking type of passed argument before munging them to all be strings through split. * lib/Automake/tests/Condition.pl (test_basics): Also test -human. (test_merge): New function, test -merge, -merge_conds, -strip. *

[PATCH 4/5] Coverage for DisjConditions.pm.

2009-10-18 Thread Ralf Wildenhues
* lib/Automake/tests/DisjConditions.pl (test_basics): Increase test coverage: test -human, -merge, -simplify, -multiply. * lib/Automake/tests/DisjConditions-t.pl (test_basics): Likewise changes, but including state copies across thread creation. * lib/Automake/tests/DisjCon2.pl: New test. *

[PATCH 5/5] Simplify Variable::_check_ambiguous_condition.

2009-10-18 Thread Ralf Wildenhues
* lib/Automake/Variable.pm (_check_ambiguous_condition): No need to check for $def since ambiguous_p returns an empty $message if there is no other condition which is ambiguous to $cond. Signed-off-by: Ralf Wildenhues ralf.wildenh...@gmx.de --- ChangeLog|5 +