From: Vincent Pit <vince@profvince.com>
Date: Mon, 28 Jul 2008 16:14:09 +0000 (+0200)
Subject: Also fool B::Deparse::single_delim when patchlevel >= 31268. Make Config a dependency
X-Git-Tag: v0.02~4
X-Git-Url: http://git.vpit.fr/?a=commitdiff_plain;h=f92b75386a7476c420e66629a5a57d1882c0c2f4;p=perl%2Fmodules%2FB-RecDeparse.git

Also fool B::Deparse::single_delim when patchlevel >= 31268. Make Config a dependency
---

diff --git a/META.yml b/META.yml
index bc040de..7c4e4d1 100644
--- a/META.yml
+++ b/META.yml
@@ -10,6 +10,7 @@ distribution_type:   module
 requires:     
     B::Deparse:                    0
     Carp:                          0
+    Config:                        0
 meta-spec:
     url:     http://module-build.sourceforge.net/META-spec-v1.3.html
     version: 1.3
diff --git a/Makefile.PL b/Makefile.PL
index 065172c..a569c7b 100644
--- a/Makefile.PL
+++ b/Makefile.PL
@@ -25,8 +25,9 @@ WriteMakefile(
     ABSTRACT_FROM => 'lib/B/RecDeparse.pm',
     PL_FILES      => {},
     PREREQ_PM     => {
-        'Carp'       => 0,
-        'B::Deparse' => 0
+        'B::Deparse' => 0,
+        'Config'     => 0,
+        'Carp'       => 0
     },
     dist          => {
         PREOP        => 'pod2text lib/Variable/Magic.pm > $(DISTVNAME)/README; '
diff --git a/lib/B/RecDeparse.pm b/lib/B/RecDeparse.pm
index fb712d9..da731d9 100644
--- a/lib/B/RecDeparse.pm
+++ b/lib/B/RecDeparse.pm
@@ -4,6 +4,7 @@ use strict;
 use warnings;
 
 use Carp qw/croak/;
+use Config;
 
 use base qw/B::Deparse/;
 
@@ -43,6 +44,13 @@ The L<B::RecDeparse> object constructor. You can specify the underlying L<B::Dep
 
 =cut
 
+use constant {
+ # p31268 made pp_entersub call single_delim
+ FOOL_SINGLE_DELIM =>
+     ($^V ge v5.9.5)
+  || ($Config{perl_patchlevel} && $Config{perl_patchlevel} >= 31268)
+};
+
 sub _parse_args {
  croak 'Optional arguments must be passed as key/value pairs' if @_ % 2;
  my %args = @_;
@@ -97,8 +105,7 @@ sub init {
 
 my $key = $; . __PACKAGE__ . $;;
 
-# p31268 made pp_entersub call single_delim
-if ($^V ge v5.9.5) {
+if (FOOL_SINGLE_DELIM) {
  my $oldsd = *B::Deparse::single_delim{CODE};
  no warnings 'redefine';
  *B::Deparse::single_delim = sub {
@@ -138,10 +145,10 @@ sub pp_gv {
   ++$self->{brd_cur};
   $body = 'sub ' . $self->indent($self->deparse_sub($gv->CV));
   --$self->{brd_cur};
-  if ($^V lt v5.9.5) {
-   $body .= '->';
-  } else {
+  if (FOOL_SINGLE_DELIM) {
    $body = $key . $body;
+  } else {
+   $body .= '->';
   }
  }
  return $body;
@@ -167,7 +174,7 @@ An object-oriented module shouldn't export any function, and so does this one.
 
 =head1 DEPENDENCIES
 
-L<Carp> (standard since perl 5), L<B::Deparse> (since perl 5.005).
+L<Carp> (standard since perl 5), L<Config> (since perl 5.00307) and L<B::Deparse> (since perl 5.005).
 
 =head1 AUTHOR