Sync Module-CoreList in Maintainers.pl with CPAN

Merge fix for File::stat's -x and -X for root, and test improvements.

lib/File/stat.t shouldn't test -A $^X

Testing this fails intermittently on darwin, which has POSIXly-correct atime
semantics. When tests run in parallel, the atime of $^X will update whenever
another perl process starts, which is a race condition with this test.

Fix File::stat's -x and -X for root for directories and executable files.

Previously File::stat's overloaded -x and -X operators did not give
the correct results for directories or executable files when running as
root. They had been treating executable permissions for root just like for
any other user, performing group membership tests etc. for files not owned
by root. They now follow the correct Unix behaviour - for a directory they
are always true, and for a file if any of the three execute permission bits
are set then they report that root can execute the file. Perl's builtin
-x and -X operators, added in Perl 2, have always been correct.

Test that File::stat exports stat by default, overriding the builtin.

Also, add a test that the array produced by File::stat::stat is identical to
the list produced by the builtin.

lib/File/stat.t should also test the -X operators on directories.

In lib/File/stat.t, permute the test file's mode for more thorough testing.

For each mode bit set, and all bits clear, test each filetest operator with
and without a File::stat object. Previously the only test was for the mode
the file was created with.

Refactor lib/File/stat.t for robustness.

Use CORE::stat instead of stat, to be clear which is the builtin, and which
is the routine that File::stat is prepared to export. As it is now
unambiguous which is which, remove comments that annotated each use.

Use isa_ok() in place of ok() to test the return value of File::stat::stat.
Use is_deeply() in place of is() and arrays interpolated into strings.

Move the data driven loop that tests most of the -X operators into a
function test_X_ops(), and use this to test both the tempfile
(non-executable) and $^X (executable).

Put the first sanity test of File::stat::stat inside a block so that its
lexicals don't leak.

*Actually* test that bug 20011110.104 is fixed.

Bug ID 20011110.104 (RT #7896) was fixed by commit 2f173a711df6278f in Nov
2001, but the test that commit added never actually tested this.

The initial problem was that the new code, as written, used C<stat>,
intending that to call File::stat::stat(). However the refactoring of the
test script (all part of the same commit) from C<use File::stat;> to
C<use_ok( 'File::stat' );> (not in a BEGIN block) intentionally eliminated
the export of &File::stat::stat. This means that plain C<stat> is the
core builtin.

Fixing this as-is to File::stat::stat() won't help, as tests have
subsequently been added earlier in the script that trigger the autoloading
of Symbol by File::stat (commit 83716b1ec25b41f2 in Feb 2002). Moving the
tests earlier won't help now that the test uses File::Temp, as that uses
IO::Seekable which uses IO::Handle, which unconditionally loads Symbol.

The simplest solution seems to be to move the test to its own file.

Simplify lib/File/stat.t by using a tempfile as the test victim.

The previous code had got very gnarly trying to use a file from the
distribution for the test file, attempting to cope with
1) Other programs reading the file and hence the atime updating
2) The perl interpreter reading the file and hence the atime updating :-)
3) Building with -Dmksymlinks meaning that the file is a symlink

All these problems and work arounds simply *vanish* if we use a tempfile.
This will also enable us to change its mode in the future for better testing.

Remove needless skip tests from lib/File/stat.t

The stat builtin is always implemented, so no need for an eval {} test to
check for this. The presence or absence of <sys/stat.h> at the C level isn't
going to affect the outcome of this test.

Both have been in the test script since it was added (as t/lib/filestat.t)
by commit f7a45afb043dafc5 in May 2001. I suspect that both were copied
from one of the Net::* test scripts which was being used as the template.

In lib/File/stat.t, test everything with and without use filetest "access".

Previously the use filetest "access" tests were separate, and didn't test
all the "should not warn" cases. By moving them into the main data-driven
loop it's trivial to test everything.

Also test that all the correct errors are seen on VMS, and not seen anywhere
else.

Bring the joy of strict (and warnings) to lib/File/stat.t

Replace repetitive code in lib/File/stat.t with a data driven loop.

Convert lib/File/stat.t from a complex plan calculation to done_testing().

