Added -forced-filename option to the P::C::Document contstructor.
This allows you to give the Document a filename in situations where
the Document is being constructed from something that doesn't already
have a filename (like a reference to a scalar containing actual source
code).

RT #61311: Subroutines::ProhibitUnusedPrivateSubroutines dies on
"&_name" call

The problem was that the return value of sprevious_sibling() was being
incorrectly tested.

About 12 lines into sub _find_sub_reference_in_document(), and again
about 18 lines in, the code 'defined $prior' occurs. The 'defined'
should be removed; that is, the code should just be '$prior'.

The erroneous code assumed that the PPI method sprevious_sibling()
returned undef if there was no previous sibling in the parse tree. In
fact, it is documented as returning a false value.

Add stop words to P::C::P::ControlStructures::ProhibitPostfixControls.

Move the Changes comment on commit 3919 to the 'Bugs' section (what was
I thinking?) and added the RT number.

RT#61071: OptionsProcessor vs perl 5.8.3 Exporter

Updated inc/Perl/Critic/BuildUtilities.pm to require Exporter version
5.63, to get a version that understands export tags in positions other
than leading ones. Another solution was to move all the tags to the
leading position, but I thought it would be difficult to ensure that the
code stayed that way. I could also have gone with version 5.59 (the
first dual-lifed version), but the CPAN testers say 5.63 (the current
version) works back to perl 5.6.1.

Add "when" support to ControlStructures::ProhibitPostfixControls.

