cmpVERSION.pl and test.pl now handle checkouts from git-worktree.

pp_tie should completely reset the underlying hash's iterator state.

Previously it would mangle it, resetting EITER but not RITER, meaning that
after untie continuing iteration would be inconsistent - normally it would
carry on exactly where it left off, but if iteration had been in the middle
of a chain of HEs, it would skip the rest of the chain.

Fixes GH #19077

Tests for tied hashes using test.pl

t/op/tie.t uses run_multiple_progs() to run each test in an individual perl
interpreter. This is robust, but slow and doesn't give much flexibility in
test diagnostics.

Hence this test file, for test cases that can safely run in the same
interpreter, and where we'd can use like() etc for better diagnostics.

Don't call HvHASKFLAGS_on() in S_hv_delete_common().

We're not changing any keys on the hash, so if it didn't have any keys with
flags before the delete, it won't have any afterwards either.

I added this line in Nov 2003 as part of commit cd6d36ac47e47079:
    Move the negative key -> utf8 flag conversion out to hv_delete

I don't think that it was correct then, and I don't think that it's correct
now.

pp_tie should use HvEITER_get() instead of HvEITER().

The former doesn't make a function call.

Add /^vgcore.*/ to the list of podcheck.t filename exclusions.

Regularly named core files were in the list, but valgrind's variant were not.

Display the completion ETA once we've done at least 20%

Print count of done vs. total tests and percentage from Porting/bench.pl

op.h: define missing BASEOP fields (op_sibparent,op_targ)

sv.c: Removed a redundant SvNV_nomg call in sv_vcatpvfn_flags.

SvNV_nomg(argsv) is redundant here because its NV is already loaded
to the local variable "nv".

Exporter: Document non-inheriting as default mechanism

utf8.c: Make new static fcn more flexible

This commit allows this function to be called with NULL parameters when
the result of these is not needed.

utf8.c: White-space, comment only

utf8.c: Rmv no longer needed speed-up code

The code this commit removes made sense when we were using swashes, and
we had to go out to files on disk to find the answers.  It used
knowledge of the Unicode character database to skip swaths of scripts
which are caseless.

But now, all that information is stored in C arrays that will be paged
in when accessed, which is done by a binary search.  The information
about those swaths is in those arrays.  The conditionals removed here
are better spent in executing iterations of the search in L1 cache.

utf8.c: Split a static fcn

This adds a new function for changing the case of an input code point.
The difference between this and the existing function is that the new
one returns an array of UVs instead of a combination of the first code
point and UTF-8 of the whole thing, a somewhat awkward API that made
more sense when we used swashes.  That function is retained for now, at
least, but most of the work is done in the new function.

utf8.c: Use porcelain libc case changing fcn

The fancy wrapper macro that does extra things was being used, when all
that is needed is the bare libc function.  This is because the code
calling it already wraps it, so avoids calling it unless the bare
version is what is appropriate.

regexec.c: White-space/comment only

regexec.c: regrepeat() switch() on target utf8ness

Many of the cases in the main switch() statement in regrepeat() begin
with

    if (utf8_target)

or similar.  These branches can be saved, and the code easier to
understand by including the utf8ness in the case statements, as this
commit does; taking advantage of the changes in the previous commit.

regnodes.h: Add new set of defines

There currently is a set of defines for switching on the op code of the
regnode, and another set for switching on that plus the utf8ness of both
the target and pattern.  This latter set is useful in one routine in
regexec.c where all three are important.  It allows fewer branches to
have to be done by encoding where to go into the case: value.

There are other similar switches in regexec.  The one in regrepeat()
doesn't care very much about the utf8ness of the pattern.

This commit adds a third set of defines for use by that switch, which
will be used by the next commit.

Replace FIXME in the MANIFEST with the correct description for the file.

We have 117 instances of the text FIXME in the core source code - it's not
actually good choice of text for a placeholder meaning "fix this before
pushing", as it's easy to miss the one that was added with this intent.

My mistake, spotted by ilmari.

Pre-extend hashes in list assignment before assigning to them.

We know how many pairs of keys/values are on the stack, so pre-extend the
hash to the appropriate size, instead of letting the hash needlessly re-size
one or more times during the assignment loop.

Pre-extend new anonymous hashes before assigning to them.

We know how many pairs of keys/values are on the stack, so pre-extend the
hash to the appropriate size, instead of letting the hash needlessly
re-size one or more times during the assignment loop.

Use sv_inc() for the env de-dup hash in S_init_postdump_symbols().

This replaces an hv_exists/hv_store pair with a single LVALUE hv_fetch.

Comment about some subtleties in S_init_postdump_symbols().

Move some variables in S_init_postdump_symbols() to a tighter scope.

Pre-size the %ENV hash before populating it from environ.

