speed up AV and HV clearing/undeffing

av_clear(), av_undef(), hv_clear(), hv_undef() and av_make()
all have similar guards along the lines of:

    ENTER;
    SAVEFREESV(SvREFCNT_inc_simple_NN(av));
    ... do stuff ...;
    LEAVE;

to stop the AV or HV leaking or being prematurely freed while processing
its elements (e.g. FETCH() or DESTROY() might do something to it).

Introducing an extra scope and calling leave_scope() is expensive.
Instead, use a trick I introduced in my recent pp_assign() recoding:
add the AV/HV to the temps stack, then at the end of the function,
just PL_tmpx_ix-- if nothing else has been pushed on the tmps stack in the
meantime, or replace the tmps stack slot with &PL_sv_undef otherwise
(which doesn't care how many times its ref count gets decremented).

This is efficient, and doesn't artificially extend the life of the SV
like sv_2mortal() would.

This commit makes this code around 5% faster:

    my @a;
    for my $i (1..3_000_000) {
        @a = (1,2,3);
        @a = ();
    }

and this code around 3% faster:

    my %h;
    for my $i (1..3_000_000) {
        %h = qw(a 1 b 2);
        %h = ();
    }

t/op/read.t: test with zero-length buffer

This test file had:

    my (@values, @buffers) = ('', '');

which isn't doing what the author probably intended. Instead it's testing
twice for the same zero-length value, and not testing at all for a zero
length buffer.

Net::Ping: avoid stderr noise in tests

Send status info to STDOUT rather than STDERR.

Add perldelta entries for

8b0c3377906a6f991cd6c21a674bf9561d85e3cb
b09ed995add057b8e0b51964b48ef1d1cc3c9c91

tiearray.t - more fine-grained DESTROY counts

It currently tests once near the end of the script that DESTROY has been
called 3 times. Instead test after each individual scope exit
where we expect DESTROY to be called

reindent tiearray.t

The indents in this test file were a mixture of 1,2,4, and most
importantly, 0 - which made it very hard to visually see the scope
of lexical vars.

Reindent to a standard 4-char indent.

Also delete trailing whitespace from lines.

Whitespace-only changes

Handle list assignment in list context better

In something like

    sub inc { $_++ for @_ }
    inc(($a,$b,$c,$d) = (10,20))

The four scalar vars will end up with the values (11,21,1,1), with
the list assign op returning 4 lval scalars to be passed to inc.

However, when the LHS includes a hash or array, any 'empty' scalars weren't
being returned. This:

    inc(($a,$b,@array,$c) = (10))

used to leave $b and $c undefined. With this commit, they are both set to
1.

This change broke some tests in hashassign.t, which were added in 2012
by commit v5.17.6-295-gb1babc5. Half these tests were commented as

    # this arguable, but this is how it works

and they all tested that a scalar following a hash wasn't returned in
lvalue context; i.e. they all assumed that

    inc((%h, $x) = (...))

*wouldn't* increment $x. This commit causes $x to be incremented, and
I've changed the failing tests.

It also adds an EXTEND(SP,1) in the scalar case; otherwise with
scalar( () = @a) and empty @a, it could in theory push the '0' return
value off the end of the stack.

Better optimise array and hash assignment

[perl #127999] Slowdown in split + list assign

Re-implement the code that handles e.g.

    (..., @a) = (...);
    (..., %h) = (...);

to make it a lot faster - more than reversing a performance regression
introduced in 5.24.0 - and fix some bugs. In particular, it now
special-cases an empty RHS, which just clears the aggregate, e.g.

    (..., @a) = ()

Getting list assignment correct is quite tricky, due to the possibility of
premature frees, like @a = ($a[0]), and magic/tied values on the LHS or
RHS being triggered too soon/late, which might have side-effects.  This
often requires making a copy of each RHS element (and indeed for assigning
to an array or hash, the values need copying anyway). But copying too soon
can result in leaked SVs if magic (such as calling FETCH()) dies. This
usually involves mortalising all the copies, which slows things down.

Further, a bug fix in 5.24.0 added the SV_NOSTEAL flag when copying SVs.
This meant in something like @a = (split(...))[0,0], where the two SvTEMPs
on the RHS are the same, the first copy is no longer allowed to steal the
PVX buffer, which would have made the second SV undef. But this means that
PVX buffers are now always copied, which resulted in the slowdown seen in
RT #127999.

Amongst the general rewriting and optimising, this commit does the
following specific things to boost performance (and fix RT #127999).

* If the SVs on the RHS are non-magical SvTEMPs with a ref count of 1, then
the SV isn't copied; instead it is stored directly in the array/hash. This
more than undoes the cost of SV_NOSTEAL.

* The tmps stack is now used as a temporary refcounted version of the
argument stack frame, meaning that args placed there will be freed on
croak.  In something like @a = (....), each RHS element is copied, with
the copy placed on the temps stack. Then @a is cleared. Then the elements
on the tmps stack are stored in the array, and removed from the temps
stack (with the ownership of 1 reference count transferring from the temps
stack to the array). Normally by the time pp_aassign() returns, there is
nothing left on the tmps stack and tmps_free() isn't called - this is the
novel element that distinguishes this from the normal use of mortalising.

* For hash assignment, the keys and values are processed in separate
loops, with keys not normally being copied.

* The ENTER/SAVEFREESV(ary/hash)/LEAVE has been removed, and the array or
hash kept temporarily alive by using the temps stack along with all the
other copied SVs.

* The main 'for each LHS element' loop has been split into two loops: the
second one is run when there no more RHS elements to consume. The second
loop is much simpler, and makes things like @a = () much faster.

Here are the average expr::aassign:: benchmarks for selected perls
(raw numbers - lower is better)

          5.6.1    5.22.0    5.24.0    5.25.5      this
         ------    ------    ------    ------    ------
    Ir   1355.9    1497.8    1387.0    1382.0    1146.6
    Dr    417.2     454.2     410.1     411.1     335.2
    Dw    260.6     270.8     249.0     246.8     194.5
  COND    193.5     223.2     212.0     207.7     174.4
   IND     25.3      17.6      10.8      10.8      10.0

COND_m      4.1       3.1       3.1       3.7       2.8
 IND_m      8.9       6.1       5.5       5.5       5.5

And this code:

    my @a;
    for my $i (1..10_000_000) {
        @a = (1,2,3);
        #@a = ();
    }

with the empty assign is 33% faster than blead, and without is 12% faster
than blead.

bench.pl: fix --sort and --compact options

They were using the full pathname of the perl executable to index into
a hash indexed by label.

OpenBSD 6 still does not support returning pid, gid or uid with SA_SIGINFO

make do "a\0b" fail silently instead of throwing (RT #129928)

Also remove the label/goto from CLEAR_ERRSV because labels have function
scope, which means you couldn't use CLEAR_ERRSV more than once per
function without getting a "duplicate label" error.

RT#129229: move sort_manifest() into its own library

This means that the MANIFEST.srt target in the Makefile no longer needs
to load a library that depends on Cwd (and other potentially-dynamic
modules). That in turn fixes a missing-dependency bug in the Makefile.

hv.h: rework HEK_FLAGS to a proper member in struct hek

Move the store of HEK_FLAGS off the end of the allocated
hek_key into the hek struct, simplifying access and providing
clarity to the code.

What is not clear is why Nicholas or perhaps Jarkko did
not do this themselves. We use similar tricks elsewhere,
so perhaps it was just continuing a tradition...

One thought is that we often have do strcmp/memeq on these
strings, and having their start be aligned might improve
performance, wheras this patch changes them to be unaligned.
If so perhaps we should just make flags a U32 and let the
HEK's be larger. They are shared in PL_strtab, and are
probably often sitting in malloc blocks that are sufficiently
large enough that making them bigger would make no practical
difference. (All of this is worth checking.)

[with edits by Yves Orton]

(perl #129130) make chdir allocate the stack it needs

chdir with no argument didn't ensure there was stack space available
for its result.

META.json regen

(Porting/makemeta is not run by 'make regen', apparently)

Scalar-List-Utils: regen customized.dat

Scalar-List-Utils: customize Maintainers.pl

Scalar-List-Utils: customized VERSION bump

Scalar-List-Utils: netbsd-vax: no inf/nan

Scalar-List-Utils: netbsd-vax: no inf/nan

Math-Complex: regen customized.dat

Math-Complex: customize Maintainers.pl

Math-Complex: customized VERSION bump

Math-Complex: netbsd-vax: no inf

Math-Complex: netbsd-vax: no inf, different range

Math-Complex: netbsd-vax: no inf

Math-BigInt: regen customized.dat

Math-BigInt: Maintainers.pl customization

Math-BigInt: customized VERSION bump

Math-BigInt: netbsd-vax: different float range

JSON-PP: regen customized.dat

JSON-PP: customize Maintainers.pl

JSON-PP: reformat in preparation for next change

JSON-PP: customized VERSION bump

JSON-PP: netbsd-vax: different float range

JSON-PP: netbsd-vax: different float range

op.c: silence compiler warning in fold_constants()

op.c: In function ‘S_fold_constants’:
op.c:4374:28: warning: argument ‘o’ might be clobbered by ‘longjmp’ or ‘vfork’ [-Wclobbered]
 S_fold_constants(pTHX_ OP *o)
                            ^

This warning occurs for non-volatile local variables where the compiler
can't prove that their value doesn't change between setjmp and longjmp
(because the modified value may only be stored in a register which
longjmp overwrites). Adding 'const' apparently convinces the compiler
that no such modification occurs.

Check for echo in new Net::Ping tests.

Net::Ping->new() with no parameters calls

  getservbyname('echo', 'tcp');

and falls down hard if that call does not succeed.  It would be
unusual to be running an echo service on any well-maintained
system on any platform.  On VMS at least, doing a name lookup on
a service that is not configured and enabled returns an error, and
thus any test that uses this ancient default will fail.  So skip
those tests if the getservbyname() fails.

POSIX version bump

vax-netbsd: no nan

vax-netbsd: do not tempt fp overflow, which will SIGFPE

The 3e100 seems to have no special meaning, except being in floating point.

vax-netbsd: no PL_inf/PL_nan to export

vax-netbsd: do not test for inf/nan

vax-netbsd: test non-IEEE-754-ness only once

vax-netbsd: set the RETVAL even if unused

Configure: signbit scan assuming too much

It was assuming a negative zero, which is an IEEE-754 only concept.

There is no need to assume the negative zero for the correct
functioning of the signbit, however.

vax-netbsd: another negative zero assumption

Though this had been masked by NV_INF guard.

vax-netbsd: Negative zero is only a thing IEEE 754.

So don't assume it universal in the signbit emulation.

For non-IEEE-754, negative zero is equal to the positive zero.

(Another matter is that signbit() should have been found
in the Configure scan; it does exist in the netbsd/vax.)

vax-netbsd: expand lround() and signbit testing

The 1994b214 made the test dependent on Tie::Hash::NamedCapture.

Which is not available under miniperl.

To add to the fun, tie() is partly compile-time, so just simple
skipping is not enough.

t/uni/overload.t: Skip hanging test on FreeBSD

This is a test that overflows the C stack though infinite
recursion. Unfortunately, FreeBSD doesn't seem to overflow the
stack as much as it overflows main memory. On my FreeBSD 12 VM,
it uses all available swap and hangs. This patch skips it only
for FreeBSD.

APItest/t/utf8.t: Fix a test on EBCDIC

I forgot to translate this to EBCDIC, so created massive failures there,
obscuring the real bugs.

Move _pDEPTH and _aDEPTH after config.h.

Otherwise DEBUGGING may not be defined yet (at least it isn't on
VMS).

Update Module::Corelist with entries for 5.25.7

t/op/blocks.t: tests for RT #113934

t/op/goto.t: tests for RT #45091

t/lib/overload_fallback.t: tests for RT #43356

t/op/threads.t: tests for RT #41121

t/op/threads.t: tests for RT #36664

t/re/subst.t: tests for RT #23624

t/re/pat.t: tests for RT #21491

t/lib/warnings/op: tests for RT #6870

t/op/local.t: tests for RT #7615

t/io/socket.t: tests for RT #7614

t/op/local.t: tests for RT #7411

t/op/caller.t: tests for RT #7165

t/op/die.t: tests for RT #4821

t/lib/warnings/toke: tests for RT #4346

t/op/attrs.t: tests for RT 3605

Maybe this should be in a different file?

t/op/bless.t: tests for RT #3305 and RT #3306

t/uni/overload.t: test for RT #3270

t/uni/overload.t: test for RT 3054: might segfault.

This one may be a bit dangerous. It is also one of many bugs
involving a segfault due to a C stack overflow.

t/op/blocks.t: add test for RT #2917

t/op/blocks.t: add test for RT #2754

t/op/for.t: RT #2166: Actually run the test so we know if behavior changes

t/op/local.t: Unknown RT#, but appears to be fixed. Blame says not edited since 2005.

t/op/for.t: RT #1085: ticket 'resolved' but test was still 'todo'

t/op/attrs.t: Fixup for 7fe45fb9 - should be a semicolon

pp_ctl.c: silence compiler warning about mixing (un)signed types

Commit d081a35540aca5fe changed PADOFFSET (the type of op_targ) to
SSize_t (a signed type). It used to be unsigned.

pp_ctl.c: In function ‘S_doeval_compile’:
pp_ctl.c:3350:31: warning: signed and unsigned type in conditional expression [-Wsign-compare]
        ? oldcurcop->cop_hints : saveop->op_targ;
                               ^

the test for #129897 was missing the min-mod (?) on the dot star

A plain .* will make the pattern match skip the bug we
are testing for. We need .*? to see the difference

Fixed:
./perl -Ilib -le'"riiaan"=~/(.*?(a(a)|i(i))n)/ and print "$2-$3-$4-$1"'
aa-a--riiaan

Broken:
perl -le'"riiaan"=~/(.*?(a(a)|i(i))n)/ and print "$2-$3-$4-$1"'
aa-a-i-riiaan

Maybe the test could make this more explicit. Not sure if its
worth it.

Thanks to Aaron Crane for noticing the test didn't do what it was
meant to. Bad Yves.

Bump version numbers ready for 5.25.7

New perldelta for 5.25.7

Tick the 5.25.6 release

Add epigraph for 5.25.6

Merge branch 'release-5.25.6' into blead

Add 5.25.6 release to perlhist

Finalise perldelta for 5.25.6

Update Module::CoreList for 5.25.6

Update 5.25.6 release date in Module::CoreList

Revert "Include time.h when testing for clock_xxx functions and syscalls"

This was meant to go on my smoke-me branch, not blead.

This reverts commit 1946ca1569f5d47e6ceffab716bce3a4169646fe.

Update Archive-Tar to CPAN version 2.14

  [DELTA]

2.14  20/10/2016
- Fix roundtrip test when tar executable is absent

perldelta: delete unneeded sections

perldelta: draft for commits up to 3cc6a05eed

Include time.h when testing for clock_xxx functions and syscalls

perly.y: remove redundant NULL casts

libperl.t: treat i686 arch the same as x86

toke.c: remove redundant (OP *) casts

toke.c: get rid of "if (0)"

utf8n_to_uvchr(): Reduce chances of reading beyond buffer

utf8n_to_uvchr() can be called incorrectly, leading it to believe the
buffer is longer than it actually is.  But often, it will be called with
NUL terminated strings, so it can reduce it's chances of being fooled by
refusing to read beyond a NUL.  The NUL will terminate any UTF-8 byte
sequence, and the only reason to read beyond it would be to print all
the expected bytes in the sequence.

This commit is not the final word, but it is an easy fix for a common
case.

podcheck.t: .core dump files don't contain pod

even if they have pod lines in them.