Add '_' to the tr/// that counts arguments in the subroutine prototype,
since this is now valid (meaning, '$ but use $_ if no argument
specified').

Make Subroutines::ProhibitManyArgs understand prototype grouping. That
is, prototype (\[$@%]) specifies one argument, not three. Tests for this
added to .run file.

Reduce use of PPI overloads i
ControlStructures::ProhibitPostfixControls.

Bump trunk version of Perl-Critic to 1.110.

Update Changes on trunk to match what was released in 1.109.

It turns out that the original complaint in RT #3077 for
RequireInterpolationOfMetachars dinging use of overload.pm really was
a general issue that the policy shouldn't complain about '${}' or
'@{}' because they're invalid syntax.

Fix detection of q<'@whatever'> in
ValuesAndExpressions::RequireInterpolationOfMetachars found due to the
Email::Address change.

Added method Perl::Critic::Document->ppix_regexp_from_element(), as a
caching wrapper for PPIx::Regexp->new(), inspired by find() and (more
recently) _nodes_by_namespace().

Rewrote all uses of PPIx::Regexp->new_from_cache() and
PPIx::Regexp->new() (found in
RegularExpressions::ProhibitComplexRegexes) in terms of
ppix_regexp_from_element().

Got rid of PPIx::Regexp->flush_cache(), which is no longer needed, and
which I was never really happy with. With the cache on the document, it
Just Works, stays as long as it is needed, and automatically goes away
when it is no longer needed.

This means 'use PPIx::Regexp' only appears in Perl::Critic::Document.

Change the test in t/08_document.t which tests
Perl::Critic::Document->uses_module() when there are no include
statements. The original (with an empty document) tickles what just
became RT #60671 in the PPI queue. The minimal way to avoid this
appeared to be using a document consisting of a single space.

It turns out that if you call PPI::Document->index_locations() on a
document with no tokens at all, it gets confused, and that's all RT
#60671 is about. Adam hadn't revoked my commit bit yet, so I committed
a proposed fix.

Since PPIx::Regexp is no longer optional, some of the tests that use it
no longer need separate .t files. These are moved into the .run files.
Unfortunately it looks like I forgot to add
t/20_policy_prohibit_unused_variables.t to subversion in the first
place, but that's where the tests in the .run file came from.

Added version to 'use PPIx::Regexp' in lib/Perl/Critic.pm.

Add failing test for
ValuesAndExpressions::RequireInterpolationOfMetachars for punctuation
variables (it fails to complain about q<'@_'>) and fix a small
self-compliance issue in the Policy itself.

Add "subscripted" to xt/author/40_stop_words.

Fix missing import of PPIx::Regexp in Perl::Critic.

Make xt/author/94_includes.t be more descriptive of what it's testing.

Note ValuesAndExpressions::RequireInterpolationOfMetachars fix in
Changes.

Make PPIx::Regexp, Perl::Tidy, Pod::Spell, and Text::ParseWords
non-optional.

Fix ValuesAndExpressions::RequireInterpolationOfMetachars with respect
to Email::Address 1.890.  This consists of checking for the presence
of an "@" anywhere in the value and removal of a bad test: "@{}" and
"${}" are syntax errors.

Move IPC::Open2 from the recommended to required dependencies since
it's core.

Realized I could simplify the code in commits 3886 and 3888 (both to do
with having P::C::P::RegularExpressions::ProhibitUnusedCapture look
inside regular expressions) because PPIx::Regexp::Token::Interpolation
is a subclass of PPIx::Regexp::Token::Code, and so I could just look for
the latter, which may appear anywhere in the regexp.

Yes, this proves I don't know my way around my own code. Sigh.

Variables::ProhibitUnusedVariables now looks inside regular expressions
if PPIx::Regexp can be loaded.

Have Subroutines::ProhibitUnusedPrivateSubroutines look inside regular
expressions if PPIx::Regexp is available.

Added the line number of the offending '=head1 NAME' to the
P::C::P::Documentation::RequirePackageMatchesPodName violation
description, using the same dodge Elliot used for
Documentation::RequirePodLinksIncludeText. This is kind of for RT
#59176, though that ticket asked for a way to jigger the line number in
the exception.

#59065 Suggested policy: prohibit conditional use statements

Modules::ProhibitContitionalUseStatements and the associated .run file
were attached to the RT ticket.  These are committed with very minor
changes:
* Added =for stopwords evals;
* Dropped trailing space in a few places.

These changes were made to satisfy authortest.

Update inc/Perl/Critic/BuildUtilities.pm to require version 0.010 of
PPIx::Regexp, since that is needed for the
P::C::P::RegularExpressions::ProhibitUnusedCapture ability to find
captures in the replacement portion of s/.../.../e.

This really should have been in commit 3888, but I forgot. Sorry.

Have RegularExpressions::ProhibitUnusedCapture account for capture
variables used in the replacement portion of s/.../.../e.

#60179: BuiltinFunctions::ProhibitStringyEval dies on eval "#...";

The code assumed that a PPI::Document manufactured from the argument to
a stringy eval would have significant children. But Sullivan Beck, in
Date::Manip 5.54, found a reason to do a stringy eval on a comment. This
resulted in a fatal error in the policy.

The fix was simply to check to see if the first significant child was
actually defined, and report a violation if not. The .run file was
updated as well. If the patch is not in place, the .run file experiences
a fatal error, not a test failure. Caveat tester.

Added support for @-, @+, and their English equivalents. This only works
for literal integer subscripts, though. There appears to be no way for
static analysis to take account of $^N and its English equivalent.

Forgot to put the RT number on the ProhibitUnusedCapture fix committed
as svn revision 3884. It's RT #60002.

RT #60002 - RegularExpressions::ProhibitUnusedCapture does not
understand English equivalents of capture variables.

The variable of immediate concern is $LAST_PAREN_MATCH. This is now
recognized provide the document includes 'use English;'.

Add new policy InputOutput::RequireEncodingWithUTF8Layer. This complains
if it sees ':utf8' rather than ':encoding(utf8)' in open() or binmode().
It is severity 5.

The issue is that ':utf8' flags the string as UTF8 without validating
it. Invalid UTF8 can have bad effects in your code. Exhibit "A" is an
exploit reported on PerlMonks where a script running with -T read input
and sanitized it with m/^(\w+)$/, after which $1 contained shell
meta-characters. This leads to the same kind of problems that the
two-argument open() has, and that policy is severity 5.

On the mailing list, Andy Lester suggested having the "policy warning
against checking $@" recommend Try::Tiny. In the belief that this means
P::C::P::ErrorHandling::RequireCheckingReturnValueOfEval, I have done my
best (or worst) to implement the suggestion there.

RT #59268: Documentation::RequirePodSections should not complain until the first line of POD

Saved the first PPI::Token::Pod containing a '=head1', for use as the
pod-of-record. Reported the error relative to this pod-of-record, so
that the disabling annotation can come before the __END__, the __DATA__,
or the pod-of-record, whichever comes first.

Updated the .run file to test for disablement. Yes, it failed before
applying the patch to the policy.

And holy smokes, Sandy, he remembered to update the Changes file!

RT 59176: "no critic" for pod after __END__

The problem turned out to be that the determination of the end of the
scope of the Pod::Critic::Annotation was short-stopped because PPI
parses the __END__ as a PPI::Statement::End, and everything after that
is descended from the PPI::Statement::End. The solution was to descend
into the PPI::Statement::End (if it was the last thing found) and
continue the analysis. I suppose it's still possible to short-stop the
annotation, but I can't think of a way to do it.

t/03_annotations.t was updated to check for the ability to extend
annotations into the __END__ of the document.

Astoundingly, the Changes file was also updated.

Specify a minimum version of version.pm in BuildUtilities.  RT #58952.

Nit-pick the docs in P::C::P::RegularExpressions::RequireDotMatchAnything

Handle the absence of include statements in
P::C::Document::uses_module().

Remove debugging code from P::C::Document.

Fix stupid bug in P::C::Document::_nodes_by_namespace().

Stupid just-prior-to-commit change to Test::P::C::Policy broke
everything.  :]

