regcomp.c: Don't repeat macro definition

Use a macro from within another macro instead of repeating the expansion
of it.

doop.c: Comment, white-space only

This removes an obsolete comment

perlop: Mention the word diamond wrt <>, <<>>

There really is no pod that refers to the common names of these
operators.

regexec.c: Use withinCOUNT()

This is faster, and clearer

regexec.c: Clarify comment

regcomp.c: Don't allow \p{Is_Is_...}

Only one 'Is_' prefix should be allowed.

canonicalize more contributor names and addresses

Add support for Bison versions up to 3.7

This requires copying the `YY_CAST` and `YY_ATTRIBUTE_UNUSED` macros
from the generated code, and extracting the `yysymbol_kind_t` enum if
it's defined.

We must also handle token type names with escaped double-quotes in
them, since it now names the `YYEOF` and `YYUNDEF` tokens `"end of
file"` and `"invalid token"` instead of `$end` and `$undefined`,
respectively.

Bump minimum required Bison version to 2.4

This lets us replace the deprecated `%pure-parser` directive with
`%define api.pure`, and get rid of some other conditional code.

Bison is only required for developers hacking on the grammar, since we
check in the generated code.  Bison 2.4 was released in 2008, and is
included in operating systems as old as Red Hat Enterprise Linux 6.

use PERL_REVISION in os2/Makefile.SHs

XS-APItest: use PERL_VERSION_GE helper

Also adjust t/xsub_h.t to use $^V.

Hash-Util: use PERL_VERSION_LT macro

Use PERL_REVISION in toke.c for -E switch

This allows to use the current features bundle
defined by the current version of Perl.

Teach Porting/pod_lib.pl to parse perldelta from any major version

Widen the feature bundle hint mask to 4 bits

VMS only uses two bits for its hints, so we can steal one more for the
feature bundles.  This is necessary if we want to remove features from
a future bundle, e.g. `indirect` and `switch`.

Plan9/9front i386 fixes

Restore 'our $VERSION' as a single line

This was changed recently to avoid potential
warnings when using the variable outside of a BEGIN
block. As this is a non issue at this date,
let's restore the original syntax to avoid breaking
poor man parsing for VERSION which can exist somewhere.

References: #18024

threads: use PERL_VERSION compare macros

Use new core PERL_VERSION compare macros.
When bumping thread with this change
we would need to make sure to use the last
version of ppport.h

Storable: use PERL_COMPARE macros

When bumping Storable with this change
we would need to make sure to use the last
version of ppport.h

XSUB.h: fix MARK and items variables inside BOOT XSUBs

ax was incremented by Perl_xs_handshake() and because of that
MARK and items were off by one inside BOOT XSUBs.

fixes #17755

Devel-PPPort: Bump to upstream at 7180c297

These changes from Devel-PPPort are adding
support for Perl 7.0 and fixes the testsuite
with a Perl 7.0.0 binary.

We would have to publish a new version of
Devel-PPPort.

Revert "Revert "Debugging GH #17671.""

This reverts commit 83aa75213860c4aa879853b1bac9e58a116214ab.

That commit was itself a reversion, hoping that various changes had
fixed a bug.  But it has cropped up, so the bug isn't fixed, and so this
is the reversion of the reversion.

Fixup handy.h to use PERL_VERSION macro

handy.h was recently updated in 4a1bbd3d but
this is not using the accurate definitions
of the new semantic versions introduced as part of #18020

They are:
- PERL_VERSION_MAJOR
- PERL_VERSION_MINOR
- PERL_VERSION_PATCH

More preparation for 7.0 in versioning

This modifies the macros introduced in
9b6e95106e18e4fd33678d0df9b2293fd4f8ab9c that allow one to determine how
the version of the perl currently being compiled compares to an
arbitrary one.  The modification is based on changes in the plan,
including renaming of variables.

This also introduces the use of '*' for the subversion (or micro as it
is now called) to make conversion from the soon-to-be deprecated
PERL_VERSION macro more rote, with less thinking involved.

