Don't use locale definitions unless within scope

Prior to this patch, stringification of an NV used the current locale's
decimal point character, even outside the scope of a 'use locale'.  This
is contrary to the documentation (though one example in perllocale
omitted the 'use locale') and can lead to unexpected results.

There was one test in the core that relied on the old behavior, and
maybe more in CPAN.  This patch is being made early in 5.19 to see what
breaks.  I do believe though that any breakage is trumped by the
principal that locale rules should only be used if locales are
explicitly requested.

PATCH: [perl #108378] [perl #115800]

This patch solves two tickets.  Both are a result of the stringification
of a floating number being sticky, so that the character representing
the decimal point may be from an old locale.  The patch solves this by
not retaining the stringification of NVs.

perllocale: Nits

regcomp.c: Reorder tests to avoid throwing away work.

Prior to this patch, we did work based on a test that could be thrown
away as a result of the next test.  This reorders so that doesn't happen

Set Config's *extensions and *_ext vars correctly for Win32 static builds

When building with ALL_STATIC=define on Win32, we arranged for Encode's
sub-extensions to get built (in win32/config_sh.PL), but neglected to list
them in the relevant Config variables, causing test failures in
t/porting/FindExt.t and lib/Config/Extensions.t. Moving the special logic
for Encode into win32/FindExt.pm fixes that.

Fix a related test failure regarding Sys-Syslog (not built on Win32) while
we're at it.

Update HTTP-Tiny to CPAN version 0.031

  [DELTA]

  0.031     2013-06-16 23:18:18 America/New_York

    [FIXED]

    - Fixed bug receiving 0-length content bodies

Update HTTP-Tiny to CPAN version 0.030

  [DELTA]

  0.030     2013-06-13 11:46:15 America/New_York

    [FIXED]

    - Requests with the empty string as body content no longer generate
      'content-type' and 'content-length' headers.

Update Getopt-Long to CPAN version 2.40

  [DELTA]

  Changes in version 2.40
  -----------------------

  * Fix bug #86095 Possible problem with aliases

  * Fix bug #85362 typo fix

Small corrections in perlsyn, less one identified by tomc++.

For: RT #118495

Update Unicode-Collate to CPAN version 0.98

  [DELTA]

  0.98  Sat Jun 15 19:44:06 2013
    - typo (see [rt.cpan.org #85655] typo fixes)

pad.c: Correct comment added by dd2155a49b

pp.c: pp_ref: remove null checks

There is no way a null can get on to the stack here.  The only piece
of code that puts nulls on the stack is pp_coreargs, which doesn’t do
that for functions with a _ prototype.

perlfunc: mention that cho(m)p resets the iterator

doop.c: Correct introductory text

do_chomp has not been in that file since 81745e4ea46c8.

pp.c: Eliminate unnecessary dORIGMARK from pp_anonlist

This extra protection was added in commit 44a8e56aa0 or d976ac8220
(I’m a little confused as to the history here).

At the time, av_make really could reallocate the stack.  That changed
in commit efaf36747029.

Now there is no need for dORIGMARK and ORIGMARK.  (ORIGMARK is more
expensive than MARK since it looks up PL_stack_base and recalculates
where we are on the stack, based on a recorded offset.  MARK simply
stores a pointer to where we want to be.)

op.c: Don’t reallocate a temp for lvalue funcs

There is no reason why keys, substr, pos and vec that can occur in
lvalue context need to reliquish ownership of the scalar in the
scratchpad used to return the value in rvalue context (the TARG) and
then have a new one allocated.

Also, the new TARG was marked PADMY, so, if the lvalue op got freed
(e.g., if it was inside if(0){...}), then its pad slot would remain
unusable by code subsequently compiled in the same sub, since pad_free
only turns off the PADTMP flag, not the PADMY flag, and it would hence
appear to pad_alloc to be still in use.  So it would waste a small
amount of memory in rare cases.  (Based on reading code, not on a
gdb session.)

op.c: add assertion

Update ExtUtils-MakeMaker to CPAN version 6.68

  [DELTA]

6.68 Fri Jun 14 23:26:11 BST 2013
    No changes from 6.67_05

6.67_05 Thu Jun 13 21:52:46 BST 2013
    Doc fixes:
    * RT#86007 - Restore meaning for divorced sentence

6.67_04 Mon Jun 10 20:18:25 BST 2013
    Bug fixes
    * Address RT#85406, where specifying 'meta-spec' in
      META_[ADD|MERGE] would remove all prereqs (bingos)

6.67_03 Wed Jun  5 22:03:28 BST 2013
    Doc Fixes
    * Document how to specify meta-spec in META_MERGE (bingos)

6.67_02 Sun Jun  2 18:27:45 BST 2013
    Bug Fixes
    * Allow v-prefixed version strings once more (bingos)
    * Typos fixed (David Steinbrunner)
    * Resolve test failure with latest CPAN::Meta

6.67_01 Thu Apr 25 21:03:58 BST 2013
    Doc Fixes
    * Change references to makemaker.org in the docs
      (Reported as RT#83246 by dolmen)

    VOS Fixes
    * 'core' files are keep files (*.kp) on vos, adjust the
      'clean' target to account for this (Paul Green)

    Win32 Fixes
    * Increase dmake MAXLINELENGTH to 800000 (RT#77215) as
      per kmx's recommendations (kmx)

    Cygwin Fixes
    * Allow linking of Cygwin libraries (Reini Urban)

note that \s matching \cK is marked experimental

perlexperiment: version of acceptance for \N{name}

perlexperiment: ithreads seemed accepted in 5.8.0

perlexperiment: long doubles are not experimental

reorder experiments by version (mostly)

the code that used hstat, istat, qstat was removed in 6fd8c33a

tboffset hasn't been used since 7196fc2f

prevent a couple of porting test issues under PERL_UNICODE

cmpVERSION.pl: get_file_from_git() was returning a character string
which was then passed to MM->parse_version(), implicitly using
PerlIO::scalar to parse the content.

test_bootstrap.t complains since some of the test files aren't valid
UTF-8.

remove reference to experimental code never in core

...at least not anywhere I can find it

Fix inclusion of Compress/Raw/Bzip2 and Encode in MinGW ALL_STATIC build

The build was previously failing with various undefined references because
there were some clashes in the names of the object files extracted into
STATICDIR (e.g. Compress/Raw/Bzip2 and Compress/Raw/Zlib both have object
files called compress.o).

The fix is to individually extract each static extension library and
archive its object files into the static perl library, rather than doing
them all together, and also to use ar's "quick append" mode rather than
"replace" mode otherwise we still won't get multiple object files with the
same names included in the static perl library as required.

note warning categories in perlexperiment

perldelta for the exec warning category change

fix category on "Statement unlikely to be reached"

The warning is documented as an "exec" category warning, but the
implementation mixes checking for exec and syntax.  This makes
only the exec category apply.

Eliminate macro for OpenVMS debugger [perl #118447].

We need to do whatever the configuration says we are doing via
$Config{dbgprefix} and/or $Config{usevmsdebug} for extension
building to work, so it makes sense for the top-level build to
also base what it is doing on the configuration choice rather
than on the special invocation of MMK with /MACRO=__DEBUG__=1.

So this patch makes the top-level build do what we've configured
to do and eliminates the instruction to use the macro.

Possessive and non greedy quantifier modifiers are mutually exclusive

When I added support for possessive modifiers it was possible to
build perl so that they could be combined even if it made no sense
to do so.

Later on in relation to Perl #118375 I got confused and thought
they were undocumented but legal.

So to prevent further confusion, and since nobody has every mentioned
it since they were added, I am removing the unusued conditional
build logic, and clearly documenting why they aren't allowed.

Don't ignore signals on pclose

This is a bug, per POSIX

Merge the re-ordering of when in the build write_buildcustomize.pl is run.

write_buildcustomize.pl is now run as part of the rule that builds miniperl.
Hence if you've built miniperl, you've now always got lib/buildcustomize.pl
too, which simplifies Makefile dependency rules.

write_buildcustomize.pl is now more robust against errors, and miniperl will
abort if a corrupt lib/buildcustomize.pl is found.

None of these changes should have any visible effect outside of the build.

In miniperl, if lib/buildcustomize.pl fails to load, exit with an error.

Unlike sitecustomize.pl, if lib/buildcustomize.pl exists, it should not fail.
If it does, it should be reported immediately as a build failure.

This makes the build process more robust.

write_buildcustomize.pl now test loads the generated lib/buildcustomize.pl

And deletes it if it encounters an error whilst loading it. A non-functional
lib/buildcustomize.pl will cause the build to fail with seemingly unrelated
errors. Deleting it and exiting with an error should make the cause of build
failures obvious.

write_buildcustomize.pl no longer writes to STDOUT

write_buildcustomize.pl now opens lib/buildcustomize.pl itself, instead of
writing to STDOUT and relying on the Makefile to set up redirection. This
means that an empty lib/buildcustomize.pl is not created if
write_buildcustomize.pl fails to compile (for whatever reason), and permits
write_buildcustomize.pl to delete (or attempt to delete) the output file if
it detects an error.

Hard code the output file name (lib/buildcustomize.pl), as it's the same on
all platforms, and @ARGV is already used to optionally pass a directory for
write_buildcustomize.pl to change to before running.

Experimentation suggests that various make utilities don't delete a file
created by redirection even if an error occurs. Hence this should be more
robust.

Add -f to the miniperl commandline when running write_buildcustomize.pl to
avoid reading in any existing lib/buildcustomize.pl. write_buildcustomize.pl
doesn't need the setup provided by lib/buildcustomize.pl, and running it
might cause errors which prevents writing out a correct version, making an
incomplete build harder to recover from.

Remove the Icwd Makefile macros as lib/buildcustomize.pl now sets this up.

Now that lib/buildcustomize.pl is built at the same time as building
miniperl, it will always have added paths to Cwd into @INC. Hence there's no
longer a need for Makefile macros to do this.

On Win32 we can't eliminate $(ICWD) completely as it's also being used for some
invocations of the (real) perl binary. Only miniperl loads buildcustomize.pl
to set up @INC to include paths for the initial locations of modules such
as Cwd.

Generate lib/buildcustomize.pl at the same time as the miniperl executable.

Doing them together ensures that we always have lib/buildcustomize.pl
available. This simplifies things. The seemingly cranky ordering of having
miniperl notionally depend on lib/buildcustomize.pl, and the rule for
lib/buildcustomize.pl actually also building miniperl permits the rest of
the Makefile to depend on (the obvious) miniperl, not the obscure
lib/buildcustomize.pl

Benchmark.t: fix skip() call

Benchmark.t: make calibration a test

As a temporary measure, make a calibration failure not only a skip but a
failed test too, so I can see whether the real test fails more often in
smokes than the calibration.

do not warn when optimizing away /x{0,0}?+/ and /x{0,0}+/

In c37d14f947f7998211b0455e453160fb7e15b22e Karl fixed an issue
reported in [perl #118375] "5.18 regex regression Quantifier follows nothing in regex"
but he fixed only the non-greedy modifier mentioned in the ticket,
and did not include support for the other quantifier modifiers like the
non-greedy possessive (stupid but not illegal), and the possessive
(useful) modifiers.

Hopefully this covers them all.

Note that because Karl already included support for m/x {0,0} ?/x
I have done so as well for the new cases. I do not necessarily endorse
the idea that it is legal or should be tested for. I am inclined to
think that '{0,0}?+' should be indivisible even under /x.

perldelta: add entries gleaned from commit messages

Corrections to 0511665b4b

- require './test.pl' in a BEGIN block, rather than use './test.pl'.
- Ensure the tests are done in the same program that emits the plan()
  otherwise one copy of test.pl thinks tests were planned but not done
  and the other thinks tests were done without being planned.
- Fix indentation.

converted t/win32.t/system* to test.pl style

Benchmark.t: consistent delta comparing

when we compare the number of iterations done in 3 seconds with
3 * (the number of iterations done in 1 second), the comparison's
effective delta of toleration depended on which value was larger:
if a > b, it tested for a/b <= 1.666; if b > a, it tested for b/a < 1.4.

Make it consistently 1.4 (or 1+$DELTA to be precise).

Benchmark.t: disdplay calibration diag output

When the notorious test 15 fails, show the ratio of our earlier 3sec and
1sec calibration in the diag output.

release schedule: fix years for 5.20

Oops.

Merge branch 'delta-deprecate' into blead

This replaces the %deprecated hash in Module::CoreList with a
Module::CoreList::TieHashDelta, making it easier to keep track of
what changed where, and easier to skim the source to see changes.

make Porting/corelist.pl emit delta-style %deprecated

add Mike Doherty to the AUTHORS file

Module::CoreList: changelog for delta %deprecated

re-add the deprecation data for soundex and checktree

Make %Module::CoreList::deprecated a delta hash

This hash tracks core module deprecations, which becomes very
verbose if listing out the deprecated modules for every perl
version. Instead, we can track the delta for each release,
using Module::CoreList::TieHashDelta to inflate as needed.

The depcrecated hash matches the current data in blead, which
might not be correct.

Fixes #118215: https://rt.perl.org/rt3//Public/Bug/Display.html?id=118215

perldelta: Correct misstatement

I forgot that two of the case changing functions were already documented;
so the change in 5.19.1 is to add documentation for more.

Branch prediction hint for SvREFCNT_dec

When decrementing the refcount on an SV, we have to branch on whether or
not we've reached a refcount of 0. But a quick instrumentation shows
that in virtually any program small or large, the number of decrements
without freeing greatly outweighs the number of decrements that cause a
free. Therefore, it's a net win to suggest to the compiler to emit a
branch prediction hint to go into the just-decrement branch.

Remove magic literal constant in favor of named constant

For clarity only

updates to release schedule

...still needs more updating and scheduling

Merge the refactoring of toke.c's S_force_word.

The C level changes are invisible to anything outside of toke.c.
Nothing should notice this change.

Inline a subset of S_force_word() into the KEY_format section of Perl_yylex().

In code handling formats, Perl_yylex() calls S_force_word() at a point where
it has already done half the work that S_force_word() does. The validation
Perl_yylex() has already passed, along with the normalisation performed by
S_scan_word() mean that all it actually needs from S_force_word() is the
token forcing. Inlining these lines decouples the code.

toke.c: Remove the allow_initial_tick hack from S_force_word.

Over the years, every caller which used this hack had it progressively
turned off. Prior to this commit, only one call remained, which
ostensibly handled this case:

format 'STDOUT = ...

However, turns out that even there it was superflous, since a scan_word
a dozen lines before will've already turned all ticks into double
colons.

Eliminate the last call to S_force_word() passing allow_initial_tick as TRUE.

Turns out that that final place using the allow_tick hack could get a tick,
because it was using the original buffer, rather than the already processed
identifier from scan_word.

Test that C<format ::Foo> is identical to C<format Foo>

This wasn't being explicitly tested.

Test that C<format 'Foo> is identical to C<format Foo>

When declaring a format, using a leading package separator requires careful
handling in the parser, to avoid confusion with a subroutine of the same
name.

prevent lib/File/stat.t from aborting at END on cygwin

cygwin appears to be unique in that it uses fast_abs_path() for
abs_path() and vaguely follows POSIX semantics on directory permissions.

Previously the tempdir created would be chmod() to 0600 (non-executable)
and fast_abs_path() would croak when it couldn't chdir into the
directory.

perldelta: typo

perldelta: ">" was missing its "C<"

pod/perldelta: Add note about documenting toLOWER() etc

Quantifier follows nothing in regex

This changes the code so that any '?' immediately (subject to /x rules)
following a {0} quantifier is absorbed immediately as part of that
quantifier.  This allows the optimization to NOTHING to work for
non-greedy matches.

Thanks to Yves Orton for doing most of the leg work on this

correct omission of File::CheckTree and Text::Soundex from deprecations

perldelta for debugger @_ fix

Fix the mutability of @_ in perl -d.

With a test.

See Father C.'s comment on RT #118169.

Fix for RT #118169

https://rt.perl.org/rt3/Public/Bug/Display.html?id=118169

subroutine arguments are no longer shown in perl debugger ( x @_ in perl
-d) .

Regression from perl-5.16.x due to the perl -d refactoring. Fixed with a
test.

Benchmark.t: test for inconsistent clock

test 15 has been failing intermittently in smokes for ages now.
This does countit(3, ...) and countit(1, ...) and checks that the first
takes approx three times longer than the second.

This commit adds in near the beginning a crude timing loop that directly
uses times() rather than anything in the Benchmark module, and that does
approx 1s and 3s of loops, and if the results aren't consistent, sets a
global flag, $INCONSISTENT_CLOCK, that causes timing-sensitive tests to be
skipped.

For now it only skips test 15. If this is successful, I'll look to
expand it to other failing tests like 128/129.

On Win32, i_rpcsvcdbm should be 'undef' as there is no <rpcsvc/dbm.h>

3 canned Win32 config files had i_rpcsvcdbm as 'define', and consequently
their generated headers had I_RPCSVC_DBM defined. This symbol means that
a header file <rpcsvc/dbm.h> exists and should be included. However, it's
only used by ODBM_File, which is not built on Win32, so nothing ever noticed
the error.

The errors in the header files seem to date from commit 0a753a764065f226
("[inseparable changes from patch from perl5.003_23 to perl5.003_24]" in
Jan 1997), which added the Win32 port. The config file, win32/config.w32,
has i_rpcsvcdbm as 'undef' but the header file win32/config.h is
inconsistent, with I_RPCSVC_DBM defined.

The errors in the canned configs seem to date from commit 68dc074516a6859e
("[inseparable changes from match from perl-5.003_93 to perl-5.003_94]" in
March 1997), where the value of i_rpcsvcdbm was changed in win32/config.w32

RMG: need to update Module::CoreList again after blead release

perldelta for recent BUILD_STATIC/ALL_STATIC changes

perldelta: DB has not been removed

perldelta for double quantifier warnings (688e039)

perldelta for File::Spec->tmpdir and changing env (#88940)

Increase $File::Spec::VERSION to 3.41

to match the version number increases in some of the submodules and
to make the perldelta entry easier to describe.

perldelta for PL_stashcacheh and glob-to-glob assignment (210fdecd4)

perldelta for freed stashes (#117941)

perldelta for sort/require CORE:: (#24482)

perldealt for delete local $ENV{x} leaking (9332b95f7)

perldelta for (?[]) fixes

perldelta for #117595 (-l warning)

perldelta for deparsing ${#}{foo} (#117531)

perldelta for JS in perltrap

perldelta for #117311/B::Deparse -l and FF

perldelta: typo

perldelta for lexsub syntax errors (3a74e0e282c)

perldelta for two lexsub fixes

perldelta for constant inlining (70e5f2b5)

perldelta for deparsing for my $x (#117081)

perldelta for 3036c853/\N{} deprecations

perldelta for #118159 (SvTRUE on ""/1 dualvar)

perldiag: Document ‘Can't call mro_isa_changed_in’

perlfunc: Document fallback to "top" format

perl 4’s perl.man says this:

By default the top‐of‐page format is "top", but it may be set to the
format of your choice by assigning the name to the $^ variable.

This is still the case, but is completely undocumented in perl 5.

In perl 5 the presence of a <FILEHANDLE_NAME>_TOP format will over-
ride "top", as will $^, but "top" is still checked for in the
absence of those.

This should be documented.  One thing I do mention here that perl 4
omits is that it is "top" in the current package.

I also tweaked the wording of the preceding sentence.