Fix misspellings in documentation. Correct spelling of name to 'De
Morgan'.

Committer: Ivan Panchenko is now a Perl author

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

Document and test command-line switch '-?'

Replace 'slave' with 'client'

This commit moves us a step forward on eliminating 'master/slave' terminology.

Its scope is limited to those files within the Perl 5 core distribution which
are not CPAN-upstream.  Within that scope, we leave untouched the use of the
term 'slave' in epigraphs and other literary quotations and its use in data
coming from outside sources (e.g., Unicode code points).

Increment $VERSION in lib/perl5db.pl.  Revisions per rjbs feedback.

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

sv_upgrade: c99 compound literals to initialize xpvav/xpvhv

For both the AV and HV cases, the new body is Zero()ed but then various
struct members are set to non-zero values. Now that we support parts of
c99, it seems more efficient to use compound literals to initailize the
struct members.

With gcc v8.3.0, the compiled function is smaller by 25 instructions.
sv.o is slightly smaller, but the final perl binary size is unchanged.

sv_upgrade: remove redundant AvALLOC(sv) = 0

xav_alloc is a member of the xpvav struct. The instance modified
here would have been Zero()ed several lines previously, so both
sides of this if() branch should be no-ops.

With gcc 8.3.0 the compiled function is about 6 instructions smaller.

use is_deeply instead of eq_ Test::More functions

the eq_set, eq_hash, and eq_array functions in Test::More are
discouraged to use, and is_deeply is recommended. Ref:

https://metacpan.org/pod/Test::More#Discouraged-comparison-functions

The reason for this is that, if the tests fail, is_deeply has much
better diagnostics.

The other thing is that is_deeply is a test function directly, where
eq_hash and such need to be wrapped inside ok(). This is easy to forget
-- proof of this is in Benchmark.t, where we had this code, that did not
test anything:

    eq_set([keys %$got], [qw(Foo Bar Baz)], 'should be exactly three objects');

It is now replaced by:

    is_deeply([sort keys %$got], [sort qw(Foo Bar Baz)], 'should be exactly three objects');

this commit replaces all usage of eq_set, eq_hash, and eq_array in lib/
and ext/ for tests that use Test::More.

One small exception is where a negated test is used; Test::More does not
have is_not_deeply() or such. Test2 has `isnt()` for this, but that is
not in core. In those cases, we still keep using the eq_ operators.

Add last-resort implementation for POSIX::FLT_ROUNDS.

POSIX::FLT_ROUNDS now emulates (more) standard behavior that
reflects the current rounding mode set by fesetround(),
even when compiled with GCC whose FLT_ROUNDS is wrongly fixed to 1.

Also add tests for this.

POSIX: FLT_ROUNDS is now an XS function rather than a constant

POSIX::FLT_ROUNDS used to be an NV constant, but C FLT_ROUNDS is
actually an integer and not a constant (C99 seems to say that
it reflects the current rounding mode set by fesetround()).

Bump version number of ext/POSIX

Add option '-?' as a synonym to -h

-? is a common paradigm for finding the usage of a program.  Prior to
this commit, doing so on perl would tell you it is illegal and suggest
-h.  This commit allows someone using this paradigm  to skip the second
step

Data-Dumper: Fixes for EBCDIC

This changes to properly calculate the control characters on EBCDIC
systems.  There should be no change to ASCII doe.

Comments are updated and corrected to properly describe the EBCDIC
situation.

The tests change to use the character \xB6, as that means the same in
both character sets, and so works on both without special casing.

Bumped Module-CoreList ready for 5.35.8

Bump the perl version in various places for 5.35.8

new perldelta for 5.35.8

recorded 5.35.7 release as done in the release schedule

Recorded epigraph for 5.35.7

Merge branch 'release-5.35.7' into blead

Added 5.35.7 to perlhist

Updated perldelta with changes to core modules

Update Module::CoreList for 5.35.7

added some entries to perldelta after trawling the git log

Updated dual-life modules for 5.35.7

perldelta entry for the fix for unbounded memory consumption for hashes

This is a fix for a bug introduced two months ago in v5.35.5.

Mention builtin in perldelta

Document that all of builtin is currently experimental

dist/IO/IO.xs: Rmv defns that ppport.h supplies

No sense trying to derive them ourselves.

No need to Renew() HvARRAY() to the same size in S_hv_auxinit()

Fix a thinko/missed simplification that should have been part of
commit 53a41f9c2c0671d8:
    Inline the xhv_aux struct in the main hash body

    For now, memory for this structure is always allocated, even though it
    isn't always flagged as being present.