Make Test::P::C::Policy well behaved and not export anything by
default.

Code cleanup of ProhibitMismatchedOperators after applying patch from
RT #58751.

Apply the patch from RT #58751 to ProhibitMismatchedOperators
unmodified (modulo trailing whitespace).  Will need cleanup.

Create P::C::Document::uses_module().

Forgot to add PPIx::Utilities::Node to required modules in
BuildUtilities.

Create namespaces() and subdocuments_for_namespace() on
P::C::Document.

Make Test::Deep required so that I can use it for P::C::Document
testing.

Updated P::C::P::ControlStructures::ProhibitMutatingListFunctions to
recognize that s///r (introduced in Perl 5.13.2) does not modify its
argument (instead, it returns the modified string). There is no test for
Perl version, because s///r fails to parse under earlier Perls.

Added a test to the .run file, to check for this.

Updated the Changes file (will wonders never cease!)

Change xt/author/80_policysummary.t to match revision 3847, which
changed '[ Severity #]' to '[ Default severity #]'.

Change example in P::C Developer to say "Default severity" instead of
"Severity" in order to match the recent change to PolicySummary.

Change the PolicySummary to use short names for link names.

Change "[Severity #]" in the PolicySummary to "[Default severity #}".
Also, mention "perlcritic --profile-proto".

Add link to Try::Tiny in RequireCheckingReturnValueOfEval.

Add release description to Changes.

Bump P::C version to 1.108.

Fix CORE_DEVELOPER.pod now that the PodSummary build is fixed.

Update Changes into something readable by users for the 1.108 release.

Change "Repository" to "repository" in (MY)?META.yml.

Document Task::Weaken dependency.

Add dependency upon Task::Weaken (even though I think PPI requires it)
to the prerequisites due to
http://www.cpantesters.org/cpan/report/07450890-b19f-3f77-b713-d32bba55d77f

Set date for 1.107_001 release in Changes.

Add allow-unsafe to ProfilePrototype.

Small comment change in Subroutines::ProhibitAmpersandSigils and code
reformat.

Rename the --unsafe option to --allow-unsafe.

Move Email::Address from the optional to the required modules in the
P::C POD.

Theoretically fix the issue with compling P::C::Module::Build at
configure time due to PolicySummaryGenerator dependencies.

Get rid of use_ok() everywhere except t/00_modules.t and
xt/author/82_optional_modules.t.

Update to TODO.pod about dumping IO::String.

Fix false statement in POD about ## no critic paying attention to
blocks.

Fix links to deprecated modules in Modules::ProhibitEvilModules.

