[MERGE] add+use PL_curstackinfo->si_cxsubix field

Makes determining context at runtime faster (e.g. last statement in a
sub).

perldelta for PL_curstackinfo->si_cxsubix

Perl_gimme_V(): assume caller always provides cxt

So we don't need to check whether (cxstack[cxix].blk_gimme & G_WANT),
just use it. Replace the check with an assertion.

add Perl_gimme_V() static inline fn for GIMME_V

This function makes use of PL_curstackinfo->si_cxsubix to avoid the
overhead of a call to block_gimme() when the context of the op is
unknown.

add PL_curstackinfo->si_cxsubix field

This tracks the most recent sub/eval/format context pushed onto the
context stack. Then make dopopto_cursub use it.

The previous value is saved in the cxt struct, and is restored whenever
the context is popped.

This adds a tiny overhead for every sub call, but speeds up other
operations, such as determining the caller context when returning a
value from a sub - this has to be dpne for every sub call where the last
expression is context sensitive, so its often a win.

add dopopto_cursub() macro to pp_ctl.c

short for dopoptosub(cxstack_ix), a common idiom in that file.

set VOID on OP_ENTER

The OP_ENTER planted at the start of a program (and possibly elsewhere)
gets left as UNKNOWN context rather than VOID context, due to op_scope()
not honouring the current context.

Fixing this makes things infinitesimally faster.

VanL is now a perl author

update perl5-porters list information

perldelta: Add note that documentation has been added to perlapi,perlintern

Add check for malloc_usable_size (Richard Leach)

perldelta for 2cb5a7e8af11

(perl #125557) correctly handle overload for bin/oct floats

The hexfp code doesn't check that the shift is 4, and so also
accepts binary and octal fp numbers.

Unfortunately the call to S_new_constant() always passed a prefix
of 0x, so overloading would be trying to parse the wrong number.

Another option is to simply allow only hex floats, though some work
was done in 131894 to improve oct/bin float support.

regcomp.sym: Fix comment

The length of an EXACTish node is the same bits as the FLAGS field in
other nodes; it doesn't "precede the length", as previously claimed.

Merge branch 'embed.fnc' into blead

This series of commits slightly improves XS code name space pollution
from the core, and better documents what is happening.

It adds a new C flag to embed.fnc which doesn't change the pollution
problem, but makes it easier for downstream tools, like Devel::PPPort.

Some of the inline.h functions didn't even have prefixes, contrary to
our policy of having functions visible outside the core have a 'Perl_'
prefix.

The branch also documents a couple of API functions and macros.

And better error checking of embed.fnc entries is now done.

regen/embed.pl: Enforce some flag restrictions

embed.fnc now has been updated to not have these flaws, so can now
enforce them for the future.

regen/embed.pl: Update a branch test

The two flags are now mutually exclusive, so no need to test for both.

embed.fnc: M and o flags don't make sense together

The M flag does what the o flag except add an incorrect warning.  So
remove the 'o' which gets rid of the warning.

embed.fnc: 'm' and 'p' flags don't make sense together

This is a macro, so doesn't have a Perl_ long form function call.
Also forbid 'm' and 'S', but there weren't any current uses of that.

perlapi: Properly document Perl_custom_op_xop()

It requires the prefix and a thread context parameter.

autodoc.pl: Special case macros whose name begins with Perl_

The usage example for these will have the thread context parameter
unless there is no thread context.

embed.fnc: Add some comment about name space pollution

embed.fnc: E, X flags don't make sense with m

Macros can't currently be restricted in scope.

Document IS_SAFE_SYSCALL

perlapi: Improve description of is_safe_syscall()

Two parameters weren't described, nor the warnings category used.

embed.fnc: Add p flag for is_utf8_non_invariant_string()

This is because it has the X flag, which means the function is visible
on ELF systems.

Add embed.fnc entry for ReANY

This inline function should have an entry.

Add embed.fnc entries for TOPMARK, POPMARK

Add Perl_ to SvAMAGIC _off,_on names

These function names need a Perl_ prefix to avoid namespace pollution.

Add embed.fnc entries for Cv inline fcns

and change the name of one.  CvDEPTH shouldn't have a trailing 'p' to
indicate private memeber access.  It may do so internally, but the name
shouldn't indicate that.

Fix up get_regex_charset_name()

This inline function should not be visible outside certain core (and
ext/re) functions.  Make that happen.

inline.h: Move some fcn '{' to column 1

Traditionally, functions in perl have been written so their initial '{'
is in column 1 after the prototype.  This makes grepping easier.  This
commit brings 3 functions in this file into conformance.

Make PadnameIN_SCOPE() name begin with S_

All functions should have a prefix.

Some SvREFCNT_foo are no longer macros

but are inline functions.  They should be listed in embed.fnc.

Strip leading underscore _variant_byte_number()

Names with leading underscores are reserved for the C implementers.

Change name of _utf8_to_uvchr_buf()

A function name with a leading underscore is not legal in C.  Instead
add a suffix to differentiate this name from an otherwise identical one.

Strip leading underscore from is_utf8_char_helper()

Leading underscored name are reserved for the C implementers

inline.h: Change fcn name prefix from S_ to Perl_

This is being done only for those functions that don't have a guard
preventing them from being seen outside of the Perl core.

Talking to Tony Cook, we agreed that this was a good idea for two
reasons:

1) The 'Perl_' prefix does not pollute XS caller's name space.  The 'S_'
   one could be argued that it doesn't much either, but it does more so
   than 'Perl_', and the next reason is the clincher:
2) It allows us to change our minds about whether a function should be
   static inline or not, without affecting callers who use the Perl_
   form, which they would be accustomed to anyway if they're using the
   full name form.

regen/embed.pl: Generally generate an ARGS_ASSERT

Prior to this commit, if you changed a pointer argument to allow it
being a NULL, you had to go in and also remove the function's
PERL_ARGS_ASSERT macro call.  Now, such macros are generally defined
even if they are empty, so you can add such a call to any function, and
it will always compile.  If there is no assertion needed, the macro
expands to nothing.

Thus now, the use of these macros is optional when no assert is needed;
but still required by the porting test args_assert.t when an assert is
needed.

The generation of such a macro (when its use is optional) can be
suppressed for individual entries by the new G flag in embed.fnc, also
added by this commit.

regen/embed.pl: Wrap #if..#endif differently

By surrounding the interior stuff with these in one statement, it makes
it easier to change that interior stuff without having dangling
statements.

This makes no difference in the current generated output

embed.fnc: Mark atof2, atof3 for internal use only

Now that we have Strtod, my_strtod, those should be used instead.

sv.h: SVt_INVLIST is core-only

embed.fnc: Change some 'x' marked fcns to 'C'

The previous commit added the C flag, which is better to use in many
instances than x.  This commit changes the x entries in embed.fnc that
I'm pretty sure would be better off being C, and removes some 'x'
entries for internal only functions

Add 'C' flag to embed.fnc, for "core-only" fcns

Previously the x flag was (mis-)used for this purpose.  Some functions
that are accessible by XS writers shouldn't be used by them, such as
helper functions that an inline public function calls.  This new flag
alows that to be specified, without having to mark the function as
experimental, which could be misleading to later core maintainers

embed.fnc: Clarify comments, and note 'u' flag exceptions

The next release of Devel::PPPort will know how to handle all these
unorthodox functions, so no need to declare them as such, and in fact
they shouldn't be declared so.

mathoms.c: Add comment

t/re/anyof.t: Fix test names

This previously duplicated some boiler plate in the test name

regcomp.c: Fix -Dr bug

If dumping the program and a single range crosses the border between
being in the bitmap and not, the range must be split at the border
because the output has separate text for things in the bitmap vs. those
not.

I'm not sure that there is a situation where this currently occurs, but
it will so with a future commit

regcomp.c: Collapse some code

These case statements are all repeated in the code for bracketed
character classes, and mean the same thing.  That code knows a bunch of
things for optimizing.  No need to duplicate that.  Instead, pretend
these are being called within brackets, and call the code to handle
that case, which will generate the proper ops.  This now follows the
example of Unicode properties which have long been processed by
pretending they are surrounded by [...]

Generalize inRANGE()

I figured out a way to make this work generally.  I've also tested this
vs what some modern compilers do under -O2.  It seems this macro is
slightly better.

Add withinCOUNT() macro and change inRANGE to use it

This uses just one conditional to see if a value is between low and
(low + n).

perl.h: Fix typo in comment

Avoid panic when last s///g is tainted and utf8

RT #134409

