This is a live mirror of the Perl 5 development currently hosted at https://github.com/perl/perl5
t/re/regexp.t: Remove extra semicolons from output.
authorDan Collins <dcollinsn@gmail.com>
Sat, 18 Jun 2016 21:24:22 +0000 (17:24 -0400)
committerTony Cook <tony@develop-help.com>
Mon, 11 Jul 2016 00:29:53 +0000 (10:29 +1000)
The $code segment in t/re/regexp.t contains an extra ';' for the first iteration
of each test, due to how 'study' and 'utf8::upgrade' testing is implemented.
Since this is the only test likely to fail (what with study being a no-op), this
is almost always line noise. This patch removes that ';'.

It would be nice to remove the back to back newlines as well.

t/re/regexp.t

index 38a24d5..5ec6e5c 100644 (file)
@@ -358,8 +358,8 @@ foreach (@tests) {
         }
     }
 
         }
     }
 
-    for my $study ('', 'study $subject', 'utf8::upgrade($subject)',
-                  'utf8::upgrade($subject); study $subject') {
+    for my $study ('', 'study $subject;', 'utf8::upgrade($subject);',
+                  'utf8::upgrade($subject); study $subject;') {
        # Need to make a copy, else the utf8::upgrade of an already studied
        # scalar confuses things.
        my $subject = $subject;
        # Need to make a copy, else the utf8::upgrade of an already studied
        # scalar confuses things.
        my $subject = $subject;
@@ -367,7 +367,7 @@ foreach (@tests) {
        my ($code, $match, $got);
         if ($repl eq 'pos') {
             $code= <<EOFCODE;
        my ($code, $match, $got);
         if ($repl eq 'pos') {
             $code= <<EOFCODE;
-                $study;
+                $study
                 pos(\$subject)=0;
                 \$match = ( \$subject =~ m${pat}g );
                 \$got = pos(\$subject);
                 pos(\$subject)=0;
                 \$match = ( \$subject =~ m${pat}g );
                 \$got = pos(\$subject);
@@ -376,7 +376,7 @@ EOFCODE
         elsif ($qr_embed) {
             $code= <<EOFCODE;
                 my \$RE = qr$pat;
         elsif ($qr_embed) {
             $code= <<EOFCODE;
                 my \$RE = qr$pat;
-                $study;
+                $study
                 \$match = (\$subject =~ /(?:)\$RE(?:)/) while \$c--;
                 \$got = "$repl";
 EOFCODE
                 \$match = (\$subject =~ /(?:)\$RE(?:)/) while \$c--;
                 \$got = "$repl";
 EOFCODE
@@ -386,14 +386,14 @@ EOFCODE
                # Can't run the match in a subthread, but can do this and
                # clone the pattern the other way.
                 my \$RE = threads->new(sub {qr$pat})->join();
                # Can't run the match in a subthread, but can do this and
                # clone the pattern the other way.
                 my \$RE = threads->new(sub {qr$pat})->join();
-                $study;
+                $study
                 \$match = (\$subject =~ /(?:)\$RE(?:)/) while \$c--;
                 \$got = "$repl";
 EOFCODE
         }
         else {
             $code= <<EOFCODE;
                 \$match = (\$subject =~ /(?:)\$RE(?:)/) while \$c--;
                 \$got = "$repl";
 EOFCODE
         }
         else {
             $code= <<EOFCODE;
-                $study;
+                $study
                 \$match = (\$subject =~ $OP$pat) while \$c--;
                 \$got = "$repl";
 EOFCODE
                 \$match = (\$subject =~ $OP$pat) while \$c--;
                 \$got = "$repl";
 EOFCODE