avoid identical stack traces

GH #15109

The output of caller() (e.g. as produced by carp::Confess) produces
multiple identical outputs when within a nested use/require. This is
because at the time of calling the 'BEGIN { require ... }', PL_curcop is
set to &PL_compiling, which is a fixed buffer within the interpreter,
whose individual file and line fields are saved and restored when doing
a new require/eval. This means that within the innermost require,
PL_compiling has file:lineno of the innermost source file, and multiple
saved PL_curcop values in the context stack frames all point to the same
&PL_copmpiling.  So all levels of the stack trace appear to come from the
innermost file.

This commit fixes this (after a fashion) by, at the start of calling a
BEGIN, making PL_curcop point to a temporary copy of PL_compiling
instead.

This is all a bit of a hack.

Enable Cygwin smoking on all kinds of branches

Remove bad "git switch" command

"git switch" isn't supported by Github git anyway

IO: update the ChangeLog after a (trial) release of IO to CPAN

perldelta for a559786348e0

perldelta for e6c7056ba6f1

un-TODO pl_check.t and remove io/handle.t

io/handle.t depends on IO::Handle using the PL_check hack, but
Nicholas's back portable fix no longer uses that

Fix threaded perl detection, thanks to James Keenan.

Move PL_check to the interp vars to fix threading issues

Fixes issue #14816

provide a test for the PL_check mis-behaviour

Nicholas Clark's fix for IO makes the test in niner's patch
meaningless, so test it separately.

Note that G_RETHROW is documented

This is for Devel::PPPort.

Fix where we look for the Sun/Solaris/Developer Studio compiler

The movable feast that is the sun/solaris/developer compiler

perlpacktut: fix broken url

This is my best guess as for what it's meant to point to.

Reported in https://github.com/OpusVL/perldoc.perl.org/issues/81

regexec.c: Clarify comment

Rmv leading underscore from macro name

These are illegal in C, but we have plenty of them around; I happened
to be looking at this function, and decided to fix it.  Note that only
the macro name is illegal; the function was fine, but to change the
macro name means changing the function one.

Only allow punct delimiter for regex subpattern

The experimental feature that allows wildcard subpatterns in finding
Unicode properties, is supposed to only allow ASCII punctuation for
delimitters.  But if you preceded the delimitter by a backslash, the
check was skipped.  This commit fixes that.

It may be that we will eventually want to loosen the restriction and
allow a wider range of delimiters.  But until we have valid use-cases
that would push us in that direction, I don't want to get into
supporting stuff that we might later regret, such as invisible
characters for delimitters.  This feature is not really required for
programs to work, so I don't view it as necessary to be as general as
possible.

Clarify in documentation that sv_isa_sv does not invoke magic

APItest.xs: Increase version

APItest.xs: Fix compile warning: format ‘%d’ expects argument of type ‘int’, but argument 2 has type ‘line_t {aka long unsigned int}’

Use "%u" format modifier instead of "%d" as line_t is unsigned. Type line_t
is of U32 type and therefore casting it to unsigned int is safe on most
platforms.

Enable smoke-testing on all branches

Previously, branch names that contain a "/" (as in "smoke-me/Corion-foo")
were skipped

https://help.github.com/en/actions/automating-your-workflow-with-github-actions/workflow-syntax-for-github-actions#filter-pattern-cheat-sheet

Fix superfluous article and comma splice

Add the `isa` operator

Adds a new infix operator named `isa`, with the semantics that

  $x isa SomeClass

is true if and only if `$x` is a blessed object reference that is either
`SomeClass` directly, or includes the class somewhere in its @ISA
hierarchy. It is false without warning or error for non-references or
non-blessed references.

This operator respects `->isa` method overloading, and is intended to
replace boilerplate code such as

  use Scalar::Util 'blessed';

  blessed($x) and $x->isa("SomeClass")

regcomp.c: Make some refcnt changes NN

to save a few cycles

re/regexp_unicode_prop.t: Add comment

regcomp.c: White-space only

Indent after the previous commit introduced a new block around this
code.

PATCH GH #17025 \p{user-defined} overrides official Unicode

Prior to this patch, they only sometimes overrode.

