update perldelta for 5.37.6

Update Module::CoreList for 5.37.6

locale.c: Don't Safefree temps

Commit 22392ba2f8a changed the querylocale_foo macros to return a
mortalized copy instead of having to free things explicitly.  This was
done because the various code paths are complicated, and I was noticing
various leaks that would require a bunch of #ifdef's to fix.

That commit meant that all the existing free's of the results needed to
be removed.  This one was overlooked at the time.  Reported in
<f47046d6-7e35-301d-10f-9160b34ca2b6@uwe-dueffert.de>, not in github

Devel::PPPort/module3: eliminate the always true warning for the return

In this case we're testing that the result is always true, and it
happens the compiler realizes that the return value is always true,
even in isolation.

Returning the variable that we've already set to point to
PL_bufptr eliminates the warning.

The compiler is still smart enough to eliminate the comparison
on optimized builds.  Some linter may still complain about it.

GvNAME() always returns a true value

gcc 12 was complaining that GvNAME() always returns a true value,
and that's also true.

Omit such uses in conditions.

change HvENAME_HEK() to HvENAME_HEK_NN() where NULLs would crash anyway

gcc 12 was producing a confusing message complaining that
references to with hek_key[] were beyond the end of the array,
even though a properly HEK has bytes beyond the first we declare.

From experimentation I theorize the confusing message was produced
because HvENAME_HEK() can return a NULL pointer, and the array
pointed to by any NULL pointer is zero length, producing the
array bounds warning we were seeing.

Fixed by changing each hv_(fetch|delete)hek() call to use the
HvENAME_HEK_NN() macro variant, which doesn't include an explicit
NULL return value.

mro_method_changed_in() was a bit special, it evaluated the
hv_fetchhek() before the check for an anonymous stash, so I reordered
the code to take advantage of C99, checking the assertions before we
dereference the stash pointer, checking we have a name before trying
to look it up.

only fully calculate the stash (effective) name where needed

gcc 12 was complaining that evaluating (somehekptr)->hek_key
was always true in many places where HvNAME() or HvENAME() was
being called in boolean context.

Add new macros to check whether the names should be available and
use those instead.

clarify that every tag is required to be annotated
- while the instructions were already clear at this point, accidents with lightweight (unannotated) tags have happened

relax the assertion in SSPTR() and SSPTRt()

I added assertions in SSPTR() and SSPTRt() to try to ensure the new
SSize_t result from SSNEW*() wouldn't be truncated, but this
was too strict, requiring that the size matched exactly.

Relax this to allow the input offset to be the same or larger than
SSize_t.

Fixes #20504.

Skip one test under miniperl

threads module not yet available.

For: # https://github.com/Perl/perl5/issues/20524

dist/Tie-File/t/29a_upcopy.t: double timeout to 10

Some of these tests (which involve moving blocks of data around in a
file) are occasionally randomly timing out on some slow smokers.
Double the timeout from 5s to 10s and see if the problem goes away.

dist/Tie-File/t/29a_upcopy.t add test descriptions

Getopt::Long: synch with CPAN version 2.54

From Changes:

* Fix compatibility with Perl 5.8.

Getopt::Long: sync with CPAN version 2.53

From Changes:

* Improve parsing of float numbers.
  https://rt.cpan.org/Ticket/Display.html?id=133216

* Fix Version/HelpMessage -message argument.
  https://rt.cpan.org/Ticket/Display.html?id=133963

* Added 'starter' method to the callback object. This method returns the
  starter (e.g. '--' or '-') of the option as used by the user.

* Fix problem with Pod::Usage argument in examples/skel2.pl.

* Enhanced option:default spec to octal, hex and binary.

Fix & test useless-anonymous-subroutine warnings.

Issue #20384: This fixes a breakage from
35458d36d3c9fcf47064e4379e15276057b0bca5 that Devel::WatchVars’s test suite
caught. A test is added to Perl’s own test suite to prevent regressions.

Fail attempts to pass anonsub to \$ prototype.

Issue #20384

ParseXS - version bump to 3.48 and test perlxs.pod has the right version

The version in the pod has been long wrong. We are on 3.48 now, it was 3.13_01.

ParseXS - make testing easier

Use warn instead of print STDERR, and provide a way to make
errors trigger a die instead of an exit(1).