Fix github action testing of pull requests on cygwin

The GITHUB_REF head on the remote doesn't reliably contain GITHUB_SHA
when testing pull requests. These PR merge SHAs are likely transient.
Checkout whatever the fetch retrieved from GITHUB_REF instead.

util.c: Reorder two functions

This will make some future commits a little less noisy

perl.h: Add comment

regcomp.c: Fix grammar in comment

Switch EXTENDED_TESTING logic to avoid failure notifications.

This sets a variable to skip or run jobs in the github workflow rather
using a workflow failure to stop early. Failing the workflow sends
unnecessary notifications.

split IRC notification between blead changes and non blead changes

Use GitHub action to send IRC notifications

This is a first iteration advertising to #p5p-commits.

The action for 'inBlead' needs to be altered.
Once stable, I would also port the existing bot
to this workflow so anyone can tweak/adjust it.

Try using hasConflicts tag

The action cannot find the tag using a space.

perl.h: Fix comment

utf8.h: Add comment

regcomp.c: Use get_cvs(), a clearer short cut

os2.c: Use PerlEnv_getenv

which has added protections beyond plain getenv()

Reposition tests for formats

Formats can be declared anywhere within a file, but, per
pod/perlform.pod, it's best to declare them near where they are being
used or referenced.

For: https://github.com/Perl/perl5/issues/18006

t/op/while.t: Correct two typos

In commit 138b823057 (written Nov 12 2009 but not committed until Jan 8
2011), contributor apparently intended to declare two scalar variables
with 'my': "my ($got_var, $got_i)".  However, the variables declared
were arrays: "my (@got_var, @got_i)".

These two variables were not subsequently used within the block and, in
principle, could be removed without affecting the test results.  But
since the contributor's intent was to make the file somewhat more
strict-compliant, it makes more sense to change the variables to
scalars, after which the variables are actually used within the block
and all tests continue to pass.

fix typo in example

thanks Deji Adegbite! via https://github.com/OpusVL/perldoc.perl.org/issues/145

Document warning 'readline() on unopened filehandle %s'

Place it in category: (W unopened).

Signed-off-by: James E Keenan <jkeenan@cpan.org>
For: https://github.com/Perl/perl5/issues/17935

Fix Math-Complex issue in test suite

By enabling strict and warnings in the testsuite
we could notice that t/Complex.t was testing
some functions not exported by Math::Complex:
- _stringify_cartesian
- _stringify_polar

This commit enable strict/warnings for the testsuite
and fix the test issue.

The customized VERSION is also bumped.

Automatically tags Pull Requests with conflicts

The goal is too quickly identify the pending pull
requests with conflicts.

