Sync Devel::PPPort with CPAN version 3.63

Partially Revert "skip using gcc brace groups for STMT_START/END"

This reverts the portion of commit
6d66b125bb339b62c0d8d4283d3fe576795764ce that modified Devel::PPPort.
It turns out that it broke that module when the module is run on Perl
5.13.0 (and presumably any earlier Perl).

perlapi: Consolidate and revise pod for toCASE_CHANGEfoo()

This finishes the work started by
3cb048e540f681a46e641b466030b55feaf5d337 for toLOWERfoo, using the same
paradigm, for the rest of the case changing macros and functions.

Sven Kirmess is now a Perl author

Bump if.pm version

Put back the old url for unicode.org (in lib/unicore) since there is now a redirection

Refine the changes to perlintro.pod after feedback and discussion.

Following useful feedback and suggestions from Neil Bowers, Jason McIntosh
and Dan Book, try to make the sentances shorter and clearer. Hopefully also
avoid some potential abmiguities about "special variables" not just being
all-punctuation or scalars.

Three small changes to perlintro.pod

Simplify the text about declaring variables with my. The previous text said
both that "you don't have to pre-declare" and "you have to declare them",
where it wasn't clear that the former was about types, and the latter about
variables. The new phrasing tries to be clearer.

Note that special variables can also be "all capitals", not just
"punctuation". Remove the in joke about "line noise" because it adds little,
but many people use it as a negative trope about Perl.

Remove mention of given/when. We should not be promoting something we
intend to change. We should have done this years ago when we made it
experimental.

Add missing word in 'if's documentation

CPAN and Core match at Data-Dumper-2.183

Sync DB_File with CPAN version 1.856

From Changes:

1.856 22 June 2021

   * Ensure that all *.bak files are cleaned up
     https://github.com/pmqs/DB_File/pull/9
     96bcf494625897ab4fcda322481dd6f730bdb668

Merge branch '18780-compound-token-split-by-macro' into blead

skip using gcc brace groups for STMT_START/END

This warns (and warns a lot) on clang, and since these are documented
to only work to make a single statement, so there's little value to
allowing them to work in an expression.

An alternative would be to disable GCC brace groups on clang, but
these are used extensively in DEBUGGING builds to add extra checks
in sv.h.

Haiku BFS does not store atime and always return current time

Increment $VERSION throughout

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

Re-arrange subs in calling order

I.e., first pod2html(), then its internal subs in order of (potential)
call.

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

get working directory on a non existing path returns a value and no errno on Haiku

av_new_alloc: revise description of size parameter

