Create new perldelta

Update epigraphs.pod

Merge branch 'release-5.21.7' into blead

add new release to perlhist

Update perldelta

Update Module::CoreList for 5.21.7

Skip multideref for PL_check[OP_HELEM] tinkering

We already do it when PL_check[OP_AELEM] is modified.  Do it
also for PL_check[OP_HELEM].  This will make it easier to get
Data::Alias working.

dump.c: Suppress extra sigils in multideref dump

This:

  my $z; my @y; $y[$z]

included

  <+> multideref($@y[$$z]) sK ->6

in its -MO=Concise output.

dump.c:unop_aux_stringify: Use newSVpvn_flags

newSVpvn_flags with SVs_TEMP takes less machine code than
sv_2mortal(newSVpv()).

Allow multideref for $x[CONST->$*]

Allow multideref for $x[($pkg)]

The parens don’t need to disable the optimisation.

Fix assertion failure with $x[CONST->[0]]

Fix assertion failure with $x[($_)]

Allow multideref for $x[($lex)]

The parens don’t need to disable the optimisation.

(The OPpLVAL_INTRO flag [indicating my or state] has no run-time
effect for state vars, so we could ignore it and optimise, but it
would make deparsing inordinately complex, and no-one would write
$x[state $y] anyway.)

protect RExC_naughty changes behind macros

Avoid overflow, and add a bit of explanation.

PerlIO::scalar: skip the 4GB seek test if off_t is too small

01234567890123456789012345678901234567890123456789
pp.h: Remove stack-popping from void overload code

There is no need to pop the stack in void context, as every void-context
op is followed by something that resets the stack.