In a repeated substitution, where the replacement is an expression,
and when the last replacement value is both tainted and utf8, and
everything earlier has been plain, and the final string is suitably
shorter than the original, a panic resulted:

    sv_pos_b2u: bad byte offset, blen=1, byte=6

This is because when at the end, taint magic is being being added to
the target of the s///, the target SV has already had its buffer updated
with the shorter result string, but still has the pos() magic set which
corresponded to the original longer string (this pos value would, in the
normal flow of things, be reset shortly afterwards).

One quirk of sv_magic(), which adds any sort of magic including taint
magic, is that it always checks for the presence of pos() magic, and if
so, converts the byte to utf8 offset if necessary. This was seeing the
invalid pos() offset and panicing.

The check was added by v5.19.3-111-g25fdce4a16:
    "Stop pos() from being confused by changing utf8ness"
It seems like a bit of hack to recalibrate pos() each time sv_magic()
is called, but I've left that alone (sleeping dogs and all that) and
instead added a hack in the taint code path in pp_substcont to reset pos
before setting taint.

perldelta for 1e1b6d926a0c8361bc2ebd2b6630ac97988335e3

perldelta for 00ea29f1a18d6cb17456df2e33f105a90ddc8c3a

Update Test-Simple to CPAN version 1.302168

1.302168  2019-09-06 07:40:18-07:00 America/Los_Angeles

    - Fix Typo in a Test2::API::Breakage warning (Thanks E. Choroba)
    - Delay loading of Term::Table until needed (Thanks Graham Knop)

1.302167  2019-08-23 14:07:58-07:00 America/Los_Angeles

    - add test2_is_testing_done api method
    - Fix string compare warning

Update Scalar-List-Utils from 1.50 to 1.52

This updates the following modules from 1.50 to 1.52:

List::Util
List::Util::XS
Scalar::Util
Sub::Util

Update Module-Metadata to CPAN version 1.000037

  [DELTA]

1.000037  2019-09-07 18:32:44Z
  - add decode_pod option for automatic =encoding handling

Update ExtUtils-MakeMaker to CPAN version 7.38

  [DELTA]

7.38 Wed 11 Sep 10:01:46 BST 2019

    No changes since v7.37_04

7.37_04 Thu 22 Aug 15:20:34 BST 2019

    Bug fixes:
    - Fix static linking on macOS

7.37_03 Sat Aug  3 12:37:29 BST 2019

    Enhancements:
    - Improved mandoc section detection

7.37_02 Thu Jun 27 11:10:39 BST 2019

    Test fixes:
    - Fix test failures in 02-xsdynamic.t on Android native builds
      Github issue #337

