Note SV_UTF8_NO_ENCODING exists, but deprecated

Note GIMME is deprecated

embed.fnc: Mark various save_foo as Core only

There are documented macros for each, like SAVEBOOL, that one is
supposed to use for this functionality.

embed.fnc: Mark unlnk as Core only

The macro UNLINK is what one is supposed to use for this functionality,
(though that isn't documented either)

embed.fnc: Mark sys_init3?, sys_term as Core only

The ALL-CAPS macro names are what one is supposed to use for this
functionality.

embed.fnc: Mark gimmve_V as Core only

The macro GIMME_V is what one is supposed to use for this functionality.

embed.fnc: ckwarn(), ckwarn_d() not API

These are just for the macros ckWARN..., ckWARN_d...

embed.fnc: Mark freetmps as Core only

The macro FREETMPS is what one is supposed to use for this functionality.

embed.fnc: Mark foldEQ_latin1 as Core only

This is a helper for internal use, as it isn't a complete solution to
the problem.

hv_common, hv_common_key_len aren't API

They are helper functions called from various functions that are API.

embed.fnc: delimcpy_no_escape is now documented

Indicate this to silence an autodoc warning;

This commit also fixes the ragged right edge of continuations lines by
adding white-space only

ninstr(): Use memchr instead of a loop

This function is perl's memmem if that function isn't available.  This
commit replaces its loop with a call to memchr, which is a C89 function.

Fix leak GH #18054

This was a simple matter of one path failing to free the memory.

t/comp: use \Q\E to avoid manual escaping

This is avoiding some manual escaping,
fixes some test where '.' was not escaped
and also improve readability of the unit test itself.

remove noise from porting/regen.t

diag() is for reporting information on exceptional events, note()
is more appropriate.

The debug output this produces is still visible in a verbose test
run.

perldelta updates

Update Devel-PPPort to release 3.60

This is mainly restoring the PERL_BCDVERSION macro.
PERL_BCDVERSION is used by multiple CPAN distributions
and should not have been renamed.

porting/pod_rules.t: Add a hint comment

This comment would have saved me hours of debugging if it had been there
when I started.

list assign in list context: honour LHS undef

GH #16685

In

    @a = ($x, undef, undef) = (1))

@a should have 3 elements. v5.25.6-79-gb09ed995ad broke this and was
returning one element.

The fix is simple: that previous commit made it so that elements were
pushed back onto the stack only if they weren't immortal, so
&PL_sv_undef was getting skipped. Make it so they always are.

list assign in list context was over-optimising

GH #17816

This code:

    my $x = 1;
    print (($x, undef) = (2 => $x));

was printing "22" when it should have been printing "21".
An optimisation skips the 'common values on both sides' test
when the LHS of an assign only contains a single var; as the example
above shows, this is not sufficient.

This was broken by v5.23.1-202-g808ce55782

This commit fixes it by counting undef's on the LHS towards the var
count if they don't appear first.

add a default enabled feature "multidimensional"

Like "indirect" this feature is enabled by default and enables the
perl4 hash-based multidimensional array emulation documented under
$; in perlvar.

perlfunc.pod - when sleep() is called with a negative int or 0

Intended to address #18010

IO::Handle: Fix a spurious error reported for regular file handles

89341f87 fix for GH #6799 introduced a regression when calling error()
on an IO::Handle object that was opened for reading a regular file:

$ perl -e 'open my $f, q{<}, q{/etc/hosts} or die; print qq{error\n} if $f->error'
error

In case of a regular file opened for reading, IoOFP() returns NULL and
PerlIO_error(NULL) reports -1. Compare to the case of a file opened
for writing when both IoIFP() and IoOFP() return non-NULL, equaled
pointer.

This patch fixes handling the case of the NULL output stream.

GH #18019

Add a hint to t/porting/regen.t

Also use 'lib' instead of '.'
when running the regen script with tap output
abort on the first failure so we advertise
the command to run to fix the issue.

Provide a better recommendation for 'Run only under a shell'

Using an unset variable hides the true intent and
also requires an extra backslash `\$running_under_some_shell`
when used in heredoc.

Note that this could also lead to mistake when using
`\$` in a regular Perl program, as this would be true
and not false as it should be.

Stop recommending the use of an undefined variable for
the shell fallback. Use '0', with a comment
making clear the goal of 'if 0'.

Rewrite delimcpy_no_escape()

This was doing a byte-by-byte search and copy.  We can make things
simpler and faster by
    1) only looking within the available space region
    2) using memchr for seach
    3) using Copy to copy the whole thing at once.

Use more idiomatic Chinese in README.cn; and it is in UTF-8

Update MANIFEST and pod/perl.pod to correspond.

(Commiter did the perl.pod update)

Update Devel-PPPort to release 3.59

Note that test files are not under file version
control anymore as they are generated files.

Time-HiRes uses PERL compare macro

Time-HiRes: update Changes and promote github URLs

myconfig.SH: remove leftover bincompat5005 entry

t/op/args.t: remove obsolete 5005threads comment

B: Remove 5005threads vestiges from tests

Also remove long-obsolete comment about `usethreads` vs
`use(i|5005)threads`.  It is no longer possible to configure perl with
"threads" without an actual implementation, and ponie is long dead.

Regenerate Porting/Glossary

Several recent commits adding new Configure probes have missed
regenerating the Glossary.

fix build for the two SipHash variations

fixes #16471

fix `l $var` where $var is a lexical variable

As with `i $obj` the DB::Obj in the call stack prevented DB::eval
from compiling/executing in the context of the debugged code.

fix C<i $obj> where $obj is a lexical

the DB::eval function depends on the special behaviour of eval ""
within the DB package, which evaluates the string within the context
of the first non-DB sub or eval scope, working up the call stack.

The debugger refactor moved handling for the 'i' command from the
DB package to the DB::Obj package, so the eval in DB::eval was
working in the context of the DB::Obj::cmd_i function, not in the
calling scope.

Fixed by moving the handling for the i command back to DB.

Fixes #17661.

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