search

doy / package-stash-xs

faster and more correct implementation of the Package::Stash API
4 stars 3 forks source link

I.. I have no idea what to even call this #3

Open karenetheridge opened 10 years ago

karenetheridge commented 10 years ago

the original report: http://www.cpantesters.org/cpan/report/41acd3f2-238d-11e4-9431-1786c7ccabfc

@haarg boiled this down to:

use strict;
use warnings FATAL => 'all';

use Package::Stash::XS;
unshift @INC, sub {
  "a" =~ /a/;
  return;
};

Package::Stash::XS->new("abc::def");
doy commented 10 years ago

This is called "regular expression matching is not reentrant in 5.8". I'm not particularly interested in trying to solve this myself, but if anyone has any non-insane patches, I'd be willing to look at them.

doy commented 10 years ago

See also: https://rt.cpan.org/Public/Bug/Display.html?id=74151

haarg commented 10 years ago

The modules getting loaded, causing the @INC hook to be triggered, are utf8.pm and utf8_heavy.pl. Could _is_valid_module_name either avoid using UTF8, or preload the modules?

ribasushi commented 4 years ago

@haarg it seems "the cabal" has perms for this, can it ship somehow...?