Prior to that commit the Renew() in S_hv_auxinit() had been needed because
the memory that HvARRAY() points to needed to be expanded from "just the
array of HE pointers" to "that, plus space for the xhv_aux struct.

With that commit, the xhv_aux struct was no longer stored in the same
memory block, so there was no need to reallocate storage. Hence the
Renew() should be completely removed. What that commit actually did was
neatly change the parameters to the Renew() call such that it reallocated
storage to the exact same size, which isn't wrong, but is makework. Oops.

Return the "hv with aux" PVHV bodies to the correct arena

Move the assignments to SvFLAGS(sv) after the check for SvOOK(), and use
arena_index for the array index to PL_body_roots. Without both fixes,
del_body() would always be called with &PL_body_roots[SVt_PVHV], which is
wrong if the body was allocated from &PL_body_roots[HVAUX_ARENA_ROOT_IX].

PVHV body handling was changed recently by commit 94ee6ed79dbca73d:
    Split the XPVHV body into two variants "normal" and "with aux"

    Default to the smaller body, and switch to the larger body if we need to
    allocate a C<struct xpvhv_aux> (eg need an iterator).

    This restores the previous small size optimisation for hashes used as
    objects.

as part of changing where memory for C<struct xpvhv_aux> is allocated.

The new implementation uses two sizes of "bodies" for PVHVs - the default
body is unchanged, but when a hash needs a C<struct xpvhv_aux>, it now
swaps out the default body for a larger body with space for the struct.

(Previously it would reallocate the memory used for HvARRAY() to make space
for the struct there - an approach which requires more tracking of changed
pointers.)

That commit was subtly buggy in that on hash deallocation it didn't *return*
hash bodies to the correct arena. As-was, it would return both sizes of hash
body to the arena for the default (smaller) body. This was due to a
combination of two subtle bugs

1) the explicit reset of all SvFLAGS to SVf_BREAK would implicitly clear
   SVf_OOK. That flag bit set is needed to signal the different body size
2) The code must call del_body() with &PL_body_roots[arena_index], not
   &PL_body_roots[type]
   type is SVt_PVHV for both body sizes. arena_index is SVt_PVHV for the
   smaller (default) bodies, and HVAUX_ARENA_ROOT_IX for the larger bodies.

There were no memory errors or leaks (that would have been detectable by
tools such as ASAN or valgrind), but the bug meant that larger hash bodies
would never get re-used. So for code that was steady-state creating and
destroying hashes with iterators (or similar), instead of those bodies being
recycled via their correct arena, the interpreter would need to continuously
allocate new memory for that arena, whilst accumulating ever more unused
"smaller" bodies in the other arena.

I didn't spot this bug whilst reviewing commit 8461dd8ad90b2bce:
    don't overwrite the faked up type details for hv-with-aux

    CID 340472

which is in the same area of the code, but fixes a related problem.

Curiously when reporting CID 340738:

    Assigning value "SVt_IV" to "arena_index" here, but that stored value is
    overwritten before it can be used

Coverity also didn't spot that the code in question was unconditionally
unreachable, and warn about that, or modify its warning to note that.

(For the code prior to this commit SvOOK(sv) at that point could only ever
be false. This is obscured thanks to macros, but all these macros are
unconditionally defined to the same values, so static analysis should be
able to infer that they are actually constant, and reason accordingly.
To be fair, this is *hard*. But not impossible. Hence I infer that static
analysis tools still have room for improvement, and can make "mistakes".)

Ironically, defining PURIFY would hide this bug, because that disables the
use of arenas, instead allocating all bodies explicitly with malloc() and
releasing them with free(), and free() doesn't need to know the size of the
memory block, hence it wouldn't matter that the code had that size wrong.

Don't clear the flag bit SVf_OOK in hv_undef_flags()

Now that HvAUX(hv) is allocated as part of the HV's body, it's imperative
to retain the flag that indicates which size of body has been allocated.

Otherwise, if we clear the flag, sv_clear() believes that the body is the
smaller of the two possible sizes, and so returns it to the correct arena
for that body size. This doesn't result in out-of-bounds memory reads, or
even memory leaks (when full cleanup runs with PERL_DESTRUCT_LEVEL=2),
because all bodies are still correctly tracked. However, without this bug
fixed, code that takes the same hash and repeatedly

1) assigns entries
2) iterates the hash
3) undef %hash

will request one new (larger) hash body for each iteration of the loop,
but each time return it to the arena for the smaller hash bodies, with the
result that the interpreter will steadily request ever more memory for
arenas for larger hash bodies, until memory is exhausted.

