Change handy.h macro names to be C standard conformant

C reserves symbols beginning with underscores for its own use.  This
commit moves the underscore so it is trailing, which is legal.  The
symbols changed here are most of the ones in handy.h that have few uses
outside it.

Minor update to perl -h to display newly added -g option.

regcomp.c: Add ASSERT

Mohammad S Anwar is now a Perl author

Note gotchas, workarounds in STMT_START..STMT_END

Discourage use of GCC brace groups

These days, static inline functions are the better bet, as you don't
have to maintain two code paths.

Make fc(), qr//i thread-safe on participating platforms

A long standing bug in Perl that has gone undetected is that the array
is global that is created when changing locales and tells fc() and qr//i
matching what the folds are in the new locale.

What this means is that any program only has one set of fold definitions
that apply to all threads within it, even if we claim that the locales
are thread-safe on the given platform.  One possibility for this going
undetected so long is that no one is using locales on multi-threaded
systems much.  Another possibility is that modern UTF-8 locales have the
same set of folds as any other one.

It is a simple matter to make the fold array per-thread instead of
per-process, and that solves the problem transparently to other code.

I discovered this stress-testing locale handling under threads.  That
test will be added in a future commit.

In order to keep from having a dTHX inside foldEQ_locale, it has to have
a pTHX_ parameter.  This means that the other functions that function
pointer variables get assigned to point to have to have an identical
signature, which means adding pTHX_ to functions that don't require it.
The bodies of all these are known to the compiler, since they are all
in inline.h or in the same .c file as where they are called.  Hence the
compiler can optimize out the unused parameter.

Two calls of STR_WITH_LEN also have to be changed because of C
preprocessor limitations; perhaps there is another way to do it that I'm
unfamiliar with.

mg.c: Fix to compile under g++

Commit 00a5df846e035280750985222a693ac58022ee36 fixed some c undefined
behavior, but it will not compile with g++, because a goto crosses
initialization.

https://stackoverflow.com/questions/14274225/statement-goto-can-not-cross-variable-definition

and its links sort of explains why

This commit works around this limitation

Restore wrongly deleted pod for sv_pvutf8

Remove pod for non-existent functions

7008caa915ad99e650acf2aea40612b5e48b7ba2 removed several deprecated
functions, but did not remove all the pods thereof.

toke.c: Remove Undefined C behavior

Spotted by clang 14.

mg.c: Remove Undedfined C behavior

Spotted by clang-14.

op.c: Improve comments

Remove support for Ultrix

Ultrix has been removed.  Ultrix was the native Unix-like operating
system for various Digital Equipment Corporation machines.  Its final
release was in 1995.

perlapi: Document hv_name_set

SvGETMAGIC: evaluate its argument just once

This required making it into an inline function.  I tried using
STMT_START{ ... } STMT_END, which should work since it has a void
return, but there were places where it was used in a comma operator, and
those did not compile.

Capitalize Perl

Capitalize Perl when referring to the language (rather than the interpreter).

Also pluralize "built-ins" and format "CORE" package.

POSIX.xs: White-space only

Properly indent some nested preprocessor directives

POSIX.xs: Use macro to reduce complexity

This #defines a macro and uses it to populate a structure, so that
strings don't have to be typed twice.

regexec.c: Handle Turkish locale if large ANYOF bitmap

Perl defaults to the bitmap for ANYOF nodes being for the lowest 256
characters, but it is possible to compile the bitmap to be up to size
2**16.  Doing so, prior to this commit, broke Turkish locale handling.

regexec.c: Avoid using a more general fcn

The regnodes in the ANYOFH series by definition don't have a bitmap used
in the other ANYOF-type nodes.  Instead they have an inversion list.  We
can avoid the overhead of calling the general function that looks first
in the bitmap.

locale.c: Replace most #ifdef DEBUGGING lines

THe previous commit enhanced the DEBUG macros so that they contain the
logic that previously had to be done with conditional compilation
statements.  Removing them makes the code easier to read.

DEBUG_L now also looks at environment variable

Because locale initialization happens before command line processing,
one can't pass a -DL argument to enable debugging of locale
initialization.  Instead, an environment variable is read then, and is
used to enable debugging or not.  In the past, code specifically had to
test for this being set.  This commit changes that so that debugging can
automatically be enabled without having to write special code.  Future
commits will strip out those special checks.

XSLoader: convert from Test::More to internal test helpers

The XSLoader tests need to play with the XS bits of various modules,
which can interfere with testing modules like Test::More. For example,
Test::More now loads Time::HiRes. This results in redefinition warnings,
and could lead to more serious problems.

Avoid this by creating some test helpers inside the XSLoader test, and
using those rather than Test::More.

The helpers implemented include roughly the same features used by the
test itself, so that the impact on the rest of the test code is minimal.

Fix building with clang-cl

As mentioned in https://lists.llvm.org/pipermail/llvm-dev/2015-July/088122.html
and https://github.com/llvm/llvm-project/issues/24625 building with clang-cl currently fails due to the declaration of __PL_nan_u.
By declaring it like this, cl and clang-cl are happy to parse it.

