fix 5.16 crash due to qw() list being readonly

diff --git a/Changes b/Changes
index f4307f7..4f53a99 100644 (file)
--- a/Changes
+++ b/Changes
@@ -1,3 +1,4 @@
+  - fix 5.16 crash due to qw() list being readonly
 1.004003 - 2012-11-10
   - check only once for presence of extra testing prereqs
   - explicitly specify no dynamic_config in META

diff --git a/lib/strictures.pm b/lib/strictures.pm
index 714faaa..21bd8b4 100644 (file)
--- a/lib/strictures.pm
+++ b/lib/strictures.pm
@@ -46,13 +46,16 @@ sub import {
     $extra_load_states ||= do {
 
       my (%rv, @failed);
-      for my $mod (qw(indirect multidimensional bareword::filehandles)) {
+      foreach my $mod (qw(indirect multidimensional bareword::filehandles)) {
         eval "require $mod; \$rv{'$mod'} = 1;" or do {
           push @failed, $mod;
 
           # courtesy of the 5.8 require bug
-          $mod =~ s|::|/|g;
-          delete $INC{"$mod.pm"};
+          # (we do a copy because 5.16.2 at least uses the same read-only
+          # scalars for the qw() list and it doesn't seem worth a $^V check)
+
+          (my $file = $mod) =~ s|::|/|g;
+          delete $INC{"${file}.pm"};
         };
       }