Add the PERL_INDIRECT_PM_DISABLE environment variable

diff --git a/MANIFEST b/MANIFEST
index 1bf2e07965f6f644c6768f11eb3554e421ac8fe0..5090e230d1dbeca0d81bf85884691b25bfcdd3a6 100644 (file)
--- a/MANIFEST
+++ b/MANIFEST
@@ -10,6 +10,7 @@ samples/indirect.pl
 t/00-load.t
 t/10-args.t
 t/11-line.t
+t/12-env.t
 t/20-good.t
 t/21-bad.t
 t/22-bad-mixed.t

diff --git a/lib/indirect.pm b/lib/indirect.pm
index 27c37b469cfcd4808f8a14341b4578c0c7ee8f63..c6ae98a86c16164e4c6e690478d85ce23dcacd32 100644 (file)
--- a/lib/indirect.pm
+++ b/lib/indirect.pm
@@ -58,8 +58,12 @@ This module is B<not> a source filter.
 =cut
 
 BEGIN {
- require XSLoader;
- XSLoader::load(__PACKAGE__, $VERSION);
+ if ($ENV{PERL_INDIRECT_PM_DISABLE}) {
+  *_tag = sub ($) { 1 };
+ } else {
+  require XSLoader;
+  XSLoader::load(__PACKAGE__, $VERSION);
+ }
 }
 
 =head1 METHODS
@@ -153,6 +157,16 @@ The default warning/exception message thrown when an indirect call on an object
 
 The default warning/exception message thrown when an indirect call on a block is found.
 
+=head1 ENVIRONMENT
+
+=head2 C<PERL_INDIRECT_PM_DISABLE>
+
+If this environment variable is set to true when the pragma is used for the first time, the XS code won't be loaded and, although the C<'indirect'> lexical hint will be set to true in the scope of use, the pragma itself won't do anything.
+This is useful for disabling C<indirect> in production environments.
+
+Note that clearing this variable after C<indirect> was loaded has no effect.
+If you want to reenable the pragma later, you also need to reload it by deleting the C<'indirect.pm'> entry from C<%INC>.
+
 =head1 CAVEATS
 
 The implementation was tweaked to work around several limitations of vanilla C<perl> pragmas : it's thread safe, and doesn't suffer from a C<perl 5.8.x-5.10.0> bug that causes all pragmas to propagate into C<require>d scopes.

diff --git a/t/12-env.t b/t/12-env.t
new file mode 100644 (file)
index 0000000..2e3584b
--- /dev/null
+++ b/t/12-env.t
@@ -0,0 +1,19 @@
+#!perl
+
+use strict;
+use warnings;
+
+use Test::More tests => 3;
+
+{
+ local $ENV{PERL_INDIRECT_PM_DISABLE} = 1;
+ my $err = 0;
+ my $res = eval <<' TEST_ENV_VARIABLE';
+  return 1;
+  no indirect hook => sub { ++$err };
+  my $x = new Flurbz;
+ TEST_ENV_VARIABLE
+ is $@,   '', 'PERL_INDIRECT_PM_DISABLE test doesn\'t croak';
+ is $res, 1,  'PERL_INDIRECT_PM_DISABLE test returns the correct value';
+ is $err, 0,  'PERL_INDIRECT_PM_DISABLE test didn\'t generate any error';
+}