Only set IOKp, not IOK on $) and $(.

Issue #18955: This will prevent serializers from serializing these
variables as numbers (which loses the additional groups).

This restores behaviour from 5.16.

Fix syntax error wwhen NAN_COMPARE_BROKEN

http://nntp.perl.org/group/perl.perl5.porters/26082

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

Use lowercase for lexical variables -- even filehandles

Per rjbs code review in https://github.com/Perl/perl5/pull/18950

After eval-ing $VERSION, assign return value back to $VERSION

Remove superfluous assignment to @ISA.

Per rjbs code review in https://github.com/Perl/perl5/pull/18950

Pod-Html: rename Auxiliary.pm to Util.pm

Per rjbs code review in https://github.com/Perl/perl5/pull/18950

Keep $VERSION in synch

Introduce refine_parser()

Further encapsulation of code internal to pod2html().

At this point we face obstacles:  %Pages; $output.

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

Introduce generate_cache() and identify_input()

Further encapsulation of code to improve readability.

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

Rename one method per suggestion from rjbs

Introduce internal sub set_Title();

Encapsulate more code within pod2html(). The return value is for
convenience.  It's the setting inside $globals that counts.

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

Introduce internal sub refine_globals()

Encapsulate! Encapsulate!

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

Create internal sub parse_input_for_podtree()

This clears up a bit of semantic confusion.  We were using one lexically
scoped variable -- my $parser -- to hold two different objects: one for
parsing input, one for writing output.  We can encapsulate the working
of the input parser to make the code more readable.

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

Remove last hard-tabs from Html.pm

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

Create sub write_file()

We now start placing parts of sub pod2html() into separate subs so that
we reduce the length of pod2html(), making it more readable and
(ultimately) preparing for method calls.

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

Introduce Pod::Html::Auxiliary

This package will hold helper subroutines used within the main package
or in tests.  They can be placed in a separate module and imported into
Pod::Html because they won't depend on having the globals passed as an
argument.  They will also be potentially independently testable.

Start with html_escape().

Move anchorify(), htmlify() to Auxiliary.pm.  Also _unixify -- now as unixify().

Move relativize_url() to Auxiliary.

Move usage() to Auxiliary.

Move trim_leading_whitespace to Auxiliary.

Move parse_command_line() to Auxiliary.

Keep porting tests happy. Increment $VERSION.  Run: ./perl -Ilib
regen/lib_cleanup.pl

anchorify.t, eol.t:  Correct excessive corrections.

Standardize setting of $VERSION.

Start refactoring lib/Pod/Html.pm

Group all the "global" variables into a hashref $globals.  This variable
will be the return value of init_globals() within pod2html(), will be
passed through, and augmented by, parse_command_line() and then used
through the balance of pod2html().

In essence, so far this is just a renaming of variables.

Remove declarations of superseded variables.

Pass $globals to get_cache(), cache_key() and load_cache().

Reduce default arguments for Testing:_prepare_argstable()

Touch-up t/htmlview.t

Introduce Testing::record_state_of_cache()

A method to assist in debugging cache problems.

Should assist in resolving https://github.com/Perl/perl5/issues/12271.

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

perldelta for e1d3ed996ab

add a note to perldoc -f each on using anon hashes/arrays

warn when using each on an anonymous hash or array

We've had three tickets over the years where the user has been confused
by the behaviour of each on an anonymous array or hash, there's no way
to tell if other users have been struck by the same issue, so make it
easier to diagnose by producing a warning.

Eliminate '-Wstring-compare' build-time warning in re_comp.c.

Per discussion with TonyC, use of the 'GCC' versions of two macros here
is appropriate because that version is documented to handle CLANG as
well.  Using these macros preserves an optimization added in d0f83c3 in
2009.

PR GH 18948 for issue GH 17015.

Data::Dumper 2.183 was released on 2021-07-05.

Fix version numbers and attribution in the Data::Dumper changes file.

Neil Bowers spotted that the entries for the two recent development releases
were wrong (my copy paste error).

A couple of previous changes were attributed to me by mistake, because I had
updated the changes file - now fixed. Add attributions for several earlier
changes, based on blead's git history.

Bump Data::Dumper's $VERSION and update Changes, for a dev release.

Change dumper.t to done_testing() from plan() and an explicit test count.

As our minimum perl version is now 5.8.1, that comes with a Test::More that
is new enough to support done_testing(). This avoids frustrations with
cherry-picks between branches ignoring the hunk that updates the test count,
because both "sides" made the same textual change, and hence getting it
"wrong" (make test fails) because both sides meant "+6", not "this new count".

Also fix up some whitespace, and remove a long obsolete XXXif.
(Added by c4cce84862ab583d, but the corresponding close was already removed
by commit 531383e69c1fff07)

Refactor the variable name code in Dumpxs to avoid repeated SvPV* calls.

This fixes a really subtle bug whereby Dumpxs would not recognise variable
names if they were generated by references with string overloading, NVs,
and potentially other strange things. Seems that no-one has ever hit this.

Also eliminate the need for a large temporary scratch buffer by using
sv_setpvf().

In Data_Dumper_Dumpxs, rename name to name_sv.

Bump Data::Dumper's $VERSION and update Changes, for a dev release.

Data::Dumper was erroneously stringifying qr'$foo' as qr/$foo/

qr// vs qr'' only affects whether variable interpolation happens in the
parser. The compiled regex doesn't record which quote style it used. It
turns out that $ always interpolates, unless it is at the end of the string
or followed by ) or |. Otherwise, except in qr'', dollar-anything will
interpolate that punctuation or regular variable. (Possibly violating
strict)

Meaning that if we see an unescaped $ that isn't at the end of the string,
isn't followed by ) and isn't followed by |, then either the regular
expression was written as qr'', *or* there was variable interpolation
where the interpolated value was that literal dollar sign.

We can exploit variable interpolation to generate a regex in qr// form
equivalent to one written without interpolation in qr'' form, which is
very useful as the XS code needs to use \x{} style escapes for non-ASCII
literals in the regex. This approach suggested by Eirik Berg Hanssen.

Bug reported as CPAN #84569

GH #18047: Initial phrasing for =head5 and =head6

Perl_pad_alloc: simplify av_fetch() calls

The nature of PL_comppad allows this.

Add inline av_fetch_simple and av_store_simple functions

State that the -s command line switch is discouraged; mention Getopt::Long

We really should discourage the use of -s.

Also clarify that the variables are created in the main package. It's not
obvious whether this is obvious - implying that it needs saying.

Data::Dumper 2.182 was released on 2021-06-30.

