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 (file)
--- 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)