This is a live mirror of the Perl 5 development currently hosted at https://github.com/perl/perl5
Output "read only" editor blocks from regen_perly.pl
authorNicholas Clark <nick@ccl4.org>
Sun, 23 Jan 2011 16:25:30 +0000 (16:25 +0000)
committerNicholas Clark <nick@ccl4.org>
Sun, 23 Jan 2011 18:45:58 +0000 (18:45 +0000)
Use safer_open() and read_only_bottom_close_and_rename() from regen_lib.pl
Consistently use 3 argument open and lexical file handles.

A side effect of this change is that the generated files are no longer made
read-only on disk - if this is desirable, then probably better to change
regen_lib.pl so that all generated files are made read-only.

perly.act
perly.h
perly.tab
regen/regen_lib.pl
regen_perly.pl

index fc30958..ae8e330 100644 (file)
--- a/perly.act
+++ b/perly.act
@@ -1,3 +1,9 @@
+/* -*- buffer-read-only: t -*-
+   !!!!!!!   DO NOT EDIT THIS FILE   !!!!!!!
+   This file is built by regen_perly.pl from perly.y.
+   Any changes made here will be lost!
+ */
+
 case 2:
 #line 142 "perly.y"
     {
@@ -1702,3 +1708,5 @@ case 2:
 
       default: break;
     
+
+/* ex: set ro: */
diff --git a/perly.h b/perly.h
index 9695a06..25f5864 100644 (file)
--- a/perly.h
+++ b/perly.h
@@ -1,3 +1,9 @@
+/* -*- buffer-read-only: t -*-
+   !!!!!!!   DO NOT EDIT THIS FILE   !!!!!!!
+   This file is built by regen_perly.pl from perly.y.
+   Any changes made here will be lost!
+ */
+
 #ifdef PERL_CORE
 /* A Bison parser, made by GNU Bison 2.3.  */
 
@@ -232,3 +238,5 @@ typedef union YYSTYPE
 
 
 
+
+/* ex: set ro: */
index 59e88ae..2e4c30c 100644 (file)
--- a/perly.tab
+++ b/perly.tab
@@ -1,3 +1,9 @@
+/* -*- buffer-read-only: t -*-
+   !!!!!!!   DO NOT EDIT THIS FILE   !!!!!!!
+   This file is built by regen_perly.pl from perly.y.
+   Any changes made here will be lost!
+ */
+
 #define YYFINAL  14
 /* YYLAST -- Last index in YYTABLE.  */
 #define YYLAST   2683
@@ -1066,3 +1072,5 @@ static const toketypes yy_type_tab[] =
   toketype_opval, toketype_opval, toketype_opval, toketype_opval, toketype_opval, toketype_opval, toketype_opval,
   toketype_opval
 };
