bump $Storable::VERSION and update ChangeLog

minimally document the stacksize tool

move Storable CAN_FLOCK computation into XS

At build time, Storable would check if it could use flock based on some
Config values, and generate the final Storable.pm file using tha value.
This was done to avoid needing to load Config at runtime.  This adds
complexity to the build process.

A simpler option is to build the constant in the XS code.  This means
the build process can be entirely standard for XS.

Some slight adjustments to the code are needed to load the XS module at
BEGIN time, so that the contant is available in the rest of the file.

remove remains of Storable stack size detection

At build time, Storable used to detect the stack limits and store them.
This was removed in 2a0bbd31bbcc94c7c511f9575df619c5fdf3164c.  That
removed the Limit module and the setup for building it, but left behind
the script used to detect the limits.

Clean up the script and some related bits in the Makefile.PL.

TonyC: retain stacksize, so end users can use it to choose limits

perldelta for d1bc97feec1a

(perl #133695) "0".."-1" should act like 0..-1

Previously, *any* string beginning with 0, including the string "0"
itself, would be subject to the magic string auto-increment, instead of
being treated like a number. This meant that "-2".."-1" was the same as
-2..-1 and "1".."-1" was the same as 1..-1, but "0".."-1" was the same
as "0".."99".

This patch fixes that inconsistency, while still allowing ranges like
"01".."31" to produce the strings "01", "02", ... "31", which is what
the "begins with 0" exception was intended for.

This patch also expands the documentation in perlop and states the rules
for the range operator in list context with both operands being strings
more explicitly.

See also #18165 and #18114.

bump $PerlIO::via::VERSION

PerlIO::Via: check arg is non-NULL before using it.

I can't find any code in core that ends up calling the _pushed handler
with arg == NULL, but PerlIO_push() is API, and there might be
CPAN or DarkPAN code out there that does, escpecially since there's
a check for arg being non-NULL further down.

CID 169261.

Storable: make count large enough

AvARRAY() could be very large, and we check for that at line 3807,
but int was (potentially) too small to make that comparison
meaningful.

CID 174681.

locale.c: Stop Coverity warning

Coverity is right, so re-order these clauses.  This code is executed
only if some very strange error occurs.

Revert "Revert "postpone perl_parse() exit(0) bugfix""

This reverts commit 2773b4f50f991900e38d33daace2b9c6a0902c6a.

I haven't made much progress in resolving the problems this produces
downstream, so rather than leaving it broken, I'll revert it until
they can be solved.

avoid use-after free in /(?{...})/

RT #134208

In something like

    eval { sub { " " }->() =~ /(?{ die })/ }

When the match string gets aliased to $_, the SAVE_DEFSV is done after the
SAVEDESTRUCTOR_X(S_cleanup_regmatch_info_aux).  So if croaking, the SV
gets SvREFCNT_dec()ed by the SAVE_DEFSV, then S_cleanup_regmatch_info_aux()
manipulates the SV's magic.

This doesn't cause a problem unless the match string is temporary, in
which case the only other reference keeping it alive will be removed
by the FREETMPs during the croak.

The fix is to make sure an extra ref to the sv is held.

include a trailing \0 in SVs holding trie info

RT #13427

TRIE_STORE_REVCHAR() was creating SvPV()s with no trailing '\0'. This
doesn't really matter given the specialised use these are put to, but
it upset valgrind et al when perl was run with -Drv which printf("%s")'s
the contents of the string.

Update Win32 to CPAN version 0.53

  [DELTA]

0.53  [2019-08-05]
        - improve Win32::GetOSDisplayName
        - added Win2016/2019 detection and version information by
          Richard Leach [PR/15]
        - Include wchar.h to allow building with g++ by Tony Cook [rt#127836]

0.52_02 [2018-11-02] by Reini Urban
        - added () usage croaks.
        - Fixed a -Warray-bounds buffer overflow in LONGPATH,
        - Fix various -Wunused warnings
          and two -Wmaybe-uninitialized.

0.52_01 [2017-11-30]
        - add missing const

op_dump(): display a wild parent pointer.

Normally the PARENT is displayed only for top-level ops: lower-level
ops have the obvious parent.

This commit adds a check that a lower-level op has a valid parent (i.e.
non-null, points to an op with OPf_KIDS and one of the kids is us).
If these checks fails, the raw content of op_sibparent is displayed.

I18N::LangTags: minor clarification

Don't imply that there are other undocumented env vars in play

I18N::LangTags: document IGNORE_WIN32_LOCALE

I18N::LangTags: bump version

[MERGE] slim down opslot structure.

When OPs are allocated from a slab (the normal case), what are actually
allocated from the slab are opslot structs, which consist of an OP plus
two pointers.

The branch reduces those two pointers to two U16s, which saves 8 bytes
per op on a 64-bit system.

A further 8 bytes could be saved if those two U16s were included as
extra fields in the OP structure (and the opslot struct disposed of),
but unfortunately too much code does things like Zero(o) or Copy(o1,o2)
which would obliterate the allocation info contained in those two U16s.

Perl_opslab_force_free() adjust loop test

Formerly, slots were allocated within a slab, but leaving the very top
word in the slab as a NULL pointer which appeared as a fake slot so that
a 'while (slot->opslot_next)' loop would stop. Since opslot_next has
been eradicated and the NULL is no longer allocated, the loop condition
for scanning all slots can be simplified slightly (with no change in
functionality).

OPSLOT: replace opslot_next with opslot_size

Currently, each allocated opslot has a pointer to the opslot that was
allocated immediately above it. Replace this with a U16 opslot_size field
giving the size of the opslot. The next opslot can then be found by
adding slot->opslot_size * sizeof(void*) to slot.

This saves space.

struct opslot: document a field better

opslabs: change opslab_first to opslab_free_space

Currently a OPSLAB maintains a pointer to the lowest allocated OPSLOT
within the slab (slots are allocated downwards). Replace this pointer
with a U16 indicating how many pointer-sized words are free below the
lowest allocated slot.

OPSLAB: always have opslab_size field

Currently this struct only has the opslab_size field on debugging
builds. Change it so that this field is always present. This will make
it easier to not need a fake partial OPSLOT at the end of the slab with
a NULL opslot_next field, which will in turn simplify converting
opslot_next into  U16 size field shortly.

make opslot_slab an offset in current slab

Each OPSLOT allocated within an OPSLAB contains a pointer, opslot_slab,
which points back to the first (head) slab of the slab chain (i.e. not
necessarily to the slab which the op is contained in).

This commit changes the pointer to be a 16-bit offset from the start of
the current slab, and adds a pointer at the start of each slab which
points back to the head slab.

The mapping from an op to the head slab is now a two-step process: use
the op's slot's opslot_offset field to find the start of the current
slab, then use that slab's new opslab_head pointer to find the head
slab.

The advantage of this is that it reduces the storage per op.  (It
probably doesn't make any practical difference yet, due to alignment
issues, but that will will be sorted shortly in this branch.)

Perl_Slab_Alloc(): rename 'slab' to 'head_slab'

Rename this local var to better identify that it always points to the
first slab in the slab chain, rather than to the current slab.

autodoc.pl: Forget heuristics, we have a flag

We know if something is a macro because of the 'm' flag.  Don't use the
fallible heuristics to try to determine this.

autodoc.pl: Warn if entry missing d flag but has docs

If you have documention, you're supposed to say so, so that the tools
will work properly.

embed.fnc: Note functions that have documentation

A bunch of functions actually were documented, but this fact was omitted
in embed.fnc.

numeric.c: Fix typo in pod

embed.fnc: Fix typo in comment

pad.h: Parameters to many functions are ptrs

The apidoc entries for 22 functions forgot that the parameters were
pointers to the type.  Since these are macros, this error wasn't caught
until Devel::PPPort tried to generate a test case for them.

autodoc.pl: Fix typo in comment

Fix typo in STORE_LC_NUMERIC_SET_TO_NEEDED_IN docs

[perl #134172] perldelta

[perl #134172] Avoid multiple checks of IN_LC(LC_NUMERIC)

This adds new API macros STORE_LC_NUMERIC_SET_TO_NEEDED_IN and
WITH_LC_NUMERIC_SET_TO_NEEDED_IN that accept a precalculated value
for the hints checks of IN_LC(LC_NUMERIC).

[perl #134172] restrict scope of locale changes during sprintf

In some environments we must hold a mutex for the duration of a temporary
locale change, so we must ensure that mutex is released appropriately.
This means intervening code must not croak, or otherwise bypass the
unlock.

In sv_vcatpvfn_flags(), that requirement was violated when attempting
to avoid multiple temporary locale changes by collapsing them into
a single one. This partially undoes that to fix the problem, while
still attempting to retain some of the benefits by caching the
expensive hints check.

perldelta for 8348ac19a3c3

(perl #134310) don't confuse S_no_op()

This code could previously set PL_bufptr so it points after the
returned s pointer, which S_no_op() asserts on for the test case.

Since setting PL_bufptr here is useful when producing the error
message, just make sure we restore it.

Recommend chdir after chroot more strongly

On Thu, Jul 25, 2019 at 09:35:44AM -0600, Theo de Raadt wrote:
> It is mandatory because otherwise a program-user can arrange for cwd
> to be outside the jail, and utilize that fact to pivot, and in some
> program path utilizations the chroot then becomes not just pointless..
> it is worse than normal, because it creates a dual-namespace view of
> the filesystem, that is a condition that program and libraries are not
> prepared to operate in.

https://marc.info/?l=openbsd-misc&m=156406908805913&w=2

vms/vms.c: fix unused relational comparison result

Closes [perl #134307].

Configure: reuse "cached" value of "none" for dir

When value of a directory (i.e. man1dir) is set to "none" because of:
- '-D' option set on cmdline (i.e. -Dman1dir=none) OR
- because value is reused from previous configure run

And when the prefix is different from the previous configure run(*) then
keep the value of "none". predefined/cached value does not contain the
old prefix so there is no harm in keeping it.

(*): due to another bug this also happens when prefix ends with a
     trailing slash

See https://rt.perl.org/Ticket/Display.html?id=134284

Remove trailing '/' from prefix

Change how/when the trailing '/' from a prefix is removed.

The original code had two problems:

1) it only stripped the trailing slash when value was predefined
   (either from -Dprefix=/foo/bar/ or from a previous configure run)
2) when it did strip the value then it also caused 'oldprefix' to
   be set. This in turns causes other predefined (/cached) paths to
   be ignored.

See https://rt.perl.org/Public/Bug/Display.html?id=134284

embed.fnc: foldEQ has documentation

embed.fnc: Certain fcns don't have Perl_ prefix

This fixes embed.fnc to match the reality.

embed.fnc: Fix wrong comment

embed.fnc: Clarify wording

An 'A' flagged element is accessible everywhere.  I think that is
clearer than the previous 'Available'

Document my_sprintf as deprecated

It turns out that Devel::PPPort needs a documentation entry for this
function in order to not have to special case it.  So, add a brief
explanation as to why its deprecated.

handy.h: Add missing character classification synonym

This should complete the set.

perldelta for 43e5ab2e34fe

I don't think f670c2bb3dd57 needs a perldelta entry.

avoid synthesizing locale names with newlines in them

while debugging some strange behaviour on Win32 I tried dumping
locale names in _trylocale() and saw names go past like:

sv_fi.15
          <-- newline added at the end
sv_fi.15
.UTF-8
sv_fi.15
.65001
sv_fi.15
.ACP
sv_fi.15
.OCP
sv_fi.15
.1252

Win32: set FOO lists values for all variables starting with FOO

I'd been testing setting FOOBAR to Unicode values, so this test was
failing due to the extra variable.

Fix obsolete mention if is_sig in S_pending_ident comment

Commit 49fb862041f1f74fafbc91afa553624c43078261 replaced the parameter
with a pseudo-keyword value KEY_sigvar in PL_in_my, but forgot to
update the comment.

[perl #132474] Add function for parsing sub signatures

This exposes a GRAMSUBSIGNATURE top-level production from perly.y for
toke.c to consume, which parses a subroutine signature, inside the
parens.

This needed a small change to the existing rules, to pull out a rule
that handles all of the insides of the parens but not the parens
themselves.

(h/t to ilmari for the suggestion)

perldelta for e3f4f3212908, 33617017c38f, cfb8d85df6eb

bump $File::Find::VERSION

for the win32 test change

bump $Devel::PPPort::VERSION

for the mktests change

File-Find/t/find.t: reindent after adding a SKIP block

writing to the root directory requires elevated privileges

on modern Win32, and testing might be done without such elevation.

I'll re-indent in the next commit

ensure mktests.PL produces the same output cross-platform

With the change to always generate the tests, this was producing
modified files on Win32, with the path in the header and the line
endings changing.

perldelta for 85d4e0a35b2d

(perl #134291) propagate non-PVs in $@ in bare die()

add perlweb update to RMG

e.g. as done at https://github.com/perlorg/perlweb/pull/289

perldelta for 6b62d97b14ca2894417d0297a435c63e056e31fe

Fix pod formatting

&gt; is an HTML entity, not pod.

Committer: increment $VERSION

For: RT # 134300

Update next major release number in Porting/todo.pod

update 'say' docs to better represent reality

Get dist/Devel-PPPort/t/uv.t passing

Module::CoreList 5.20190720 is now on CPAN

Update Module::CoreList for 5.31.3

Bump version to 5.31.3

Create new perldelta for 5.31.3

Tick off 5.31.2

Add epigraph for 5.31.2

5.31.2 today

Finalize perldelta

Update Module::CoreList for 5.31.2

perldelta tidy-ups

VC6 support is likely to be removed soon

perldelta - Document a couple more changes

No need to list CUSTOMIZED files for anything but upstream=>CPAN modules

The vast majority of upstream=>BLEAD modules having modifications over the
given CPAN release aren't listed in Maintainers.pl. Net::Ping was the only
anomaly, and was out of date too -- t/001_new.t and t/010_pingecho.t are
also modified but weren't listed.

Revert the Module::CoreList part of 7872aa47f6

5.31.1 contained B::Op_private 5.031001 and Module::CoreList* 5.20190620

Revert the Module::CoreList part of e01077d94e

5.15.0 contained CGI::Util 3.53
5.21.7 contained File::Spec 3.53
5.31.1 contained Devel::PPPort 3.53

perldelta - Remove boilerplate stuff

perldelta - Documentation and Diagnostics

perldelta - Modules and Pragmata

Document known problems on Windows

Fix upgrade notes in INSTALL

Each BLEAD release is potentially not binary compatible with the previous
release.

RMG - Fix a couple of typos

Config::Perl::V is ahead of CPAN

Correct an epigraph link

Fill in missing epigraphs links

(perl #134221) MSVC doesn't define O_ACCMODE

perldelta for ae73d7ec2329, 0424723402ef, 74b421cc877e

(perl #134221) preserve O_APPEND for open ... undef calls.

For most systems this means switching from mkstemp() to mkostemp(),
which Zefram helpfully added an emulation for.

For Win32 the append seems to happen in the POSIX emulation, not in
WriteFile(), so we can simply supply the flags to _open_osfhandle().

After some discussion with Craig Berry, the VMS version of the code
now uses largely the same mkostemp() code, with some special code in
the mkostemp() emulation to allow marking the file to be deleted
automatically when closed.

(perl #134221) support O_APPEND for open ..., undef on VMS

VMS doesn't allow you to delete an open file like POSIXish systems
do, but you can mark a file to be deleted once it's closed, but
only when you open it.

Since VMS doesn't (yet) have mkostemp() we can add our own flag to
our mkostemp() emulation to pass the necessary magic to open() call
to delete the file on close.

(perl #134221) support append mode temp files on Win32 too

(perl #134221) support append mode for open .. undef

Update IPC-Cmd to CPAN version 1.04

  [DELTA]

1.04 Sat Jul 13 10:08:08 BST 2019

  Bug fixes:
  - Fixed usage of setsid()

Support Bison version 3.1 through 3.4

No significant changes in the generated code since 3.0.