From: Stefano Lattarini Date: Fri, 12 Aug 2011 09:03:46 +0000 (+0200) Subject: tap: improve some comments in the TAP driver X-Git-Tag: ng-0.5a~89^2~101^2~10 X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=cc485aba6babbe23c0b67eb28eb6181f18fedb98;p=thirdparty%2Fautomake.git tap: improve some comments in the TAP driver * lib/tap-driver (handle_tap_plan): Improve comments describing possible errors in a "plan with SKIP" directive. --- diff --git a/ChangeLog b/ChangeLog index 9a1d6011e..d346d05a2 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,9 @@ +2011-08-12 Stefano Lattarini + + tap: improve some comments in the TAP driver + * lib/tap-driver (handle_tap_plan): Improve comments describing + possible errors in a "plan with SKIP" directive. + 2011-08-12 Stefano Lattarini tap: non-zero exit status after "Bail out!" should not be reported diff --git a/lib/tap-driver b/lib/tap-driver index b7a8ffbe3..0f4eb84d4 100755 --- a/lib/tap-driver +++ b/lib/tap-driver @@ -369,12 +369,12 @@ sub handle_tap_plan ($) # So, if we find it after having already seen at least one TAP result, # set a flag signaling that no more TAP results are acceptable. $tap_stopped = 1 if $testno >= 1; - # If the plan contains a SKIP directive, and it's not an error, we - # want to report it as a particular kind of SKIP result. - # If "$testno > 0", we have is an error that will be automatically - # dealt with later, so don't worry about it here. - # If "$plan_seen" is true, we have an error due to a repeated plan, - # and that has already been dealt with above. + # If $testno > 0, we have an error ("too many tests run") that will be + # automatically dealt with later, so don't worry about it here. If + # $plan_seen is true, we have an error due to a repeated plan, and that + # has already been dealt with above. Otherwise, we have a valid "plan + # with SKIP" specification, and should report it as a particular kind + # of SKIP result. if ($plan->directive && $testno == 0 && !$plan_seen) { my $explanation = $plan->explanation ?