Currently the module code is written as though the only way
it will be used is via the xsubpp script, so the library does
annoying things like calling exit() instead of die() to signal
an exception. It also uses print STDERR instead of warn,
which means the test code can't just use a $SIG{__WARN__} hook
to see its warnings, and instead has to include PrimitiveCapture
in the t directory. These two things combine annoyingly in our
test code such that when you break the module you can see tests
exiting early, but with no useful diagnostics as to why.

This patch reworks this to use "warn" instead of print STDERR,
and to provide a way to enable the use of "die" instead of exit.
Thus making debugging failing tests far easier.

ParseXS - add support for elifdef and elifndef

The upcoming C++23 and C23 standards add #elifdef, #elifndef.

ParseXS - allow symbolic alias of default function

Also normalize warnings. It used to be if you created an alias
of the root function (0) no warning would be produced. Now
we will produce a warning, but we also allow symbolic references
to defuse the warning.

ParseXS - better support for duplicate ALIASes

Sometimes you *want* to create multiple names for the same
functionality, but doing so with the ALIAS functionality requires
awkward workarounds. This adds a new "symbolic alias" that does
not warn on dupes as creating a dupe is its whole point. For a
symbolic alias the value is the name of an existing alias.

This also cleans up some of the warnings related to aliases so
we distinguish between when a duplicate is truly ignored or
where it overrides a previous value. And deal with a few other
edge cases properly.

ParseXS - handle #else and #endif without blank line prefixes

This patch makes it possible to omit some of the whitespace around
preprocessor directives. It teaches fetch_para() to understand
that a #else or #endif directive that does not end a #if that
was seen in the current "paragraph" should not be parsed as part
of that paragraph. This means that a conditional block that defines
the same sub under different define conditions need not have extra
whitespace after each sub definition.

ParseXS - sort things that might produce output

Make sure our output is deterministic.

configure.gnu forwards extra arguments to Configure

except when it doesn't (the -- arguments)

This came up during discussion of #20502, but is not a fix for it.

Whitespace alignment fix in regen/feature.pl

Update AUTHORS and .mailmap with my (Mattia Barbon) current address

SvPVutf8_nomg - assign string length to the len parameter

This is consistent with other SvPV* variants that take a len parameter.

utf8::upgrade: Don't coerce undef arg

This fixes GH #20419

Figure out I32df, U32uf, etc. in Configure rather than in perl.h

These macros were defined in perl.h using preprocessor conditionals,
but determining wheter I32 is "int" or "long" is pretty hard with
preprocessor, when INTSIZE == LONGSIZE.  The Configure script
should know exact underlying type of I32, so it should be able to
determine whether %d or %ld shall be used to format I32 value
more robustly.

Various pre-configured files, such as uconfig.h, are updated to
align with this.

Perl doesn't appear to be included in the ibb any more

The "ibb-perl" link prompts for a login, though other links like
"https://hackerone.com/perl" load without a login.

Perl also isn't listed under the ibb project itself at
"https://hackerone.com/ibb?type=team"

Tie-File: report test timeouts to STDERR

Currently some tests have a 5 second timeout. If this is exceeded,
the test number is output as "not ok", along with "# Timeout" to
STDOUT. Change the latter to output to STDERR along with the test
filename and test number, for easer debugging of smoke logs.

(Really these test files need upgrading to use Test::* but that's a job
for another day and another person....)

skip testing .github/README.md if doesn't appear to be a README

git on windows at least of 2.37.1 does not checkout symbolic links
as symbolic links, leaving them as files containing the filename
linked to.

This meant this test against .github/README.md failed, since it only
contained "../README": nothing that appears to be a copyright message

I considered testing that .github/README.md was a symbolic link
but 8975221916 suggests that it may eventually become more distinct
from the root README file, so instead check its size.

regcomp.c - add a PARNO() macro to wrap the ARG() macro

We used the ARG() macro to access the parno data for the OPEN
and CLOSE regops. This made it difficult to find what needed to
change when the type and size or location of this data in the
node was modified. Replacing this access with a specific macro
makes the code more legible and future proof.

This was actually backported from finding everything that broke
by changing the regnode type for OPEN and CLOSE to 2L and moving
the paren parameter to the 2L slot. We might do something like this
in the future and separating the PARNO() macros from their
implementation will make it easier.

regcomp.c - correct comment

Devel-PPPort: bump version

Devel-PPPort: fix STMT_START and STMT_END to not warn on clang

