Factor most of the ARGV filtering logic into its own sub

diff --git a/Makefile.PL b/Makefile.PL
index ac429aaa05c6765ea0aa33b06c9d6c30229a31ce..49e3969ceef0de4cf502d38bc9823d22c61c382a 100644 (file)
--- a/Makefile.PL
+++ b/Makefile.PL
@@ -72,24 +72,35 @@ sub check_exe {
  return $exe;
 }
 
-my $pp;
-for my $i (0 .. $#ARGV) {
- my $arg = $ARGV[$i];
- if ($arg =~ /^PP=(.*)/) {
-  my $val = $1;
-  if (do { no warnings 'numeric'; int $val } or $val =~ /^(?:y|yes)$/i) {
-   print "Forcing the pure-Perl implementation from the arguments passed to Makefile.PL.\n";
-   $pp = 1;
-   last;
+sub filter_argv {
+ my ($var, $code) = @_;
+
+ my $ret;
+
+ for my $i (0 .. $#ARGV) {
+  my $arg = $ARGV[$i];
+  if ($arg =~ /^\Q$var\E=(.*)/) {
+   my $val = $1;
+   $ret = $code->($val);
+   $ARGV[$i] = undef;
+   last if $ret;
   }
-  $ARGV[$i] = undef;
  }
-}
 
-@ARGV = grep defined, @ARGV;
+ @ARGV = grep defined, @ARGV;
+
+ return $ret;
+}
 
 my ($cc, $ld);
-unless ($pp) {
+my $pp = filter_argv PP => sub {
+ my ($val) = @_;
+ return (do { no warnings 'numeric'; int $val } or $val =~ /^(?:y|yes)$/i)
+        ? 1 : 0;
+};
+if ($pp) {
+ print "Forcing the pure-Perl implementation from the arguments passed to Makefile.PL.\n";
+} else {
  $cc = check_exe('C compiler', 'CC', 'cc', 'CC');
  if (defined $cc) {
   my ($vol, $dir, $file) = File::Spec->splitpath($cc);