Spelling fixes and stopwords.

Adding a few stopwords

Add a little explanation about the way we've mishandled
the parsing of "## no critic" restrictions.

Added new policy: ProhibitReturnSort, as suggested by Ulrich Wisser.

I hadn't anticipated the problem with false-positives due to wantarray().
I don't know if this issue is a release-blocker.  What do you think?

Now that we have ProhibitUselessNoCritic, we can
scratch one more item off the TODO list!

Can scratch one item off the todo list...

Added idea about refactoring F<perlcritic> to the TODO list.

Test all the standard Capitalization schemes.  Regexes aren't tested
yet, nor are all kinds of names, e.g. labels.

Progress on Capitalization tests.

Update Changes.

Add exclude_functions parameter to RequireCheckedSyscalls.

Mention parent in the POD for ProhibitExplicitISA.

Mention IPC::System::Simple in the ProhibitBacktickOperators POD.

Handle standard filehandle names in NamingConventions::Capitalization
by not reinventing the wheel.

POD updates in NamingConventions::Capitalization.

Handle labels in NamingConventions::Capitalization.

Re-characterizing the "## no critic" comments/pragmas as "annotations" throughout the code and documentation.

Renaming test scripts for readability, as Elliot had done for some of the others.

Committing the rest of my work on refactoring the handling of the
"## no critic" annotations.  This completes the integration with the
new Annotation class.

I also created a new ProhibitUselessNoCritic policy, which uses the
new capabilities of P::C::Document and P::C::Annotation to effectively
replace the --warn-about-useless-no-critc option.  Thanks Elliot for i
suggesting that we implement this as a Policy instead of an option.

Next, I want to change all the code and documentation to recharacterize
the "## no critic" thingies as "annotations".  Also need to write some
unit tests for P::C::Annotation and P::C::Document.  I've had success
using Test::Class at work lately, so I'm thinking of using it here.

Add optional modules used in tests to 82_optional_modules.t.

Get rid of most conditional loading of modules in xt/author tests.  If
you're an author, these modules are required, not optional.  Plus, it
makes the code simpler.

Change regex delimiters in P::C::Document in order to passify editor
syntax highlighting.

Bump P::C version number for developer release.  Unfortunately, I
screwed up last time and left a zero out of the version, so this is
version 1.093_02 instead of 1.093_002.

Update Changes for developer release.

Fix self compliance of P::C::(Violation|Annotation).

Mark failing test for CodeLayout::ProhibitParensWithBuiltins as TODO
so that I can do a developer release.

Do the period chomping right

Add another cruft file to the SKIP list

I think Schwern mention the double periods appearing at the
end of some violation descriptions and/or explanations.  So
we strip out whatever punctuation the author provided, and
let the user define their own, using a format.

Offering a new Perl::Critic::Annotation class for review.
The idea here is to define an object that represents the
"## no critic" markers that appear in the file.  Policies
can then interact with these objects via P::C::Document.

For example, ProhibitUnrestrictedNoCritic would look something
like this:

sub violates {
    my ( $self, $doc, undef ) = @_;

    # If for some reason $doc is not a P::C::Document, then all bets are off
    return if not $doc->isa('Perl::Critic::Document');

    my @violations = ();
    for my $annotation ($doc->annotations()) {
        if ($annotation->disables_all_policies()) {
            my $token = $annotation->token();
            push @violations, $self->violation($DESC, $EXPL, $token);
        }
    }

    return @violations;
}

I've completed most of the work to integrate this with the existing
code.  But I won't commit it until everyone has reviewed this idea.

Comments wanted!

Holy smokes, it works!

Using the --warn-about-useless-no-critic option, I found several places
where we were disabling too many policies, or were disabling them
due to bugs that have since been fixed.  This is a pretty cool feature,
but I have reservations about making it into a regular Policy.  Since
we use pattern-matching to select the Policies, it is really easy for
someone to disable a *future* policy.  So I'm not sure I want to force
them to change their '## no critic' markers just because they've upgraded
P::C.

BTW, I discovered that we were using the wrong syntax in some of our
'## no critic' markers:

  ## no critic   'SomePolicy'  # wrong, disables all policies.
  ## no critic   (SomePolicy)  # ok
  ## no critic qw(SomePolicy)  # ok

We should probably emit a warning or something if we come across
a '## no critic' marker that looks wrong.

A little more tweaking.  Should probably write some unit tests here.
I've become too reliant upon the functional tests.

Fixed trailing whitespace and pod spelling

Second attempt to provide warnings when a useless "## no critic"
is found.  This time, I completely re-wrote the way that we
keep track of disabled lines.  Instead of just marking each
line that was affected by a no-critic, I keep track of each
no-critic and the region of lines that it affects.  This
makes it possible to report which no-critic markers are
not having an effect, rather than reporting all the lines
that were disabled but had no violations.

I had a hard time thinking of a good data structure
for expressing all this.  So this certianly isn't the
fastest code.  But this also isn't the slowest part
of Perl-Critic.

Along the way, I found that my strategy for finding
useless no-critic markers was fundamentally flawed --
the most important thing is to keep track of the
violations that were *not* reported.  That way,
you can tell if a no-critic marker was effective
or not.

Remove the fictional $SKIP from NamingConventions::Capitalization.

Finally able to individualy commit the Perl-Critic directory itself
with the adding of nytprof to svn:ignore.