Document the HV aux struct free/release logic in hv_undef_flags()

struct xpvhv_aux holds 4 pointers which can point to memory which needs
explicit release - document how hv_undef_flags() is directly responsible
for freeing xhv_mro_meta, indirectly responsible for freeing xhv_eiter,
and by implication can't free xhv_backreference or xhv_name_u.

utf8.c: Rmv duplicate #define.

This code is identical to a few lines above it

Update ExtUtils-MakeMaker to CPAN version 7.64

  [DELTA]

7.64  Fri 17 Dec 15:35:46 GMT 2021

    No changes since v7.63_11

7.63_11 Tue 14 Dec 16:00:11 GMT 2021

    OS390 fixes:
    - Extend prereqs sort to work on EBCDIC

7.63_10 Mon 13 Dec 16:26:49 GMT 2021

    OS390 fixes:
    - Fix dynamic loading

7.63_09 Wed  8 Dec 22:20:53 GMT 2021

    Enhancements:
    - Don't use canned libpth values

7.63_08 Sat 27 Nov 17:28:03 GMT 2021

    Correction:
    - Previous change to ${LDFLAGS) was reverted

7.63_07 Sat 27 Nov 11:34:12 GMT 2021

    Enhancements:
    - Add $(LDFLAGS) when linking binary modules

7.63_06 Wed  3 Nov 01:24:05 GMT 2021

    Bug fixes:
    - Add -rpath when compiling XS modules on macOS

7.63_05 Sat 14 Aug 09:04:08 BST 2021

    Enhancements:
    - Added CPPRUN variable

7.63_04 Wed 30 Jun 15:15:01 BST 2021

    Doc fixes:
    - Describe CCFLAGS’ default

7.63_03 Tue 22 Jun 14:39:32 BST 2021

    OS390 Enhancements:
    - Fix override xs_make_dynamic_lib() for os390

7.63_02 Thu  3 Jun 19:52:03 BST 2021

    Doc fixes:
    - Changed wording for POLLUTE

7.63_01 Tue 25 May 16:22:50 BST 2021

    Bug fixes:
    - Comparing inodes numerically is unsafe

fix struct ioinfo definition for Mingw

The "#ifndef _SAFECRT_IMPL" part made sense only for no longer
supported versions of Visual C++, but
92ca2349051b948d915e3fc721e3cbb97e7271e2 has accidentally exposed it
to Mingw builds.

Mark keywords.{c,h} as linguist-generated in .gitattributes

perlapio: binmode() and apply_layers() require pTHX

These two functions, unlike all the rest,  require the thread context
parameter to be explicitly passed.  This commit makes the  documentation
be accurate.

_MSC_VER is now always >= 1800 (i.e. Visual C++ 2013, aka Visual C++ 12.0)

Minor RMG tweaks

Minor output tweaks to sync-with-cpan

Also fix paren-less calls of subs where a filehandle could be

If a sub exists, it takes precedence over the filehandle
interpretation even without the parens:

    $ perl  -le 'sub foo { "bar" } print foo;'
    bar

Fix function calls being misinterpreted as bareword filehandles

When bareword filehandles are disabled, the parser was interpreting
any bareword as a filehandle, even when immediatey followed by parens:

    $ perl -M-feature=bareword_filehandles -le 'print foo()'
    Bareword filehandle "foo" not allowed under 'no feature "bareword_filehandles"' at -e line 1.

While with the feature enabled, it works and prints the value returned
by the function:

    $ perl  -le 'sub foo { @_ } print foo("bar")'
    bar

As for filehandles versus functions, a space before the parens makes
the difference:

    $ perl -le 'print STDOUT ("bar")'
    bar
    $ perl -le 'print STDOUT("bar")'
    Undefined subroutine &main::STDOUT called at -e line 1.

This fixes the bug by using the already-existing "immediate_paren"
variable to make it consistent when the feature is disabled.

Fixes #19271

Modernise File::Compare a little

 - Move the version to the package line and bump it.
 - Bump use version now that we're using package NAME VER.
 - Drop strict now that we get it from use version.
 - Switch to non-inheritance based Exporter usage.
 - Use defined-or operator.

perltie: Fix variable name in TIEHASH example

Merge branch 'builtin-in-core-libs' into blead

t/op/switch.t no longer needs to avoid overload.pm on miniperl

Fix indirect call of ->import

No need for avoiding overload in File::Copy

Use builtin::blessed() in overload.pm rather than using Scalar::Util

Use builtin::reftype in ext/

Porting/makerel: Don't reinvent a function