perldelta for ea43bc00e1113f

minitest_prep conflict on building $(MINIPERL_EXE)

minitest depends on $(MINIPERL_EXE) and minitest_prep, but
minitest_prep calls back into make to build lib/Config.pm which
indirectly depends on $(MINIPERL_EXE).

This can result in a race between the parallel work of the parent
make and the work of the child make, as they both try to build
$(MINIPERL_EXE) and its dependencies, causing some of the errors
described in the ticket.

Note that trying to `make -j6 minitest all` could lead to
similar conflicts, but I think that's an unlikely and not worth
supporting anyway.

Fixes #19829 (I think)

perldelta for bbaab2c3cd9

use Off_t for file offsets in sdbm.c

On Win32 long is only 32-bits (even for x86_64), which meant
file sizes were limited to 2GB.

This successfully runs the example code, and can successfully read
all the keys back.

I didn't add a test, since creating a 2GB (or 8GB for the issue test)
would be unfriendly.

handy.h: WIDEST_UTYPE is just PERL_UINTMAX_T

No need to re-derive it

perl5dp.pl: Bump version

Data::Dumper: bump version

Elvin Aslanov is now a Perl author

Fix typos in tim64.c comments

Dumper.pm: Add proper link to pod section

perl5db.pl: Make hyperlink in pod

inline.h: White space only

Add Hobbit quote to time64.c

Make use of av_push_simple in a few places.

Specifically, only where a new AV has just been created and it's therefore
obvious that av_push_simple is safe to use.

In many cases, nearby code does av_push to an existing array. It might
well be the case that these are simple arrays and av_push_simple could be
used instead. The necessary analysis has been left for the future.

Move av_new_alloc from av.c to inline.h

There was less benefit in doing this prior to newSV_type becoming an inline
function. Now that it is, inlining av_new_alloc can help compilers eliminate
redundant setting of AvMAX/AvFILLp and unnecessary branches of any following
inlined calls to av_store_simple.

inline.h: add av_push_simple

A simplified version of av_push, along the lines of av_store_simple and
av_fetch_simple. This function is trivial, but having it makes refactoring
of existing code easier to read.

For example, changing:
    av_push(some_av, val);
to:
    av_push_simple(some_av,val);

is easier to read than:
    av_store_simple(some_av, AvFILLp(some_av) + 1, val)

Appease gcc's warning that keysv may be used uninitialized in op.c

perlapi: Deprecate 5 functions

and document two of them, which I had done before I realized they were
to be deprecated.

These functions are obsolete, and stemmed from a long-solved problem in
Windows.  There are a couple of uses on cpan of a couple of them.  But
people should just use PL_opnames[], for example, instead of the
function here that returns the address of that array.

Opcode.xs now uses PL_op_name and PL_op_desc directly

This removes the last core use of the trivial accessor functions
get_op_names() and get_op_descs().

opset_len in Opcode.xs is a constant, so should be a static const

We can also remove many dMY_CXT declarations, as they are no longer needed
(and generate a now unused variable in threaded builds, hence compiler
warnings).

Previously it was part of the module's my_cxt_t, because it was a value
calculated from the interpreter variable PL_maxo. But PL_maxo itself is *not*
a variable - it was converted to a #define in Aug 2016 by commit
8d89205aa6324e7d:

    Remove PL_maxo

    We have an interpreter variable using memory, PL_maxo, which is
    defined to be the same as MAXO, a #defined constant.  As far as I can
    tell, it is never used in lvalue context, in core or on CPAN, except
    for the initialisation in intrpvar.h.

    It can simply be removed and replaced with a macro defined as equiva-
    lent to MAXO.

    It was added in this commit:

    commit 84ea024ac9cdf20f21223e686dddea82d5eceb4f
    Author: Perl 5 Porters <perl5-porters.nicoh.com>
    Date:   Tue Jan 2 23:21:55 1996 +0000

        perl 5.002beta1h patch: perl.h

        5.002beta1 attempted some memory optimizations, but unfortunately
        they can result in a memory leak problem.  This can be
        avoided by #define STRANGE_MALLOC.  I do that here until
        consensus is reached on a better strategy for handling the
        memory optimizations.

        Include maxo for the maximum number of operations (needed
        for the Safe extension).

    But apparently it is not needed for the Safe extension (tests pass
    without it).

What the author of that commit didn't realise was that Opcode had been split
out from Safe - the code in question is in this module not Safe.

ExtUtils::CBuilder - bump version

ExtUtils::CBuilder - print out commands being run in usable form

When not set to be quiet, the system commands being run are printed out
before running them. This can assist with diagnosing issues. It is
implied that this is a command that could be run by a user.

Since the command is internally run with system using list form, it does not
involve the shell, and does not need any extra quoting. The printed out
command however, would be run by a user through a shell. Just joining
the arguments with spaces will not quote them correctly for use through
a shell.

Add a new method for quoting a string for the shell. And use this method
when printing out the command line that we are running.

Module::CoreList - change github clone URL to https

GitHub no longer supports git:// cloning because it is not encrypted.

pp_unshift: av_store is often unnecessary

feature - document that use utf8 should probably be used for non-ascii delimiters

