perldelta copyediting

Revert perlfilter part of 318b791f053

The CPAN release has an old version.

whoops, should be using $new_version here

use the right default for MAP

make sure podlators is skipped, it won't be handled correctly

bump Filter::Util::Call to CPAN version 1.44

teach sync-with-cpan how to update Maintainers.pl

update sync-with-cpan to handle dists with MAPped files

ensure 'make' updates all necessary files

moving files around doesn't change the mtime, and tarballs store the
mtime to use when untarring, so it's possible to end up with an updated
file with an mtime before the corresponding file in lib/

get the dir name from the tarball filename

Filter::Util::Call comes from Filter-x.yz.tar.gz for instance, which
untars to Filter-x.yz, not Filter-Util-Call-x.yz as it was previously
assuming

allow overriding version in this branch too

not sure why this is necessary

Remove some 5.002 and 5.003 compatibility code from Class::Struct.

Add tests for all the error paths in Class::Struct.

Add tests for Class::Struct constructors called with initialiser lists.

use ppport.h for Storable on Perl 5.8.9

Storable.xs requires a definition of dVAR, which on 5.8.9 is only supplied
by ppport.h.

Merge fixes for Porting tests when building with -Dmksymlinks.

Avoid tests converting @INC to abs paths where no longer necessary.

As find_git_or_skip() no longer changes directory, no need to make paths in
@INC absolute in the various tests that use it.

Set GIT_DIR instead of changing directory in find_git_or_skip() in test.pl

Return the directory of the source tree, so that tests can change directory
to it if necessary. Modify t/porting/cmp_version.t to change directory.

This gets t/porting/utils.t passing again under -Dmksymlinks, without breaking
t/porting/cmp_version.t. The former needs to run in the build tree as it
inspects various build products. The latter needs to run in the (real)
checkout tree, else it thinks that every single file has changed (into a
symlink).

Teach Porting/cmpVERSION.pl about .pm files extracted from .pm_PL files.

Previously these were overlooked - the generated .pm file corresponding to
an .xs file would be found on disk, but as that file was not in git, the
original $VERSION could not be determined and it would be skipped. Now the
correct master file is checked, and failed if it is not updated.

Without this change, t/porting/cmp_version.t currently fails for a build
using -Dmksymlinks, due to ext/DynaLoader/dl_vms.xs having recent changes.
The test changes directory to the original git checkout, and if that
checkout is clean, there is *no* ext/DynaLoader.pm file there, which meant
that the previous version of pm_file_from_xs() would die because it could
not figure out which Perl file relates to dl_vms.xs

Update DynaLoader's VERSION after commit 7d08496d81c138d9.

In Porting/cmpVERSION.pl, treat undefined versions more rigorously.

For any changed module, if the old version is undefined or unparseable, treat
it as a SKIP, not an OK. If the old version *was* parseable, but the current
version is not, treat that as a FAIL - moving from a parseable to an
unparseable version is a regression.

In Porting/cmpVERSION.pl, increment $count in one place, unconditionally.

Previously it was only incremented if it was being used (as part of generating
TAP output). However, this was done as ++$count, necessitating using a printf
statement. By moving the increment to one place we can use print instead,
which is simpler, and avoid the risk of forgetting to increment $count if we
add a new block for different test output.

Sync Maintainers.pl with CPAN for Carp

Updated podlators to CPAN version 2.4.2

  [DELTA]

