From: Abigail <abigail@abigail.be>
Date: Wed, 1 Oct 2003 14:19:43 +0000 (+0200)
Subject: Re: [PATCH lib/warnings.t]  Adding TODO functionality to lib/warnings.
X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=commitdiff_plain;h=0b1f57502e82e7086eb37fa5981c91be8f02c535;p=p5sagit%2Fp5-mst-13.2.git

Re: [PATCH lib/warnings.t]  Adding TODO functionality to lib/warnings.
Message-ID: <20031001121943.GA29419@abigail.nl>

p4raw-id: //depot/perl@21414
---

diff --git a/lib/warnings.t b/lib/warnings.t
index 0647b55..7757173 100644
--- a/lib/warnings.t
+++ b/lib/warnings.t
@@ -76,6 +76,8 @@ for (@prgs){
         $switch =~ s/(-\S*[A-Z]\S*)/"$1"/ if $Is_VMS; # protect uc switches
     }
     my($prog,$expected) = split(/\nEXPECT\n/, $_);
+    my ($todo, $todo_reason);
+    $todo = $prog =~ s/^#\s*TODO(.*)\n//m and $todo_reason = $1;
     if ( $prog =~ /--FILE--/) {
         my(@files) = split(/\n--FILE--\s*([^\s\n]*)\s*\n/, $prog) ;
 	shift @files ;
@@ -177,12 +179,21 @@ for (@prgs){
 			 (!$option_regex && $results !~ /^\Q$expected/))) or
 	   (!$prefix && (( $option_regex && $results !~ /^$expected/) ||
 			 (!$option_regex && $results ne $expected)))) {
-        print STDERR "PROG: $switch\n$prog\n";
-        print STDERR "EXPECTED:\n$expected\n";
-        print STDERR "GOT:\n$results\n";
+        my $err_line = "PROG: $switch\n$prog\n" .
+                       "EXPECTED:\n$expected\n" .
+                       "GOT:\n$results\n";
+        if ($todo) {
+            $err_line =~ s/^/# /mg;
+            print $err_line;  # Harness can't filter it out from STDERR.
+        }
+        else {
+            print STDERR $err_line;
+        }
         print "not ";
     }
-    print "ok " . ++$i . "\n";
+    print "ok " . ++$i;
+    print " # TODO$todo_reason" if $todo;
+    print "\n";
     foreach (@temps)
 	{ unlink $_ if $_ }
     foreach (@temp_path)