This is a live mirror of the Perl 5 development currently hosted at https://github.com/perl/perl5
Avoid $` $& $' in libraries
authorIlya Zakharevich <ilya@math.ohio-state.edu>
Fri, 21 Feb 1997 02:07:30 +0000 (21:07 -0500)
committerChip Salzenberg <chip@atlantic.net>
Fri, 21 Feb 1997 16:41:00 +0000 (04:41 +1200)
Subject: Pessimal $` $& $' in libraries

Corrected:

lib/diagnostics.pm
lib/Getopt/Long.pm
lib/Pod/Text.pm
os2/OS2/REXX/REXX.pm

$' $` $& eliminated from all the known *.pm.

Btw, by this I proudly introduce usage of $+ in standard perl modules ;-).

p5p-msgid: <199702210207.VAA03560@monk.mps.ohio-state.edu>

lib/Getopt/Long.pm
lib/Pod/Text.pm
lib/diagnostics.pm
os2/OS2/REXX/REXX.pm

index f2b37e9..fa6d5bf 100644 (file)
@@ -566,7 +566,7 @@ sub GetOptions {
        my $opt = shift (@optionlist);
 
        # Strip leading prefix so people can specify "--foo=i" if they like.
-       $opt = $' if $opt =~ /^($genprefix)+/;
+       $opt = $+ if $opt =~ /^($genprefix)+(.*)/s;
 
        if ( $opt eq '<>' ) {
            if ( (defined $userlinkage)
@@ -854,19 +854,19 @@ sub GetOptions {
 
 sub find_option {
 
-    return 0 unless $opt =~ /^$genprefix/;
+    return 0 unless $opt =~ /^($genprefix)(.*)/s;
 
-    $opt = $';
-    my ($starter) = $&;
+    $opt = $+;
+    my ($starter) = $1;
 
     my $optarg = undef;        # value supplied with --opt=value
     my $rest = undef;  # remainder from unbundling
 
     # If it is a long option, it may include the value.
     if (($starter eq "--" || $getopt_compat)
-       && $opt =~ /^([^=]+)=/ ) {
+       && $opt =~ /^([^=]+)=(.*)/s ) {
        $opt = $1;
-       $optarg = $';
+       $optarg = $2;
        print STDERR ("=> option \"", $opt, 
                      "\", optarg = \"$optarg\"\n") if $debug;
     }
@@ -992,7 +992,7 @@ sub find_option {
     # Get key if this is a "name=value" pair for a hash option.
     $key = undef;
     if ($hash && defined $arg) {
-       ($key, $arg) = ($arg =~ /=/o) ? ($`, $') : ($arg, 1);
+       ($key, $arg) = ($arg =~ /(.*?)=(.*)/s) ? ($1, $2) : ($arg, 1);
     }
 
     #### Check if the argument is valid for this option ####
index 9d6636a..c62e515 100644 (file)
@@ -116,18 +116,18 @@ POD_DIRECTIVE: while (<IN>) {
        next;
     }
 
-    if (/^=for\s+(\S+)\s*/s) {
+    if (/^=for\s+(\S+)\s*(.*)/s) {
         if ($1 eq "text") {
-            print STDOUT $',"";
+            print STDOUT $2,"";
         } else {
             # ignore unknown for
         }
         next;
     }
-    elsif (/^=begin\s+(\S+)\s*/s) {
+    elsif (/^=begin\s+(\S+)\s*(.*)/s) {
         $begun = $1;
         if ($1 eq "text") {
-            print STDOUT $'."";
+            print STDOUT $2."";
         }
         next;
     }
@@ -409,7 +409,7 @@ sub clear_noremap {
             defined $HTML_Escapes{$3}
                ? do { $HTML_Escapes{$3} }
                : do {
-                   warn "Unknown escape: $& in $_";
+                   warn "Unknown escape: E<$1> in $_";
                    "E<$1>";
                }
         }
index bbae58e..0aa5b54 100644 (file)
@@ -500,7 +500,7 @@ sub unescape {
              exists $HTML_Escapes{$1}
                 ? do { $HTML_Escapes{$1} }
                 : do {
-                    warn "Unknown escape: $& in $_";
+                    warn "Unknown escape: E<$1> in $_";
                     "E<$1>";
                 } 
          } 
index 78e0cf9..114e159 100644 (file)
@@ -107,21 +107,21 @@ sub dropall
 sub TIESCALAR
 {
        my ($obj, $name) = @_;
-       $name =~ s/^[\w!?]+/\U$&\E/;
+       $name =~ s/^([\w!?]+)/\U$1\E/;
        return bless \$name, OS2::REXX::_SCALAR;
 }      
 
 sub TIEARRAY
 {
        my ($obj, $name) = @_;
-       $name =~ s/^[\w!?]+/\U$&\E/;
+       $name =~ s/^([\w!?]+)/\U$1\E/;
        return bless [$name, 0], OS2::REXX::_ARRAY;
 }
 
 sub TIEHASH
 {
        my ($obj, $name) = @_;
-       $name =~ s/^[\w!?]+/\U$&\E/;
+       $name =~ s/^([\w!?]+)/\U$1\E/;
        return bless {Stem => $name}, OS2::REXX::_HASH;
 }