Get generation of t/NamingConventions/Capitalization.run in place.
What it generates right now is static, but all the infrastructure is
in place.

Remove column numbers from =over POD directives.  The POD formatter
knows how things should be set, if it pays any attention to the number
at all.

Create P::C::Config::all_policies_enabled_or_not() and use it in
PolicySummary.pod.PL.

Add $VERSION to t/03_useless_pragmas.t and add it to the list of tests
that need to have "without optional dependencies" wrappers generated
for them.

Self-compliance.

Allow any number of leading underscores in names that are supposed to
start with lower|upper case letters.  Finish remaining self-compliance
capitalization issues.

Capitalization self-compliance in tests.

Explain new option in Changes file

Add exemptions for $TODO and $SKIP to Capitalization.

Cleaned up trailing whitespace.  Fixed typo in name of hash key

Add nytprof to the MANIFEST.SKIP files.

Don't complain about capitalization of BEGIN, UNITCHECK, etc. no
matter how things are set up on NamingConventions::Capitalization.

First attempt to warn users when they have an unecessary "## no critic"
marker in their code.  I'm not happy with the option-name that I chose.
Also, I know there are bugs related to the shebang line, and the borders of
##no critic -> ##use critic blocks.

Incorporating some of Chris' suggestions.

Fixed more self-compliance

More self compliance

Get rid of that Debian-whatever file.  I think it
gets produced by the Kwailitee tests.

Fixed up self-compliance issues in NamingConventions::Capitalization.

Fixed breakage that started when the names of some
of the test scripts were changed.

Pulled all the 'no critic' handling code out of Critic.pm
and put it into Document.pm.  The internals are still kinda
ugly, but the interface is much nicer (I think).

Just comment formatting changes.

More self-compliance.

Turn off Capitalization policy for now.  It is making too much noise

Self-compliance with ProhibitUnrestrictedNoCritic

Woops, I left a naked subroutine out there.

Also re-arranged some code slightly.  I do like to take advantage of
the postfix-if-with-flow-control-statements rule.

Vanquised can_be_disabled() method.

Resolved failing test cases for Miscellanea::ProhibitUnrestrictedNoCritic.  Next, I'll remove the can_be_disabled() method.

Add a check on myself: add a test that complains if any of the
optional modules aren't installed.

Failing test with ProhibitParensWithBuiltins that I'm not sure what to
do with.  Treat "int" as an operator?

Rename the t/20_policy_*.t tests to include underscores so that the
names are notlikethis.

Disable Modules::ProhibitEvilModules if there aren't any evil modules.

Add subroutine name to violation descriptions for
Subroutines::ProhibitExcessComplexity.  Optimistically, it handles the
case of anonymous subroutines, in case PPI ever handles them.  :]

RT #40070, thanks to Andreas Koenig.

Links to Test::P::C.

Missed RegularExpressions::ProhibitCaptureWithoutTest in earlier bugs
documentation move commit.

Add small links HTML page for holding links to *nix P::C packages and
other things.

Move policy bugs from TODO.pod to the actual Policies.  Users should
be aware of limitations.

Delete dead code from NamingConventions::ProhibitAmbiguousNames.

Code cleanups to comply with NamingConventions::Capitalization.
Amazing what some of this turned up, e.g. leftover, unused variables
from prior to conversion to use Policy Parameters.

Add exceptions for the standard tie subroutines to
NamingConventions::Capitalization and borrow code from unreleased
PPI::Statement::Variable to get real variable names in the case of
"local" statements.

And $P::C::Policy::FORMAT bites the dust too.

$P::C::Violation::FORMAT loses to NamingConventions::Capitalization.
Oh, darn.

Fix subroutine capitalization in Modules::RequireVersionVar and mark
TODO test as no longer TODO in t/Modules/RequireVersionVar.run.

Add AUTOLOAD exceptions to NamingConventions::Capitalization and
include the kind of name in violation descriptions.

Add explanation to RequireNoMatchVarsWithUseEnglish about merely not
importing any of the match variables not being good enough to prevent
performance problems.

More fully state that development is done using Module::Build.

Add some draft notes about how we do P::C releases

RT #39787 is wrong.  RequireNoMatchVarsWithUseEnglish is correct, as
is.

Handle the __PACKAGE__->_foo() case in ProtectPrivateSubs.

Add failing test for __PACKAGE__->_foo() for ProtectPrivateSubs.

Add a bunch of stuff to TODO.

Add a :single_case tag to NamingConventions::Capitalization and
rewrite its POD.

Test for "package main" in NamingConventions::Capitalization

Failing test for Modules::RequireNoMatchVarsWithUseEnglish from
RT #39787.

Add "main" to the default package name exemptions to
NamingConventions::Capitalization.

Simplify the regexes for the default global variables that are exempt
from Capitalization.

The big NamingConventions::Capitalization rejiggering.  Need to make
NamingConventions/Capitalization.run be generated.  But right now, my
brain is fried.

Add test for Statement::Include::module_version() in
xt/author/81_ppi_problems.t.

Fix self compliance in t/05_utils_ppi.t and Utils::PPI.

Add cleaned up versions of Utils::PPI::is_subroutine_declaration() and
is_in_subroutine() from Schwern.

Add a test file for things we want from PPI.

Rename Variables::ProhibitReusedVarNames to Variables::ProhibitReusedNames

Update threading comment in TODO.pod.

Add threading idea to TODO.pod.