provide MakeMaker attribute PERL_MALLOC_OK that allows extensions
[p5sagit/p5-mst-13.2.git] / lib / ExtUtils / xsubpp
index 21bbc4e..e5c7e09 100755 (executable)
@@ -6,7 +6,7 @@ xsubpp - compiler to convert Perl XS code into C code
 
 =head1 SYNOPSIS
 
-B<xsubpp> [B<-C++>] [B<-except>] [B<-typemap typemap>] file.xs
+B<xsubpp> [B<-v>] [B<-C++>] [B<-except>] [B<-s pattern>] [B<-prototypes>] [B<-noversioncheck>] [B<-nolinenumbers>] [B<-typemap typemap>] ... file.xs
 
 =head1 DESCRIPTION
 
@@ -40,6 +40,25 @@ Indicates that a user-supplied typemap should take precedence over the
 default typemaps.  This option may be used multiple times, with the last
 typemap having the highest precedence.
 
+=item B<-v>
+
+Prints the I<xsubpp> version number to standard output, then exits.
+
+=item B<-prototypes>
+
+By default I<xsubpp> will not automatically generate prototype code for
+all xsubs. This flag will enable prototypes.
+
+=item B<-noversioncheck>
+
+Disables the run time test that determines if the object file (derived
+from the C<.xs> file) and the C<.pm> files have the same version
+number.
+
+=item B<-nolinenumbers>
+
+Prevents the inclusion of `#line' directives in the output.
+
 =back
 
 =head1 ENVIRONMENT
@@ -50,30 +69,111 @@ No environment variables are used.
 
 Larry Wall
 
+=head1 MODIFICATION HISTORY
+
+See the file F<changes.pod>.
+
 =head1 SEE ALSO
 
-perl(1)
+perl(1), perlxs(1), perlxstut(1)
 
 =cut
 
-$usage = "Usage: xsubpp [-C++] [-except] [-typemap typemap] file.xs\n";
+require 5.002;
+use Cwd;
+use vars '$cplusplus';
+use vars '%v';
+
+use Config;
+
+sub Q ;
 
-SWITCH: while ($ARGV[0] =~ s/^-//) {
+# Global Constants
+
+$XSUBPP_version = "1.9507";
+
+my ($Is_VMS, $SymSet);
+if ($^O eq 'VMS') {
+    $Is_VMS = 1;
+    # Establish set of global symbols with max length 28, since xsubpp
+    # will later add the 'XS_' prefix.
+    require ExtUtils::XSSymSet;
+    $SymSet = new ExtUtils::XSSymSet 28;
+}
+
+$FH = 'File0000' ;
+
+$usage = "Usage: xsubpp [-v] [-C++] [-except] [-prototypes] [-noversioncheck] [-nolinenumbers] [-s pattern] [-typemap typemap]... file.xs\n";
+
+$proto_re = "[" . quotemeta('\$%&*@;') . "]" ;
+# mjn
+$OBJ   = 1 if $Config{'ccflags'} =~ /PERL_OBJECT/i;
+
+$except = "";
+$WantPrototypes = -1 ;
+$WantVersionChk = 1 ;
+$ProtoUsed = 0 ;
+$WantLineNumbers = 1 ;
+SWITCH: while (@ARGV and $ARGV[0] =~ /^-./) {
     $flag = shift @ARGV;
-    $spat = shift,     next SWITCH     if $flag eq 's';
+    $flag =~ s/^-// ;
+    $spat = quotemeta shift,   next SWITCH     if $flag eq 's';
     $cplusplus = 1,    next SWITCH     if $flag eq 'C++';
-    $except = 1,       next SWITCH     if $flag eq 'except';
+    $WantPrototypes = 0, next SWITCH   if $flag eq 'noprototypes';
+    $WantPrototypes = 1, next SWITCH   if $flag eq 'prototypes';
+    $WantVersionChk = 0, next SWITCH   if $flag eq 'noversioncheck';
+    $WantVersionChk = 1, next SWITCH   if $flag eq 'versioncheck';
+    # XXX left this in for compat
+    $WantCAPI = 1, next SWITCH    if $flag eq 'object_capi';
+    $except = " TRY",  next SWITCH     if $flag eq 'except';
     push(@tm,shift),   next SWITCH     if $flag eq 'typemap';
+    $WantLineNumbers = 0, next SWITCH  if $flag eq 'nolinenumbers';
+    $WantLineNumbers = 1, next SWITCH  if $flag eq 'linenumbers';
+    (print "xsubpp version $XSUBPP_version\n"), exit   
+       if $flag eq 'v';
     die $usage;
 }
+if ($WantPrototypes == -1)
+  { $WantPrototypes = 0}
+else
+  { $ProtoUsed = 1 }
+
+
 @ARGV == 1 or die $usage;
-chop($pwd = `pwd`);
-# Check for error message from VMS
-if ($pwd =~ /unrecognized command verb/) { $Is_VMS = 1; $pwd = $ENV{DEFAULT} }
-($dir, $filename) = @ARGV[0] =~ m#(.*)/(.*)#
-       or ($dir, $filename) = @ARGV[0] =~ m#(.*[>\]])(.*)#
+($dir, $filename) = $ARGV[0] =~ m#(.*)/(.*)#
+       or ($dir, $filename) = $ARGV[0] =~ m#(.*)\\(.*)#
+       or ($dir, $filename) = $ARGV[0] =~ m#(.*[>\]])(.*)#
        or ($dir, $filename) = ('.', $ARGV[0]);
 chdir($dir);
+$pwd = cwd();
+
+++ $IncludedFiles{$ARGV[0]} ;
+
+my(@XSStack) = ({type => 'none'});     # Stack of conditionals and INCLUDEs
+my($XSS_work_idx, $cpp_next_tmp) = (0, "XSubPPtmpAAAA");
+
+
+sub TrimWhitespace
+{
+    $_[0] =~ s/^\s+|\s+$//go ;
+}
+
+sub TidyType
+{
+    local ($_) = @_ ;
+
+    # rationalise any '*' by joining them into bunches and removing whitespace
+    s#\s*(\*+)\s*#$1#g;
+    s#(\*+)# $1 #g ;
+
+    # change multiple whitespace into a single space
+    s/\s+/ /g ;
+    
+    # trim leading & trailing whitespace
+    TrimWhitespace($_) ;
+
+    $_ ;
+}
 
 $typemap = shift @ARGV;
 foreach $typemap (@tm) {
@@ -83,38 +183,49 @@ unshift @tm, qw(../../../../lib/ExtUtils/typemap ../../../lib/ExtUtils/typemap
                 ../../lib/ExtUtils/typemap ../../../typemap ../../typemap
                 ../typemap typemap);
 foreach $typemap (@tm) {
-    open(TYPEMAP, $typemap) || next;
-    $mode = Typemap;
+    next unless -e $typemap ;
+    # skip directories, binary files etc.
+    warn("Warning: ignoring non-text typemap file '$typemap'\n"), next 
+       unless -T $typemap ;
+    open(TYPEMAP, $typemap) 
+       or warn ("Warning: could not open typemap file '$typemap': $!\n"), next;
+    $mode = 'Typemap';
+    $junk = "" ;
     $current = \$junk;
     while (<TYPEMAP>) {
-       next if /^#/;
-       if (/^INPUT\s*$/) { $mode = Input, next }
-       if (/^OUTPUT\s*$/) { $mode = Output, next }
-       if (/^TYPEMAP\s*$/) { $mode = Typemap, next }
-       if ($mode eq Typemap) {
-           chop;
-           ($typename, $kind) = split(/\t+/, $_, 2);
-           $type_kind{$typename} = $kind if $kind ne '';
+       next if /^\s*#/;
+        my $line_no = $. + 1; 
+       if (/^INPUT\s*$/)   { $mode = 'Input';   $current = \$junk;  next; }
+       if (/^OUTPUT\s*$/)  { $mode = 'Output';  $current = \$junk;  next; }
+       if (/^TYPEMAP\s*$/) { $mode = 'Typemap'; $current = \$junk;  next; }
+       if ($mode eq 'Typemap') {
+           chomp;
+           my $line = $_ ;
+            TrimWhitespace($_) ;
+           # skip blank lines and comment lines
+           next if /^$/ or /^#/ ;
+           my($type,$kind, $proto) = /^\s*(.*?\S)\s+(\S+)\s*($proto_re*)\s*$/ or
+               warn("Warning: File '$typemap' Line $. '$line' TYPEMAP entry needs 2 or 3 columns\n"), next;
+            $type = TidyType($type) ;
+           $type_kind{$type} = $kind ;
+            # prototype defaults to '$'
+            $proto = "\$" unless $proto ;
+            warn("Warning: File '$typemap' Line $. '$line' Invalid prototype '$proto'\n") 
+                unless ValidProtoString($proto) ;
+            $proto_letter{$type} = C_string($proto) ;
        }
-       elsif ($mode eq Input) {
-           if (/^\s/) {
-               $$current .= $_;
-           }
-           else {
-               s/\s*$//;
-               $input_expr{$_} = '';
-               $current = \$input_expr{$_};
-           }
+       elsif (/^\s/) {
+           $$current .= $_;
+       }
+       elsif ($mode eq 'Input') {
+           s/\s+$//;
+           $input_expr{$_} = '';
+           $current = \$input_expr{$_};
        }
        else {
-           if (/^\s/) {
-               $$current .= $_;
-           }
-           else {
-               s/\s*$//;
-               $output_expr{$_} = '';
-               $current = \$output_expr{$_};
-           }
+           s/\s+$//;
+           $output_expr{$_} = '';
+           $current = \$output_expr{$_};
        }
     }
     close(TYPEMAP);
@@ -124,70 +235,655 @@ foreach $key (keys %input_expr) {
     $input_expr{$key} =~ s/\n+$//;
 }
 
+$END = "!End!\n\n";            # "impossible" keyword (multiple newline)
+
+# Match an XS keyword
+$BLOCK_re= '\s*(' . join('|', qw(
+       REQUIRE BOOT CASE PREINIT INPUT INIT CODE PPCODE OUTPUT 
+       CLEANUP ALIAS PROTOTYPES PROTOTYPE VERSIONCHECK INCLUDE
+       SCOPE INTERFACE INTERFACE_MACRO C_ARGS
+       )) . "|$END)\\s*:";
+
+# Input:  ($_, @line) == unparsed input.
+# Output: ($_, @line) == (rest of line, following lines).
+# Return: the matched keyword if found, otherwise 0
+sub check_keyword {
+       $_ = shift(@line) while !/\S/ && @line;
+       s/^(\s*)($_[0])\s*:\s*(?:#.*)?/$1/s && $2;
+}
+
+
+if ($WantLineNumbers) {
+    {
+       package xsubpp::counter;
+       sub TIEHANDLE {
+           my ($class, $cfile) = @_;
+           my $buf = "";
+           $SECTION_END_MARKER = "#line --- \"$cfile\"";
+           $line_no = 1;
+           bless \$buf;
+       }
+
+       sub PRINT {
+           my $self = shift;
+           for (@_) {
+               $$self .= $_;
+               while ($$self =~ s/^([^\n]*\n)//) {
+                   my $line = $1;
+                   ++ $line_no;
+                   $line =~ s|^\#line\s+---(?=\s)|#line $line_no|;
+                   print STDOUT $line;
+               }
+           }
+       }
+
+       sub PRINTF {
+           my $self = shift;
+           my $fmt = shift;
+           $self->PRINT(sprintf($fmt, @_));
+       }
+
+       sub DESTROY {
+           # Not necessary if we're careful to end with a "\n"
+           my $self = shift;
+           print STDOUT $$self;
+       }
+    }
+
+    my $cfile = $filename;
+    $cfile =~ s/\.xs$/.c/i or $cfile .= ".c";
+    tie(*PSEUDO_STDOUT, 'xsubpp::counter', $cfile);
+    select PSEUDO_STDOUT;
+}
+
+sub print_section {
+    # the "do" is required for right semantics
+    do { $_ = shift(@line) } while !/\S/ && @line;
+    
+    print("#line ", $line_no[@line_no - @line -1], " \"$filename\"\n")
+       if $WantLineNumbers && !/^\s*#\s*line\b/ && !/^#if XSubPPtmp/;
+    for (;  defined($_) && !/^$BLOCK_re/o;  $_ = shift(@line)) {
+       print "$_\n";
+    }
+    print "$xsubpp::counter::SECTION_END_MARKER\n" if $WantLineNumbers;
+}
+
+sub merge_section {
+    my $in = '';
+  
+    while (!/\S/ && @line) {
+        $_ = shift(@line);
+    }
+    
+    for (;  defined($_) && !/^$BLOCK_re/o;  $_ = shift(@line)) {
+       $in .= "$_\n";
+    }
+    chomp $in;
+    return $in;
+}
+
+sub process_keyword($)
+{
+    my($pattern) = @_ ;
+    my $kwd ;
+
+    &{"${kwd}_handler"}() 
+        while $kwd = check_keyword($pattern) ;
+}
+
+sub CASE_handler {
+    blurt ("Error: `CASE:' after unconditional `CASE:'")
+       if $condnum && $cond eq '';
+    $cond = $_;
+    TrimWhitespace($cond);
+    print "   ", ($condnum++ ? " else" : ""), ($cond ? " if ($cond)\n" : "\n");
+    $_ = '' ;
+}
+
+sub INPUT_handler {
+    for (;  !/^$BLOCK_re/o;  $_ = shift(@line)) {
+       last if /^\s*NOT_IMPLEMENTED_YET/;
+       next unless /\S/;       # skip blank lines 
+
+       TrimWhitespace($_) ;
+       my $line = $_ ;
+
+       # remove trailing semicolon if no initialisation
+       s/\s*;$//g unless /[=;+].*\S/ ;
+
+       # check for optional initialisation code
+       my $var_init = '' ;
+       $var_init = $1 if s/\s*([=;+].*)$//s ;
+       $var_init =~ s/"/\\"/g;
+
+       s/\s+/ /g;
+       my ($var_type, $var_addr, $var_name) = /^(.*?[^& ]) *(\&?) *\b(\w+)$/s
+           or blurt("Error: invalid argument declaration '$line'"), next;
+
+       # Check for duplicate definitions
+       blurt ("Error: duplicate definition of argument '$var_name' ignored"), next
+           if $arg_list{$var_name} ++  ;
+
+       $thisdone |= $var_name eq "THIS";
+       $retvaldone |= $var_name eq "RETVAL";
+       $var_types{$var_name} = $var_type;
+       print "\t" . &map_type($var_type);
+       $var_num = $args_match{$var_name};
+
+        $proto_arg[$var_num] = ProtoString($var_type) 
+           if $var_num ;
+       if ($var_addr) {
+           $var_addr{$var_name} = 1;
+           $func_args =~ s/\b($var_name)\b/&$1/;
+       }
+       if ($var_init =~ /^[=;]\s*NO_INIT\s*;?\s*$/) {
+           print "\t$var_name;\n";
+       } elsif ($var_init =~ /\S/) {
+           &output_init($var_type, $var_num, $var_name, $var_init);
+       } elsif ($var_num) {
+           # generate initialization code
+           &generate_init($var_type, $var_num, $var_name);
+       } else {
+           print ";\n";
+       }
+    }
+}
+
+sub OUTPUT_handler {
+    for (;  !/^$BLOCK_re/o;  $_ = shift(@line)) {
+       next unless /\S/;
+       if (/^\s*SETMAGIC\s*:\s*(ENABLE|DISABLE)\s*/) {
+           $DoSetMagic = ($1 eq "ENABLE" ? 1 : 0);
+           next;
+       }
+       my ($outarg, $outcode) = /^\s*(\S+)\s*(.*?)\s*$/s ;
+       blurt ("Error: duplicate OUTPUT argument '$outarg' ignored"), next
+           if $outargs{$outarg} ++ ;
+       if (!$gotRETVAL and $outarg eq 'RETVAL') {
+           # deal with RETVAL last
+           $RETVAL_code = $outcode ;
+           $gotRETVAL = 1 ;
+           next ;
+       }
+       blurt ("Error: OUTPUT $outarg not an argument"), next
+           unless defined($args_match{$outarg});
+       blurt("Error: No input definition for OUTPUT argument '$outarg' - ignored"), next
+           unless defined $var_types{$outarg} ;
+       $var_num = $args_match{$outarg};
+       if ($outcode) {
+           print "\t$outcode\n";
+           print "\tSvSETMAGIC(ST(" , $var_num-1 , "));\n" if $DoSetMagic;
+       } else {
+           &generate_output($var_types{$outarg}, $var_num, $outarg, $DoSetMagic);
+       }
+    }
+}
+
+sub C_ARGS_handler() {
+    my $in = merge_section();
+  
+    TrimWhitespace($in);
+    $func_args = $in;
+} 
+
+sub INTERFACE_MACRO_handler() {
+    my $in = merge_section();
+  
+    TrimWhitespace($in);
+    if ($in =~ /\s/) {         # two
+        ($interface_macro, $interface_macro_set) = split ' ', $in;
+    } else {
+        $interface_macro = $in;
+       $interface_macro_set = 'UNKNOWN_CVT'; # catch later
+    }
+    $interface = 1;            # local
+    $Interfaces = 1;           # global
+}
+
+sub INTERFACE_handler() {
+    my $in = merge_section();
+  
+    TrimWhitespace($in);
+    
+    foreach (split /[\s,]+/, $in) {
+        $Interfaces{$_} = $_;
+    }
+    print Q<<"EOF";
+#      XSFUNCTION = $interface_macro($ret_type,cv,XSANY.any_dptr);
+EOF
+    $interface = 1;            # local
+    $Interfaces = 1;           # global
+}
+
+sub CLEANUP_handler() { print_section() } 
+sub PREINIT_handler() { print_section() } 
+sub INIT_handler()    { print_section() } 
+
+sub GetAliases
+{
+    my ($line) = @_ ;
+    my ($orig) = $line ;
+    my ($alias) ;
+    my ($value) ;
+
+    # Parse alias definitions
+    # format is
+    #    alias = value alias = value ...
+
+    while ($line =~ s/^\s*([\w:]+)\s*=\s*(\w+)\s*//) {
+        $alias = $1 ;
+        $orig_alias = $alias ;
+        $value = $2 ;
+
+        # check for optional package definition in the alias
+       $alias = $Packprefix . $alias if $alias !~ /::/ ;
+        
+        # check for duplicate alias name & duplicate value
+       Warn("Warning: Ignoring duplicate alias '$orig_alias'")
+           if defined $XsubAliases{$alias} ;
+
+       Warn("Warning: Aliases '$orig_alias' and '$XsubAliasValues{$value}' have identical values")
+           if $XsubAliasValues{$value} ;
+
+       $XsubAliases = 1;
+       $XsubAliases{$alias} = $value ;
+       $XsubAliasValues{$value} = $orig_alias ;
+    }
+
+    blurt("Error: Cannot parse ALIAS definitions from '$orig'")
+        if $line ;
+}
+
+sub ALIAS_handler ()
+{
+    for (;  !/^$BLOCK_re/o;  $_ = shift(@line)) {
+       next unless /\S/;
+       TrimWhitespace($_) ;
+        GetAliases($_) if $_ ;
+    }
+}
+
+sub REQUIRE_handler ()
+{
+    # the rest of the current line should contain a version number
+    my ($Ver) = $_ ;
+
+    TrimWhitespace($Ver) ;
+
+    death ("Error: REQUIRE expects a version number")
+       unless $Ver ;
+
+    # check that the version number is of the form n.n
+    death ("Error: REQUIRE: expected a number, got '$Ver'")
+       unless $Ver =~ /^\d+(\.\d*)?/ ;
+
+    death ("Error: xsubpp $Ver (or better) required--this is only $XSUBPP_version.")
+        unless $XSUBPP_version >= $Ver ; 
+}
+
+sub VERSIONCHECK_handler ()
+{
+    # the rest of the current line should contain either ENABLE or
+    # DISABLE
+    TrimWhitespace($_) ;
+    # check for ENABLE/DISABLE
+    death ("Error: VERSIONCHECK: ENABLE/DISABLE")
+        unless /^(ENABLE|DISABLE)/i ;
+    $WantVersionChk = 1 if $1 eq 'ENABLE' ;
+    $WantVersionChk = 0 if $1 eq 'DISABLE' ;
+}
+
+sub PROTOTYPE_handler ()
+{
+    my $specified ;
+
+    death("Error: Only 1 PROTOTYPE definition allowed per xsub") 
+        if $proto_in_this_xsub ++ ;
+
+    for (;  !/^$BLOCK_re/o;  $_ = shift(@line)) {
+       next unless /\S/;
+       $specified = 1 ;
+       TrimWhitespace($_) ;
+        if ($_ eq 'DISABLE') {
+          $ProtoThisXSUB = 0 
+        }
+        elsif ($_ eq 'ENABLE') {
+          $ProtoThisXSUB = 1 
+        }
+        else {
+            # remove any whitespace
+            s/\s+//g ;
+            death("Error: Invalid prototype '$_'")
+                unless ValidProtoString($_) ;
+            $ProtoThisXSUB = C_string($_) ;
+        }
+    }
+
+    # If no prototype specified, then assume empty prototype ""
+    $ProtoThisXSUB = 2 unless $specified ;
+
+    $ProtoUsed = 1 ;
+
+}
+
+sub SCOPE_handler ()
+{
+    death("Error: Only 1 SCOPE declaration allowed per xsub") 
+        if $scope_in_this_xsub ++ ;
+
+    for (;  !/^$BLOCK_re/o;  $_ = shift(@line)) {
+               next unless /\S/;
+               TrimWhitespace($_) ;
+        if ($_ =~ /^DISABLE/i) {
+                  $ScopeThisXSUB = 0 
+        }
+        elsif ($_ =~ /^ENABLE/i) {
+                  $ScopeThisXSUB = 1 
+        }
+    }
+
+}
+
+sub PROTOTYPES_handler ()
+{
+    # the rest of the current line should contain either ENABLE or
+    # DISABLE 
+
+    TrimWhitespace($_) ;
+
+    # check for ENABLE/DISABLE
+    death ("Error: PROTOTYPES: ENABLE/DISABLE")
+        unless /^(ENABLE|DISABLE)/i ;
+
+    $WantPrototypes = 1 if $1 eq 'ENABLE' ;
+    $WantPrototypes = 0 if $1 eq 'DISABLE' ;
+    $ProtoUsed = 1 ;
+
+}
+
+sub INCLUDE_handler ()
+{
+    # the rest of the current line should contain a valid filename
+    TrimWhitespace($_) ;
+    death("INCLUDE: filename missing")
+        unless $_ ;
+
+    death("INCLUDE: output pipe is illegal")
+        if /^\s*\|/ ;
+
+    # simple minded recursion detector
+    death("INCLUDE loop detected")
+        if $IncludedFiles{$_} ;
+
+    ++ $IncludedFiles{$_} unless /\|\s*$/ ;
+
+    # Save the current file context.
+    push(@XSStack, {
+       type            => 'file',
+        LastLine        => $lastline,
+        LastLineNo      => $lastline_no,
+        Line            => \@line,
+        LineNo          => \@line_no,
+        Filename        => $filename,
+        Handle          => $FH,
+        }) ;
+    ++ $FH ;
+
+    # open the new file
+    open ($FH, "$_") or death("Cannot open '$_': $!") ;
+    print Q<<"EOF" ;
+#
+#/* INCLUDE:  Including '$_' from '$filename' */
+#
+EOF
+
+    $filename = $_ ;
+
+    # Prime the pump by reading the first 
+    # non-blank line
+
+    # skip leading blank lines
+    while (<$FH>) {
+        last unless /^\s*$/ ;
+    }
+
+    $lastline = $_ ;
+    $lastline_no = $. ;
+}
+sub PopFile()
+{
+    return 0 unless $XSStack[-1]{type} eq 'file' ;
+
+    my $data     = pop @XSStack ;
+    my $ThisFile = $filename ;
+    my $isPipe   = ($filename =~ /\|\s*$/) ;
+    -- $IncludedFiles{$filename}
+        unless $isPipe ;
+
+    close $FH ;
+
+    $FH         = $data->{Handle} ;
+    $filename   = $data->{Filename} ;
+    $lastline   = $data->{LastLine} ;
+    $lastline_no = $data->{LastLineNo} ;
+    @line       = @{ $data->{Line} } ;
+    @line_no    = @{ $data->{LineNo} } ;
+
+    if ($isPipe and $? ) {
+        -- $lastline_no ;
+        print STDERR "Error reading from pipe '$ThisFile': $! in $filename, line $lastline_no\n"  ;
+        exit 1 ;
+    }
+
+    print Q<<"EOF" ;
+#
+#/* INCLUDE: Returning to '$filename' from '$ThisFile' */
+#
+EOF
+
+    return 1 ;
+}
+
+sub ValidProtoString ($)
+{
+    my($string) = @_ ;
+
+    if ( $string =~ /^$proto_re+$/ ) {
+        return $string ;
+    }
+
+    return 0 ;
+}
+
+sub C_string ($)
+{
+    my($string) = @_ ;
+
+    $string =~ s[\\][\\\\]g ;
+    $string ;
+}
+
+sub ProtoString ($)
+{
+    my ($type) = @_ ;
+
+    $proto_letter{$type} or "\$" ;
+}
+
+sub check_cpp {
+    my @cpp = grep(/^\#\s*(?:if|e\w+)/, @line);
+    if (@cpp) {
+       my ($cpp, $cpplevel);
+       for $cpp (@cpp) {
+           if ($cpp =~ /^\#\s*if/) {
+               $cpplevel++;
+           } elsif (!$cpplevel) {
+               Warn("Warning: #else/elif/endif without #if in this function");
+               print STDERR "    (precede it with a blank line if the matching #if is outside the function)\n"
+                   if $XSStack[-1]{type} eq 'if';
+               return;
+           } elsif ($cpp =~ /^\#\s*endif/) {
+               $cpplevel--;
+           }
+       }
+       Warn("Warning: #if without #endif in this function") if $cpplevel;
+    }
+}
+
+
 sub Q {
-    local $text = shift;
-    $text =~ tr/#//d;
+    my($text) = @_;
+    $text =~ s/^#//gm;
     $text =~ s/\[\[/{/g;
     $text =~ s/\]\]/}/g;
     $text;
 }
 
-open(F, $filename) || die "cannot open $filename\n";
+open($FH, $filename) or die "cannot open $filename: $!\n";
 
-while (<F>) {
-    last if ($Module, $foo, $Package, $foo1, $Prefix) =
-       /^MODULE\s*=\s*([\w:]+)(\s+PACKAGE\s*=\s*([\w:]+))?(\s+PREFIX\s*=\s*(\S+))?\s*$/;
+# Identify the version of xsubpp used
+print <<EOM ;
+/*
+ * This file was generated automatically by xsubpp version $XSUBPP_version from the 
+ * contents of $filename. Do not edit this file, edit $filename instead.
+ *
+ *     ANY CHANGES MADE HERE WILL BE LOST! 
+ *
+ */
+
+EOM
+
+print("#line 1 \"$filename\"\n")
+    if $WantLineNumbers;
+
+while (<$FH>) {
+    last if ($Module, $Package, $Prefix) =
+       /^MODULE\s*=\s*([\w:]+)(?:\s+PACKAGE\s*=\s*([\w:]+))?(?:\s+PREFIX\s*=\s*(\S+))?\s*$/;
+
+    if ($OBJ) {
+        s/#if(?:def\s|\s+defined)\s*(\(__cplusplus\)|__cplusplus)/#if defined(__cplusplus) && !defined(PERL_OBJECT)/;
+    }
     print $_;
 }
-exit 0 if $_ eq "";
-$lastline = $_;
+&Exit unless defined $_;
 
+print "$xsubpp::counter::SECTION_END_MARKER\n" if $WantLineNumbers;
+
+$lastline    = $_;
+$lastline_no = $.;
+
+# Read next xsub into @line from ($lastline, <$FH>).
 sub fetch_para {
     # parse paragraph
+    death ("Error: Unterminated `#if/#ifdef/#ifndef'")
+       if !defined $lastline && $XSStack[-1]{type} eq 'if';
     @line = ();
-    if ($lastline ne "") {
-       if ($lastline =~
-    /^MODULE\s*=\s*([\w:]+)(\s+PACKAGE\s*=\s*([\w:]+))?(\s+PREFIX\s*=\s*(\S+))?\s*$/) {
-           $Module = $1;
-           $foo = $2;
-           $Package = $3;
-           $foo1 = $4;
-           $Prefix = $5;
-           ($Module_cname = $Module) =~ s/\W/_/g;
-           ($Packid = $Package) =~ s/:/_/g;
-           $Packprefix = $Package;
-           $Packprefix .= "::" if defined $Packprefix && $Packprefix ne "";
-           while (<F>) {
-               chop;
-               next if /^#/ &&
-                   !/^#[ \t]*(if|ifdef|ifndef|else|elif|endif|define|undef)\b/;
-               last if /^\S/;
-           }
-           push(@line, $_) if $_ ne "";
-       }
-       else {
+    @line_no = () ;
+    return PopFile() if !defined $lastline;
+
+    if ($lastline =~
+       /^MODULE\s*=\s*([\w:]+)(?:\s+PACKAGE\s*=\s*([\w:]+))?(?:\s+PREFIX\s*=\s*(\S+))?\s*$/) {
+       $Module = $1;
+       $Package = defined($2) ? $2 : '';       # keep -w happy
+       $Prefix  = defined($3) ? $3 : '';       # keep -w happy
+       $Prefix = quotemeta $Prefix ;
+       ($Module_cname = $Module) =~ s/\W/_/g;
+       ($Packid = $Package) =~ tr/:/_/;
+       $Packprefix = $Package;
+       $Packprefix .= "::" if $Packprefix ne "";
+       $lastline = "";
+    }
+
+    for(;;) {
+       if ($lastline !~ /^\s*#/ ||
+           # CPP directives:
+           #   ANSI:   if ifdef ifndef elif else endif define undef
+           #           line error pragma
+           #   gcc:    warning include_next
+           #   obj-c:  import
+           #   others: ident (gcc notes that some cpps have this one)
+           $lastline =~ /^#[ \t]*(?:(?:if|ifn?def|elif|else|endif|define|undef|pragma|error|warning|line\s+\d+|ident)\b|(?:include(?:_next)?|import)\s*["<].*[>"])/) {
+           last if $lastline =~ /^\S/ && @line && $line[-1] eq "";
            push(@line, $lastline);
+           push(@line_no, $lastline_no) ;
        }
-       $lastline = "";
-       while (<F>) {
-           next if /^#/ &&
-               !/^#[ \t]*(if|ifdef|ifndef|else|elif|endif|define|undef)\b/;
-           chop;
-           if (/^\S/ && @line && $line[-1] eq "") {
-               $lastline = $_;
-               last;
+
+       # Read next line and continuation lines
+       last unless defined($lastline = <$FH>);
+       $lastline_no = $.;
+       my $tmp_line;
+       $lastline .= $tmp_line
+           while ($lastline =~ /\\$/ && defined($tmp_line = <$FH>));
+           
+       chomp $lastline;
+       $lastline =~ s/^\s+$//;
+    }
+    pop(@line), pop(@line_no) while @line && $line[-1] eq "";
+    1;
+}
+
+PARAGRAPH:
+while (fetch_para()) {
+    # Print initial preprocessor statements and blank lines
+    while (@line && $line[0] !~ /^[^\#]/) {
+       my $line = shift(@line);
+       print $line, "\n";
+       next unless $line =~ /^\#\s*((if)(?:n?def)?|elsif|else|endif)\b/;
+       my $statement = $+;
+       if ($statement eq 'if') {
+           $XSS_work_idx = @XSStack;
+           push(@XSStack, {type => 'if'});
+       } else {
+           death ("Error: `$statement' with no matching `if'")
+               if $XSStack[-1]{type} ne 'if';
+           if ($XSStack[-1]{varname}) {
+               push(@InitFileCode, "#endif\n");
+               push(@BootCode,     "#endif");
            }
-           else {
-               push(@line, $_);
+
+           my(@fns) = keys %{$XSStack[-1]{functions}};
+           if ($statement ne 'endif') {
+               # Hide the functions defined in other #if branches, and reset.
+               @{$XSStack[-1]{other_functions}}{@fns} = (1) x @fns;
+               @{$XSStack[-1]}{qw(varname functions)} = ('', {});
+           } else {
+               my($tmp) = pop(@XSStack);
+               0 while (--$XSS_work_idx
+                        && $XSStack[$XSS_work_idx]{type} ne 'if');
+               # Keep all new defined functions
+               push(@fns, keys %{$tmp->{other_functions}});
+               @{$XSStack[$XSS_work_idx]{functions}}{@fns} = (1) x @fns;
            }
        }
-       pop(@line) while @line && $line[-1] =~ /^\s*$/;
     }
-    $PPCODE = grep(/PPCODE:/, @line);
-    scalar @line;
-}
 
-while (&fetch_para) {
+    next PARAGRAPH unless @line;
+
+    if ($XSS_work_idx && !$XSStack[$XSS_work_idx]{varname}) {
+       # We are inside an #if, but have not yet #defined its xsubpp variable.
+       print "#define $cpp_next_tmp 1\n\n";
+       push(@InitFileCode, "#if $cpp_next_tmp\n");
+       push(@BootCode,     "#if $cpp_next_tmp");
+       $XSStack[$XSS_work_idx]{varname} = $cpp_next_tmp++;
+    }
+
+    death ("Code is not inside a function"
+          ." (maybe last function was ended by a blank line "
+          ." followed by a a statement on column one?)")
+       if $line[0] =~ /^\s/;
+
     # initialize info arrays
     undef(%args_match);
     undef(%var_types);
@@ -196,38 +892,69 @@ while (&fetch_para) {
     undef($class);
     undef($static);
     undef($elipsis);
+    undef($wantRETVAL) ;
+    undef(%arg_list) ;
+    undef(@proto_arg) ;
+    undef($proto_in_this_xsub) ;
+    undef($scope_in_this_xsub) ;
+    undef($interface);
+    $interface_macro = 'XSINTERFACE_FUNC' ;
+    $interface_macro_set = 'XSINTERFACE_FUNC_SET' ;
+    $ProtoThisXSUB = $WantPrototypes ;
+    $ScopeThisXSUB = 0;
 
-    # extract return type, function name and arguments
-    $ret_type = shift(@line);
-    if ($ret_type =~ /^BOOT:/) {
-        push (@BootCode, @line, "", "") ;
-        next ;
+    $_ = shift(@line);
+    while ($kwd = check_keyword("REQUIRE|PROTOTYPES|VERSIONCHECK|INCLUDE")) {
+        &{"${kwd}_handler"}() ;
+        next PARAGRAPH unless @line ;
+        $_ = shift(@line);
     }
-    if ($ret_type =~ /^static\s+(.*)$/) {
-           $static = 1;
-           $ret_type = $1;
+
+    if (check_keyword("BOOT")) {
+       &check_cpp;
+       push (@BootCode, "#line $line_no[@line_no - @line] \"$filename\"")
+         if $WantLineNumbers && $line[0] !~ /^\s*#\s*line\b/;
+        push (@BootCode, @line, "") ;
+        next PARAGRAPH ;
     }
+
+
+    # extract return type, function name and arguments
+    ($ret_type) = TidyType($_);
+
+    # a function definition needs at least 2 lines
+    blurt ("Error: Function definition too short '$ret_type'"), next PARAGRAPH
+       unless @line ;
+
+    $static = 1 if $ret_type =~ s/^static\s+//;
+
     $func_header = shift(@line);
-    ($func_name, $orig_args) =  $func_header =~ /^([\w:]+)\s*\((.*)\)$/;
-    if ($func_name =~ /(.*)::(.*)/) {
-           $class = $1;
-           $func_name = $2;
-    }
+    blurt ("Error: Cannot parse function definition from '$func_header'"), next PARAGRAPH
+       unless $func_header =~ /^(?:([\w:]*)::)?(\w+)\s*\(\s*(.*?)\s*\)\s*(const)?\s*$/s;
+
+    ($class, $func_name, $orig_args) =  ($1, $2, $3) ;
+    $class = "$4 $class" if $4;
     ($pname = $func_name) =~ s/^($Prefix)?/$Packprefix/;
-    push(@Func_name, "${Packid}_$func_name");
-    push(@Func_pname, $pname);
+    ($clean_func_name = $func_name) =~ s/^$Prefix//;
+    $Full_func_name = "${Packid}_$clean_func_name";
+    if ($Is_VMS) { $Full_func_name = $SymSet->addsym($Full_func_name); }
+
+    # Check for duplicate function definition
+    for $tmp (@XSStack) {
+       next unless defined $tmp->{functions}{$Full_func_name};
+       Warn("Warning: duplicate function definition '$clean_func_name' detected");
+       last;
+    }
+    $XSStack[$XSS_work_idx]{functions}{$Full_func_name} ++ ;
+    %XsubAliases = %XsubAliasValues = %Interfaces = ();
+    $DoSetMagic = 1;
+
     @args = split(/\s*,\s*/, $orig_args);
     if (defined($class)) {
-       if (defined($static)) {
-           unshift(@args, "CLASS");
-           $orig_args = "CLASS, $orig_args";
-           $orig_args =~ s/^CLASS, $/CLASS/;
-       }
-       else {
-           unshift(@args, "THIS");
-           $orig_args = "THIS, $orig_args";
-           $orig_args =~ s/^THIS, $/THIS/;
-       }
+       my $arg0 = ((defined($static) or $func_name eq 'new')
+                   ? "CLASS" : "THIS");
+       unshift(@args, $arg0);
+       ($orig_args = "$arg0, $orig_args") =~ s/^$arg0, $/$arg0/;
     }
     $orig_args =~ s/"/\\"/g;
     $min_args = $num_args = @args;
@@ -235,17 +962,18 @@ while (&fetch_para) {
            if ($args[$i] =~ s/\.\.\.//) {
                    $elipsis = 1;
                    $min_args--;
-                   if ($args[i] eq '' && $i == $num_args - 1) {
+                   if ($args[$i] eq '' && $i == $num_args - 1) {
                        pop(@args);
                        last;
                    }
            }
-           if ($args[$i] =~ /([^=]*\S)\s*=\s*(.*)/) {
+           if ($args[$i] =~ /^([^=]*[^\s=])\s*=\s*(.*)/s) {
                    $min_args--;
                    $args[$i] = $1;
                    $defaults{$args[$i]} = $2;
                    $defaults{$args[$i]} =~ s/"/\\"/g;
            }
+           $proto_arg[$i+1] = "\$" ;
     }
     if (defined($class)) {
            $func_args = join(", ", @args[1..$#args]);
@@ -254,14 +982,29 @@ while (&fetch_para) {
     }
     @args_match{@args} = 1..@args;
 
+    $PPCODE = grep(/^\s*PPCODE\s*:/, @line);
+    $CODE = grep(/^\s*CODE\s*:/, @line);
+    # Detect CODE: blocks which use ST(n)= or XST_m*(n,v)
+    #   to set explicit return values.
+    $EXPLICIT_RETURN = ($CODE &&
+               ("@line" =~ /(\bST\s*\([^;]*=) | (\bXST_m\w+\s*\()/x ));
+    $ALIAS  = grep(/^\s*ALIAS\s*:/,  @line);
+    $INTERFACE  = grep(/^\s*INTERFACE\s*:/,  @line);
+
     # print function header
     print Q<<"EOF";
-#XS(XS_${Packid}_$func_name)
+#XS(XS_${Full_func_name})
 #[[
 #    dXSARGS;
 EOF
+    print Q<<"EOF" if $ALIAS ;
+#    dXSI32;
+EOF
+    print Q<<"EOF" if $INTERFACE ;
+#    dXSFUNCTION($ret_type);
+EOF
     if ($elipsis) {
-       $cond = qq(items < $min_args);
+       $cond = ($min_args ? qq(items < $min_args) : 0);
     }
     elsif ($min_args == $num_args) {
        $cond = qq(items != $min_args);
@@ -275,10 +1018,15 @@ EOF
 #    *errbuf = '\0';
 EOF
 
-    print Q<<"EOF";
-#    if ($cond) {
-#      croak("Usage: $pname($orig_args)");
-#    }
+    if ($ALIAS) 
+      { print Q<<"EOF" if $cond }
+#    if ($cond)
+#       Perl_croak(aTHX_ "Usage: %s($orig_args)", GvNAME(CvGV(cv)));
+EOF
+    else 
+      { print Q<<"EOF" if $cond }
+#    if ($cond)
+#      Perl_croak(aTHX_ "Usage: $pname($orig_args)");
 EOF
 
     print Q<<"EOF" if $PPCODE;
@@ -288,82 +1036,34 @@ EOF
     # Now do a block of some sort.
 
     $condnum = 0;
-    if (!@line) {
-       @line = "CLEANUP:";
-    }
+    $cond = '';                        # last CASE: condidional
+    push(@line, "$END:");
+    push(@line_no, $line_no[-1]);
+    $_ = '';
+    &check_cpp;
     while (@line) {
-       if ($_[0] =~ s/^\s*CASE\s*:\s*//) {
-           $cond = shift(@line);
-           if ($condnum == 0) {
-               print "    if ($cond)\n";
-           }
-           elsif ($cond ne '') {
-               print "    else if ($cond)\n";
-           }
-           else {
-               print "    else\n";
-           }
-           $condnum++;
-       }
-
-       if ($except) {
-           print Q<<"EOF";
-#    TRY [[
+       &CASE_handler if check_keyword("CASE");
+       print Q<<"EOF";
+#   $except [[
 EOF
-       }
-       else {
-           print Q<<"EOF";
-#    [[
-EOF
-       }
 
        # do initialization of input variables
        $thisdone = 0;
        $retvaldone = 0;
        $deferred = "";
-       while (@line) {
-               $_ = shift(@line);
-               last if /^\s*NOT_IMPLEMENTED_YET/;
-               last if /^\s*(PPCODE|CODE|OUTPUT|CLEANUP|CASE)\s*:/;
-               ($var_type, $var_name, $var_init) =
-                   /\s*([^\t]+)\s*([^\s=]+)\s*(=.*)?/;
-               # Catch common errors. More error checking required here.
-               blurt("Error: no tab in $pname argument declaration '$_'\n")
-                       unless (m/\S+\s*\t\s*\S+/);
-               # catch C style argument declaration (this could be made alowable syntax)
-               warn("Warning: ignored semicolon in $pname argument declaration '$_'\n")
-                       if ($var_name =~ s/;//g); # eg SV *<tab>name;
-               # catch many errors similar to: SV<tab>* name
-               blurt("Error: invalid $pname argument name '$var_name' (type '$var_type')\n")
-                       unless ($var_name =~ m/^&?\w+$/);
-               if ($var_name =~ /^&/) {
-                       $var_name =~ s/^&//;
-                       $var_addr{$var_name} = 1;
-               }
-               $thisdone |= $var_name eq "THIS";
-               $retvaldone |= $var_name eq "RETVAL";
-               $var_types{$var_name} = $var_type;
-               print "\t" . &map_type($var_type);
-               $var_num = $args_match{$var_name};
-               if ($var_addr{$var_name}) {
-                       $func_args =~ s/\b($var_name)\b/&\1/;
-               }
-               if ($var_init !~ /^=\s*NO_INIT\s*$/) {
-                       if ($var_init !~ /^\s*$/) {
-                               &output_init($var_type, $var_num,
-                                   "$var_name $var_init");
-                       } elsif ($var_num) {
-                               # generate initialization code
-                               &generate_init($var_type, $var_num, $var_name);
-                       } else {
-                               print ";\n";
-                       }
-               } else {
-                       print "\t$var_name;\n";
-               }
-       }
+       %arg_list = () ;
+        $gotRETVAL = 0;
+
+       INPUT_handler() ;
+       process_keyword("INPUT|PREINIT|INTERFACE_MACRO|C_ARGS|ALIAS|PROTOTYPE|SCOPE") ;
+
+       print Q<<"EOF" if $ScopeThisXSUB;
+#   ENTER;
+#   [[
+EOF
+       
        if (!$thisdone && defined($class)) {
-           if (defined($static)) {
+           if (defined($static) or $func_name eq 'new') {
                print "\tchar *";
                $var_types{"CLASS"} = "char *";
                &generate_init("char *", 1, "CLASS");
@@ -377,7 +1077,8 @@ EOF
 
        # do code
        if (/^\s*NOT_IMPLEMENTED_YET/) {
-               print "\ncroak(\"$pname: not implemented yet\");\n";
+               print "\n\tPerl_croak(aTHX_ \"$pname: not implemented yet\");\n";
+               $_ = '' ;
        } else {
                if ($ret_type ne "void") {
                        print "\t" . &map_type($ret_type) . "\tRETVAL;\n"
@@ -385,146 +1086,263 @@ EOF
                        $args_match{"RETVAL"} = 0;
                        $var_types{"RETVAL"} = $ret_type;
                }
-               if (/^\s*PPCODE:/) {
-                       print $deferred;
-                       while (@line) {
-                               $_ = shift(@line);
-                               die "PPCODE must be last thing"
-                                   if /^\s*(OUTPUT|CLEANUP|CASE)\s*:/;
-                               print "$_\n";
-                       }
+
+               print $deferred;
+
+        process_keyword("INIT|ALIAS|PROTOTYPE|INTERFACE_MACRO|INTERFACE|C_ARGS") ;
+
+               if (check_keyword("PPCODE")) {
+                       print_section();
+                       death ("PPCODE must be last thing") if @line;
+                       print "\tLEAVE;\n" if $ScopeThisXSUB;
                        print "\tPUTBACK;\n\treturn;\n";
-               } elsif (/^\s*CODE:/) {
-                       print $deferred;
-                       while (@line) {
-                               $_ = shift(@line);
-                               last if /^\s*(OUTPUT|CLEANUP|CASE)\s*:/;
-                               print "$_\n";
-                       }
-               } elsif ($func_name eq "DESTROY") {
-                       print $deferred;
+               } elsif (check_keyword("CODE")) {
+                       print_section() ;
+               } elsif (defined($class) and $func_name eq "DESTROY") {
                        print "\n\t";
-                       print "delete THIS;\n"
+                       print "delete THIS;\n";
                } else {
-                       print $deferred;
                        print "\n\t";
                        if ($ret_type ne "void") {
                                print "RETVAL = ";
+                               $wantRETVAL = 1;
                        }
                        if (defined($static)) {
-                           if ($func_name =~ /^new/) {
+                           if ($func_name eq 'new') {
                                $func_name = "$class";
-                           }
-                           else {
-                               print "$class::";
+                           } else {
+                               print "${class}::";
                            }
                        } elsif (defined($class)) {
+                           if ($func_name eq 'new') {
+                               $func_name .= " $class";
+                           } else {
                                print "THIS->";
+                           }
                        }
-                       if (defined($spat) && $func_name =~ /^($spat)(.*)$/) {
-                               $func_name = $2;
-                       }
+                       $func_name =~ s/^($spat)//
+                           if defined($spat);
+                       $func_name = 'XSFUNCTION' if $interface;
                        print "$func_name($func_args);\n";
-                       &generate_output($ret_type, 0, "RETVAL")
-                           unless $ret_type eq "void";
                }
        }
 
        # do output variables
-       if (/^\s*OUTPUT\s*:/) {
-               while (@line) {
-                       $_ = shift(@line);
-                       last if /^\s*CLEANUP\s*:/;
-                       s/^\s+//;
-                       ($outarg, $outcode) = split(/\t+/);
-                       if ($outcode) {
-                           print "\t$outcode\n";
-                       } else {
-                               die "$outarg not an argument"
-                                   unless defined($args_match{$outarg});
-                               $var_num = $args_match{$outarg};
-                               &generate_output($var_types{$outarg}, $var_num,
-                                   $outarg); 
-                       }
-               }
+       $gotRETVAL = 0;
+       undef $RETVAL_code ;
+       undef %outargs ;
+        process_keyword("OUTPUT|ALIAS|PROTOTYPE"); 
+
+       # all OUTPUT done, so now push the return value on the stack
+       if ($gotRETVAL && $RETVAL_code) {
+           print "\t$RETVAL_code\n";
+       } elsif ($gotRETVAL || $wantRETVAL) {
+           # RETVAL almost never needs SvSETMAGIC()
+           &generate_output($ret_type, 0, 'RETVAL', 0);
        }
+
        # do cleanup
-       if (/^\s*CLEANUP\s*:/) {
-           while (@line) {
-                   $_ = shift(@line);
-                   last if /^\s*CASE\s*:/;
-                   print "$_\n";
-           }
-       }
+       process_keyword("CLEANUP|ALIAS|PROTOTYPE") ;
+
+       print Q<<"EOF" if $ScopeThisXSUB;
+#   ]]
+EOF
+       print Q<<"EOF" if $ScopeThisXSUB and not $PPCODE;
+#   LEAVE;
+EOF
+
        # print function trailer
-       if ($except) {
-           print Q<<EOF;
+       print Q<<EOF;
 #    ]]
+EOF
+       print Q<<EOF if $except;
 #    BEGHANDLERS
 #    CATCHALL
 #      sprintf(errbuf, "%s: %s\\tpropagated", Xname, Xreason);
 #    ENDHANDLERS
 EOF
+       if (check_keyword("CASE")) {
+           blurt ("Error: No `CASE:' at top of function")
+               unless $condnum;
+           $_ = "CASE: $_";    # Restore CASE: label
+           next;
        }
-       else {
-           print Q<<EOF;
-#    ]]
-EOF
-       }
-       if (/^\s*CASE\s*:/) {
-           unshift(@line, $_);
-       }
+       last if $_ eq "$END:";
+       death(/^$BLOCK_re/o ? "Misplaced `$1:'" : "Junk at end of function");
     }
 
     print Q<<EOF if $except;
 #    if (errbuf[0])
-#      croak(errbuf);
+#      Perl_croak(aTHX_ errbuf);
 EOF
 
-    print Q<<EOF unless $PPCODE;
+    if ($ret_type ne "void" or $EXPLICIT_RETURN) {
+        print Q<<EOF unless $PPCODE;
 #    XSRETURN(1);
 EOF
+    } else {
+        print Q<<EOF unless $PPCODE;
+#    XSRETURN_EMPTY;
+EOF
+    }
 
     print Q<<EOF;
 #]]
 #
 EOF
+
+    my $newXS = "newXS" ;
+    my $proto = "" ;
+
+    # Build the prototype string for the xsub
+    if ($ProtoThisXSUB) {
+       $newXS = "newXSproto";
+
+       if ($ProtoThisXSUB eq 2) {
+           # User has specified empty prototype
+           $proto = ', ""' ;
+       }
+        elsif ($ProtoThisXSUB ne 1) {
+            # User has specified a prototype
+            $proto = ', "' . $ProtoThisXSUB . '"';
+        }
+        else {
+           my $s = ';';
+            if ($min_args < $num_args)  {
+                $s = ''; 
+               $proto_arg[$min_args] .= ";" ;
+           }
+            push @proto_arg, "$s\@" 
+                if $elipsis ;
+    
+            $proto = ', "' . join ("", @proto_arg) . '"';
+        }
+    }
+
+    if (%XsubAliases) {
+       $XsubAliases{$pname} = 0 
+           unless defined $XsubAliases{$pname} ;
+       while ( ($name, $value) = each %XsubAliases) {
+           push(@InitFileCode, Q<<"EOF");
+#        cv = newXS(\"$name\", XS_$Full_func_name, file);
+#        XSANY.any_i32 = $value ;
+EOF
+       push(@InitFileCode, Q<<"EOF") if $proto;
+#        sv_setpv((SV*)cv$proto) ;
+EOF
+        }
+    } 
+    elsif ($interface) {
+       while ( ($name, $value) = each %Interfaces) {
+           $name = "$Package\::$name" unless $name =~ /::/;
+           push(@InitFileCode, Q<<"EOF");
+#        cv = newXS(\"$name\", XS_$Full_func_name, file);
+#        $interface_macro_set(cv,$value) ;
+EOF
+           push(@InitFileCode, Q<<"EOF") if $proto;
+#        sv_setpv((SV*)cv$proto) ;
+EOF
+        }
+    }
+    else {
+       push(@InitFileCode,
+            "        ${newXS}(\"$pname\", XS_$Full_func_name, file$proto);\n");
+    }
 }
 
 # print initialization routine
-print qq/extern "C"\n/ if $cplusplus;
+
+print Q<<"EOF";
+##ifdef __cplusplus
+#extern "C"
+##endif
+EOF
+
 print Q<<"EOF";
 #XS(boot_$Module_cname)
+EOF
+
+print Q<<"EOF";
 #[[
 #    dXSARGS;
 #    char* file = __FILE__;
 #
 EOF
 
-for (@Func_name) {
-    $pname = shift(@Func_pname);
-    print "    newXS(\"$pname\", XS_$_, file);\n";
-}
+print Q<<"EOF" if $WantVersionChk ;
+#    XS_VERSION_BOOTCHECK ;
+#
+EOF
+
+print Q<<"EOF" if defined $XsubAliases or defined $Interfaces ;
+#    {
+#        CV * cv ;
+#
+EOF
+
+print @InitFileCode;
+
+print Q<<"EOF" if defined $XsubAliases or defined $Interfaces ;
+#    }
+EOF
 
 if (@BootCode)
 {
     print "\n    /* Initialisation Section */\n\n" ;
-    print grep (s/$/\n/, @BootCode) ;
-    print "    /* End of Initialisation Section */\n\n" ;
+    @line = @BootCode;
+    print_section();
+    print "\n    /* End of Initialisation Section */\n\n" ;
 }
 
-print "    ST(0) = &sv_yes;\n";
-print "    XSRETURN(1);\n";
-print "}\n";
+print Q<<"EOF";;
+#    XSRETURN_YES;
+#]]
+#
+EOF
+
+warn("Please specify prototyping behavior for $filename (see perlxs manual)\n") 
+    unless $ProtoUsed ;
+&Exit;
 
 sub output_init {
-    local($type, $num, $init) = @_;
+    local($type, $num, $var, $init) = @_;
     local($arg) = "ST(" . ($num - 1) . ")";
 
-    eval qq/print " $init\\\n"/;
+    if(  $init =~ /^=/  ) {
+       eval qq/print "\\t$var $init\\n"/;
+       warn $@   if  $@;
+    } else {
+       if(  $init =~ s/^\+//  &&  $num  ) {
+           &generate_init($type, $num, $var);
+       } else {
+           eval qq/print "\\t$var;\\n"/;
+           warn $@   if  $@;
+           $init =~ s/^;//;
+       }
+       $deferred .= eval qq/"\\n\\t$init\\n"/;
+       warn $@   if  $@;
+    }
+}
+
+sub Warn
+{
+    # work out the line number
+    my $line_no = $line_no[@line_no - @line -1] ;
+    print STDERR "@_ in $filename, line $line_no\n" ;
+}
+
+sub blurt 
+{ 
+    Warn @_ ;
+    $errors ++ 
 }
 
-sub blurt { warn @_; $errors++ }
+sub death
+{
+    Warn @_ ;
+    exit 1 ;
+}
 
 sub generate_init {
     local($type, $num, $var) = @_;
@@ -533,55 +1351,76 @@ sub generate_init {
     local($ntype);
     local($tk);
 
-    blurt("'$type' not in typemap"), return unless defined($type_kind{$type});
+    $type = TidyType($type) ;
+    blurt("Error: '$type' not in typemap"), return 
+       unless defined($type_kind{$type});
+
     ($ntype = $type) =~ s/\s*\*/Ptr/g;
-    $subtype = $ntype;
-    $subtype =~ s/Ptr$//;
-    $subtype =~ s/Array$//;
+    ($subtype = $ntype) =~ s/(?:Array)?(?:Ptr)?$//;
     $tk = $type_kind{$type};
     $tk =~ s/OBJ$/REF/ if $func_name =~ /DESTROY$/;
-    $type =~ s/:/_/g;
+    $type =~ tr/:/_/;
+    blurt("Error: No INPUT definition for type '$type' found"), return
+        unless defined $input_expr{$tk} ;
     $expr = $input_expr{$tk};
     if ($expr =~ /DO_ARRAY_ELEM/) {
+        blurt("Error: '$subtype' not in typemap"), return 
+           unless defined($type_kind{$subtype});
+        blurt("Error: No INPUT definition for type '$subtype' found"), return
+            unless defined $input_expr{$type_kind{$subtype}} ;
        $subexpr = $input_expr{$type_kind{$subtype}};
        $subexpr =~ s/ntype/subtype/g;
        $subexpr =~ s/\$arg/ST(ix_$var)/g;
        $subexpr =~ s/\n\t/\n\t\t/g;
-       $subexpr =~ s/is not of (.*")/[arg %d] is not of $1, ix_$var + 1/g;
+       $subexpr =~ s/is not of (.*\")/[arg %d] is not of $1, ix_$var + 1/g;
        $subexpr =~ s/\$var/${var}[ix_$var - $argoff]/;
        $expr =~ s/DO_ARRAY_ELEM/$subexpr/;
     }
+    if ($expr =~ m#/\*.*scope.*\*/#i) { # "scope" in C comments
+       $ScopeThisXSUB = 1;
+    }
     if (defined($defaults{$var})) {
            $expr =~ s/(\t+)/$1    /g;
            $expr =~ s/        /\t/g;
            eval qq/print "\\t$var;\\n"/;
+           warn $@   if  $@;
            $deferred .= eval qq/"\\n\\tif (items < $num)\\n\\t    $var = $defaults{$var};\\n\\telse {\\n$expr;\\n\\t}\\n"/;
-    } elsif ($expr !~ /^\t\$var =/) {
+           warn $@   if  $@;
+    } elsif ($ScopeThisXSUB or $expr !~ /^\t\$var =/) {
            eval qq/print "\\t$var;\\n"/;
+           warn $@   if  $@;
            $deferred .= eval qq/"\\n$expr;\\n"/;
+           warn $@   if  $@;
     } else {
            eval qq/print "$expr;\\n"/;
+           warn $@   if  $@;
     }
 }
 
 sub generate_output {
-    local($type, $num, $var) = @_;
+    local($type, $num, $var, $do_setmagic) = @_;
     local($arg) = "ST(" . ($num - ($num != 0)) . ")";
     local($argoff) = $num - 1;
     local($ntype);
 
+    $type = TidyType($type) ;
     if ($type =~ /^array\(([^,]*),(.*)\)/) {
            print "\tsv_setpvn($arg, (char *)$var, $2 * sizeof($1)), XFree((char *)$var);\n";
+           print "\tSvSETMAGIC($arg);\n" if $do_setmagic;
     } else {
-           blurt("'$type' not in typemap"), return
+           blurt("Error: '$type' not in typemap"), return
                unless defined($type_kind{$type});
+            blurt("Error: No OUTPUT definition for type '$type' found"), return
+                unless defined $output_expr{$type_kind{$type}} ;
            ($ntype = $type) =~ s/\s*\*/Ptr/g;
            $ntype =~ s/\(\)//g;
-           $subtype = $ntype;
-           $subtype =~ s/Ptr$//;
-           $subtype =~ s/Array$//;
+           ($subtype = $ntype) =~ s/(?:Array)?(?:Ptr)?$//;
            $expr = $output_expr{$type_kind{$type}};
            if ($expr =~ /DO_ARRAY_ELEM/) {
+               blurt("Error: '$subtype' not in typemap"), return
+                   unless defined($type_kind{$subtype});
+                blurt("Error: No OUTPUT definition for type '$subtype' found"), return
+                    unless defined $output_expr{$type_kind{$subtype}} ;
                $subexpr = $output_expr{$type_kind{$subtype}};
                $subexpr =~ s/ntype/subtype/g;
                $subexpr =~ s/\$arg/ST(ix_$var)/g;
@@ -589,41 +1428,58 @@ sub generate_output {
                $subexpr =~ s/\n\t/\n\t\t/g;
                $expr =~ s/DO_ARRAY_ELEM\n/$subexpr/;
                eval "print qq\a$expr\a";
+               warn $@   if  $@;
+               print "\t\tSvSETMAGIC(ST(ix_$var));\n" if $do_setmagic;
            }
            elsif ($var eq 'RETVAL') {
-               if ($expr =~ /^\t\$arg = /) {
+               if ($expr =~ /^\t\$arg = new/) {
+                   # We expect that $arg has refcnt 1, so we need to
+                   # mortalize it.
+                   eval "print qq\a$expr\a";
+                   warn $@   if  $@;
+                   print "\tsv_2mortal(ST(0));\n";
+                   print "\tSvSETMAGIC(ST(0));\n" if $do_setmagic;
+               }
+               elsif ($expr =~ /^\s*\$arg\s*=/) {
+                   # We expect that $arg has refcnt >=1, so we need
+                   # to mortalize it!
                    eval "print qq\a$expr\a";
+                   warn $@   if  $@;
                    print "\tsv_2mortal(ST(0));\n";
+                   print "\tSvSETMAGIC(ST(0));\n" if $do_setmagic;
                }
                else {
+                   # Just hope that the entry would safely write it
+                   # over an already mortalized value. By
+                   # coincidence, something like $arg = &sv_undef
+                   # works too.
                    print "\tST(0) = sv_newmortal();\n";
                    eval "print qq\a$expr\a";
+                   warn $@   if  $@;
+                   # new mortals don't have set magic
                }
            }
            elsif ($arg =~ /^ST\(\d+\)$/) {
                eval "print qq\a$expr\a";
-           }
-           elsif ($arg =~ /^ST\(\d+\)$/) {
-               eval "print qq\a$expr\a";
-           }
-           elsif ($arg =~ /^ST\(\d+\)$/) {
-               eval "print qq\a$expr\a";
+               warn $@   if  $@;
+               print "\tSvSETMAGIC($arg);\n" if $do_setmagic;
            }
     }
 }
 
 sub map_type {
-    local($type) = @_;
+    my($type) = @_;
 
-    $type =~ s/:/_/g;
-    if ($type =~ /^array\(([^,]*),(.*)\)/) {
-           return "$1 *";
-    } else {
-           return $type;
-    }
+    $type =~ tr/:/_/;
+    $type =~ s/^array\(([^,]*),(.*)\).*/$1 */s;
+    $type;
 }
 
+
+sub Exit {
 # If this is VMS, the exit status has meaning to the shell, so we
-# use a predictable value (SS$_Abort) rather than an arbitrary
-# number.
-exit $Is_VMS ? 44 : $errors;
+# use a predictable value (SS$_Normal or SS$_Abort) rather than an
+# arbitrary number.
+#    exit ($Is_VMS ? ($errors ? 44 : 1) : $errors) ;
+    exit ($errors ? 1 : 0);
+}