Also use HvTOTALKEYS() instead of HvKEYS(), as the latter makes a check for
placeholders, which here cannot make a difference.

Hash::Util::FieldHash fix_trigger can pre-size the hash before use.

This is only called by CLONE (and hence only matters for ithreads), but it's
an improvement, and small and self-contained.

Pre-extend the ISA hash in Perl_mro_get_linear_isa().

S_mro_get_linear_isa_c3() now uses sv_inc_nomg().

Previously it was coding the increment "longhand", to avoid bugs in 5.6.x.
However, this XS code is now exclusively in the core, and 5.6.x is long
dead, so there's no need to work around it.

The "longhand" code might *seem* to be more efficient (due to inlining some
logic) but it still required at least one call to a function in sv.c
(so no real change in generated code size) and that call was for the
increment 0 => 1, so likely called a lot, so no significant change in total
calls made.

Use an LVALUE hv_fetch() in S_sequence_num() instead of fetch/store.

Better put a date in it

Bump the perl version in various places for 5.35.4

Merge branch 'porting-core-cpan-diff-20210805' into blead

Porting/core-cpan-diff: Remove '--binary' on *BSDs

Other things being equal, Porting/core-cpan-diff defaults to passing '-u
--binary' to the 'diff' command.  It turns out that, though '--binary'
is not a *documented* option for GNU diff, it is present in the source
code (rjbs research in
https://github.com/Perl/perl5/pull/19027#issuecomment-898901007).  The
'--binary' option clearly does *not* work on FreeBSD and probably
doesn't work on other *BSDs.  On those systems, running the program
spews more than 80,000 lines of error output.

Code revised to remove '--binary' on *BSDs.  Documentation adjusted.

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

Merge branch 'pod-html-refactoring-5-of-5' into blead

Pod::Html: correct line dropped in 13f1edeb52

As noted by @Grinnz in
https://github.com/Perl/perl5/pull/19050#discussion_r688968921.

Pod::Html: remove debugging artifact

As noted by @Grinnz in https://github.com/Perl/perl5/pull/19050#discussion_r688969025.

Pod::Html: remove debugging artifact

As noted by @Grinnz in
https://github.com/Perl/perl5/pull/19050#discussion_r688969538.

Pod::Html: assign directly to array ref

This restores the thrust of c0ac28175b, which got zapped when resolving
merge conflicts.

Pod-Html: remove commented-out code

Make use of Exporter consistent.

Introduce $self->{Pages}

htmldir3.t: 3rd test unit is now expected to fail.  So now the test file
demonstrates that the unit fails whether it's placed before the second
unit or after.

Signed-off-by: James E Keenan <jkeenan@cpan.org>

Move $output into the object

Correct dropped variable declaration.

Signed-off-by: James E Keenan <jkeenan@cpan.org>

Replace _save_pages() with _transform()

_save_pages() is a Pod::Simple::Search method which takes two specific
arguments.  From the point of view of making Pod-Html OO, it was
problematic because (a) it used a variable ($Podroot) defined outside of
its scope and (b) it assigned to a variable (%Pages) defined outside of
its scope.  It was therefore resistant to encapsulation.

Experimentation showed that if we use the return value of
Pod::Simple::Search::survey(), we could parse that hashref using what
was the guts of _save_pages() and assign explicitly to %Pages.

TODO:  Move %Pages within the Pod::Html object.  Handle $object
properly.

Signed-off-by: James E Keenan <jkeenan@cpan.org>
Remove merge conflict marks, commented-out code, as noted by rjbs in
https://github.com/Perl/perl5/pull/19050#discussion_r693559914.

Convert 3 more subroutines to method calls

refine_parser(); feed_tree_to_parser(); write_file()

Signed-off-by: James E Keenan <jkeenan@cpan.org>

Turn 3 more internal subroutines into method calls

identify_input(); parse_input_for_podtree(); set_Title_from_podtree();

Signed-off-by: James E Keenan <jkeenan@cpan.org>

Have generate_cache() use object

Still have to deal with %Pages being outside the object.

Signed-off-by: James E Keenan <jkeenan@cpan.org>

Make get_cache(), cache_key() and load_cache() use object

Signed-off-by: James E Keenan <jkeenan@cpan.org>

File::Spec::Unix is not needed within package Pod::Html

It's only needed within Pod::Simple::XHTML::LocalPodLinks, so let's
remove it from the first package.

Signed-off-by: James E Keenan <jkeenan@cpan.org>

Start to make Pod::Html object-oriented.

For the time being, at least, we'll have pod2html() call a constructor.
All subs up through refine_globals have been turned into method calls.

Signed-off-by: James E Keenan <jkeenan@cpan.org>

Increment $VERSION to 1.33 in all .pm files

perlfunc 'ref' and UNIVERSAL: document isa operator alternative

isUTF8 DFA: change while {} to do {} while;

This saves a conditional in many cases.  Core Perl doesn't call this
on an empty string, so the first test that it is empty is redundant.

We can't guarantee this for non-core calls, so the conditional is made
explicit for them.

update the range of versions we support and do not support

Rmv redundant API info for isUTF8_char_flags

This resolves GH #19069

add epigraph for 5.35.3

Text-Tabs+Wrap: Sync with CPAN version 2021.0814

Here be corelist

Before filling the lookup hash for opcode names, pre-expand it.

We know the size needed (about 400 entries), so doing this saves several
automatic resizes when populating it.

Delete Perl_ptr_table_clear, marked as deprecated since v5.14.0

make way for 5.35.4

tick off release 5.35.3

add new release to perlhist

finalize perldelta

Update Module::CoreList for 5.35.3

update AUTHORS

more perldelta updates

perlop: Fix indented here-doc empty line description

This bug was introduced by c275db86a9.

Spotted by Karen Etheridge

add some missing perldelta entries

collapse 3 whitespace to 2

fix pod markup

Added my choice of epigraph for 5.35.2

perldelta for sort.pm becoming a no-op.

Also fix a typo in sort's Pod.

Add top-level Code of Conduct document

This commit adds a top-level file that points readers to the actual
Standards of Conduct in perlpolicy.pod.  It also makes a point to
distinguish between what the Perl Steering Council is responsible for
and what other community groups are responsible for.

pp_defined: modify OP_DEFINED paths to use TOPs and RETSETs

perldelta updates

also sort the module entries.

bump $Time::HiRes::VERSION

for the Makefile.PL change

Perldelta that «use v5.35» enables warnings

mktables: Generate =head1 NAME line in Name.pm

All .pm files are supposed to have this line.  So far this hasn't been
necessary for this file, but future commits will require it.

Document v5.35 warnings-on-by-default

Test new warnings enabled by default

Add asher mancinelli to AUTHORS

Enable warnings on «use v5.35»

Add "run makedepend in parallel" to perldelta

Add utf8_to_utf16

Improve utf16_to_utf8_reversed()

Instead of destroying the input by first swapping the bytes, this calls
a base function with the order to use.  The non-reverse function is
changed to call the base function with the non-reversed order.

lib/unicore/mktables: correct sub signatures in 2 locations

Then, re-run regen/mk_invlists.pl and regen/regcharclass.pl and commit
changes in headers.

Simplify utf16_to_utf8()

A previous commit has simplified uvoffuni_to_utf8_flags() so that it is
hardly more than the code in this function.  So strip out the code and
replace it by a call to uvoffuni_to_utf8_flags().

Merge branch 'pod-html-refactoring-4-of-5' into blead

Pod::Html::Util: small documentation changes

In part suggested by rjbs code review.

Pod::Html: assign directly to array ref

... rather than to the array which is referenced.  (Corrected response
to rjbs comment in
https://github.com/Perl/perl5/pull/18977#discussion_r671734180)

Pod-Html: Simplify the sub which processes options

Pod::Html::Util::parse_command_line() was doing too much.  It should
wrap around GetOptions(), call the usage() routine if needed, then
return a hashref of options to its caller.

Its caller -- Pod::Html::pod2html() -- should do the heavy lifting
involved in processing the options and inserting the data into the
globals hashref.

Hence, we rename Pod::Html::Util::parse_command_line to
Pod::Html::Util::process_command_line() and simplify its functionality.
In lib/Pod/Html.pm we import process_command_line(), call it and pass
its return value onto a new subroutine, process_options(), that
populates $globals.

Document Pod::Html::Util subroutines.

utf8.c: Rmv #undef

This is unnecessary in a .c file, and the code it referred to has been
moved away.

utf8.c: Use macros instead of reinventing them

utf8.c: Refactor is_utf8_char_helper()

Now that the DFA is used by the only callers to this to eliminate the
need to check for e.g., wrong continuation bytes, this function can be
refactored to use a switch statement, which makes it clearer, shorter,
and faster.

The name is changed to indicate its private nature

Make macro isUTF8_CHAR_flags an inline fcn

This makes it use the fast DFA for this functionality.

is_utf8_valid_partial_char_flags: Use DFA

The DFA macro for determining if a sequence is valid UTF-8 was
deliberately made general enough to accommodate this use-case, in which
only a partial character is acceptable.  Change the code to use the DFA.

The helper function's name is changed to indicate it is private

utf8.c: Rmv EBCDIC dependency

There are new macros that suffice to make the determination here.

utf8.c: Rmv an EBCDIC dependency

This is now generated by regcharclass.pl

Check off 5.35.2 in release schedule

perldelta: make note of "switch" leaving the v5.36 bundle