Reformatted Data-Dumper Changes as per CPAN::Changes::Spec

This means that tools can parse this, and in particular MetaCPAN
will list the changes when you look at the dist/release page for
Data-Dumper.

dist/Data-Dumpers/Changes used to be in pod, with a known pod
issue, so that's now dropped.

Updated AUTHORS and checkAUTHORS, as my github account has
an email address not currently known, but it's my prime
email address for Perl stuff.

Dumper.xs: Add comment

Dumper.xs: Output orphaned EBCDIC control as octal

This makes the code simpler, and removes the need to worry about and
comment on EBCDIC.

On ASCII machines there are the C0 controls, the C1 controls, and DEL,
which isn't technically in either set.  The C0 and DEL controls are
treated as low ordinal, and output using octal notation.  This commit
has no behavior changes on ASCII platforms.

On EBCDIC machines, there are 1-1 mappings to the entire set of 65 ASCII
controls.  All but one are in a single block and have been output using
octal.  This commit doesn't change the behavior of the 64 single-block
controls.

There is a lone control that isn't adjacent to the others, orphaned.
This commit's only effect is to cause it to be displayed using octal
instead of hex.  I believe the simplification of the code warrants this
change.

On extant EBCDIC platforms that Perl supports, this control is 0xFF,
named EO or EIGHT ONES, and is somewhat like DEL on ASCII platforms,
which we already display as octal, even though it is much higher ordinal
than any other control displayed as octal.

Dumper.xs: Revise calculation of needed size

This changes the calculation of how large a buffer is needed to use a
loop instead of a chain of conditionals.  I think it is easier to read,
certainly it is tidier, and has the added (small) benefit that it
doesn't give up and choose the worst case scenario for large code
points.

Dumper.xs: isDIGIT() is now in ppport.h

So use it.  This makes the code cleaner, and removes a conditional
(unless the compiler optimizer figures out to do so anyway).

Dumper.xs: Port ESC_NATIVE back on EBCDIC

Wrap its definition with LATIN1_TO_NATIVE() which will automatically
port it back as far as possible.

This is the only use of ESC_NATIVE on cpan, so I don't think it's worth
putting it in ppport.h

Bump Data::Dumper's $VERSION and update Changes, for a dev release.

Report Data::Dumper problems to Perl 5 Github issues queue

In the commit message for 29c66c43dc9b80118284c89a37f090d4f2fada58,
Nicholas Clark wrote:

    Really we should decide *which* bugtracker is canonical for Data::Dumper
    (and will be checked and acted upon) and then record that metadata in this
    Makefile.PL so that [metacpan.org, etc., can] link to it.

Let's provide that information based on the examples of other
blead-upstream distros under `dist`, `dist/Module-CoreList/Makefile.PL`
and `dist/PathTools/Makefile.PL`.

Remove references to Macintosh/Mac OS in perlfunc.pod and perlintro.pod

Both of these were about Classic, not Darwin.

The relevant Mac OS text in perlrun referenced from perlintro was removed
in 2009 by commit 78aea6ff0e955611:
    Remove Mac OS classic instructions from perlrun

utf8_length: Fix undefined C behavior

In C the comparison of two pointers is only legal if both point to
within the same object, or to a virtual element one above the high edge
of the object.

The previous code was doing an addition potentially outside that range,
and so the results would be undefined.

perldiag: numer -> number typo fix

Pod::Simple: synch with CPAN version 3.43

Remove :win32 PerlIO layer

Merge branch 'pod2html-single-tempdir-20210510' into blead

