regcomp.c: Replace code with fcn that does the same thing

This makes one less place that has to know certain details.  Since the
function is inline, there should be no change in the compiled code

utf8.h: Update outmoded comment

utf8.c: Rename macro and move to utf8.h, and use it in regcomp.c

This hides an internal detail

invlist_inline.h: Two symbols no longer needed in utf8.c

This is because the code using them has been moved to regcomp.c
in cef721997e14497f2fbc4be17ab736ad7ddfda29

Add inline function to hide implementation details

t/re/regexp_unicode_prop.t: Add tests for run-time

User-defined \p{} properties can be referred to in a regex pattern
compilation before their definition is encountered.  This is supposed to
work, and their definitions get compiled when needed at run-time.  But
there was only one test that this worked.  This commit restructures
things so that every user-defined property is compiled into a pattern
before its definition is known, and also into another pattern after its
definition is known.

This removes the need to special case the earlier single one that did
this test.

t/re/regexp_unicode_prop.t: Reorder a few tests

This is based on knowledge of how the code works, to stress it a little
more.

t/re/regexp_unicode_prop.t: Avoid a TEST crash if failure

This changes the logic to not do a 'like' unless we know that the
operand is a valid pattern.

t/re/regexp_unicode_prop.t: Add check no warnings generated

t/re/regexp_unicode_prop.t: Extract code into a function

This is in preparation for it to be called from a 2nd place

t/re/regexp_unicode_prop.t: White-space only

Indent some lines that a future commit will surround with a block,
and outdent others where a block will be removed

regcomp.c: Use isFOO_A instead of isFOO

They evaluate to the same thing, but this change emphasizes that only
ASCII characters can pass.

regcomp.c: Change parameter name

This makes it clearer what variable this length is for

inline.h: Use 'do {} while' instead of 'while {}'

t/re/regexp_unicode_prop.t: Make sure returns true

Handle 3 digit exponents in \p{nv=%e}

Prior to this commit, Windows machines which format %e to have a minimum
of 3 digits would fail in looking up the Numeric value properties, and
would revert to the old swash method of doing it.

mktables: Handle platforms with 3 digit exponents

C99 says there shouldn't be more than 2 digits in an exponent unless
needed.  But Windows uses three.  This messes some stuff up that is
expecting two.  Change to remove leading zeros so that only two digits
are used.  This allows mktables to properly operate on Windows.

Update Time-Piece to CPAN version 1.33

  [DELTA]

1.33    2018-08-18
        - Allow objects in overloaded methods

perldelta for 56191551147fe888e34f15ef84f08013f4fbb5aa

Synch with CPAN Test-Simple 1.302140.

Addressing RT 133382.

perlre: Add some clarifying script run documentation

perlre, perlrecharclass: Add examples

This adds more concrete cases of how mixed script digits can be
hazardous.

Fix script run bug '1' followed by Thai digit

This does not have a ticket, but was pointed out in
http://nntp.perl.org/group/perl.perl5.porters/251870

The logic for deciding if it was needed to check if a character is a
digit was flawed.

regexec.c: Use a macro for clarity

This commit #defines a macro and uses it, which makes the code easier to
understand.

regexec.c: Fix UNLIKELY() scope

The expression as a whole is unlikely to be true, not just the portion
that was marked so.

Releaser for 5.29.2

