NAME

Perl::Critic::Policy::Subroutines::ProtectPrivateSubs - Prevent access to private subs in other packages.

AFFILIATION

This Policy is part of the core Perl::Critic distribution.

DESCRIPTION

By convention Perl authors (like authors in many other languages) indicate private methods and variables by inserting a leading underscore before the identifier. This policy catches attempts to access private variables from outside the package itself.

The subroutines in the POSIX package which begin with an underscore (e.g. POSIX::_POSIX_ARG_MAX) are not flagged as errors by this policy.

CONFIGURATION

You can define what a private subroutine name looks like by specifying a regular expression for the private_name_regex option in your .perlcriticrc:

[Subroutines::ProtectPrivateSubs]
private_name_regex = _(?!_)\w+

The above example is a way of saying that subroutines that start with a double underscore are not considered to be private. (Perl::Critic, in its implementation, uses leading double underscores to indicate a distribution-private subroutine-- one that is allowed to be invoked by other Perl::Critic modules, but not by anything outside of Perl::Critic.)

You can configure additional subroutines to accept by specifying them in a space-delimited list to the allow option:

[Subroutines::ProtectPrivateSubs]
allow = FOO::_bar FOO::_baz

These are added to the default list of exemptions from this policy. Allowing a subroutine also allows the corresponding method call. So FOO::_bar in the above example allows both FOO::_bar() and FOO->_bar().

HISTORY

This policy is inspired by a similar test in B::Lint.

BUGS

Doesn't forbid $pkg->_foo() because it can't tell the difference between that and $self->_foo().

SEE ALSO

Perl::Critic::Policy::Variables::ProtectPrivateVars

AUTHOR

Chris Dolan <cdolan@cpan.org>

COPYRIGHT

Copyright (c) 2006-2011 Chris Dolan.

This program is free software; you can redistribute it and/or modify it under the same terms as Perl itself. The full text of this license can be found in the LICENSE file included with this module.