[perl #77680] add pointers to a few other perl called subs

briefly document DB::lsub

test and briefly document DB::goto

[perl #77680] perlsub pointers to sub names used by perl itself

Tony Cook: convert from a diff to a commit

perldelta for 5f39160d

[perl #79074] make sure HeSVKEY_force() preserves UTF8ness (pvn)

HeSVKEY_force() is only used in two places in core.

In the first case, the key is always stored as a SV (when handling tie
magic, since NEXTKEY can only return a SV)

The second case is in B::HE, but I don't see a way to create a B::HE object
from a hash.

clarify that the new rand() isn't cryptographically secure

so people won't read too much into the change

Note new blead customizations to JSON::PP and Test::Simple

perldelta - Fix syntax errors

Remove the 'first-come' status from Porting/Maintainers.pl

This has outlived its usefulness and we have agreement on p5p and irc to
drop it. Many thanks to the CPAN authors of those modules formerly listed
as 'first-come' for being sufficiently responsive to patches sent upstream!

perldelta - Update new/changed diagnostics section

Also tweak some formatting in perldiag.pod itself.

perldelta - Update new/changed documentation section

perldelta - Fill in a TODO item for a module, which I forgot earlier

perldelta - New test scripts

perldata.pod: Mention what version introduced kv slices

op.h: Describe entersub and rv2cv flags

Entersub ops can turn into rv2cv ops at compile time.  It is important
that possible flag conflicts are handled properly.  Since it took me
a while to figure out that there were no bugs here, here are my own
notes, cleaned up and nicely formatted.

Use dev version for JSON::PP

Fix precedence issue in JSON::PP

Use dev version for Test::Builder et al.

It has changes in blead that are not on CPAN yet.  The number
0.98_05 is already used on CPAN.

op.c: Warn on "return $a or $b" [perl #59802]

Add a warning for the (likely) unintended use of "return $a or $b"
(and similar expressions), which perl parses as "(return $a) || $b"
(which is effectively just "return $a;").

Note this warning is triggered by some modules (e.g. Test::Builder).
These are not fixed by this commit.

Signed-off-by: Niels Thykier <niels@thykier.net>

Test::Builder: Fix "return $a or $b" precedence issue

perl parses "return $a or $b" effectively as "return $a;", which is
not what was intended.

Signed-off-by: Niels Thykier <niels@thykier.net>

perldelta for e9827b374

Remove -lbsd from optional libs on GNU/Hurd too

sv.h: Make BmUSEFUL the same type on debug/non-debug builds

408dc2ec1c7 made in an IV (signed) on debugging builds but a UV
(unsigned) on regular builds.

perldelta - Module upgrades

perldelta - A little copy-editing

perldelta - Restore 525553cafc, accidentally undone by e7f2d65136

ExtUtils::ParseXS: bump version of related modules

9b2d7314c3d60 bumped $VERSION in ParseXS.pm from 3.22 to 3.23,
but left the other .pm files at .22. This stops ExtUtils::ParseXS from
loading, so bump all the other modules in this distribution too.

perldelta for 429a25554/#28380,#114024

perldelta for b846c0c86e5

perldelta for e15ca23a92

Increase $ExtUtils::ParseXS::VERSION to 3.23

There has been another change (e15ca23a9) since the last CPAN
release.  Would that cmp_version.t could catch this!

perldelta for ad39f3a27

perldelta for eaaaaa32882752

perldelta for fdc185567

known_pod_issues: Add Devel::CallParser

perldelta for 6ddcf93b

perldelta: two more to-do items

perldelta for 8c9d3376fbf

perldelta for dd2a7f9048

perldelta for 7601007

perldelta for 65c68e1718

perldelta for ffdb8b167e/#3643

perldelta for 25502127/#115768

perldelta for cbd58baf59

perldelta: A to-do item

perldelta for 66230c862b

perldelta for 25375124d/#119311

perldelta: correct re description

perldelta for 35995e5c

perldelta for 232af1f839

perldelta for regexps and 64-bit strings

perldelta for 25fdce4a16

perldelta for 83694831

perldelta for c70927a6ffc3ca

perldelta for #119161

perldelta for bf1b738b6/#118931

perldelta for d4a823b3

perldelta for d4a823b39

perldelta for bbfdc870/#118691

perldelta for ce0d59fdd1c/#7503,#118753

perldelta for 1b268608/#118753

perldelta for 4f62cd62/#118693

perldelta for 3a6ce63

perldelta for 25451ceff7

perldelta: missing fullstop

perl5200delta: Devel::NYTProf is fixed

5.06 works with bleadperl.

Reduce false positives for @hsh{$s} and @ary[$s] warnings

This resolves tickets #28380 and #114024.

Commit 95a31aad5 did something similar to this for the new %hash{...}
syntax.  This commit extends it to @ slices and combines the two
code paths.

The heuristics in toke.c can easily produce false positives.  So the
op is flagged as being a candidate for the warning.  Then when op.c
has the op tree available, it examines it to see whether the heuristic
may have been a false positive.

This avoids bugs with qw "foo bar baz" and sub calls triggering
the warning.

The source code is no longer available for the warning, so we recon-
struct it from the op tree, skipping the subscript if it is anything
other than a const op.

This means that @hash{$foo} comes out as @hash{...} and @hash{foo} as
@hash{"foo"}.  It also meeans that @hash{"]"} is displayed correctly
instead of as @hash{"].

Commit 95a31aad5 also modified the heuristic for %hash{...} to exempt
qw altogether.  But it did not exempt it if it was preceded by a tab.
So this commit rectifies that.

This commit also improves the false positive detection by exempting
any ops returning lists that can get past toke.c’s heuristic.  I went
through the entire list of ops, but I may have missed some.

Also, @ slices on the lhs of = are exempt, as they change the context
and are hence actually useful.

dump.c: Dump SLICEWARNING flag for kv[ah]slice, too

I accidentally omitted this in commit 95a31aad5.

Move ‘Scalar value better written as’ tests from toke to op

I am about to move the warning itself from toke.c to op.c.  Moving the
tests, too, makes them easier to find for future maintainters who see
it in op.c.

op.c: Factor out common varname code

Upgrade Encode from version 2.54 to 2.55

Remove a __BORLANDC__ stowaway

Support for __BORLANDC__ was axed (actually, chainsawed) in 378eeda70c.

Fix the VC6 build on Windows following commit 3be8f09452

The VC6 compiler has slightly flaky support for __int64 and chokes on util.c
when using the 64-bit implementation of drand48, so switch it to the 32-bit
version instead.

fix O_CREAT without O_TRUNC in cpan/autodie/t/utf8_open.t

Awaiting upstream application as [CPAN RT#87237].

Test kv slices in places that expect %{}

op.c:leave_scope: use mg_free before sv_force_normal

This is part of ticket #119295.

Commit 7fa949d (5.19.3) allowed copy-on-write with compile-time con-
stants.  That caused this to fail:

use Variable::OnDestruct:
{
 my $h = "foo";
 on_destruct $h, sub { warn defined $_[0] ? $_[0] : "undef" };
 $x++; # prev statement must not be last in the block
}

It prints undef instead of foo.

It turns out this is much older:

use Variable::OnDestruct;
{
 my $h = __PACKAGE__;
 on_destruct $h, sub { warn defined $_[0] ? $_[0] : "undef" };
 $x++; # prev statement must not be last in the block
}

This one prints undef starting with 5.17.3 (a6d7a4ac1).

But even before that, this prints undef:

use Variable::OnDestruct;
{
 my $h = \1;
 on_destruct $h, sub { warn defined $_[0] ? $_[0] : "undef" };
 $x++; # prev statement must not be last in the block
}

In all these cases, the scalar is getting undefined before free magic
triggers (Variable::OnDestruct uses free magic).

Usually when a scalar is freed, the magic is triggered before anything
else.  When a lexical scalar is ‘freed’ on scope exit (cleared for
reuse on scope entry), the order is different.  References, globs and
copy-on-write scalars become undefined (via sv_force_normal) before
magic is triggered.

There is no reason for the order to be different here, and it causes
unpredictable behaviour (you never know when you will or will not have
a cow).  So change the order in scope exit to match regular freeing.

#ifdef DEBUGGING, not QDEBUGGING, in regcomp.c

[Merge] Key/value hash and array slices

Document the %hash{$scalar} warning

Fewer false positives for %hash{$scalar} warning

Instead of warning in the lexer, flag the op and then warn in op.c,
when the op tree is available, so we don’t end up warning for actual
lists or for sub calls.

Also, only warn in scalar context, as in list context $hash{$scalar}
and %hash{$scalar} do different things.

In op.c we no longer have easy access to the source code, so recon-
struct the hash/array access based on the op tree.  This means
%hash{foo} becomes %hash{"foo"}.  We only reconstruct constant keys,
so %hash{++$x} becomes %hash{...}.  This also corrects erroneous
dumps, like %hash{"} for %hash{"}"}.

Instead of triggering the warning solely based on the op tree, we
still keep the heuristic in toke.c, so that common workarounds for
that warning (e.g., {q<key>} and {("key")}) continue to work.

The heuristic in toke.c is tweaked to avoid warning for qw().

In a future commit I plan to extend this to the existing @array[0] and
@hash{key} warnings, to avoid false positives.

pp.c: Allow 64-bit stack offsets in kv[ah]slice

Improve new slice docs in perldata.pod

perldata.pod: Capitalise =head3 consistently

perldata.pod: consistent spaces after dots

pp.c:pp_kvhslice: remove unused var

Increase $B::Deparse::VERSION to 1.23

Increase $Opcode::VERSION to 1.26

Add new opcodes to Opcode.pm

Sort MANIFEST

Use diag_listed_as two suppress two diag.t failures

Add two missing perldiag entries

Hard-code op name in ck_exists|delete errors

The reason is manifold:
• ck_exists and ck_delete are each only ever called with one op type,
  so we can hard-code the op, which takes less code.
• The messages in perldiag are more helpful if they include
  the op name.
• Changing them in op.c too avoids the need for
  diag_listed_as clutter.

(I know, two of these are not listed in perldiag yet.  I’m get-
ting to that.)

Suppress a warning in kvhslice.t

The bugginess of the ‘Scalar value better written as...’ warning
applies to %new{silce} syntax, too.

update manifest

short documentation for key/value hash/array slices

test index/value array slices

test key/value hash slices

index/value array slice operation

kvaslice operator that imlements %a[0,2,4] syntax which
result in list of index/value pairs. Implemented in
consistency with "key/value hash slice" operator.

scalar value %x{'a'} better written as $x{'a'}