PATCH: [perl #41202] text->float gives wrong answer

This changes to use Perl_strtod() when available, and that turns out to
be the key to fixing this bug.

S_mulexp10() is removed from embed.fnc to avoid repeating the
complicated prerequisites for defining Perl_strtod().  This works
because this static function already was defined before use in
numeric.c, and always called in full form without using a macro.

James Keenan fixed a file permissions problem originally introduced by
this commit, but the fix has been squashed into it.

perl.h - mingw-w64 builds use __mingw_strtold instead of strtold

There are bugs in strtold().

James Keenan fixed a file permissions problem originally introduced by
this commit, but the fix has been squashed into it.

lib/locale.t - no longer special case quadmath builds

The previous commit now has these respect locale

numeric.c: Move some code around

For readability, this changes

if (f00) {  MANY LINES } else { FEW LINES }

to

if (! f00) { FEW LINES } else { MANY LINES }

numeric.c: White-space only

Remove trailing blanks, add a few blank lines

numeric.c: Quadmath now honors LC_numeric

Previously the code compiled under quadmath did not bother to check for
locale.  Fixing this was a simple matter of a little rearrangement.

(perl #133422) handle Off_t smaller than size_t

time::HiRes: don't truncate nanosec utime

When passed a floating point atime/mtime value, T::HR::utime()
was converting it into two longs: secs and nsec. But the nanosec value
was calculated using a final NV to long cast, which truncates any
fractional part rather than rounding to nearest. Use a 0.5 addition to
force rounding.

This was manifesting as a test in lib/File/Copy.t failing to preserve
the same mtime after a couple of round trips with utime() and stat().

In particular, the test was attempting to set an mtime to the literal
floating-point value

    1000000000.12345

This value can't be represented exactly as an NV, so was actually
(under -Dquadmath)

1000000000.1234499999999999999999999568211720247320

which was (using truncation) being converted into the two sec/nsec
longs:

    1000000000, 123449999

After this commit, it instead correctly gets converted to

    1000000000, 123450000

Upgrade HTTP::Tiny from version 0.070 to 0.076

Upgrade Filter::Util::Call from version 1.58 to 1.59

PA-RISC is obsolted

Itanium in not "new". It is newer than PA-RISC though, and PA-RISC is
getting rarer

Update HP-UX readme

• Porting center released 5.28.0 :)
• EOL matrix link was dead

Time-HiRes/t/itimer.t: avoid race condition.

This test script sets a repeating interval timer going, and after 4
'ticks' (SIGVTALRM), disables the timer (by setting it to zero).

The main loop which does CPU burning, does a getitmer() every now and
again, and when the value is zero, assumes the signal handler has
disabled the timer,  and so finishes.

The trouble was that it was checking the 'time left', which can reach
zero because the interval timer has counted down to zero, and the signal
handler is about to be called, but the interval hasn't been reset back
to 0.4s yet.

i.e. the code doesn't distinguish between "timer disabled" and "timer
just reached zero".

In that scenario, the cleanup code in the test script disables the
SIGVTALRM handler while the timer is still active, and so the process
gets killed if another signal is raised.

This commit changes the test to check the second value returned by
getitmer() for being zero rather than the first - the second being the
repeat interval, whichb is always 0.4 until the timer is disabled.

dump.c: Fix space before tabs

These generate git warnings

Use sv_catpvs where appropriate vs sv_catpv

This moves calculations definitely to compile time; some optimizing
compilers may already do this, but some may not.

perlapi: Fix up pod for utf8n_to_uvchr_error()

There are two return flags signalling that the input UTF-8 was malformed
by being too short.  This commit adds detail comparing and contrasting
the meanings of the two

perlapi: Fix up SV handling pod

The pod for this has gotten screwed up over the releases, with a section
entitled SV-Body Allocation containing a bunch of SV manipulation
functions, and another section with that name containing still more with
no real rhyme or reason as to why one would be in one vs the other.  And
one function was in both places.  This commit consolidates all these
into one section entitled "SV Manipulation Functions"

perlapi: Clarify sv_insert()

regcomp.c: Add comments

perlunicode: Clarifiy user-defined props

perl.h: Use TAINT_get, instead of PL_tainting.

The former is designed to be compilable out.

perl.c: Use TAINT_get, instead of PL_tainting.

The former is designed to be compilable out.

The HP-UX optimizer fails again (on Itanium only this time)

mathoms.c: Use my_strnlen, not strnlen

Spotted by H. Merijn Brand.

Make utf8_to_uvchr() slightly safer

Recent commit aa3c16bd709ef9b9c8c785af48f368e08f70c74b made this
function safe if the input is a NUL-terminated string.  But if not, it
can read past the end of the buffer.  It used as a limit the maximum
length a UTF-8 code point can be.  But most code points in real-world
use aren't nearly that long, and we know how long that can be by looking
at the first byte.  Therefore, use the length determined by the first
byte as the limit instead of the maximum possible.

Revise kludge for [perl #133136]

Lexical::SealRequireHints has a test that is looking for internal Perl
behavior that has changed, and the maintainer has been unresponsive.
Since this test is high in the cpan river, in order to get 5.28
released, a kludge was added that used the old behavior.  But that old
behavior is about to be completely removed, so this does a different
kludge that loads utf8.pm, and that is what the .t is actually
expecting.  This leaves us free to remove the behavior without affecting
the module.

mathoms.c: Make safer a deprecated function

This function is deprecated because it is lacking a length parameter, so
malformed UTF-8 may cause it to read beyond the buffer.  This commit
causes it to not read beyond a NUL character, which makes it safe for
the common case that the input is a C string.

utf8.c: Make safer some deprecated functions

This function is a common function, only called from various deprecated
functions, but they may be moved to ppport.h.  They are lacking a length
parameter, so malformed UTF-8 may cause them to read beyond the buffer.
This commit causes this common function to not read beyond a NUL
character, which makes all its callers safe for the common case that
the input is a NUL-terminated C string.

utf8.c: Avoid some unnecessary work

The code changed by this commit used to check that the input was valid
UTF-8, and if so, calculated the code point, using a fast function
that doesn't do any error checking.

However, the changes earlier in 5.29 make the time spent in checking for
validity hardly less than the time spent in calculating the code point
at the same time.  So this commit switches to calculating the code point
from the start, avoiding a second pass through the byte string.

utf8.c: Remove unused parameters

These functions used to have swashes, but now all have inversion lists
instead, so we can get rid of the code that handled their swashes.

utf8.c: Remove common code

Call another function that incorporates the same code that is removed by
this commit.

mktables: Some tests are invalid

These tests have been wrongly passing.  A future commit will change
that.

utf8.c: Make safer a deprecated function

This function is only called from deprecated functions, but they may be
moved to ppport.h.  It is lacking a length parameter, so malformed UTF-8
may cause it to read beyond the buffer.  This commit causes it to not
read beyond a NUL character, which makes it safe for the common case
that the input is a C string.

utf8.c: Make unused parameter signal its status

Change the string in this string parameter to indicate it is unused,
like the similar calls nearby.

Initialize some inversion lists at start-up.

The time it takes for these is trivial, since they are using compiled-in
Unicode values.

inline.h: Init a variable to silence stupid compilers

In looking at smokes, I noticed some compilers wrongly warn that this
variable is used uninitialize.  It's easiest to humor them.

Avoid extra newlines on VMS in t/opbasic/arith.t

Separate print statements through a pipe with autoflush on cause
extra newlines due a bug in the VMS pipe implementation, and were
causing this test to fail with an "unexpected output" message. So
consolidate into a single print statement.

Avoid extra newlines on VMS in t/op/local.t.

Separate print statements through a pipe with autoflush on cause
extra newlines due a bug in the VMS pipe implementation, and were
causing this test to fail with an "unexpected output" message. So
consolidate into a single print statement.

Move Unicode \p{} definitions to regcomp.c

These are only used in compiling patterns.  They previously were placed
in utf8.c because they are large, and there is a copy of regcomp.c in
ext/re, so they would have use twice the space.

This commit changes things so that they only are used and defined in
regcomp.c, (not re_comp.c) so that duplication does not occur.  They are
accessed only from one function, and that is also moved from utf8.c to
regcomp.c, only compiled in regcomp.c, and referred to as an external by
re_comp.c

I had to change the names of the table.  Previously they started with
'PL_' in case any got exposed, but globvar.t mindlessly assumes that any
such variables in the file regcomp.c are globals, and wrongly complains.
It was easier to just change the prefix to 'UNI_' instead.

A few tables are used in regexec.c, and are duplicated in re_exec.c.
Things could be adjusted so that only one copy is used.  I tried this,
but the tables are far more intertwined in regexec.c functions than
the ones changed in this commit, as only a single function accesses
these.  Thus doing this would be a lot harder, and the payback isn't all
that much.  I started work to make them EXTCONSTs, and then discovered
the intertwining, but left in that work, unused.

ext/POSIX/t/posix.t - Fix 'strtold works' test

t/run/locale.t - Add missing SKIP:{} block

Configure: redirect try.c errors to /dev/null

perldelta for 028f02e7e97a, 3d5e9c119db6 and bbc9927bf24e

win32: fix argumentless sleep()

Before this commit, pause() was implemented as a macro calling
win32_sleep((32767L << 16) + 32767), which was causing the following
problems with argumentless sleep():

- it was sleeping for 4262198 seconds (~50 days) instead of forever
- it was triggering "sleep(2147450879) too large" warning

This commit implements pause() as a proper function which calls
win32_msgwait() with an INFINITE timeout.

[perl #133376]

(perl #133314) always close the directory handle on clean up

Previously the directory handle was only closed if the rest of the
magic free clean up is done, but in most success cases that code
doesn't run, leaking the directory handle.

So always close the directory if our AV is available.

(perl #133314) test for handle leaks from in-place editing

bump $Storable::VERSION

since I made a substansive change to stacklimit

(perl #133411) don't try to load Storable with -Dusecrosscompile

Fix setting GCCTARGET in win32/makefile.mk when CCTYPE is set to GCC

This just copies a piece of logic from win32/GNUmakefile to get GCCTARGET
set when CCTYPE is already set to GCC.

fix version autodetect for Visual C 64bit edition

-replace *= with := in makefile.mk because we dont want to run
 "gcc.exe -dumpmachine" multiple times
-64 bit Visual C has the major version number at 1 "token" less than
 32 bit edition, so the autodetect code was saying VC 2005 64b was
 CCTYPE="MSVC-60" because shell var %i was "00" in the "set /A", VC 2005
 32b wasn't affected, MS when they created 64 bit VC took some liberties
 in changing the version string, rumor has it many legacy code bases do a
 string test for "80x86", so that is why it was never updated by MS. I
 think detecting 64b VC version numbers has been broken since day 1 of
 the autodetect code that I wrote in commit ca30c090c3 . I just never
 tested it on 64b before.
-copy the "info" target from GNUMakefile to makefile.mk for debugging
 reasons, mostly so I can verify this revised autodetect logic works but
 since the target uses alot of CPU/IO/many procs launched to compute the
 version and many cmd.exes are launched for all those echo line, dont by
 default execute the "info" target from "all" target on dmake makefile.mk
 to save build time. Copying "info" target also reduces diff-ness between
 the 2 makefiles. Maybe one day remove running the info target by dafault
 on a target "all" from GNUmakefile too, but not in this commit.
-consolidate the recipe lines for "info" target so less cmd.exe procs are
 launched by the make tools, unroll echo CCTYPE into both branches. For
 me with dmake, the "info" targets execution time went from 398ms to 141 ms
-there was a skip of "VC" version vs "cl.exe" version between VC 2013 aka
 VC 12 and VC 2015 aka VC 14, so at cl.exe ver 19 the skip must be
 accounted for, also starting in VC 2017, the MINOR version number started
 being relevent since MS stopped ticking up the MAJOR version number so
 take the "X" from AB.XY.CD.EF and append it to the version number, the 2nd
 number in "tokens" sets %j var to be that number, but dont look at minor
 number for older VCs since the only old "AB.10" VC compiler, which is
 VC 2003 aka 7.10 and VC 2002 aka 7.00 are treated identically and combined
 to 7.00 for perl purposes
-fixed a bug that a console without gcc and without vc in the PATH
 autodetected to a gibberish MSVC version instead of erroring to the user

Bump Time::HiRes $VERSION since its XS was changed in the previous commit

remove distros from needing ppport.h in core

These changes are towards a goal of removing mkppport one day as part of
a optimization for parallel building (less deps needed before XS
extensions are built). This is a follow-on patch to the dummy ppport.h
trick in Perl #128438 ticket that had issues.

-for Time::HiRes the DEFINE() in Makefile.PL can't be be in sub init()
 because that sub isn't called on windows
-SLU has not needed ppport.h in core since commit 5e99e069f5
 aka 1.47 import from cpan
-Cwd used to be a no ppport.h since 3.25, in 3.30_02 the code (Cwd.xs) was
 deleted/broken under commit 9bc94e3dae, or changes file
 "Remove more special logic required for core perl.". Put back the Cwd.xs
 code and remove Cwd from the ppport.h list

t/porting/manifest.t - Don't strip out MANIFEST.SKIP files

The previous commit's addition of Devel-PPPort's MANIFEST.SKIP caused this
test to fail because it was stripping that file out of the list of files
tracked by git and then complaining that it wasn't in the list of files
tracked by git!

We only have two MANIFEST.SKIP files in core. They're both listed in the
MANIFEST and they're both tracked by git, so nothing special needs doing
here.

Add dist/Devel-PPPort/MANIFEST.SKIP to MANIFEST

Revert "Delete MANIFEST.SKIP from dist/Devel-PPPort for now"

This reverts commit 077685406080f6c6256731ba0cc7c07e8b91e160.

remove all exported symbols from GCC win32 miniperl

Similar to commit a19baa613 and same reasoning. Makes binary smaller/more
chances for optimization but mingw linker is still much less smart than
the VC linker.

Remaining problems are that GCC miniperl.exe still links to WS2_32.dll
(winsock library) yet the win32_* socket functions have nothing calling
them.

GCC 4.9.2 i686-w64-mingw32

miniperl.exe 1660KB before this no exports commit
miniperl.exe 1584KB after this no exports commit

Don't compile S_env_alloc() when not used

RT #133360

I recently added S_env_alloc() as a helper function for
Perl_my_setenv(); but it's not needed on all platforms, so #ifdef it to
only be compiled on platforms where its needed.

Avoids a compiler warning.

Fix trivial compilation warning in Data::Dumper

Apparently a declaration of a static function that's never defined can
count as an unused function.

embed.fnc: Rmv wrong marker that a fcn has docs

This was a copy paste error.  This is a function that should not be
called directly, and isn't documented.

lib/warnings.pm: Bump version to 1.43

lib/Config/Extensions.pm: Bump version to .03

[MERGE] Data::Dumper fixes for older Perl versions

Notably, this restores deparsing support in the XS dumper.

Data::Dumper: restore XS-mode deparse support in older Perls

Data::Dumper: handle incomplete support for Unicode glob names

Before version 5.16, Perl didn't have full support for Unicode in glob
names. This change allows Data::Dumper's tests to pass in Perl 5.8 through
5.14.

Data::Dumper: ensure character-classification macros use U8

This fixes some tests on older Perls.

dist/Data-Dumper/t/dumper.t: clarify some test names

dist/Data-Dumper/t/dumper.t: add missing test name

dist/Data-Dumper/t/indent.t: avoid scalar split

Older versions of Perl yield a warning for this usage.

Porting/epigraphs.pod: fix a typo in an old epigraph

Ensure perlmodlib.pod contains all non-dual-life modules

The pod/perlmodlib.pod file is generated at build time by pod/perlmodlib.PL,
which looks at the POD of library and pragma files to find "=head1 NAME"
directives. It reads each file a paragraph at a time ($/ = ""), and considers
each individual paragraph as potentially containing POD.

However, if it finds no "=head1 NAME" entry in a module, it silently ignores
it. That's reasonable for dual-life modules — there's often a good reason
why a particular file has no POD — but for single-life modules, this could
well indicate an issue.

In particular, commit effd17dc012719d584aa712c6c7bd5dc142885b6, as part of
rearranging the contents of lib/warnings.pm, inadvertently changed the
context of its "=head1 NAME" line so that it's no longer the start of a
paragraph. This had the effect of removing warnings.pm from perlmodlib; that
gap was pointed out by pink_mist++ on IRC.

The fix for that is trivial: add a blank in the appropriate place in
regen/warnings.pl, and regen.

The change to pod/perlmodlib.PL to die on missing "=head1 NAME" lines has
also yielded a second module (Config::Extensions) that was missing from
perlmodlib for the same reason; that's also fixed in this commit.

regen/mk_invlists.pl: Collapse unused boundary values

Each Unicode property that specifies a boundary conditions, like
Word_Break, partitions all the Unicode code points into equivalence
classes.  So, for example, combining marks are placed into the Extend
class, because they are usually used to extend the previous character
and don't stand on their own.  mk_invlists.pl creates a boolean table of
all pairwise combinations of these classes, so that it knows by simple
lookup if the first character is class X and the next character is class
Y, if a break is permitted between these.

However, in some cases the answer isn't as simple as this, and other
means such as the characters in the vicinity of X and Y must be used to
disambiguate.  In these cases the table value in the cell (X,Y) isn't a
boolean, but is some other number indicating some specially crafted code
section to execute to resolve the issue.

Over the years, Unicode has tended to subdivide partitions into smaller
ones, as they've refined their algorithms.  But with Unicode 11, they
used another method and actually removed partitions.  Rather, they
retain the partitions, but no code point actually takes on the value of
an obsolete partition.

In order to not have to change the algorithm unnecessarily between
Unicode releases (who knows, they might change their minds, and
unobsolete these next time), mk_invlists has just kept the tables
around, but those cells won't ever get accessed because no code point in
the current release evaluates to them.

But that makes the tables unnecessarily large.  We can achieve the same
thing by mapping each unused equivalence class to the same value, which
we call 'unused'.  The algorithms that refer to the obsolete partitions
go through the data assigning values to the cells, but now the cells
overlap, since all obsolete classes map to the same row or column.  Thus
the data is total garbage.  But that doesn't matter, since that row or
column is never read by the data in the Unicode release the table is
constructed for.

mk_invlists also can compile older Unicode releases, and this makes
those tables smaller than before, with all unused classes in a
given release collapsed into a single row and single column of (unused)
garbage.

regen/mk_invlists.pl: Make adjacent comment and its code

regen/mph.pl: Add comments

These are from http://nntp.perl.org/group/perl.perl5.porters/251395

Use Unicode 11.0

This completes the process of upgrading to Unicode 11.0.