ext/Pod-Html/t/*.t: Have each file run in tempdir

Until now, the tests for this library ran inside the ext/Pod-Html
directory in the core distribution.  Since the tests necessarily create
files while the harness runs, there was a possibility for race
conditions, which meant that we had to have a workaround in t/harness to
have the tests in this directory not run in parallel.

This commit restructures the testing process so that each test program
gets its own tempdir in which to create files and directories.

The test architecture is now found in t/lib/Testing.pm.
This module will export two functions on demand that are used in the
individual test files.  They guarantee that all files created when
running a test program will be created in a temporary directory rather
than underneath the core distribution.

t/pod2html-lib.pl is now superseded by t/lib/Testing.pm and can be
removed.  The workaround has been removed from t/harness.

Change directory name from testdir/ to corpus/.  So as to more precisely
describe what the directory's purpose is.

Keep porting tests happy. Update MANIFEST and known_pod_issues database.

Acknowledge Craig Berry's recommendation in earlier branch

This should help in the resolution of
https://github.com/Perl/perl5/issues/12271.

Correct typo spotted by rjbs.

Copy the test bootstrapping order from t/TEST to t/README

Clarify what the ordering constraints are and why the need to exist.

Avoid some conditionals in is...UTF8_CHAR()

These three functions to determine if the next bit of a string is UTF-8
(constrained in three different ways) have basically the same short
loop.

One of the initial conditions in the while() is always true the first
time around.  By moving that condition to the middle of the loop, we
avoid it for the common case where the loop is executed just once.  This
is when the input is a UTF-8 invariant character (ASCII on ASCII
platforms).

If the functions were constrained to require the first byte pointed to
by the input to exist, the while() could be a do {} while(), and there
would be no extra conditional in calling this vs checking if the next
character is invariant, and if not calling this.  And there would be
fewer conditionals for the case of 2 or more bytes in the character.

Makefile.SH: Fixes for z/OS

This removes two things that are causing problems:
    1) a space in a parameter
    2) DPERL_EXTERNAL_GLOB=''

fix new libperl.t 'no symbols' warning on darwin

A change in October 2019 added handling for the case that 'nm' might
emit the error message 'no symbols' (apparently erroneously. Apple seems
to have included a fix to upstream LLVM to change the message to be a
file reference followed by ': no symbols'. For libperl.t, the message is
typically '../libperl.a:perlapi.o: no symbols'. This change handles both
the old and current behavior.

Add David Marshall to AUTHORS.

perlio.c: Don't wrongly conditionally compile

The code is using '#ifdef EBCDIC' when it has nothing to do with EBCDIC;
but is a proxy for how things work on IBM boxes that use EBCDIC.

utfebcdic.h: Remove obsolete #defines

For a couple of releases now, these have been uncalled

Update hints/os390.sh

This modernizes this hints file enough that it actually works on a real
live system.

hints/os390.sh: White-space/comment only

Dead link on ccl4, not found anywhere else but in webarchive. Use webarchive for l-optperl.html

oreilly.com.tw no longer exists

oreilly.com.tw no longer exists and jp and cn are give right afterwards

Committer: update t/porting/known_pod_issues.dat

Improve perlxs grammar

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

Remove inheritance from Exporter in ext/ modules

Inheriting from Exporter adds several subroutines that are not really
needed by these modules.

The remaining uses of inheritance from Exporter are:

re.pm - this uses export_to_level(), which really, really wants
the inheritance

# Conflicts:
# ext/File-Find/lib/File/Find.pm
# ext/File-Glob/Glob.pm
# ext/GDBM_File/GDBM_File.pm
# ext/Opcode/Opcode.pm
# ext/Pod-Html/lib/Pod/Html.pm

perlnewmod.pod: Remove dead link and make a header line stand out more.

Remove undeclared $Is_VMS from DynaLoader_pm.PL

The variable appears to have been unused since 1c7f9087b1206cc7,
and the addition of strict in f0e0f25d5ce9118d61837a3 makes the
generated DynaLoader.pm fail to compile. But only on VMS.

ensure types of PL_nan and PL_inf are consistent

Different types for a variable across compilation units can in
theory cause undefined behavior.  In this case I think it's unlikely
to cause an issue, but LTO builds also (correctly) complain about
the type mismatch, so ensure they match.

Update perlgov.pod with Steering Council Election Results

Add a "Show Config" step to the GitHub CI config.

Hopefully this will make it easier to figure out the cause of test failures.

Correct grammar/spelling problems

As spotted by Tux

Fix definition of ITEM_NOT_FOUND for pre-1.13 versions.

Raise version number in ext/GDBM_File/GDBM_File.pm

Fix GDBM_File to compile with version 1.20 and earlier

* ext/GDBM_File/GDBM_File.xs (ITEM_NOT_FOUND): Define conditionally,
depending on the GDBM_VERSION_MAJOR and GDBM_VERSION_MINOR.
Don't assume GDBM_ITEM_NOT_FOUND is a define (it isn't since
gdbm commit d3e27957).

Change the "New issue" page to point to the RFC process.

perldelta for fe991866a0

perldelta for 2b96d013a

fix magic and upgraded SV handling for setsockopt()'s OPTVAL

The code here checked SV flags before fetching magic, potentially
getting confused if magic fetched changed flags.

This also fixes handling for upgraded SVs, but I'm not sure that can
be tested sufficiently portably.

perl5db: add tests for v command

bump perl5db.pl $VERSION to 1.61

fix comment.

lib/perl5db.pl: cmd_l() no longer exists.

Subroutine cmd_l() no longer exists.
Now we should call cmd_l_main().

Correct typo in inline comment