Bump P::C version to 1.107_001.

Remove scary warning from INSTALL.

Update the optional module discussion in CORE_DEVELOPER to match the
list of modules that are not part of the standard optional ones.

Stop all conditional module usage under xt/.  If you're an author,
you've got to run everything.

Change Modules::ProhibitEvilModules to complain about modules
deprecated by p5p.

(Try to) make P::C::P::Subroutines::RequireFinalReturn understand a
final given/when statement. The intent is that this is acceptable if and
only if there is a default block and all branches have a final
unconditional return.

Some simple cases have been added to the .run file, and (glory be!) the
Changes file was updated.

RT #57818 - DEVELOPER.pod typo in link to ValuesAndExpressions::ProhibitConstantPragma policy

Small typo in RequireFinalReturn POD.  RT #57803

Tell policy BuiltInFunctions::ProhibitLvalueSubstr not to complain when
there is a low-precedence operator between the substr() and the
assignment operator.

Note that the modified code assumes that anything with the same
precedence as '=' is an assignment operator. The hash that drives the
precedence_of() subroutine contains a bunch of things at the same
precedence as '='. All appear to be things like '||=', which PPI
decomposes into '||' and '=' anyway. But if someone's religion wants a
test for a literal '=', I will be glad to oblige.

Have policy TestingAndDebugging::RequireUseStrict accept 'use 5.011' or
greater as equivalent to 'use strict'.

Expand list of modules needed for authortest, by actually trying to run
the authortest suite on a virgin Perl.

Updated lib/Perl/Critic/CORE_DEVELOPER.pod to try to document the
modules needed over and above the Perl::Critic requirements to get
authortest to run.

Bug #57375 for Perl-Critic: [PATCH] Fix spelling error in the documentation

Accepted attached patch.

Put 1.106 changes to Changes on trunk.

49501  From Debian: RequirePackageMatchesPodName: doesn't use nearest package

Allow any package (rather than nearest) to match the pod NAME. "Nearest"
was not used because of the practice of putting all pod at the end. The
policy still only checks the first NAME section, if more than one
exists, but since I know of no examples of this, I figure it can wait.

Update P::C::Utils::is_unchecked_call() to include chmod in the set of
builtins covered by autodie.

Changed violation description to "Quotes used with a string containing
no non-whitespace characters" in
ValuesAndExpressions::ProhibitEmptyQuotes to address RT #56623.

Wordsmith my recent Changes updates.

RT #56619 redux

After discussion on the mailing list, added option
allow_in_main_unless_in_subroutine to
P::C::P::ErrorHandling::RequireCarping to control whether 'warn' or
'die' are allowed unconditionally in main:: (unless in a subroutine).
This option is false by default.

The configuration documentation was updated not only to cover this, but
to try to make explicit its interaction with
allow_messages_ending_in_newlines (it's ||, not &&).

The original work was committed as revision 3799.

RT #56667: No "$VERSION" variable found (false positive)

Wordsmithed $DESC and POD in P::C::P::Modules::RequireVersionVar to
try to make explicit the fact that a lexical $VERSION is not
acceptable either to Perl's version system or to this policy. Better
wordsmithing welcome.

RT #56619: "die" used instead of "croak" (possibly false positive)

Allow 'warn' or 'die' in name space main::, provided it is not in a
subroutine. This implementation per the BUGS section of the
P::C::P::ErrorHandling::RequireCarping POD (except, of course that it no
longer says that).

Some tests added to RequireCarping.run to exercise the new code, and
(glory be!) Changes updated to say in general what was done.

RT #56627: Too many arguments (sub prototype, false positive)

If the subroutine had a prototype, the previous code computed the number
of arguments as the number of characters in the prototype, less two for
the parens. The submitted case had embedded whitespace, which is not
usual, but other characters (semicolons, backslashes, square brackets)
also trigger the problem. The fix was to use tr/// to count the
characters that actually represent arguments.

'warnings' pragma was declared as dependency twice.

Don't generate test file for 20_policy_require_interpolation_of_metachars.t.

Bumped version number to 1.105_03.