Since 7169efc77525df70484a824bff4ceebd1fafc760, perl's core STMT_START
and STMT_END macros no longer try to use brace groups, due to the
warnings they can generate and their very limited usefulness.

That commit also changed Devel::PPPort to remove the use of brace groups
in STMT_START/STMT_END. That led to errors in older perls, so it was
partly reverted in e08ee3cb66f362c4901846a46014cfdfcd60326c. Since then,
various other macros have been improved to properly work with brace
groups enabled or disabled.

We can now remove the brace group using variant of STMT_START/STMT_END,
which will silence the warnings from clang.

Devel-PPPort: use inline function for croak_sv without brace groups

The macro for croak_sv when brace groups are not enabled uses
STMT_START/STMT_END. This works when used as a statement, but would
break if used as part of an expression.

To resolve this, change that macro to use a function. As a static inline
function with a namespaced name, it shouldn't cause problems with
namespace pollution.

pod/perlhacks.pod - document TEST_ARGS and make win32 perl use it too

Win32 has had TEST_SWITCHES and TEST_FILES for ages. Unix has
TEST_ARGS. This extends Win32 to support TEST_ARGS as well. I
havent made the contrary change to the unix code, anyone using
TEST_SWITCHES is already using it on Win32.

ParseXS - fix todo tests to match the correct filename

The original regex was off and showed this test as passing when
it was actually failing, which is what we expect and why the test
is marked as TODO. The test accidentally doubled the extension
expected in the filename, and this patch corrects that mistake.

perldelta for 545d499045, 42d0708bf6, a1aa0ce9693, 4f8b3850b207

bump $Storable::VERSION

make store_hook() handle regular expression objects

Previously this would complain it didn't know the object type
when preparing to call STORABLE_freeze.

improve error reporting by store_hook() on an unknown type

This produced an opaque message when it was asked to freeze an
object of an unsupported type.  Changes:

- replace the opaque internal object type number with the typical perl name of the
  type ("GLOB" instead of "8")
- include the class of the object being frozen to identify which class needs work
- include name of the function we're trying to call to do the freeze,
  if possible

allow porting/corelist.t to be run from root

Test case where different functions are declared

In the first sample .xs file supplied for this branch, a function named
'dbh' was declared twice, once inside an 'ifdef', once outside.  This
led me to think that that dual use of the string 'dbh' was the problem
EUPXS was stumbling upon.

However, that appears to be incorrect.  If we supply a second sample .xs
file in which we use two different function names, we still get the
undesired warning.

Add TODO-ed test for duplicate warning

Add test file contributed in GH 19661

buildtoc: Add error check

If something goes wrong, this wouldn't have known it

Update Data::Dumper synopsis

Add `my` to synopsis.

Committer: Data-Dumper: Update $VERSION in POD.  The release date no
longer appears in the POD, so let's remove the comment admonishing us to
update it.

For: https://github.com/Perl/perl5/pull/20224

regen/regcomp.pl - add more detail on what the script does and list its inputs properly

Previously we were only showing one of the inputs that generates
regnodes.h, now we show all of them. Also add some paragraphs explaining
a bit more detail what the files contain and how we process.

Committer: Small grammatical correction, per
https://github.com/Perl/perl5/pull/20288#discussion_r968344745

Generate opcode.h, opnames.h

Refactor opcode.pl - with c99 array init index comments are not necessary

Refactor opcode.pl - use c99 array initialization (using op enum)

Refactor opcode.pl - use INIT() macro instead of #ifdef/#endif

Refactor opcode.pl - let INIT() macro to accept expressions with comma(s)

Refactor opcode.pl - B/Op_private.pm - move generator code to function

Refactor opcode.pl - pp_proto.h - move generator code to function

Refactor opcode.pl - opnames.h - unglobalize output handle

Refactor opcode.pl - opnames.h - move epilogue into function

Refactor opcode.pl - opnames.h - move opcode predicates into function

Refactor opcode.pl - opnames.h - move enum generator into function

Refactor opcode.pl - opcode.h - unglobalize output handle

Refactor opcode.pl - opcode.h - move PL_opargs into function

Refactor opcode.pl - opcode.h - move PL_check into function

Refactor opcode.pl - opcode.h - move PL_ppaddr into function

Refactor opcode.pl - opcode.h - move opnames into function

Refactor opcode.pl - opcode.h - move defines into function

Refactor opcode.pl - opcode.h - move prologue into function

Refactor opcode.pl - required little bit newer perl