It turns out the functionality here duplicates a pre-existing tool,
which this commit converts to use instead.

Porting/makrel: White space only

Porting/makerel: Add xlation of UTF-16 files

Porting/makerel: Avoid hardcoding UTF-EBCDICS

Instead, this uses the fundamental constants and derived values copied
from utf8.h, making things shorter

Porting/makerel: White-space/comment only

Prepare for a future commit that will add a surrounding block.

Porting/makerel: Add use strict/warnings

Porting/makerel Make EBCDIC files writable

Some of them need to be changed after generation

t/TEST: white space only

t/TEST: print Skip msg for EBCDIC skipped tests

mktables: Use builtin::refaddr

Now that this function is available in miniperl, mktables can use it to
avoid a bunch of visually distracting 'no overloading' calls.

mktables: Don't calculate some unused values

These apparently were once needed, but no longer.

Update compile and bind options for z/OS (os390)

This update enables us to build EBCDIC static/dynamic
and 31-bit/64-bit addressing mode Perl. The number of
tests that pass is consistent with the baseline before
these updates, namely:

For blead.31.dynamic.ebcdic
  Configured using -Dusedl

  Failed 98 tests out of 1939, 94.95% okay.
  Elapsed: 1038 sec
  u=18.13  s=6.04  cu=535.69  cs=178.56  scripts=1939  tests=1035071

For blead.64.dynamic.ebcdic
  Configured using -Dusedl -Duse64bitall

  Failed 102 tests out of 1941, 94.74% okay.
  Elapsed: 1057 sec
  u=19.49  s=6.49  cu=543.00  cs=181.00  scripts=1941  tests=1053149

These changes also provide the base support to be able to provide
ASCII static/dynamic and 31-bit/64-bit addressing mode Perl.

Description of changes:

Makefile.SH
  Changes were made to the os390*) case specific part of the code.
  Support was added for the 64-bit DLL path because the original
  only had support for 31-bit.

hints/os390.sh
  This is the largest set of changes:
   - Compilation and Link options were added for ASCII and 64-bit
   - A z/OS specific check was added to determine if the Perl
     code being built is ASCII or EBCDIC. The check works by
     looking at the first character of the shell script to see
     if it is an ASCII or non-ASCII character. If ASCII, then
     the build is deemed to be ASCII. If not ASCII, it is assumed
     to be EBCDIC.
   - Cleanup was performed to remove code for z/OS systems that are
     no longer supported, simplifying the file (e.g.
        "`uname -v`x`uname -r`" in 02x0[89].*|02x1[0-9].*|[0-9][3-9]x*)
     which would only be true on unsupported, very old pre-z/OS systems
   - The compiler has been changed from xlc to c99. Both are available
     as priced features of the operating system, and the c99 compiler is
     a better 'fit' for options processing, being more consistent with
     c99 on other platforms.
   - Suppressing warning messages for CCN3159 were added because the
     1-bit bitfields flagged due to a smaller-than-int data type are
     harmless.
   - Several feature test macros were added to bring the compilation up
     to a modern level to enable Perl to take advantage of capabilities
     available on all supported z/OS 2.4 and up systems.
   - Removed the -Wl,EDIT=NO because debug information is no longer stored
     in an area that will be loaded into memory, but is now stored in a
     NOLOAD section. So - while this does mean that the binary on disk is a
     little 'fat', what is loaded into memory is not, and it means that
     people can have better problem determination tools available even on
     production Perl distributions.

Mike Fulton is now a Perl author

Simplify IV abs operation in pp_abs.

Transforming -iv into "(UV)-(iv + 1) + 1" can avoid signed integer
overflow even if iv was IV_MIN in 2's compelement representation
(as long as iv < 0), so that it makes special treatment for IV_MIN
unnecessary and saves one conditional jump.

Unify SETu() calls in pp_abs.

SETu() is a relatively complex macro and calling it in multiple places
will lead bloating of compiled code.  Unifying them with intermediate
variable may save a few hundred bytes.

Replace SvUV_nomg(sv) with SvUVX(sv) in pp_abs.

Under the conditions SvIOK(sv) and SvIsUV(sv), SvUV_nomg(sv) is
equivalent to SvUVX(sv).  This change will eliminate unreachable code
and may save a few dozen bytes of code size.

Remove unnecessary braces in t/op/array.t

Use builtin::reftype/refaddr in .t files

Use builtin::weaken() in .t files in lib/ + ext/

Use builtin::weaken() in t/

Merge branch 'builtin-ref-utils' into blead

Give blessed() the same TRUEBOOL optimisation that ref() has in boolean contexts