regcomp.c: Replace a loop with strcspn()

A later commit will add a terminating character for the span, so we use
this function instead of memchr.

Correct typo:  the warning has 'statement' in singular

fix Github URL in message posted to #p5p-qa

the repository key is deprecated, and we now produce the correct
full URL, so the comment is meaningless

re-work travis cases to test more variety

I expect a -DPERL_GLOBAL_STRUCT_PRIVATE build to catch most
-DPERL_GLOBAL_STRUCT issues, and these are very rarely used options
anyway, so eliminate the second.

Expand the usesitecustomize case to also check long doubles and
an English Unicode locale.

Replace the removed case to check quadmath, cbacktrace and a non-English
Unicode locale.

regcomp.c: Replace a loop with memchr

regcomp.c: After AvARRAY(), don't use av_store

We are manimpulating the array directly here, so don't revert to going
through the regular interface, as not all the hand-offs have been done

regcomp.c: White space only

regcomp.c: Use inRANGE macro

This is faster and clearer

t/op/die.t: 'use utf8'

This file is encoded in UTF-8, even though it didn't say it was.

handy.h: Fix typo in little-used macro

This only affected an EBCDIC build where no other tools are around.
This is no longer really used.  It was used to bootstrap EBCDIC when
first porting to it (after the 5.8 series).

ext/XS-APItest/t/utf8_warn_base.pl: Fix for EBCDIC

There were some flaws here that showed up in EBCDIC testing.

ext/XS-APItest/t/utf8_warn_base.pl: Don't recalculate value

This value is already available in a different variable

ext/XS-APItest/t/utf8_warn_base.pl: Fix comment typos

regen/ebcdic.pl: Fix bug in decoding some high code points

This affected non-allocated or little-used high code points.

regen/ebcdic.pl: Comments only: typo, add column headings

Fix UTF8_IS_START on EBCDIC

Update IO-Compress to CPAN version 2.093

Update Compress-Raw-Bzip2 to CPAN version 2.093

Update Compress-Raw-Zlib to CPAN version 2.093

Update IO-Compress to CPAN version 2.092

  [DELTA]

  2.092 4 December 2019

      * No changes

Update Compress-Raw-Bzip2 to CPAN version 2.092

  [DELTA]

  2.092 4 December 2019

      * No changes

Update Compress-Raw-Zlib to CPAN version 2.092

  [DELTA]

  2.092 4 December 2019

      * No Changes

Local variable 'last' hides a parameter of the same name

LGTM static analysis identified a situation where it recommended
renaming a variable.

See: https://lgtm.com/projects/g/Perl/perl5/alerts/?mode=tree&severity=recommendation&ruleFocus=2156240606

Better variable name per khw review

Update link to Birrell's threads paper

Local variable 'imode' hides a parameter of the same name

LGTM static code analysis of Perl 5 source code issued this
recommendation.  Implement fix.

https://lgtm.com/projects/g/Perl/perl5/rev/ae73d7ec2329275a2dba4be24415743f884d9dfd

perldelta for 5e91ca38cafa2b006f7671e941287e05d31c2d97

perldelta for 41fb88e3240, b34c1a7ee25

updates to $^H{feature_validname} now update cop_features

This removes $^{FEATURE_BITS}, since it's no longer needed.

tests for the faster features problem sprout pointed out

Correct inconsistent version numbers in English.pm

