Eliminate sub ProtoString(), which was used in only one location.
authorJames E. Keenan <jkeenan@cpan.org>
Sat, 27 Mar 2010 02:44:35 +0000 (22:44 -0400)
committerSteffen Mueller <smueller@cpan.org>
Tue, 12 Jul 2011 18:53:52 +0000 (20:53 +0200)
dist/ExtUtils-ParseXS/lib/ExtUtils/ParseXS.pm

index 53525e1..1c07c3a 100644 (file)
@@ -1176,8 +1176,9 @@ sub INPUT_handler {
     }
     $var_num = $args_match{$var_name};
 
-    $proto_arg[$var_num] = ProtoString($var_type)
-      if $var_num;
+    if ($var_num) {
+      $proto_arg[$var_num] = $proto_letter{$var_type} || "\$";
+    }
     $func_args =~ s/\b($var_name)\b/&$1/ if $var_addr;
     if ($var_init =~ /^[=;]\s*NO_INIT\s*;?\s*$/
       or $in_out{$var_name} and $in_out{$var_name} =~ /^OUT/
@@ -1628,12 +1629,6 @@ EOF
   return 1;
 }
 
-sub ProtoString ($) {
-  my ($type) = @_;
-
-  $proto_letter{$type} or "\$";
-}
-
 sub check_cpp {
   my @cpp = grep(/^\#\s*(?:if|e\w+)/, @line);
   if (@cpp) {