Change 15009 by jhi@alpha on 2002/03/04 17:24:30

        Integrate ext/Encode/compile from perlio.

Affected files ...

.... //depot/perl/ext/Encode/compile#29 integrate

Differences ...

==== //depot/perl/ext/Encode/compile#29 (xtext) ====
Index: perl/ext/Encode/compile
--- perl/ext/Encode/compile.~1~ Mon Mar  4 10:30:06 2002
+++ perl/ext/Encode/compile     Mon Mar  4 10:30:06 2002
@@ -6,6 +6,7 @@
 use strict;
 use Getopt::Std;
 my @orig_ARGV = @ARGV;
+my $perforce  = '$Id$';
 
 # These may get re-ordered.
 # RAW is a do_now as inserted by &enter
@@ -120,12 +121,13 @@
 my %opt;
 # I think these are:
 # -Q to disable the duplicate codepoint test
+# -S make mapping errors fatal
 # -q to remove comments written to output files
 # -O to enable the (brute force) substring optimiser
 # -o <output> to specify the output file name (else it's the first arg)
 # -f <inlist> to give a file with a list of input files (else use the args)
 # -n <name> to name the encoding (else use the basename of the input file.
-getopts('QqOo:f:n:',\%opt);
+getopts('SQqOo:f:n:',\%opt);
 
 # This really should go first, else the die here causes empty (non-erroneous)
 # output files to be written.
@@ -416,6 +418,8 @@
  $encoding{$name} = [$e2u,$u2e,$erep,$min_el,$max_el];
 }
 
+
+
 sub compile_enc
 {
  my ($fh,$name) = @_;
@@ -441,6 +445,7 @@
   my $v = hex($def);
   $rep = &$type_func($v & 0xFF, ($v >> 8) & 0xffe);
  }
+ my $errors;
  my $seen;
  # use -Q to silence the seen test. Makefile.PL uses this by default.
  $seen = {} unless $opt{Q};
@@ -455,8 +460,9 @@
     {
      # So why is it 1% faster to leave the my here?
      my $line = <$fh>;
-     die "Line should be exactly 65 characters long including newline"
-       unless length ($line) == 65;
+     $line =~ s/\r\n$/\n/;
+     die "$.:${line}Line should be exactly 65 characters long including
+     newline (".length($line).")" unless length ($line) == 65;
      # Split line into groups of 4 hex digits, convert groups to ints
      # This takes 65.35                
      # map {hex $_} $line =~ /(....)/g
@@ -483,6 +489,7 @@
              {
                warn sprintf("U%04X is %02X%02X and %04X\n",
                             $val,$page,$ch,$seen->{$uch});
+               $errors++;
              }
            else
              {
@@ -505,6 +512,7 @@
   } while --$pages;
  die "\$min_el=$min_el, \$max_el=$max_el - seems we read no lines"
    if $min_el > $max_el;
+ die "$errors mapping conflicts\n" if ($errors && $opt{'S'});
  $encoding{$name} = [$e2u,$u2e,$rep,$min_el,$max_el];
 }
 
End of Patch.

Reply via email to