[perl #123458] list cx re::regexp_pattern($nonre)

It was returning (undef) in list context, though it was documented to
return the empty list.

universal.c:re_regexp_pattern: Mention GIMME_V once

While it was only being called once, it occurred in two code paths.
Pulling it out of the main if-block reduces the size of universal.o.
Before and after:

$ ls -l universal.o
-rw-r--r--  1 sprout  staff  33700 Dec 19 16:05 universal.o
$ ls -l universal.o
-rw-r--r--  1 sprout  staff  33692 Dec 19 16:13 universal.o

pp_sys.c:pp_readdir: Call GIMME_V just once

pp_ctl.c: Only do GIMME_V once in pp_caller

This shrinks the machine code.

Before and after:

$ ls -l pp_ctl.o
-rw-r--r--  1 sprout  staff  96156 Dec 19 16:05 pp_ctl.o
$ ls -l pp_ctl.o
-rw-r--r--  1 sprout  staff  96044 Dec 19 16:09 pp_ctl.o

Use GIMME_V in preference to GIMME

GIMME_V is a simpler macro that results in smaller machine code.

GIMME does not distinguish between scalar and void context.  The two
instances of GIMME == G_SCALAR that I changed (which used to match
void context too, but no longer do) are in code paths unreachable in
void context, so we don’t need to check for it.

wantarray.t: $got before $expected

When the tests fail, the diagnostic output is very confusing
otherwise.

wantarray.t: Test logops at sub exit

wantarray.t: Remove unused var

It stopped being used in dab34d0f01.

fix integer overflow in S_regpiece().

RExC_naughty is incremented when nasty bits of regex are found.
If at the end of compilation, its > 01, then PREGf_NAUGHTY is set on the
pattern.

However, some bits of S_regpiece on detecting naughiness, double or nearly
double RExC_naughty, quickly resulting in overflow of the I32.

I've fixed it by skipping the doubling when RExC_naughty is large,
but I don't know whether the doubling is conceptually wrong in the first
place.

Found by -fsanitize=undefined.

fix integer overflow in S_study_chunk().

It was adding to delta even when delta was already SSize_t_MAX

This triggered it: /.*(ab|abc)/.

Found by -fsanitize=undefined.

fix integer overflow in S_study_chunk().

It was adding SSize_t_MAX to data->last_start_max when
data->last_start_max was already SSize_t_MAX.

This triggered it: /(x+y)+/.

Found by -fsanitize=undefined.

fix integer overflow in S_scan_commit().

offset_float_max could end up as SSize_t_MAX+1.

This triggered it: /^x(ab|c.+)$/.

Found by -fsanitize=undefined.

add asan_ignore - which errors to ignore

"clang -fsanitize=undefined" produces lots of false positives.
This file allows certain functions to be excluded from checking.
Use it as:

    clang -fsanitize=undefined -fsanitize-blacklist=`pwd`/asan_ignore

op.c: better casting fix

My commit 0cb87cd1a of a few minutes ago broke g++. Hopefully this
commit won't break anything else.

fix a couple of win32 op.c warnings

These warnings appear on win32 smokes.

op.c(8600) : warning C4244: 'function' : conversion from 'cv_flags_t' to 'char'
, possible loss of data
op.c(11905) : warning C4146: unary minus operator applied to unsigned type, res
ult still unsigned

Since I'm not running windows, I can't confirm that this commit shuts up
these warnings, but I think think I've understood them correctly.

ODBM: fix a compiler warning.

This has been applied blind since my system doesn't have libodbm,
but it matches exactly the cast added to NDBM's typemap a few years ago to
fix a similar warning.

NDBM: silence 'unused var' warnings x 2

(I've done this blind because I don't have libndbm on my system)

B.xs: remove redundant assignment

Follow-up to 500f40f555ee.

ExtUtils::Embed: silence test warning

From the added code comments:

    XXX DAPM 12/2014: ExtUtils::Embed doesn't seem to provide API access
    to $Config{optimize} and so compiles the test code without
    optimisation on optimised perls. This causes the compiler to warn
    when -D_FORTIFY_SOURCE is in force without -O. For now, just strip
    the fortify on optimised builds to avoid the warning.

Also allow /\N{}/

See 4cbd7e223 and ticket #123417.

If a charnames handler returns the empty string for a particular name,
then the regular expression stringifies with an empty \N{} in it.
This needs to round-trip properly (eval "/$qr/"), just like \N{U+...}.

perldelta for 1e2dd519

perldelta for 3b91d897

perldelta for 1f3063473

perldelta for 9d52f6f3f87

perldelta for 84ee769f11

include reference to metacpan.org; refer to the module-authors list instead of modules, and include a link to the subscription page

Update Unicode-Collate to CPAN version 1.09

  [DELTA]

1.09  Thu Dec 18 21:39:18 2014
    - XS: a workaround for perl 5.6.x to handle noncharacters U+FFFF etc.
      is abandoned. Perl 5.8.0 or later is recommended for handling these
      noncharacters.

perldelta for 696efa16de2c

fix PerlIO::scalar get_cnt when the file position is beyond 2GB

This caused a new test to fail on 32-bit builds.

regen pod issues

The F<>/L<> warning is bogus in this case.

[perl #123417] Allow lexer to parse \N{U+dotted.hex}

This ‘unoffical’ notation is used in stringifying regular expressions
that contain named sequences, so that qr/\N{foo}/ stringified can be
incorporated into another regular expression in a different scope and
still mean the same thing.

This also needs to work with eval "/$that_qr/".  I didn’t because the
lexer rejected this syntax.

perldelta for 63d073d27fe5, 1d050e5534ce

don't allow a negative file position on a PerlIO::scalar handle

previosly seek() would produce an error, but would still make the\
file position negative.

[perl #123443] avoid overflowing got into a negative number

Update Filter::Util::Call to CPAN version 1.51

  [DELTA]

1.50 2014-06-04 rurban
----

  * Do not re-bless already blessed filter_add arguments into the callers package.
    Fixes RT #54452
  * t/z_pod-coverage.t: omit empty Filter::decrypt (also fixes RT #84405)
  * Fix Perl Compiler detection in Filter::decrypt

1.51 2014-12-09 rurban
----

  * Minor -Wall -Wextra cleanups by jhi and me. Fixes RT #100742
  * Updated Copyright years
  * Document and warn about its limitations

Sort perldiag

perldelta typos

Fix bad read in pad.c:cv_undef

When freeing a sub, we can’t assume an entry named "&" contains a CV.
It may instead be a weak reference to a format or named sub, or undef
if such a reference went stale, in which case we don’t want to mess
with CvOUTSIDE pointers.

This bug probably goes back to v5.17.1-213-ge09ac07, when weak refer-
ences started being stored in "&" pad entries.  It didn’t start trig-
gering AddressSanitizer failures until it was extended to named subs,
in v5.21.6-386-ga70f21d.

improve xs_handshake() diag message

The important part of the error message is that the binaries are
mismatched; the details of the handshake keys are an implementation
detail.

Or to put it another way, when someone mixes up their paths, getting
something like

    Fcntl.c: Invalid handshake key got 0xcf80000 needed 0xd700000, binaries are mismatched

Is a bit scary and confusing. This is hopefully (slightly) less scary:

    Fcntl.c: loadable library and perl binaries are mismatched (got handshake key 0xcf80000, needed 0xd700000)

xs_handshake(() clean up desciption

Clean up the description of this function; in particular, say at the top
what the function is for; fix typos; and generally improve the readability
of the text.

Template::Declare's tests now pass under 5.22-to-be

Remove duplicate apidoc entries

Modify apidoc.pl to warn about duplicate apidoc entries, and
remove duplicates for av_tindex and toLOWER_LC

update perldelta for UNOP_AUX and OP_MULTIDEREF

INSTALL: warning text has changed

Some text tells you about

    Note (probably harmless): No library found for -lsomething

warnings, but the actual warning has now changed to

    Warning (mostly harmless): No library found for -lsomething

Deparse constant Perl subs

Subs like sub f () { 42 } stopped being emitted, probably in
v5.21.3-638-g2eaf799, when such subs started being stored as simple
scalar refs in the stash.

VMS does have fsync, so configure accordingly.

And has had since roughly the beginning of time, and has even been
using it since right around then in vms/vms.c.  But it doesn't work
on read-only filehandles, so skip the test that expects that.

Clean up VMS includes in ext/Errno/Errno_pm.PL.

There are currently (and for many years now) no viable alternatives
to the headers that come with the operating system, regardless of
compiler.

Obsolete workaround in ext/SDBM_File/sdbm/Makefile.PL.

Hasn't been necessary or relevant for some years.

Add Template::Declare to perl5220delta

for #123364.

Storable.xs: introduce SEEN*_NN

Introduce _NN versions of the SEEN() and SEEN0() macros, that
don't bother to check whether their first arg is null.

The initial motivation for this was to to silence a compiler warning
about a conditional always being true in SEEN(&PL_sv_undef,...), but
I've also applied it to all places in the code where it's clear that the
arg can't be null, e.g.

    sv = NEWSV();
    SEEN(sv,...);

At it happens, "places where" currently equates to every usage of
SEEN/SEEN0 in the source code.

perldiag.pod: fix typo

toupper/lower: avoid sign warnings

Perl's toLOWER_LC() etc macros are specified as having U8 arg and return,
while the underlying macro may call the OS's tolower() function which is
int. Stop the compiler warning about mismatched sign in conditional by
casting the result of the OS function.

[perl #123436] installperl patch related to $Config{dlext}

[perl #123438] Wrong comment style in win32/win32.h

[perl #123020] Scalar cx for lhs of void (...)x...

No sane code calls x in void context (except at the end of a subrou-
tine), but we have to handle it anyway.

Previously, the left-hand parenthesized operand to (...)x... would be
evaluated in the context in which the current sub was called.  I.e.,
if the last statement in the current sub is called in last context,
then then lhs of (...)x... a hundred lines earlier (say we have a
big sub) is called in list context, even though it is completely
unrelated.

Since the left-hand operand could be called in any of the three con-
texts when x itself was in void context, I could choose whatever I
wanted when making it consistent.  Scalar context makes the most sense
to me, because x falls back to string repeat when not in list context,
and because void context makes stack handling complex.

Unfortunately, this does not fix the context if x occurs at the end of
a subroutine.  That will be a lot trickier to fix.  (Hence, the com-
plex stack handling in pp_repeat must remain.)

Don’t ignore rv2gv when deparsing readline

Unlike most handle ops, readline does not implicitly add an rv2gv op.

In fact, there is a difference in behaviour, depending on whether
there is an rv2gv op present.  An explicit * (rv2gv) will force vivi-
fication of the typeglob, but ‘readline’ alone won’t vivify it.

So B::Deparse should not ignore the rv2gv kid of readline.

Previously, readline(*$foo) (vivifying) would deparse as <$foo> (non-
ivifying), and readline *{$foo} would deparse as readline do { $foo }
(same behavioural difference again).

Deparse: Restore proto checks for coderef2text

This fixes #123435 (I hope).

Before v5.21.6-485-gf2279a6, B::Deparse, when deparsing sub calls,
would look in the stash for a sub with a prototype, and use that
prototype for deparsing.  (It would only do this for subs that it
itself is not going to deparse.)  That caused this to deparse
incorrectly:

BEGIN {
   require Test::More;
   is("foo", "foo");
}

Because of the presence of the prototyped &main::is by the time
B::Deparse is invoked, it would deparse the is() sub call without
parentheses, resulting in invalid code.

So I changed it to trust only those subs that it has already deparsed.

Now, with coderef2text, B::Deparse doesn’t deparse any subs except the
one passed to it.  So *all* prototype sub calls turn into &foo, break-
ing CPAN tests that expect otherwise.  While such tests could be con-
sidered too sensitive, I don’t think that this is necessarily a sane
default for coderef2text.

Ideally we should provide the option to tell coderef2text which
subs to trust, the default being to trust all of them.  But for
now I will just implement that default, which means reverting
v5.21.6-485-gf2279a6 *for coderef2text*, but not when deparsing the
main program.

Deparse.pm: Remove obsolete comment about globals

Knowing that object fields were globals is no longer useful, unless
one is intimately familiar with a very old version of B::Deparse and
wants to find his way around the new one.  (Good luck!)

Furthermore, several of these ‘former globals’ were never globals,
because they came after this comment.

Skip obsolete skip in io_xs.t.

The VAXC compiler and its compatibility library VAXCRTL are long
since obsolete.

perldelta for 3f6e436ff088

also move the B::Deparse fix under B::Deparse

document + improve ancient optimization in ParseXS

From commit 8990e30710 "perl 5.0 alpha 6". The DESTROY XSUB with T_PTROBJ
as INPUT arg was refusing to obey my custom typemap for T_PTROBJ, but all
other XSUBs followed the custom T_PTROBJ override. Extend this
optimization to other rarely used typemap types and document this behavior.

Deparse: Stop =~ QR_CONST from dropping flags

Add B::REGEXP::compflags

Deparse.pm: Two more matchwords

This is *highly* important. :-)

deparse-skips.txt: op/getpid.t fails

deparse-skips.txt: More passing tests

Deparse system-BLOCK correctly

Same with exec.  Don’t put ‘do’ before the block.

This was accomplished by running it through the same code path
(indirop) that print and sort already go through.

There was a slight discrepancy between listop and indirop in the way
they handled no arguments.  Before we would get:

    () = system;
    () = exec;
    () = sort();

Instead of making the first two get the redundant parentheses, I
changed indirop, so sort lost them.

deparse-skips.txt: eval tests pass

deparse-skips.txt: Skip reason for op/each.t

pp_readline: Don’t set PL_last_in_gv to &PL_sv_undef

Code elsewhere assumes it is always a GV or NULL:

readline "foo";
my $lastfh = "${^LAST_FH}";

add cpan/Test-Simple/t/Behavior/subtest_die.t to MANIFEST

perl.pod: Use consistent perldelta order

perl.pod: Line things up vertically

Import perl5184delta

Test-Simple Version Bump, 1.301001_090 (RC10)

Don’t set PadlistMAXNAMED for single-char entries

This fixes perl #123430.

Single-character entries like "$" and "&" are not actually named
entries, but are partially treated as such for the sake of bookkeeping
and scope.  (E.g., a flipflop target must have the same lifetime as a
state variable.)

PadlistMAXNAMED is an optimisation that marks the offset of the high-
est pad slot with a name.  If there any many anonymous pad slots after
the last named one, we don’t want to search through them when looking
up a symbol.  So we mark the maximum named slot and skip half the pad
if we are lucky.

Commit v5.21.4-51-g14d9114 caused flipflop targets to be allocated as
variables named "$", causing compilation of some generated code to
slow down.  At compile time, the name pad is not extended to the
length of the pad until the end of subroutine compilation.  So prior
to 14d9114 flipflop targets would not make the name pad any longer.
Now that flipflop targets expand the name pad, stop setting
PadlistMAXNAMED, so that things are no slower than before.

This is not really the best fix, IMO, because code that is sensitive
to this will slow down dramatically if you tweak it ever so slightly
by adding a ‘my $x’ here or there.

perldelta for syber’s method stuff

b55b14d0f234ed20e6c2a0b6fd8609fa418cddf3
e3384dcea386322f748cca779e4b8d80c8644024
d648ffcb179b885089e064ec1d58c60027c80915
7d6c333c75cb0519428c389de3894edcb394d3a0
810bd8b704e337dfba3e46eaea33244c1b0afce3
c6afe6664edd0f72ba320c7d17ab247876849084
c290e187228e365708b7d5a9ae53ad10c3c002c6

This description is not precise, but trying to explain exactly which
cases have and have not been optimised would make it confusing.

perldelta for 0ed5b3c8f7

perldelta for 0175f038c506e

perldelta for 127ce1c

perldelta for f6a1dc93

perldelta for 3f193e551 and 8c2e27d21

perldelta for bae5b54e7