(perl #17844) don't update SvCUR until after we've done moving

SvCUR() before the SvGROW() calls could result in reading beyond the
end of a buffer.

It wasn't a problem in the normal case, since sv_grow() just calls
realloc() which has its own notion of how big the memory block is, but
if the SV is SvOOK() sv_backoff() tries to move SvCUR()+1 bytes, which
might be larger than the currently allocated size of the PV.

Comply old C-standart

Try to fix for MSVC

Fix AUTHORS

Update AUTHORS

Correct my_rpeep example in perlguts

The example of my_rpeep in perlguts has an issue that
it might never stop iterating if there is a cycle in
OP tree. For example in the repository

https://github.com/basiliscos/p5-PeepholeTest

it hangs when starts executing test.pl

The right traversal was originally taken from

https://metacpan.org/source/ZEFRAM/Devel-GoFaster-0.001/lib/Devel/GoFaster.xs#L330

Note range op behavior change in docs

This change documents the previous behavior of the range operator with magic string increment in Perl 5.30 and below - the change was introduced in commit d1bc97feec from GitHub #16770 (RT133695).

perl5db: allow the recorded history item length to be configured

This requires a few small changes.

1. add a new option, which I stored in what looked like one of the
   standard ways
2. only store the item in terminal history if >= this length
3. only add to @hist if >= this length
4. only display hist item if >= this length

I believe #4 is redundant and should be removed, but the code was
already effectively there.

Full revert of 894d8b10212a906402f4db9f9aac9efe9fa084fd

Revert "Small changes to test on bugzilla and improve compatibility"

This is a partial revert to remove utf8->import
which breaks Storable

This reverts commit 894d8b10212a906402f4db9f9aac9efe9fa084fd.

Add missing removal of FCRYPT macro use after rebase

Fixup for #17624

reentr.c: Prevent infinite looping

This is an easy, though  paranoid hedge to prevent something that should
never happen from causing an infinite loop if it were to happen.

Add missing MANIFEST entry from fix for debugger

Add on fix to #17901

Fixing intrpvar.h from recent merge

This was broken with the change from
a04ef3ff46f2526da1484bdd80995415ac7e1969

utf8.h: Remove obsolete macro

It turns out that this macro would have failed to compile since
commit 538b546eb0f252250a30c08e6af47d0ea7433fa1, in October 2013.  So it
is clear no one is using it.

regexec.c: Use UTF8SKIP for utf8 string

This code was advancing per-byte on a UTF-8 string, which still works,
but is slower than need be.

Revert "regcomp: use long jumps if there is any possibility of overflow"

This reverts commit 4b4b1fbd0d43429c43d5de8857f3266daba1dd66.

That fix was intended as a fail-safe, and left in 5.32.  But, other
changes in blead, chiefly always checking the return of reginsert,
should have made this commit unnecessary.  Now that it is early in the
development cycle, revert it.

regcomp.c: Improve pattern dumps

Instead of saying [x-y], say [xy].

regcomp.c: No need to test before setting

It's cheaper to just set unconditionally than add a condition to see if
it has already been done.

regcomp.c: Replace 2 branches with a single inRANGE

regcomp.c: Remove SPSTART flag

This flag is set but not examined, so is useless.  Research by Hugo
van der Sanden indicates its value has never been used.

See the thread starting at
http://nntp.perl.org/group/perl.perl5.porters/257101

regcomp.c: Remove WORST define

The is not used and is actually 0, so does nothing as a flag, except
when initializing.

See the thread starting at
http://nntp.perl.org/group/perl.perl5.porters/257101

Time-HiRes: Remove obsolete vms code

This code became irrelevant in 1996.

See https://github.com/Perl/perl5/pull/17658#pullrequestreview-377796612

reentr.c: Buffer sizes for asctime_r,ctime_r are small

The needed sizes of these are stated in the man pages, and are much
smaller than were being allocated.

No need to use locale_r with thread-safe locales

These use a completely different mechanism.

Use quoted strings to compare 5.031006 in Safe.pm

Fixup AUTHORS with recent merges

Adjust AUTHORS file with new authors

Do not load utf8_heavy.pl in Safe.pm above 5.31.6

This is a fixup for #17969 which wanted to load utf8_heavy.pl but
it is no longer available as of 5.31.6

Remove obsolete FCRYPT ifdefs and associated PL_cryptseen (#17624)

Co-authored-by: Karl Williamson <khw@cpan.org>

Haiku use finddir for default prefix in hints

Align to haiku ports (+ keep some extra) and set path dynamically

re-work opslab handling to avoid non-portable alignment assumptions

Fixes #17871

The op slab allocator code made the assumption that since OP and
hence OPSLOT contain pointers, the base of each of those would be
an integral number of sizeof(pointer) (pointer units) from the
beginning of OPSLAB.

This assumption is non-portable, and broke calculating the location
of the slab based on the address of the op slot and the op slot offset
on m68k platforms.

To avoid that, this change now stores the opslot_offset as the
offset in pointer units from the beginning of opslab_slots rather
than from the beginning of the slab.

If alignment on a pointer boundary for OPs is required, the compiler
will align opslab_opslots and since we work in pointer units from there,
any allocated op slots will also be aligned.

If we assume PADOFFSET is no larger than a pointer and requires no
stricter alignment and structures in themselves have no stricter
alignment requirements then since we work in pointer units all core
OP structures should have sufficient alignment (if this isn't true,
then it's not a new problem, and not the problem I'm trying to solve
here.)

I haven't been able to test this on m68k hardware (the emulator I
tried to use can't maintain a network connection.)

Revert "op.h: Add additional padding to struct opslab to ensure proper alignment"

This reverts commit a760468c9355bafaee57e94f13705c0ea925d9ca.

This change is fragile, the next change avoids the need for such
manual padding.

Always expose the perl_tsa_mutex_* functions when threads are enabled

These functions were only part of the API if perl is built with clang.
However perl modules built with clang still try to use them even
when perl itself is built with gcc.  This patch replaces the #ifdef PERL_TSA_ACTIVE
around these functions with defined(USE_ITHREADS) && defined(I_PTHREAD)
so they are always available when treading is enabled.  This fixes the clang build of
perl modules when perl is built with gcc.

corelist - print a better error message if unknown versions are passed to corelist --diff

report the index for not found hash/array index lookups in report_uninit()

where the index is a non-magical, non-reference variable.

This only works where the op is converted to OP_MULTIDEREF, but that
should be happening for the simple cases this handles.

An alternative would be to report the index variable name rather than
the index, but that seems less useful to me.

Porting/makerel: add cleanup option

We can now run 'Porting/makerel -c' to cleanup
and generate the tarball.

Porting/makerel - check for xz

In addition to the .tar.gz file also abort when one .xz file
already exists.

Update Net-Ping from 2.72 to 2.73

Fix #17992

This is preserving customized tests
and mark them as customized.

Clearing DB::action at the end is no longer needed

as it's cleared right after it's been run.

After running an action in the debugger, turn it off

When running with "c", there was no problem, but when running with "n"
or "s", once the action was executed, it kept executing on the
following lines, which wasn't expected. Clearing $action here prevents
this unwanted behaviour.

Fix indirect method call in ExtUtils::Constant test

It puts both "use $];" and "bootstrap $package \$VERSION;" in the
generated test module, which is going to break if we ever remove
`indirect` from the current feature bundle.

Fix by making the method call direct instead.

put version directly in Exporter::Heavy

Modules should have their version number directly in the module, not try
to pull it in from another module. The Exporter module that would be
loaded may not correspond to the Exporter::Heavy that is having its
version checked. Generally, pulling a version from another module would
fail on something like PAUSE or a CPAN client, but because Exporter is
probably already loaded, the old code would usually just pick the
version of the currently installed Exporter, rather than the one
corresponding to the Exporter::Heavy module that is being checked.

Future proof version check

Update darwin.sh

Simplify syscall check

Update deprecated syscall check for 11.x and greater

Update error message

Update AUTHORS

Add 11.x support for darwin.sh

Update regen/feature.pl

Features bundles are built on top of each other.

This change make it easier to read what features
were added from one version to another.
It also helps identify similar bundles.

Features are also now sorted on pod.

Enforce strict & warnings to generate feature.pm

IO::Socket::UNIX: synchronize behavior with module documentation (#17787)

* synchronize behavior with module documentation

IO::Socket docs states that passing Blocking => 0 will be set socket to non-blocking mode

* Update AUTHORS
* bump version

pp.c/pp_hot.c - add NV<->NV case to numerical comparison ops

Fix 404 and text in New Unicode properties section

See https://github.com/Perl/perl5/issues/17881

I found a 404, and an "old" link. I investigated.

My conclusion was UC have landed new TR18 and TR39 since text in section
  New Unicode properties Identifier_Status and Identifier_Type supported
was written.

I've guessed at a suitable update.

bump IO to 1.44

and update the ChangeLog

IO::Handle: clear the error on both input and output streams

Similarly to GH #6799 clearerr() only cleared the error status
of the input stream, so clear both.

make $fh->error report errors from both input and output

For character devices and sockets perl uses separate PerlIO objects
for input and output so they can be buffered separately.

The IO::Handle::error() method only checked the input stream, so
if a write error occurs error() would still returned false.

Change this so both the input and output streams are checked.

fixes #6799