Stories
Slash Boxes
Comments
NOTE: use Perl; is on undef hiatus. You can read content, but you can't post it. More info will be forthcoming forthcomingly.

All the Perl that's Practical to Extract and Report

The Fine Print: The following comments are owned by whoever posted them. We are not responsible for them in any way.

Module::Pluggable vs. Emacs 5 Comments More | Login | Reply /

 Full
 Abbreviated
 Hidden
More | Login | Reply
Loading... please wait.

  • Since Foo::.#Home.pm is not a valid module name, I don't think Module::Pluggable should be loading it. It's not a difficult thing to check. I'd file the bug report.

    • I concur.

      I'd expect Module::Pluggable to skip at the very least hidden files) but checking that the file name matches a legitimate module naming pattern works as well.
      • In fact, it works better, as it's a whitelist rather than a blacklist. So it won't have to be updated in the future. For example, I'd bet that you also want to ignore @._*@ (OS X droppings).
    • Hmm, I thought I was skipping non valid files. I'll have a check and see what's going on.
      • I think this line is at fault:

          next unless $plugin =~ m!(?:[a-z\d]+)[a-z\d]!i;
        It should probably be something like:

          next unless $plugin =~ m!\A [a-z]\w* (?: ::[a-z]\w* )* \z!ix;
        Does Perl allow Unicode in module names? I though not, but if so [a-z] is too restrictive
Stories, comments, journals, and other submissions on use Perl; are Copyright 1998-2006, their respective owners.