2012-06-01  Russ Allbery  <rra@stanford.edu>

  * VERSION: podlators 2.4.2 released.

  * t/text-encoding.t: Remove the test of a POD document without an
  encoding.  We previously tested that this interpreted the document
  as ISO 8859-1, but Pod::Simple behavior has changed so that the
  test started failing, plus Pod::Simple now warns about a missing
  =encoding.  (#77553)

2012-05-30  Russ Allbery  <rra@stanford.edu>

  * VERSION: podlators 2.4.1 released.

  * lib/Pod/Man.pm (parse_file): Override the Pod::Simple method to
  set output_fh to STDOUT if it's not already set.  (#77530)
  * lib/Pod/Text.pm (parse_file): Likewise.

Add perldelta entry for commit 4f46e52b00

Also wrap a long line.

perldelta update

bump version in Maintainers.pl to match what is in core

typo fix

update the core-cpan-diff section a bit

use 'undef' instead of 'UNKNOWN' for -u

check upstreams with -x too

Revert "bump Pod::Simple to CPAN version 3.22"

This reverts commit 74216f0477933a6ab02c6e6041e0dc98eb109017.

this update breaks podlators and Pod::HTML, so reverting it until i
figure out how to get those updated

bump Pod::Simple to CPAN version 3.22

bump HTTP::Tiny to CPAN version 0.022

bump Archive::Tar to CPAN version 1.88

fix error message

add use strict to perl -d

Updated PerlIO-via-QuotedPrint to CPAN version 0.07

  [DELTA]

0.07  5 April 2012
  Brought up to date with current code esthetics.  Verified it runs on
  5.14.2, threaded and unthreaded.

  28 January 2004
  Made sure the test-suite cleans up on multi-versioned systems.

Updated Memoize to CPAN version 1.03

  [DELTA]

  1.03  Sun, 22 Apr 2012 17:11:16 -0400
    * Clarified MERGE cache options and fixed bug
    * Documentation fixes

Updated Term-ANSIColor to CPAN version 3.02

  [DELTA]

  2012-03-18  Russ Allbery  <rra@stanford.edu>

  * ANSIColor.pm: Version 3.02 released.

  * ANSIColor.pm (AUTOLOAD): Only return pass-through behavior if
  the function that was called was one of our constants, rather than
  turning every unknown function in the Term::ANSIColor namespace
  into a passthrough join function when colors are disabled.
  * t/basic.t: Test proper error reporting with colors disabled.

  * ANSIColor.pm (AUTOLOAD): Preserve an existing value of $@ when
  generating a constant sub and restore it afterwards.  Diagnose
  errors in creating the constant sub and die instead of ignoring
  them.
  * t/eval.t: New test for $@ preservation.

  * ANSIColor.pm: Add italic and the ITALIC constant.  Document that
  the support for it is rare.
  * t/basic.t: Test italic and ITALIC support.

  * ANSIColor.pm: Add "mistyped" as a stopword, required by the
  latest aspell on Debian.

Updated Digest-MD5 to CPAN version 2.52

  [DELTA]

  2012-06-08   Gisle Aas <gisle@ActiveState.com>

   Gisle Aas (3):
      Wrong version number in the changelog
      The t/threads.t was missing from the MANIFEST
      Update expected digests for files

   Andrew Fresh (1):
      Remove double the

   Lyle Hopkins (1):
      Digest::Perl::MD5 OO fallback didn't work [RT#66634]

   Peter J. Acklam (1):
      Fix typos (spelling errors) in cpan/Digest-MD5/*

   Shlomi Fish (1):
      Modernize the code in the POD.

   Zefram (1):
      Makes Digest::MD5 work on Perl 5.6 [RT#75032]

Updated autodie to CPAN version 2.11

  [DELTA]

  2.11
        * DOCS: Explicitly documented that autodie is context
          unaware.  (Thanks to chromatic.)

        * TEST: Multi-arg open tests are skipped on VMS.
          (Thanks to Craig A. Berry.)

        * TEST BUGFIX recv.t shouldn't assume STDIN is a file handle.
          (Thanks to Todd Rinaldo)

        * TEST: Fixed compatibility with Carp 1.25.
          (Thanks to Olivier Mengué.)

        * INTERNAL: Exception classes are loaded more safely.
          (Thanks to Schwern)

Updated Safe to CPAN release 2.33

  [DELTA]

2.33 Tue Apr  3 2012
    - Don't eval code under 'no strict' (Father Chrysostomos)

2.32 Sat Mar 31 2012
    - Make Safe play nice with Devel::Cover

Updated Search-Dict to CPAN release 1.07

  [DELTA]

1.07 2012-04-11

  - install into sitelib for Perl 5.12+
  - use Tie::Handle to get Tie::StdHandle for older Perls
  - avoid using fc() in v5.15.0 - v5.15.7
  - fix tests for older Perls

1.06 2012-03-31

  - suppress stat() warnings on tied filehandles

1.05  2012-03-31

  - no longer requires stat() on filehandle
  - tests use Test::More
  - case folds with fc() on Perl 5.15+

Revert "B::Deparse: loopexes have list prec"

This reverts commit 6a31dbf44ee919c340a3372c95b28d581979d165.

It turns out to be wrong.  Loopexes do have low precedence, but not
as low as I thought.  Their precedence level is actually their own,
undocumented in perlop:

diff --git a/pod/perlop.pod b/pod/perlop.pod
index 3edeabd..c9a1adf 100644
--- a/pod/perlop.pod
+++ b/pod/perlop.pod
@@ -49,6 +49,7 @@ values only, not array values.
     nonassoc   ..  ...
     right      ?:
     right      = += -= *= etc.
+    nonassoc   loop exits (last, next, goto)
     left       , =>
     nonassoc   list operators (rightward)
     right      not

I am reverting this for now, till we decide what it is that
needs fixing.

perlop: Fit some verbatim lines into 79 cols

numeric.c: Fix typo, clarify comment

Fixup for bigint docs

Some copy/paste and insertion errors in bignum 0.19 left the docs in
a confusing state.

ensure correctness if sv_2mortal modifies errno

require should die if a file exists but can't be read.

See [perl #113422]. If a file exists but there's an error opening it,
we throw an exception and disregard the rest of @INC.

perldelta: entry for qr/\87/ bug fix

t/re/re_tests: Clarify comment

regcomp.c: Warn and don't emit a NUL for e.g. /\87/

/\87/ when there are fewer than 87 capture groups was being read as
/\00087/, with no warning.

regcomp.c: Add comments

The main part of this commit summarizes a huge switch statement, that I
never fully understood until I wrote the comment.

toke.c: Combine macro calls

Replace a combination of macro calls with a single one that means the
 same thing

pp_hot.c: Combine macro calls

Replace a combination of macro calls with a single one that means the
same thing

walk.c: Combine macro calls

Replace combinations of macro calls with single ones that mean the same
thing.

a2py.c: Use isWORDCHAR() instead of individ components

isWORDCHAR() is a single macro (and array lookup) that is the equivalent
of the OR of the three macros previously used.

handy.h: Add comment

perlipc typo fixes, reported by Linda Walsh (RT#113556)

typo fix, reported by Joaquin Ferrero (RT#113616)

add note about minicpan mirrors and core-cpan-diff

pp_hot.c: Add -Dr messages

This announces, when run with the appropriate debug levels, when regex
matching is skipped because we know up-front that it can't possibly
match.

PATCH: [perl #113640] Typo in perlop.pod: bignum pragma

There is no 'bitfloat' pragma

fix paren unwinding with CURLYN, CURLYM

Historically, CURLYN and CURLYM didn't restore lastparen ($+) nor
invalidate higher numbered paren slots (reg->offs[n].end = -1)
when backtracking.

Recent commit f6033a9d6c91e0c29d5aa5049361a529b5d7cdc4 fixed the lastparen
issue, but then exposed the offs[n].end issue, which had previously been
masked by the too-high lastparen value causing a previous state (such as a
branch) to conveniently wipe out the offending parens on further
backtracking.

Or to put it another way, each backtracker should be expected to clean up
his own mess, not hope that someone previous to him will do it for him.

Fix this by doing the .end = 1 stuff on CURLYN,M failure as well as
BRANCH/TRIE. For consistency, define a new macro to do it.

minicpan doesn't set up the by-authors symlink

Updated IO-Compress to CPAN version 2.052

  [DELTA]

  2.052 29 April 2012

      * IO::Compress::Zip
        Force a ZIP64 archive when it contains >= 0xFFFF entries.

      * Typos in POD
        [RT# #76130: Gunzip Pod typo in OO section: $$output instead of $$input

  2.049 18 February 2012

      * IO::Compress::Zip
        Error in t/cz-03zlib-v1.t that caused warnings with 5.15
        [RT# 110736: warnings from cpan/IO-Compress/t/cz-03zlib-v1.t]

Updated Compress-Raw-Bzip2 to CPAN version 2.052

  [DELTA]

  2.052 29 April 2012

      * No Changes

  2.049 18 February 2012

      * No Changes

Updated Compress-Raw-Zlib to CPAN version 2.054

  [DELTA]

  2.054 8 May 2012

      * Build issue on Win32
        [#77030]

  2.053 6 May 2012

      * Include zlib 1.2.7 source.

  2.052 29 April 2012

      * Fixed build issue when Perl is built with C++

  2.051 20 February 2012

      * Bug in Compress::Raw::Zlib on Windows
        [#75222]

  2.050 20 February 2012

      * Build failure on Irix & Solaris.
        [RT #69985]

  2.049 18 February 2012

      * Include zlib 1.2.6 source.

add note about metacpan to the rmg

checkAUTHORS.pl: another address for Renee Bäcker

Increase $Class::Struct::VERSION to 0.64

add tests for overridden Class::Struct accessors

[perl #29230] Class::Struct, accessor overrides not called from constructor

Class::Struct allows you to override the accessors it creates, but it
doesn't call them in its constructor.
In other words,

     $struct->field('blah');

calls my override, but

     $struct = structure->new('field' => 'blah');

doesn't. Class::Struct simply does

     $r->{'field'} = $init{'field'}

but it would be more useful if it did

     $r->field($init{'field'})

Add Rhesa Rozendaal to AUTHORS

op.c:utilize: don't allocate pegop if !PL_madskills

Just following what the FIXME note says. :-)

This shaves off a few CPU cycles.

Let hints.t run at the top level

Tests for perl #94476

configpm: comment typo

Don’t call mro_method_changed_in after newCONSTSUB

newCONSTSUB already does it itself (by calling newXS_len_flags,
which does it).

Don’t create pads for sub stubs

Two code paths, sv_2cv (for \&name) and get_cvn_flags (for
&{"name"}()) were using start_subparse and newATTRSUB to create a
subroutine stub, which is what usually happens for Perl subs (with
op trees).

This resulted in subs with unused pads attached to them, because
start_subparse sets up the pad, which must be accessible dur-
ing parsing.

One code path, gv_init, which (among other things) reifies a GV after
a sub declaration (like ‘sub foo;’, which for efficiency doesn’t
create a CV), created the subroutine stub itself, without using
start_subparse/newATTRSUB.

This commit takes the code from gv_init, makes it more generic so it
can apply to the other two cases, puts it in a new function called
newSTUB, and makes all three locations call it.

Now stub creation should be faster and use less memory.

Additionally, this commit causes sv_2cv and get_cvn_flags to bypass
bug #107370 (glob stringification not round-tripping properly).  They
used to stringify the GV and pass the string to newATTRSUB (wrapped in
an op, of all things) for it to look up the GV again.  While bug
been fixed, as it was a side effect of sv_2cv triggering bug #107370.

op.c:newPROG: Remove questionable comment

This thing about PL_eval_root going through an expensive function call
had already ceased to be the case before commit 5983a79d, which added
it.  And I think it was only ever valid for non-core code.

Use assertions for /* NOT REACHED */

to make sure it really is never reached.

sv.h: Move comment just before struct it describes

It was divorced therefrom in commit 20f4945e.

perl.c: Clarify comment

This comment was added by commit 17fbfdf.  Without looking at the
diff, it’s not clear that ‘Do this now’ refers to setting PL_curcop,
rather than destroying the main CV.  Also, there was a missing verb,
leading to confusion.  (I first inferred ‘cause’ when reading it,
but it is actually ‘expect’, based on inspecting the code in
sv.c:S_new_SV.)

Rmv mro_method_changed_in call on stub upgraded to const

If a subroutine stub exists and a new subroutine is defined with the
same name, that new subroutine’s body is attached to the stub.  So
there are no changes from the point of view of method lookup.

For most subs, mro_method_changed_in is not called in this case.

For constant subs, it was being called.

In the case where a new sub is being created, we know that
cv == GvCV(gv) and GvSTASH(gv) is set, so checking CvSTASH is
unnecessary.

gv.c: Don’t repeat GvSVn

GvSVn checks whether there is a scalar present and creates one if there
is not.  So doing GvSVn twice in a row results in a redundant check.

It has been this way since c69033f2.

perl.c:get_cvn_flags: Move a comment where it belongs

The PERL_ARGS_ASSERT_GET_CVN_FLAGS got inserted between a
comment and the code it described.

gv.c: Remove mro_method_changed_in() from gv_init

gv_init(_pvn) does not conceptually change anything.  There was
already a subroutine there before the stub was upgraded to a gv with
a real cv.

The example in the comment:

 sub Foo::bar($) { (shift) } sub ASDF::baz($); *ASDF::baz = \&Foo::bar

suggests that this was put in the wrong place to begin with.  Glob
assignment already takes care of mro_method_changed_in, so calling
it beforehand when reifying the glob is redundant.

gv.c: Don’t ENTER/LEAVE unnecessarily

Commit 756cb47730 added an if/else block in between ENTER/LEAVE, mov-
ing the existing code into the else block.  But the if code (consist-
ing of newCONSTSUB) never needed to be surrounded by ENTER/LEAVE.

Fix small Pod error introduced by commit c5e3e317152223ae.

Spotted by t/porting/podcheck.t

clean up documentation for sync-with-cpan

add Abigail to the release schedule

Updated Test-Harness to CPAN version 3.25

  [DELTA]

  3.25    2012-06-05
        - Rereleased to fix CPAN permission problem. No functional change.

  3.24    2012-06-03
        - RT #74393: corrected typo in M::B integration docs.
        - RT #63473: fix typo.
        - RT #49732: Attempt to load File::Glob::Windows to get correct
          glob semantics on Win32.
        - RT #47890: Don't use Win32::GetShortPathName.
        - RT #64404: Ignore textness ('-T') of script when reading shebang.
        - Handle the case where we don't know the wait status of the
          test more gracefully.
        - Make the test summary 'ok' line overrideable so that it can be
          changed to a plugin to make the output of prove idempotent.
        - Apply upstream patch:

            http://perl5.git.perl.org/perl.git/commit \
              /6359c64336d99060952232e7e300bd3c31afead8

          In testargs.t in Test::Harness, don't run a world-writable file.

          The test writes a file, then changes the mode, then executes it. The file needs
          to be +x to be executable (on many platforms). The file will need to be +w to
          be deletable on some platforms. But setting the file world writable just before
          running it feels like a bad idea, given that the file's name is as predictable
          as process IDs, as there's a race condition to break into the account running
          perl's tests.

Updated CPAN-Meta to CPAN version 2.120921

  [DELTA]

  2.120921  2012-04-01 15:20:24 Europe/Paris

  [REMOVED]

  - CPAN::Meta::Requirements has been split out into its own distribution
    so it can be used by CPAN.pm without requiring all of CPAN::Meta

  2.120920  2012-04-01 11:30:43 Europe/Paris

  [ADDED]

  - CPAN::Meta::Requirements now has a 'requirements_for_module' method
    to retrieve a version requirements string for a specific module
    [Leon Timmermans]

  [OTHER]

  - Parse::CPAN::Meta prerequisite bumped to 1.4403
  - JSON::PP prerequisites bumped to 2.27200
  - CPAN::Meta::YAML prerequisite bumped to 0.008

  2.120900  2012-03-30 13:15:15 Europe/Paris

  [BUGFIX]

  - CPAN::Meta::Requirements now treats undef requirements given to
    from_string_hash() as '0' and warns about it; add_string_requirements()
    dies if it does not get a requirements string to parse

Dual lifed CPAN-Meta-Requirements 2.122 as a prereq for CPAN-Meta

Update Module-Build to CPAN version 0.40

  [DELTA]

  0.40 - Fri Feb 24 18:47:48 CET 2012

  - Released 0.39_02 as 0.40 without any code changes

  0.39_02 - Thu Feb 17 00:33:18 MET 2012

  [BUG FIXES]

  - Fixed bug where modules without $VERSION might have a version of '0'
    listed in 'provides' metadata, which will be rejected by PAUSE
    [David Golden, reported by Christopher Fields]

  - Fixed bug in PodParser to allow numerals in module names
    [Tokuhirom]

  - Fixed bug where giving arguments twice led to them becoming arrays,
    resulting in install paths like "ARRAY(0xdeadbeef)/lib/Foo.pm"
    [Leon Timmermans]

  [DOCUMENTATION]

  - Fixed deviance from alphabetical order in documentation of
    constructor parameters. [Eric Brine]

  - Add documentation for configure_requires constructor parameter.
    [Eric Brine]

  - Change some of the docs' language describing relationship to
    MakeMaker. [Ken Williams]

  [OTHER]

    - List Perl-Toolchain-Gang repo as official repo

Updated CPANPLUS to CPAN version 0.9130

  [DELTA]

Changes for 0.9130      Thu May 24 22:04:10 2012
================================================
* Always re-fetch CHECKSUMS if fetchdir is set
  (Torsten Schoenfeld)

Changes for 0.9129      Wed May  9 21:22:41 2012
================================================
* Handle multiple makemakerargs and makeflags
  arguments better.
* Use File::HomeDir for home directory location
  if it is available, thanks to kmx
* Added PERL5_CPANPLUS_HOME for altering where
  the .cpanplus directory is located

Changes for 0.9128      Sat Apr 28 21:27:06 2012
================================================
* Fix the previous fix

Changes for 0.9127      Sat Apr 28 20:34:44 2012
================================================
* Silenced annoying warnings related to older
  perls and the progress indicators

Changes for 0.9126      Sat Apr 28 00:49:43 2012
================================================
* More speed enhancements to module indexing,
  thanks to Vincent Pit

Changes for 0.9125      Wed Apr 25 14:28:34 2012
================================================
* Speed enhancements to module indexing, thanks
  to Vincent Pit

Changes for 0.9124      Fri Apr  6 19:24:55 2012
================================================
* Save the history between invocations of the
  shell.

Changes for 0.9123      Fri Mar 30 16:46:52 2012
================================================
* Added support for adding blib/script to PATH

Changes for 0.9122      Wed Mar 28 21:52:38 2012
================================================
* Don't spawn a process to check whether perl
  version prereqs are satisfied

Updated HTTP-Tiny to CPAN version 0.021

  [DELTA]

  0.021     2012-05-15 22:38:57 America/New_York

  [TESTING]

  - Skip live SSL testing if $ENV{http_proxy} is set

  0.020     2012-05-14 15:24:37 America/New_York

  [TESTING]

  - Capture prerequisite versions under AUTOMATED_TESTING to help
    chase down some failures from CPAN Testers

  0.019     2012-05-14 07:14:00 America/New_York

  [ADDED]

  - Require IO::Socket::SSL 1.56 (which added SSL_hostname support) when
    doing HTTPS.  [Mike Doherty]

  [TESTING]

  - Provide better diagnostic output in t/210_live_ssl.t [Mike
    Doherty]

  0.018     2012-04-18 09:39:50 America/New_York

  [ADDED]

  - Add verify_SSL option to do more secure SSL operations, incl.
    attempting to validate against a CA bundle (Mozilla::CA
    recommended, but will attempt to find some OS bundles). Also
    add SSL_opts, which passes through IO::Socket::SSL's SSL_*
    options to control SSL verification. (GH #6, #9) [Mike Doherty]

  - Reponse hashref includes final URL (including any redirections)
    [Lukas Eklund]