Also replace use_ok (not in BEGIN) with a simple require. If the require
fails it's not worth trying to test anything else, Test::More's END
handing will automatically report a test failure, so the failure will be
recorded.

Add an editor block.

Increase $PerlIO::scalar::VERSION to 0.15

[perl #113764] Make &= duping work with PerlIO::scalar

In trying to fix bug #112780, I made in-memory handle duplication tem-
porarily hide the underlying scalar so it wouldn’t be set to the empty
string (commit 49b69fb3a).

I used PerlIO_sv_dup in j rather than PerlIOScalar_arg.  The for-
mer is usually what is called anyway.  There is only one branch of
PerlIOScalar_arg that doesn’t call PerlIO_sv_dup.  I don’t remember
what I was thinking back then, but I think I thought that branch
was there for paranoia.  But actually, it is used for "&=", so this
started failing:

open FILE, '>', \my $content or die "Couldn't open scalar filehandle";
open my $fh, ">&=FILE" or die "Couldn't open: $!";
print $fh "Foo-Bar\n";
close $fh;
close FILE;
print $content;

This commit fixes the bug in the smallest way possible, which means
switching from PerlIO_sv_dup to PerlIOScalar_arg in PerlIOScalar_arg,
which, in turn, entails fiddling with RVs.

Teach dump.c about CVf_HASEVAL

perl5171delta: caller looks beyond cur stack

perl5171delta: typo

[perl #89544] Non-eval closures don’t need CvOUTSIDE

A closure doesn’t need an outside pointer at run time, unless it has a
string eval in it.  CvOUTSIDE is only used at compilation time to look
up variables by name.

Since CvOUTSIDE is reference-counted, a closure can unnecessarily hang
on to variables it is not using (see the test in the diff).  So stop
setting it when cloning a closure, unless it is needed for eval.

closure.t: load test.pl at BEGIN time

This allows parentheses to be omitted.

RT#113730 - $@ should be cleared on "do" IO error.

[perl #113712] Don’t create stubs after errors

perl5.002beta3 (c07a80fdfe) stopped bodies of subrou-
tines from being defined after compilation errors, as in
eval "@a =~ s///; sub { die }".

But, instead of making the sub declaration not happen at all, it ended
up leaving a stub.

For a full sub declaration (body and all) to create a stub just
seems wrong.

Likewise, it would be weird if a stub declaration
after a compilation error created a stub, because then
eval "@a =~ s///; sub foo; sub bar { }" would create foo but not bar.

Similarly, a compilation error will cause ‘sub foo {}’ no suppress
‘used once’ warnings; but a lexing error won’t.

This commit fixes all this, making things consistent:  If there is a
compilation, parsing or lexing error, any kind of sub declaration that
follows is ignored.

[perl #113756] fix type of StructCopy in API documentation

perlapi currently claims StructCopy takes two structs when it really
takes two pointers.

Let while.t run from the top level

bump version to 5.17.2

don't change version numbers in this file either

comp/parser.t: Add some tests

dquote_static.c: Clarify comment

reg_mesg.t: Add tests

Refactor \x processing to single function

There are three places that process \x.  These can and did get out of
sync.  This moves all three to use a common static inline function so
that they all do the same thing on the same inputs, and their behaviors
will not drift apart again.

This commit should not change current behavior.  A previous commit
was designed to bring all three to identical behavior.

toke.c: Don't convert \x{} from native

This was and is a no-op on ASCII platforms, but on EBCDIC, when you did
e.g., a "\x{7}", it would convert that to a 127.  But it did not do this
on qr/\x{7}/, giving inconsistent results.  Now, \x{7} yields 7, and
thus is consistent, and matches the documentation.

grok_bslash_o: Don't convert from native

This was and is a no-op on ASCII platforms, but on EBCDIC, when you did
e.g., a \o{7}, it would convert that to a 127.  But it did not do this
on a \007, giving inconsistent results.  Now, \o{7} yields 7, and thus
is consistent, and matches the documentation.

PATCH: [perl #113750] re.pm clobbers $_

Thanks to Jesse Luehrs and Father Chrysostomos for testing advice.

new perldelta

the rmg should tell BLEAD-POINT to bump version at the end

add release branch steps to rmg

few rmg nits

update epigraphs.pod

Merge branch 'release-5.17.1' into blead

More tests in t/op/arith.t for constant folding of expressions containing %

Most tests for the bug resolved by commit e7311069df54baa6, including 64 bit
integer analogs of the 32 bit integer tests added by that commit.

one more place to get rid of win32/config*.gc64nox

update perlhist for 5.17.1

update Module::CoreList for 5.17.1

regen pod issues

POSIX: Sometimes `printf` beats interpolation

Tidier example code for POSIX::localeconv()

Add Daniel Perrett to AUTHORS

formatting

add acknowledgements

bump version to 5.17.1

comments in bump-perl-version

perldelta for 5c0877fa4ac0 (VMS include files).

perlhacktips: long lines

more proofreading

perlhacktips: add AddressSanitizer

[amended to fix a typo, and add an exception to the podcheck database]

we seem to have been using L<> rather than C<> here recently

none of this is important

add module updates to perldelta

change this synopsis to something that actually works

note existing smoke failures

the known problems section is also for platform specific things

Use ll rather than I64 for win32/config*.gc64 format strings

The corresponding types are long long rather than __int64 so this makes
more sense and matches proposed diffs from the land of Strawberry Perl
for supporting 32-bit builds with 64-bit ints on Windows (hopefully
coming up soon...).

Add missing dlltool default to win32/config.gc

ExtUtils::MakeMaker and ExtUtils::CBuilder both default to this anyway,
but it's good to be explicit, and reduces diffs with the .gc64 file.

perldelta updates

perldelta for 186a5ba82d58

This entry is only for the user-visible changes.

remove bogus $remainder variable from perlfunc/split

As reported by Anša Vernerová:

| Calling the third argument $remainer is simply wrong - the remainder is
| the fourth argument, which is not assigned to anything.

overloading: make qr fallback to "" better

With the re_eval jumbo fix, the behaviour of overloaded objects in
runtime patterns, such /^$overloaded/ has changed, such that the stringify
overload ("") no longer avoids the need for 'use re "eval"': for example,

    use overload "" => sub { qr/(??{1})/ }
    my $o = bless [];
    "1" =~ /^$o/;

works in 5.16.0, but dies with "Eval-group not allowed" in blead.

Change this back to the former behaviour, such that if qr and concat
ops aren't overloaded, then use "" overloading, and if the return from
that is a qr object, extract any code blocks from it.

This is achieved by:
* moving the concat/stringify code ahead of the regex block extraction
  code,
* making the overloaded stringify call be explicit (rather than
  being invoked implicitly by sv_catsv()),
* looping to re-apply overloading to any object returned by "".
* applying those last two steps in the case of a single arg too

This is a partial fix for
[perl #113682] Bleadperl v5.17.0-424-gd24ca0c breaks
    ABIGAIL/Regexp-Common-2011121001.tar.gz

In cflags.SH trim the case statement of file basenames.

Add examples showing how to use cflags.SH to tweak the compiler flags used
for individual object files.

Previously cflags.SH contained a somewhat stale pre-canned list of file
basenames including removed files such as usersub (deleted before 5.000
shipped), and a partial list of 5.000 XS extensions. Whilst it's possible
to generate the correct list in cflags by parsing MANIFEST (and adding a few
fixups), it's still not actually *useful*, as cflags gets overwritten as
soon as config.sh changes. Hence the most end-user useful solution with
minimal maintenance is to eliminate the list entirely, and document how the
user should add to it as necessary.

In pp_iter, handle end of range at IV_MAX without undefined behaviour.

The previous code assumed that incrementing a signed integer value wraps.
We're lucky that it has (so far), as it's undefined behaviour in C.
So refactor to code which doesn't assume anything.

In t/op/arith.t, tryeq_sloppy() wasn't correctly handling negative values.

The code was treating any negative value as "ok", rather than only those
within 1e-9 of the expected value. This bug has been present since
tryeq_sloppy() was added by commit 800e6488b45c59f1 in Oct 2001.

this is already mentioned at the end (and that is more correct)

remove some boilerplate

todo

consistency

combine sections

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

fix inadvertent removal during editing

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.