This is perl5 developer tool, it's safe to expect contributors have
something newer at their disposal.

Refactor opcode.pl - untabify

Moved unused variable into debug scope

Moved declaration of object p.

For: Issue: https://github.com/Perl/perl5/issues/20485
     p.r.:  https://github.com/Perl/perl5/pull/20486

regcomp.c - remove always true condition

RX_PRECOMP() has for some time been an expression that
will always be true, thus there is not need to test it.

GitHub Actions: switch from ::set-output to $GITHUB_OUTPUT

The ::set-output command has been deprecated and replaced with the
$GITHUB_OUTPUT environment file. Convert the workflows to use the newer
output mechanism.

Reference: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

scope.* - revert and rework SAVECOPWARNINGS change

We can't put PL_compiling or PL_curcop on the save stack as we don't
have a way to ensure they cross threads properly. This showed up as a
win32 t/op/fork.t failure in the thread based fork emulation layer.

This adds a new save type SAVEt_CURCOP_WARNINGS and macro
SAVECURCOPWARNINGS() to complement SAVEt_COMPILER_WARNINGS and
SAVECOMPILEWARNINGS(). By simply hard coding where the pointers should
be restored to we side step the issue of which thread we are in.

Thanks to Graham Knop for help identifying that one of my commits was
responsible.

Replaced recppush/regcppop with memcpy

Basically tested with Intel VTune to be increasing the performance of
a perl regex matching program with multiple capture groups and
recursive patterns

[With minor whitespace edits by Yves.]

perldelta for 1a2b24d857d

change the return value of SSNEW to SSize_t

The normal savestack index is an I32, but that counts in ANY
(which are typically the larger of pointer or IV sizes), this
meant is the save stack was large, but still nowhere need it's
limit, the result of SSNEW() could overflow.

So make the result SSize_t and adjust SSPTR() to match.

SSPTR() asserts to ensure the supplied type is the same size as
SSize_t to ensure callers are updated to handle the new limit.

op.c: Add a cast to silence -Wsign-compare warning