7.37_01 Fri Jun  7 11:21:39 BST 2019

    Test fixes:
    - [rt.cpan.org #129763] fixed test errors with latest Pod::Simple

PATCH: [perl #134405] Compiler warnings in blead

This was caused by functions that should have been deprecated all along,
and now are.  But they are still called in places.  In most instances
one deprecated function is a wrapper for another.  In one instance, the
calling of the function is to just make sure that mathoms.o gets linked
to.

The solution adopted here, some of which was suggested by Tony Cook, is
to use #pragmas to silence the warnings.  By doing this around the
entirety of mathoms.c, future issues will most likely be avoided.  And
there are unlikely to enough future cases outside of mathoms to cause a
problem.

Florian Weimer is now a perl author

Configure: Include <stdlib.h> in futimes check

Needed for the exit function.

README.cn: Fix pod errors

These were introduced in 4d714158728b57246e7015b4f913802c7445a80a

Merge branch 'pull/22' into blead (from https://github.com/Perl/perl5/pull/22)

Replace a few http urls with https

A few link labels are also updated.

Revise URLs in README.ko

1. Remove a dead link
2. Replace http urls with https, when available.

Revise the URLs un READEM.cn and README.tw

- link "the homepage" to www.perl.org instead of perl.com
- modify the description of www.perl.com to stop mentioning O'Reilly.
- replace a few urls with https://

Prefer https for a few important urls.

Since browsers are marking "http" as "insecure", it's generally
looking good to link to a https URL when possible.

Also, since www.perl.com is no longer operated by O'Reilly. It is probably
better to remove the name of O'Reilly from the link label.

Remove a dead link and its label.

prefer https URLs.

bump $Time::HiRes::VERSION

Time::HiRes: implement clock_gettime() and clock_getres() for win32

To make the implementation easier, the guts of gettimeofday() were
moved to a separate function named GetSystemTimePreciseAsFileTime().

[perl #134398]

(perl #133981) fix my stupid mistake

Improve grammar in pod

for  SvPV..._or_null, sv_utf8_downgraded_flags

Remove deprecated flag from sv_nosharing.

I  recently introduced this by mistake.

Merge branch 'ppport fixes' into blead

This branch contains a large number of changes related to the next
release of Devel::PPPort, which will regenerate its API list for the
first time in many years.

Documentation for 20-ish items is furnished, and the fact that some
dozens more are documented outside perlapi is now noted.

There is a new flag, h, to the =apidoc lines that indicates to hide the
documentation from perlapi, but tie it to the current place, like
perlapio.  It would be not too difficult to have such things
automagically pulled into perlapi from these, should that be desirable.

It turns out there were several undocumented conventions between D:P and
the core code, like having a parameter name of 'cast' or 'type' (in a
macro prototype apidoc line) that allowed D:P to generate proper code to
test the macro in the face of this parameter not being a legal C type.
I have documented these conventions, and added a couple more that the
next release of D:P will understand.  One of those is a parameter of
'block' as used in the recently added WITH_LC_NUMERIC_foo macros.  The
usage example in perlapi now displays correctly.

And for the few macros that I didn't add a way to test in D:P, there is
a new flag, 'u' to indicate that they are unorthodox or unconventional
in some way; that is they won't compile as-is.  This is applied, for
example, to the STR_WITH_LEN() macro which returns a comma separated
pair.

One thing that I hadn't realized before is that moving a function to
mathoms when there is no macro allowing it to be bypassed is effectively
deprecating the use of that function.  mathoms can be cleaned out at any
time, and installations may Configure to not even compile it.  I changed
to require the deprecated flag if something is in mathoms with no macro.

In an earlier commit, I had added the M flag as opposed to the m flag,
which previously had done double duty: 1) the function was implemented
only via a macro; 2) the function is a real function, but there is an
(extra) macro that you can use to get the same functionality as the
function.  I did this to help downstream processing.  And it turns out
that D:P needs that too, and needs it for functions in mathoms.  There
really is no need for an m flag with the mathoms (b) flag.  So the
existing ones that have an extra macro to bypass them are changed to use
the M flag instead of m.  This was done in an earlier commit for other
functions in this file.

embed.fnc: Clarify and fix 'b' flag description

embed.fnc: Fix typo in comment

embed.fnc: Correct grammar in comment

embed.fnc: Trivial comment only

Remove M flag from several functions

These do have an extra macro, but it's not relevant to the situation

embed.fnc: Remove experimental status from some internal fcns

These have been around long enough that they aren't going away.

embed.fnc: Restrict the 'ref' function's accessibility

This could otherwise cause name collisions.  It is only used in two .c
files in the core

Note that config.h contains usable API macros

Things like UVSIZE...

Fix documentation of WITH_LC_NUMERIC_foo

Now that Devel::PPPort can handle a macro whose argument is a {...}
block, we can appropriately document these.

Document STMT_START .. STMT_END

Now that we have the 'u' flag we can document these macros and not have
to break downstream processing.

Mark BHK macros as unorthodox

This is because they take a preprocessor token as an argument

Note that STR_WITH_LEN() is unorthodox

Now that we have the 'u' flag for macros, this one returns
a pair of values, so should be marked as such.

Change pod for macros that require "literal strings"

Now that Devel::PPPort  has the ability to handle these, we can loosen
the syntax for clarity.

embed.fnc: Clarify comments, and note 'u' flag exceptions

The next release of Devel::PPPort will know how to handle all these
unorthodox functions, so no need to declare them as such, and in fact
they shouldn't be declared so.

embed.fnc: Add 'u' flag for unorthodox parameter macros

This tells downstream processors, such as Devel::PPPort that compiling
macros like these will be won't work, as they have some issue.  For
example, aTHX_ expands to have a trailing comma.

Refer to CopLABEL_len[_flags] in pod for cop_fetch_label

Document CopLABEL[_len[_flags]]

Note that various items in perlguts are documented

perlguts: Document save_item

sv.c: Note that some SV flags are documented

Remove redundant documentation for PL_parser

Note that various XS macros are documented

Note that AvFILLp is not for public use

Note that PERL_VERSION and kin are documented