+
+/* ex: set ro: */
index 86d3b6c..2d4ceac 100644 (file)
@@ -71,7 +71,7 @@ sub safer_open {
     *{$fh}->{name} = $name;
     if (defined $final_name) {
        *{$fh}->{final_name} = $final_name;
-       *{$fh}->{lang} = ($final_name =~ /\.[ch]$/ ? 'C' : 'Perl');
+       *{$fh}->{lang} = ($final_name =~ /\.(?:c|h|tab|act)$/ ? 'C' : 'Perl');
     }
     binmode $fh;
     $fh;
index e4de1a0..00d2a59 100644 (file)
@@ -29,6 +29,7 @@
 # it may work elsewhere but no specific attempt has been made to make it
 # portable.
 
+use 5.006;
 sub usage { die "usage: $0 [ -b bison_executable ] [ file.y ]\n" }
 
 use warnings;
@@ -52,9 +53,6 @@ usage unless @ARGV==0 && $y_file =~ /\.y$/;
 (my $tab_file  = $y_file) =~ s/\.y$/.tab/;
 (my $tmpc_file = $y_file) =~ s/\.y$/tmp.c/;
 (my $tmph_file = $y_file) =~ s/\.y$/tmp.h/;
-my $new_h_file = "$h_file-new";
-my $new_act_file = "$act_file-new";
-my $new_tab_file = "$tab_file-new";
 
 # the yytranslate[] table generated by bison is ASCII/EBCDIC sensitive
 
@@ -88,28 +86,26 @@ EOF
 # creates $tmpc_file and $tmph_file
 my_system("$bison -d -o $tmpc_file $y_file");
 
-open CTMPFILE, $tmpc_file or die "Can't open $tmpc_file: $!\n";
+open my $ctmp_fh, '<', $tmpc_file or die "Can't open $tmpc_file: $!\n";
 my $clines;
-{ local $/; $clines = <CTMPFILE>; }
+{ local $/; $clines = <$ctmp_fh>; }
 die "failed to read $tmpc_file: length mismatch\n"
     unless length $clines == -s $tmpc_file;
-close CTMPFILE;
+close $ctmp_fh;
 
 my ($actlines, $tablines) = extract($clines);
 
 $tablines .= make_type_tab($y_file, $tablines);
 
-chmod 0644, $new_act_file;
-open ACTFILE, ">$new_act_file" or die "can't open $new_act_file: $!\n";
-print ACTFILE $actlines;
-close ACTFILE;
-chmod 0444, $new_act_file;
+my $read_only = read_only_top(lang => 'C', by => $0, from => $y_file);
 
-chmod 0644, $new_tab_file;
-open TABFILE, ">$new_tab_file" or die "can't open $new_tab_file: $!\n";
-print TABFILE $tablines;
-close TABFILE;
-chmod 0444, $new_tab_file;
+my $act_fh = safer_open("$act_file-new", $act_file);
+print $act_fh $read_only, $actlines;
+read_only_bottom_close_and_rename($act_fh);
+
+my $tab_fh = safer_open("$tab_file-new", $tab_file);
+print $tab_fh $read_only, $tablines;
+read_only_bottom_close_and_rename($tab_fh);
 
 unlink $tmpc_file;
 
@@ -117,17 +113,19 @@ unlink $tmpc_file;
 # C<#line 30 "perly.y"> confuses the Win32 resource compiler and the
 # C<#line 188 "perlytmp.h"> gets picked up by make depend, so remove them.
 
-open TMPH_FILE, $tmph_file or die "Can't open $tmph_file: $!\n";
-chmod 0644, $new_h_file;
-open H_FILE, ">$new_h_file" or die "Can't open $new_h_file: $!\n";
+open my $tmph_fh, '<', $tmph_file or die "Can't open $tmph_file: $!\n";
+my $h_fh = safer_open("$h_file-new", $h_file);
+
+print $h_fh $read_only;
+
 my $endcore_done = 0;
 # Token macros need to be generated manually on bison 2.4
 my $gather_tokens = ($version =~ /\b2\.4\b/ ? undef : 0);
 my $tokens;
-while (<TMPH_FILE>) {
-    print H_FILE "#ifdef PERL_CORE\n" if $. == 1;
+while (<$tmph_fh>) {
+    print $h_fh "#ifdef PERL_CORE\n" if $. == 1;
     if (!$endcore_done and /YYSTYPE_IS_DECLARED/) {
-       print H_FILE "#endif /* PERL_CORE */\n";
+       print $h_fh "#endif /* PERL_CORE */\n";
        $endcore_done = 1;
     }
     next if /^#line \d+ ".*"/;
@@ -144,16 +142,12 @@ while (<TMPH_FILE>) {
            $tokens .= "#define $tok $val\n" if $tok;
        }
     }
-    print H_FILE $_;
+    print $h_fh $_;
 }
-close TMPH_FILE;
-close H_FILE;
-chmod 0444, $new_h_file;
+close $tmph_fh;
 unlink $tmph_file;
 
-rename_if_different($new_h_file, $h_file);
-rename_if_different($new_tab_file, $tab_file);
-rename_if_different($new_act_file, $act_file);
+read_only_bottom_close_and_rename($h_fh);
 
 exit 0;