Correctly indent the body of the while{} loop in S_aassign_scan()

Also join the 'else if'

Fix indentation of a line of code in op.c to not be misleading

Rename variable in EXTEND_HWM_SET

"ix" is a common variable name in XS code, and may be passed in as
part of a parameter.

Refactor warning-generating code in two different places into one common function

The new combined function, named `S_warn_elem_scalar_context()`, is much
more meaningful and useful to become exported in an intended subsequent
change, than the previous (poorly-named) `S_op_pretty()` would have
been.

Update checksums in some generated files

These use checksums to see if the generated data could be out of date.
The new NormTest.pl wasn't counted in this, and needn't be, but
excluding it and other similar ones is more trouble than it's worth, so
make a comment to that effect and update to include the NormTest.pl
digest value.

locale.c: Declare three static arrays to be so.

perlapi: Document my_failure_exit()

Add official Unicode normalization tests

opt.t: Fix count of tests to be skipped

Comprehensive charset.t tests

These have been occasionally useful, but take quite a bit longer than
just the sameples normally used.  Enabled by environment variable
$PERL_DEBUG_FULL_TEST

Make tr/// SV compiled components ReadOnly

This could trigger some optimisations, and makes it clear to maintainers
that they do not get modified.

Remove deprecated functions

Most of these have been deprecated for a long time; and we've never
bothered to follow through in removing them.  This commit does that.

utf8.c: Minor comment changes

Add comments to taint_env, taint_proper

One is supposed to access this functionality only through the documented
uppercase-named macros.

perldelta for d6a9a133f0ec, 34e6e2fba776

remove some leftover debug output from filetime_from_time()

bump $GDBM_File::VERSION

GDBM_File: fix fetching mmapsize

Originally this came up from a Coverity complaint.

GDBM_SETMAXMAPSIZE accepts any of unsigned, unsigned long or size_t,
but GDBM_GETMAXMAPSIZE only accepts size_t.  Since this is the only
case that uses unsigned values we can safely switch it to size_t.

Unfortunately Coverity's analysis was pretty broken, it complained
about c_uv being uninitialised at the call to newSVuv, but its example
code flow went through the opt_dbname case, which sets vptr to != &c_uv
so the newSVuv() wouldn't execute anyway.

Before looking closely at the Coverity analysis and after finding the
bug fixed here I thought for a moment that Coverity had been tracing
into libgdbm, which could have caught the actual problem, but alas
that was not the case.

I expect this fix will not close CID 351943, and if it remains after
this is applied I'll close it as a false positive.

CID 351943.

perlapi: Consolidate gv_fetchmeth forms, improve doc

I read the code and added details previously missing.

regex: Create a macro to avoid some #ifdef's

Easier to read

regexec.c: Use modern name for function

The newer name is not misleading; the old name has been retained only
for backcompat; this removes its final use in the core.

regcomp,regexec: Shorten #define name

This names a flag bit whose meaning depends on context.  The previous
name contained both meanings, and was simply too long to be
comprehensible.  This commit splits it into two names, with the suffix
'_shared'.  It's not necessary to know precisely what the other meaning
is when reading the code, only that it has another meaning.

Rename internal function

The old name had a negative in it, and its easier to grok positives

regexec.c: Remove obsolete code

The results of this are no longer used

regcomp.h: Add, fix, clarify comments

regcomp.c: Move loop declaration closer to use

Now that we have c99, the loop variable can be declared in it, instead
of way before.

utf8.h: Add unsigned cast to array index parameter

this makes sure it is never negative

regcomp.c: Use mnemonic instead of what it evaluates to

The memonic was used in some places; its value in others.  This commit
brings uniformity.

perlapi: hv_stores and do_close are now documented

Update email for Mark Jason Dominus

perlapi: Document do_close

remove irrelevant mention of scalar references

Scalar references are not mentioned anywhere else in the document,
so they don't need to be mentioned here

perl5360delta: remove reduplicated "please"

We do not wish to beg...

perl5360delta: re-add a truncated line

I assume this was lost during editing.  Unhappily, this error did make
it into the release.

Regen perldebgutS

perldiag: Fix typo, collapse some msgs

Make STRLENs() available to core

This may cause problems when not used correctly; so continue to restrict
it.

regcomp.sym: Comment why no ANYOFRs node exists

t/op/vec.t: use strict, warnings

perlretut: Grammar, clarifications, white-space

handy.h: ASSERT_NOT_PTR doesn't work for void*

Add comment to that

perlapi: Document save_pushptr

perlapi: Consolidate hv_store(s?), into 1 entry,improve

The text has been expanded to discuss what happens if the any of the
pointer parameters are NULL.

APItest.xs: flush output in test_bool_internals_func

With the explicit flush this test will fail when PERLIO=stdio, because
stdio defaults to block-based buffering on pipes and therefore the
output will come out wrong.

Bump builtin.pm version

Include 'indexed' in synopsis of builtin.pm

Merge branch 'haarg/encoding-warnings-cleanup' into blead

encoding-warnings: bump version

encoding-warnings: convert tests to use Test::More rather than Test.pm