Per commit message by David Mitchell in 142a37fd (Jul 24 2013), the code
changes affecting the performance of $`, $&, $' weren't fully enabled
until 5.20.  Hence, the documentation should caution for 5.18 and
earlier.

As reported by peterdd++ in https://github.com/Perl/perl5/pull/17344

perldelta for 62fc08893a06fdb5633e7656e3c783d705682fd1

Update Test-Simple to CPAN version 1.302170

From Changes:

    1.302170  2019-12-02 13:25:48-08:00 America/Los_Angeles

    - Fix unwanted END phase event (#840)

skip code that requires dynamic loading and minitest works

fix skip for loop over locales

Any skips inside this loop was skipping over the entire loop,
not just the current iteration, from the conditions tested in
the loop this seemed incorrect (besides messing up the test count.)

POSIX isn't loadable in miniperl

move the implementation of %-, %+ into core

Previousl this could cause problems during minitest.

Fixes #17293

Regenerate Configure and friends after metaconfig changes

A probe for wcrtomb() and one for attribute always inline have been
added, and the ones for:
1) checking if there is a C backtrace facility; and
2) character data alignedness
have been revised

metaconfig.h: Add entry for wcrtomb()

metaconfig.h: Rmv temporary entries

These are no longer needed

Update documentation for UTF8f

regcomp.c: Silence compiler warning

Storable: Fix compile warning: comparison between signed and unsigned integer expressions

gcc -m32 -c   -fwrapv -fno-strict-aliasing -pipe -fstack-protector-strong -I/usr/local/include -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -D_FORTIFY_SOURCE=2 -Wall -Werror=declaration-after-statement -Werror=pointer-arith -Wextra -Wc++-compat -Wwrite-strings -O2   -DVERSION=\"3.18\" -DXS_VERSION=\"3.18\" -fPIC "-I../.."   Storable.c
Storable.xs: In function ‘retrieve_lvstring’:
Storable.xs:1238:46: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
         else if (PerlIO_read(cxt->fio, x, y) != y) {            \
                                              ^
Storable.xs:1238:46: note: in definition of macro ‘SAFEPVREAD’
         else if (PerlIO_read(cxt->fio, x, y) != y) {            \
                                              ^~

Storable: Fix compile warning: format ‘%d’ expects argument of type ‘int’, but argument 2 has type ‘I32 {aka long int}’

gcc -m32 -c   -fwrapv -fno-strict-aliasing -pipe -fstack-protector-strong -I/usr/local/include -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -D_FORTIFY_SOURCE=2 -Wall -Werror=declaration-after-statement -Werror=pointer-arith -Wextra -Wc++-compat -Wwrite-strings -O2   -DVERSION=\"3.18\" -DXS_VERSION=\"3.18\" -fPIC "-I../.."   Storable.c
Storable.xs: In function ‘get_regexp’:
Storable.xs:3446:14: warning: format ‘%d’ expects argument of type ‘int’, but argument 2 has type ‘I32 {aka long int}’ [-Wformat=]
       CROAK(("re::regexp_pattern returned only %d results", count));
              ^
Storable.xs:583:55: note: in definition of macro ‘CROAK’
 #define CROAK(x) STMT_START { cxt->s_dirty = 1; croak x; } STMT_END
                                                       ^
Storable.xs: In function ‘retrieve_regexp’:
Storable.xs:6867:16: warning: format ‘%d’ expects argument of type ‘int’, but argument 2 has type ‘I32 {aka long int}’ [-Wformat=]
         CROAK(("Bad count %d calling _make_re", count));
                ^
Storable.xs:583:55: note: in definition of macro ‘CROAK’
 #define CROAK(x) STMT_START { cxt->s_dirty = 1; croak x; } STMT_END
                                                       ^

locale.c: Use proper #ifdef to enable behavior

This changes to use USE_POSIX_2008_LOCALE instead of
HAS_POSIX_2008_LOCALE.  Rarely do they differ, but someone may choose to
configure their installation to not use these more modern functions,
even if available, perhaps because they're buggy on that system.

locale.c white space only

PATCH: GH #17081: Workaround glibc bug with LC_MESSAGES

Please see the ticket for a full explanation.  This bug has been
submitted to glibc, without any real action forthcoming so far.

This invalidates the message cache each time the locale of LC_MESSAGES
is changed, as glibc should be doing this when uselocale changes that,
but glibc fails to do so.

This patch is an extension to the one submitted by Niko Tyni++.

I don't know how to test it, since a test would rely on several
different locales in different languages being available, and that
depends on what's installed on the platform.  I suppose that one could
go through the available locales, and try to find three with different
wording for the same message.  Doing so however would trigger the bug,
and at the end, if we didn't get three that differed, we wouldn't know
we wouldn't know if it is because of the bug, or that they just didn't
exist on the system.

However, below is a perl program that demonstrated the patch worked.
You could adjust it to the available locales.  The buggy code shows the
same text for all locales.  The fixed shows three different languages.

use strict;

use Locale::gettext;
use POSIX;

$ENV{LANG} = 'C.UTF-8';

for my $lang (qw(fi_FI fr_FR en_US)) {
    $ENV{LANGUAGE} = $lang;
    setlocale(LC_MESSAGES, '');
    my $d = Locale::gettext->domain("bash");
    print $d->get('syntax error'), "\n";
}

autodoc.pl: Catch more errors

The previous commit fixed an error in an =for apidoc line.  This commit
fixes autodoc.pl to catch bugs like it, and also that there aren't
multiple places that document the same function.  This necessitated a
way to note that sample documentation wasn't the real stuff.

Fix wrong apidoc line

We don't need to switch branches in git at all

We can directly determine the (authors of) the commits
in a pull request by using the latest branch-off point
between bleadperl and the pull request branch and the
SHA that Github gives us.

Remove -Dissymlink='test -h'

This is a vestige of previously failed attempts. The previous attempts
failed due to non-Cygwin test.exe being in $ENV{PATH}.

perlguts: Add missing '=for apidoc' lines

Spotted by Pali++

POSIX/t/time.t: Reset LC_CTYPE to C

The C standard says about the format in strftime(): "The format shall be
a multibyte character sequence, beginning and ending in its initial
shift state"

Tony Cook and I interpret that as meaning that to be sure, not only does
LC_TIME have to be reset, but LC_CTYPE to account for all possible
variations in these tests.

t/loc_tools.pl: Silence 'undef' warning

regcomp.c: Silence (stupid) compiler warning

This can't really happen, but some compilers don't realize it.

Move data for PL_InBitmap to charclass_invlists.h

This makes it consistent with the other inversion lists for this sort of
thing, and finishes the fix for GH #17154

Move regex global variables to interpreter level

This is part of fixing gh #17154

This scenario from the ticket
(https://github.com/Perl/perl5/issues/17154#issuecomment-558877358)
shows why this fix is necessary:

    main interpreter initializes PL_AboveLatin1 to an SV it owns

    loads threads::lite and creates a new thread/interpreter which
    initializes PL_AboveLatin1 to a SV owned by the new interpreter

    threads::lite child interpreter finishes, freeing all of its SVs,
    PL_AboveLatin1 is now invalid

    main interpreter uses a regexp that relies on PL_AboveLatin1, dies
    horribly.

By making these interpreter level variables, this is avoided.  There is
extra copying, but it is just the SV headers, as the real data is kept
as static C arrays.

Update perlpolicy to reference GitHub issue tracker

Convert issue links from rt.perl.org to GitHub

Update Porting documents to reference GitHub issues

The version for PPPort in CPAN is 3.56

PATCH: gh #16045: add test descriptions for some basic tests

Committer's note:  I'm belatedly applying the rest of the patches from
the original ticket.

re-work the cygwin action

This does a few things:

- checks out perl under the cygwin "home" directory, since cygwin only
  tries to make POSIX permissions work under that.  This allows a test
  of the copied file mode to work in lib/File/Copy.t

- checkout using cygwin's checkout, rather than checking out using Win32
  then refreshing it.

The checkout code is a bit ugly, it might be possible to use
action/checkout's path option to fix it.

Revert "cygwin 2.1.0 can test -r just fine, incl. under admin"

and re-work it to match what the core does to check for an admin user
under Cygwin.

I misunderstood what this was testing when submitted.

Yes, -r works under admin, and returns the correct result, but this is
specifically testing the behaviour as a non-admin.

more closely match File::stat's -X behaviour to built-in on cygwin

Make FindBin dual-life

make test (via TestInit) in core and TAP::Harness set @INC

so we don't need the test code to do it

Update gitignore:

This was done by calling:

    ./perl -Ilib regen/lib_cleanup.pl

Update Maintainers.pl

Regenerate META:

Regenerate the META by calling:

    make regen_meta

Update MANIFEST:

This was done by calling:

    ./perl -Ilib Porting/pod_rules.pl

Change regex to look for t/ dir

Move FindBin to dist/

Update Storable's bugtracker

Filter::Simple: fix and sanity check for executable_no_comments

fixes #17122

bump $Storable::VERSION