| Subject: | ProhibitIndirectSyntax on dynaloader bootstrap() |
| Date: | Sun, 21 Mar 2010 09:58:45 +1100 |
| To: | bug-Perl-Critic [...] rt.cpan.org |
| From: | Kevin Ryde <user42 [...] zip.com.au> |
As an idea for a feature, ProhibitIndirectSyntax could report on the
common DynaLoader form
bootstrap Foo;
or
bootstrap Foo $VERSION;
I'd used it several times over the years by cut and paste before
realizing it was a method call instead of some magical function :-).
Anything where "bootstrap" is followed by the current package name would
be pretty unambiguous and could be a builtin.
I don't think bootstrap normally appears with anything except the
current package name either. I suppose it would work to make the call
from somewhere else, but that'd have to be rare.