From: Vincent Pit <perl@profvince.com>
Date: Tue, 1 Sep 2009 10:59:31 +0000 (+0200)
Subject: _run_test() doesn't need to conform to TAP::Harness exec callback interface anymore
X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=commitdiff_plain;h=999051eb7c6ae65cb741429d2168efaf932cecb4;p=p5sagit%2Fp5-mst-13.2.git

_run_test() doesn't need to conform to TAP::Harness exec callback interface anymore
---

diff --git a/t/TEST b/t/TEST
index 8f6882b..e293a5c 100755
--- a/t/TEST
+++ b/t/TEST
@@ -268,15 +268,11 @@ sub _after_fork {
 }
 
 sub _run_test {
-    my($harness, $test, $type) = @_;
-    if (!defined $type) {
-	# To conform to the interface expected by exec in TAP::Harness
-	$type = 'perl';
-    }
+    my ($test, $type) = @_;
 
     my $options = _scan_test($test, $type);
-
-    $test = $options->{test}; # Might have changed if we're in ext/Foo
+    # $test might have changed if we're in ext/Foo, so don't use it anymore
+    # from now on. Use $options->{test} instead.
 
     _before_fork($options);
 
@@ -486,7 +482,7 @@ EOT
 	    $te = '';
 	}
 
-        my $results = _run_test(undef, $test, $type);
+	my $results = _run_test($test, $type);
 
 	my $failure;
 	my $next = 0;