Add builtin::blessed, refaddr and reftype

Test equivalence of 'true' and 'false' to negated statements

The documentation for lib/builtin.pm asserts that 'true' is equivalent
to !!1 or !0, and that false is equivalent to !!0 or !1.  Demonstrate
that this is so.

Per review by Paul Evans in https://github.com/Perl/perl5/pull/19252,
we'll use cmp_ok() rather than is() and test for each of 'eq' and '=='.

PathTools: update bug_tracker location

Unlist Ken Williams as maintainer as requested in
https://rt.cpan.org/Public/Bug/Display.html?id=116342

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

t/perf/optcount.t for pushdefer no longer needs the $debug flag

mktables: Use mnemonic variable names

Spotted by Dagfinn Ilmari Mannsåker

Fix unicore/mktables to avoid any @_ accesses in signatured subs

Merge branch 'do-not-count-tests-if-possible' into blead

add {} around the now two lines introduced by f7f919a0

This turned out to be harmless, as sv_setpvn_fresh() does nothing
when h_addr is NULL, but the call is also unnecessary when h_addr
is NULL, and possibly confusing to future maintainers.

CID 343917

builtin.pm: Fix pod spelling, slight rewording

do not count tests, instead use done_testing()

Having to count tests is annoying for maintainers.

Also, manually updating count tests is problematic when multiple people
are working on the same code; it causes merge conflicts and recounts.

done_testing() is available since Test::More 0.88 which was released in
2009.

This commit changes all tests under lib/ that use Test::More and were
planning the number of tests.

Michiel Beijen is now a Perl author

Misc microoptimizations when dealing with new SVs

In a few places, SVs can be created more efficiently or new SVs
can be assigned to more efficiently. Small changes included:
* Use sv_setpvn_fresh instead of sv_setpvn
* Use sv_mortalcopy_flags instead of sv_newmortal + sv_setsv_flags
* newSVsv_flags instead of newSV + sv_setsv_flags
* sv_newmortal instead of sv_2mortal(newSV(0))
* Remove a SvGROW(sv, 257) following a newSV(257)

Add builtin:: funcs for handling weakrefs

Also, ensure that B::Deparse understands the OA_TARGMY optimisation of
OP_ISBOOL

Mark scripts, modules and tests as linguist-language=Perl

Otherwise GitHub thinks some of them are Raku or Prolog.

Give files autogenerated by regen/opcode.pl the linguist-generated attribute so github hides diffs

Improvements to OP_ISBOOL

 * Apply OA_RETSCALAR, OA_TARGLEX and OA_FOLDCONST flags
 * Handle both 'get' and 'set' magic

fix redundant apidoc warning

fixes #19250

Direct optree implementations of builtin:: functions

Turn builtin::true/false into OP_CONSTs

Add a dedicated OP_ISBOOL, make an efficient op version of builtin::isbool()

Merge branch 'hydahy/fix-gh19249' into blead

For: p.r. https://github.com/Perl/perl5/pull/19251;
issue: https://github.com/Perl/perl5/issues/19249

fix pod error in sv.c. Closes #19249

builtin.c: Fix C++ compilation errors

C++ does not allow `goto` across initalisations of variables that are
in scope at the label, so just get rid of the `goto`.

An alternative would be to start a new scope after the first `goto`,
so that the variables aren't in scope any more at the label, but I
think this is neater.

Add a builtin:: namespace, with true/false/isbool

This finishes the perl-visible API required for RFC 0008
  https://github.com/Perl/RFCs/blob/master/rfcs/rfc0008.md

It also begins the "builtin::" namespace of RFC 0009
  https://github.com/Perl/RFCs/blob/master/rfcs/rfc0009.md

newSVpvn_flags().. is more efficient than sv_2mortal(newSVpvn(..))

The same holds for newSVpvs* wrappers around newSVpvn* functions.

newSVpvn_flags(x, .. ,SVs_TEMP) more efficient than sv_2mortal(newSVpv(x,0))

Add a phase_name() macro.

Also C<codeword> quote "defer" and "goto"

Quote the word "defer" in exception messages about control flow

Forbid using 'goto' to jump into a 'defer' block because everything will break when it's done

update perldsc examples to be strict compliant

Where possible, update perldsc examples to use lexical variables, to be
strict compliant. Earlier code in this document specifically discusses
differences with strict, but the examples aren't really related to
anything about non-lexical variables, so they should be updated to work
with modern standards.

Some parts of the examples have not been updated because they are using
variables that are implicitly meant to come from some an outside source.

add missing brace in perldsc hash of hashes example

Fixup Corelist.pm