In 32-bit build (where IV is not wider than U32), comparing
PL_eval_begin_nest_depth and max_nest_iv would generate a warning:

    op.c:10817:42: warning: comparison of integer expressions of different signedness: ‘U32’ {aka ‘long unsigned int’} and ‘IV’ {aka ‘long int’} [-Wsign-compare]
    10817 |             if (PL_eval_begin_nest_depth >= max_nest_iv) {
          |                                          ^~

re/fold_grind.pl - rework so we don't double compile patterns

We were calling utf8::upgrade() on qr// objects, which works, in
that it stringifies the pattern, and then upgrades the string, and
the string can be used to match against. But it double compiles the
pattern, which is a bit unnecessary. For example

    $pat= qr/\x{DF}/;
    utf8::upgrade($pat); # $pat is now a string!
    "ss"=~/$pat/;        # and now $pat gets compiled again

works, but it does twice the work needed.

This was found by making utf8::upgrade() ignore refs.

cop.h - get rid of the STRLEN* stuff from cop_warnings

With RCPV strings we can use the RCPV_LEN() macro, and
make this logic a little less weird.

warnings.h - length guard cop_warnings bitvector checks

scope.* - more flexible ways to save warning bits

perldelta for 01052a1d77a5, a47628b617d1

make win32_readlink() return PrintName instead of SubstituteName

While debugging socket stat()ing I noticed that sometimes the
name returned by win32_readlink() was a full pathname rather than
the name that the link was created as.

Changing this to use the PrintName values changed win32_readlink()
to return the create as name, which seems closer to the POSIX
readlink.

Win32 stat() didn't handle AF_UNIX socket files

Unfortunately both symbolic links and sockets can only be
"statted" by opening with FILE_FLAG_OPEN_REPARSE_POINT which
obviously doesn't follow symbolic links.

So to find if a chain of symbolic links points to a socket,
is a broken chain, or loops, we need to follow the chain
ourselves.

duplicate -F switches would overwrite PL_splitstr without freeing

Of course, it wasn't safe to free it before the previous patch,
but it is now.

We freed PL_splitstr in each thread, allocate it in each thread too

I also fixed const correctness for PL_splitstr, it was initialized
to " " which required it to be const, but the value is only looked
at when PL_minus_F is true, so it wasn't necessary.

So initialize it to NULL, and duplicate the value in threads.

t/comp/require.t - add tests for module_true being off

Or the file being empty, or various other cases. We missed
a case in the earlier work on module_true.

This also includes some tweaks to make the tests pass under strict
if needed.

t/comp/require.t - add a way to disable the sleep 20

it makes developing the test script quite annoying.

t/comp/require.t - test module_true with requiring empty files

pp_ctl.c - teach module_true how to deal with blocked requires

If require encounters an @INC hook that blocks the require it
ends up with a STUB node as the optree for the LEAVEEVAL. This was
causing the module_true logic to segfault. Guarding against the
OP_STUB node fixes the problem.

This affected the tests for IO::Socket::SSL. Thanks to Graham
Knopp for the reduced case, and James Keenan for reporting it
in https://github.com/Perl/perl5/issues/20468.

This should fix the problem.

locale.c - add comments about PERL_DESTRUCT_LEVEL avoiding "leaks"

Some of the locale data is deliberately not destroyed during shutdown.
Setting PERL_DESTRUCT_LEVEL=2 causes them to be cleaned up. If someone
ends up debugging under the address sanitizer and ends up at this logic
then the comments will help them avoid some frustration.

Also add a STRLEN member to ANY

Add Size_t and SSize_t members to ANY

Convert tabs to whitespaces in  union any {}  definition

op.c - use refcounted pv pointers for cop_warnings

this allows multiple ops to share the same underlying
warning bit vector. the vectors are not deduped, however
they are efficiently copied and shared.

regen/scope_types.pl - add tool to manage the scope type defines

In scope.c and scope.h there are various defines that must be coordinated
with the contents of an array, and comments and ids which might have to be
mass changed manually in some circumstances. As this is error prone this
patch adds a new regen script to ensure it is kept in sync and to make the
process of adding new types trivial. The data that drives the script is
kept in the scripts __DATA__ section.

scope_types.h - new header file for scope type data

Next patch this data will be autogenerated.

cop.h - add support for refcounted filenames in cops under threads

We have a weird bifurcation of the cop logic around threads. With
threads we use a char * cop_file member, without it we use a GV * and
replace cop_file with cop_filegv.

The GV * code refcounts filenames and more or less efficiently shares
the filename amongst many opcodes. However under threads we were
simplify copying the filenames into each opcode. This is because in
theory opcodes created in one thread can be destroyed in another. I say
in theory because as far as I know the core code does not actually do
this. But we have tests that you can construct a perl, clone it, and
then destroy the original, and have the copy work just fine, this means
that opcodes constructed in the main thread will be destroyed in the
cloned thread. This in turn means that you can't put SV derived
structures into the op-tree under threads. Which is why we can not use
the GV * stategy under threads.

As such this code adds a new struct/type RCPV, which is a refcounted
string using shared memory. This is implemented in such a way that code
that previously used a char * can continue to do so, as the refcounting
data is located a specific offset before the char * pointer itself.
This also allows the len data to embedded "into" the PV, which allows
us to expose macros to acces the length of what is in theory a null
terminated string.

    struct rcpv {
        UV      refcount;
        STRLEN  len;
        char    pv[1];
    };
    typedef struct rcpv RCPV;

The struct is sized appropriately on creation in rcpv_new() so that the
pv member contains the full string plus a null byte. It then returns a
pointer to the pv member of the struct. Thus the refcount and length and
embedded at a predictable offset in front of the char *, which means we
do not have to change any types for members using this.

We provide three operations: rcpv_new(), rcpv_copy() and rcpv_free(),
which roughly correspond with newSVpv(), SvREFCNT_inc(), SvREFCNT_dec(),
and a handful of macros as well. We also expose SAVERCPVFREE which is
similar to SAVEGENERICSV but operates on pv's constructed with
rcpv_new().

Currently I have not restricted use of this logic to threaded perls. We
simply do not use it in unthreaded perls, but I see no reason we
couldn't normalize the code to use this in both cases, except possibly
that actually the GV case is more efficient.

Note that rcpv_new() does NOT use a hash table to dedup strings. Two
calls to rcpv_new() with the same arguments will produce two distinct
pointers with their own refcount data.

Refcounting the cop_file data was Tony Cook's idea.

mg.c/perl.c/scope.c - fixup mangled indentation and whitespace

Various code related to set_and_free_cop_warnings was terribly mangled
as far as whitespace goes. This patch cleans it up so it is readable and
correctly indented. Whitespace only changes.