Eliminate 'our' $printed_name
authorJames E. Keenan <jkeenan@cpan.org>
Sun, 28 Mar 2010 17:31:29 +0000 (13:31 -0400)
committerSteffen Mueller <smueller@cpan.org>
Tue, 12 Jul 2011 18:53:53 +0000 (20:53 +0200)
Replace it with a 'my' variable declared inside INPUT_handler().
Permissible, because $printed_name is explicitly passed to subs
called from within INPUT_handler() and only there.

dist/ExtUtils-ParseXS/lib/ExtUtils/ParseXS.pm

index 7cb2d5f..28d0ea8 100644 (file)
@@ -42,7 +42,7 @@ our (
   @line_no, 
   %XsubAliases, %XsubAliasValues, %Interfaces, @Attributes, %outargs, 
   $deferred, $gotRETVAL, $condnum, $cond,
-  $RETVAL_code, $printed_name, $func_args, @XSStack, $ALIAS, 
+  $RETVAL_code, $func_args, @XSStack, $ALIAS, 
 );
 our ($DoSetMagic, $newXS, $proto, $Module_cname, $XsubAliases, $Interfaces, $var_num, );
 
@@ -1167,6 +1167,7 @@ sub INPUT_handler {
     # XXXX This check is a safeguard against the unfinished conversion of
     # generate_init().  When generate_init() is fixed,
     # one can use 2-args map_type() unconditionally.
+    my $printed_name;
     if ($var_type =~ / \( \s* \* \s* \) /x) {
       # Function pointers are not yet supported with &output_init!
       print "\t" . &map_type($var_type, $var_name, $self->{hiertype});