Merge the refactored S_hsplit() and Perl_hv_ksplit() and related cleanups.

Perl_hv_ksplit() now calls S_hsplit(), avoiding duplicating the code to
split hashes when they grow.

In Perl_hv_common(), call S_clear_placeholders() directly.

This saves one call to HvPLACEHOLDERS_get().

Clarify why hv_common() tries to clear placeholders before calling hsplit().

Which makes me realise that if we clear placeholders, we may be able to
avoid the need to split at all.

(In fact, as hv_common() only adds one key to the hash, under the current
definition of DO_HSPLIT() which only considers total number of keys,
clearing any placeholder is going to be enough to drop the total number of
keys, and so no longer trigger the split. But we'll leave the code making a
second check, to avoid a tight coupling with the internals of DO_HSPLIT().)

In S_hsplit(), replace a for with a do/while, as the loop runs at least once.

Seems pointless to check the exit condition before any iterations, when we
know that it will always be false the first time.

Replace the bulk of Perl_hv_ksplit() with a call to S_hsplit().

The code duplication that introduced hv_ksplit() as a fork of hsplit() back
with commit 72940dca186befa0 in Sept 1996 is finally healed.

Tweak S_hsplit() to return early if there are no keys to process.

This mimics the behaviour in Perl_hv_ksplit().

Also remove a vestigial comment. The code it relates to was removed in
commit 7dc8663964c66a69 in Nov 2012.

Pass the current and desired hash sizes to S_hsplit().

Whilst this is slightly more work for its existing two callers, it will
permit Perl_hv_ksplit() to also call it.

Use STRLEN for the parameters, and change a local variable from I32 to
STRLEN to match.

Move the code handling allocating a new buffer earlier in Perl_hv_ksplit().

This makes the rest of the code of Perl_hv_ksplit() closer to that of
S_hsplit().

Refactor the loop logic in S_hsplit() and Perl_hv_ksplit() to converge.

Making the code as similar as possible will make it simpler to merge the two.

Move the call to hv_clear_placeholders() from hsplit() to hv_common().

The relevant code calls Perl_hv_clear_placeholders() at split time, if there
are still placeholders left over from a (previously) restricted hash.
There are two callers to S_hsplit(), one from the regular HV code, and one
from the shared string table code. As the shared string table can't contain
placeholders, only the other call site could trigger this condition, so move
the code there. This simplifies S_hsplit(), and will make splitting the
shared string table marginally faster.

ignore some files from Module-Pluggable

Upgrade Module-Pluggable to v4.7

Preserved old makefile as some of its logic is in Build.PL now which we dont use.

Note CPAN release for Data::Dumper

Update Data::Dumper version and release date

Update Data::Dumper changelog

Fix redeclaration compiler errors on EBCDIC

This patch was posted in
http://markmail.org/message/pwjxbxnlazvxgsyw

Get regcomp.c to compile under EBCDIC

There were a couple of syntax errors here.

The committer modified the patch slightly based on a p5p exchange.

Add, fix comments

Unicode/UCD.pm: Fix undef bug

This only happens should Perl be compiled on the very first Unicode
release, which is extremely unlikely, but fix it anyway.

unwrap #define argument list for REXEC_TRIE_READ_CHAR

Apparently this breaks the z/OS 1.13 c preprocessor. (Yay, now I can say I have written code for z/OS!)

makedepend.SH: Deal with continuation lines

makedepend was failing to take into account the fact that preprocessor
directives, like any line, may be continued onto succeeding lines by a
trailing backslash.  This led to errors in the C preprocessing, which were
suppressed on all but os390 systems.o

This commit alleviates that, but it is not a complete solution.  It
deletes C comments, but this being sed, it doesn't catch all nuances of
C comments.  We only care, however, about comments on preprocessor
directive lines, and this is good enough to work on the current Perl
without generating any errors, on my system.

Spotted by John Goodyear.

makedepend.SH: Preprocessor directives don't have to start at column 1

Like, apparently, the person(s) who originally wrote this script, I
thought all preprocessor directives had to start in column 1.

It turns out that restriction was relaxed in C89; so that they may be
indented.

Spotted by John Goodyear.

makedepend.SH: Make compiler errors show correct line number

Instead of inserting "#line 1", insert "#line 2" to account for the
inserted line.

Spotted by John Goodyear.

Move (?[]) doc to perlrecharclass

This is a form of bracketed character class, and so should be documented
in the same pod.

Document \s change for VT, commit 075b9d7d9a6d4473b240a047655e507c8baa6db3

perlintro: Typo and stop verbatim wrapping

perlhacktips: Note existence of regen/unicode_constants.pl

Handle undef values in magic.t on VMS.

888a67f6b95603701 added a new test that failed on VMS because it
expected the value of an undefined element in %ENV to be an empty
string, whereas on VMS it's actually a one-byte string containing
a null.  This is made possible by the fact that the logical name
API uses strings of known lengths, and made necessary by the fact
that a zero-length value (equivalence name in VMS parlance) is
not allowed.

So patch up the test implementation to handle this special case.

add array DELETE and EXISTS to perlfunc

[perl #116579]
These methods weren't listed in the list of methods for tied arrays

Mark v5.17.9 as done in the release schedule

Remove trailing white-space from RMG document

PATCH: [perl #116899]: jump on uninitialised value

The culprit had nothing to do really with the accuesed commit.
The function S_cl_or() tries to take the union of the code points
matched by its two inputs.  Both of those nodes must be ANYOF-like
(for bracketed character classes and synthetic start classes).  These
come in two flavors, one having extra fields in the struct after the
other one's.  That is used for locale ANYOFs and the synthetic start
class.  The first paramter to cl_or() is always one of these extended
ANYOFS, but the second parameter may be the shorter form

The function was failing to check if the second one was the longer form
before reading data from beyond the short-form's struct.  This could
cause a segfault, but that wasn't the symptom here.  Instead it copied
that data to the other paramter's struct.  valgrind had set that data to
indicate it was uninitialized, so when later it was accessed, we got
this error.

During much of the 5.17 series until the failing commit, more ANYOF
nodes were the larger size.  I presume that is why this commit showed up
the problem.

regcomp.c: Rmv comment ref to obsolete regnode

Abolish STRANGE_MALLOC. Now all malloc()s are considered strange :-)

STRANGE_MALLOC was added in 5.002 beta 1 (4633a7c4bad06b47) as part of an
work around for typical mallocs, which had a bad interaction with perl's
allocation needs. Specifically, repeatedly extending an array and then
creating SV heads (such as when reading lines of a file into an array)
could end up with each reallocation for the array being unable to extend in
place, needing a fresh chunk of memory, and the released memory not being
suitable for use as more SV heads, so sitting unused. The solution was for
perl to recycle the old array body as SV heads, instead of returning it to
the system, passing the memory from the the AV code to the SV code using
offer_nice_chunk(), PL_nice_chunk and PL_nice_chunk_size.

STRANGE_MALLOC was actually a signal that the malloc() didn't need
protecting from itself, and to disable the work around.

offer_nice_chunk(), PL_nice_chunk and PL_nice_chunk_size were removed by
commit 9a87bd09eea1d037 in Nov 2010, without any ill effects, hence the
code used when STRANGE_MALLOC was *not* defined is essentially doing extra
work for no benefits.

From the lack of problems reported, one can assume that in the intervening
15 years malloc technology has got significantly improved, and it is probably
better to be honest with it, rather than trying to second guess it.

Hence remove all the non-STRANGE_MALLOC code, and leave everyone using the
much simpler code. See also
http://www.xray.mpe.mpg.de/mailing-lists/perl5-porters/2005-11/msg00495.html
http://www.xray.mpe.mpg.de/mailing-lists/perl5-porters/2013-01/msg00126.html

perlfunc: clarify that "do $file" isn't really like "eval `cat $file`"

Closes RT#116713.

Update Maintainers.pl for Safe 2.35

Upgrade to Safe 2.35 from CPAN

Simplify the defintiion of optimization flags in Windows makefiles

The setting of -O1 for release mode builds can be factored out of the
x86 and x64 cases following commit 688ea162f4.

PATCH: [perl #116787] -Ox on Win64 Visual C 64 bit is flawed, switch to -O1

-Ox is missing -GF and -Gy and -Gs
http://blogs.msdn.com/b/branbray/archive/2005/07/08/437078.aspx
http://msdn.microsoft.com/en-us/library/8f8h5cxt%28v=vs.90%29.aspx
http://msdn.microsoft.com/en-us/library/59a3b321%28v=vs.90%29.aspx

So the remaining question is, use -O1 or use -O2 on VC 64? VC 32 bit uses
-O1, and that is also slightly faster on x64 too.

add a missing 5 to the release announcement template

Amend 'new perldelta' advice for Win32

test_porting is supported as target for both gcc and VC++ builds

win32/Makefile since 247c55c00b852ec1da004aff2c0e2d4b07af55ab

win32/makefile.mk since 106cf76e392294bc0fe10a3efa306d011f3b2e3c

Also skip io/eintr.t on MidnightBSD

Bump to v5.17.10

New perldelta template for v5.17.10

Update epigraphs with the one for v5.17.9

Finalise perldelta with updated module information

Add new release to perlhist

Updated Module::CoreList for v5.17.9

Updated ACKNOWLEDGEMENTS in perldelta

suppress "experimental" warnings for my $_

put an experimental warning on lexical topic

update release schedule for a volunteer

Remove more perldelta boilerplate

Perldelta note for undeprecating my $_

Un-deprecate the support for lexical $_

Remove boilerplate from perldelta

[perl #72892] Perl should support PERLmmnnLIB environment variable

This patch to perlrun.pod expands the documentation of the directory
hierarchy searched under the locations specified by PERL5LIB.  It now
explicitly mentions the version- and architecture-specific directories
added, and also notes that $Config{inc_version_list} directories are
added.  These additions were suggested by Damon Atkins.

Silence irrelevant stderr output from op/magic.t

Some 'set' commands in this test are intended to verify that an environment
variable is not set, which causes the command to send a warning message to
stderr.

Fix failing op/magic.t test 157

Change 888a67f6b9 added manipulation of __NoNeSuCh2, but it gets
accidentally picked up by the 'set' command looking for __NoNeSuCh in test
157 because 'set' will display the value of *all* variables whose prefix
matches the name given to the command.

Eliminate 'swap' from struct regexp_internal.

It's been unused since commit e9105d30edfbaa7f in July 2009.

Remove the check for SVt_BIND from SvOK().

The original plan to use SVt_BIND to implement read-only aliases to
read-write values is unlikely to happen. More importantly, it's not going
to happen within a maint branch, so there's no reason to have the code
"just in case" it does. The code can easily be re-instated in blead if it
is needed in future. Nothing on CPAN is relying on this code. (Almost no
code on CPAN even references SVt_BIND.)

This effectively reverts part of commit 1cb9cd5016282146 from Dec 2006.

[perl #116865] Upgrade to Thread::Queue 3.02

Attached patch updates Thread::Queue to v3.0.2 in preparation for a
CPAN release.

Adds a new method to Thread::Queue to dequeue items with a timeout
feature.  This addition was suggested by Andreas Huber.

Signed-off-by: Chris 'BinGOs' Williams <chris@bingosnet.co.uk>

re/charset.t, re/fold_grind.t: Note C locale can match 128-255

Craig Berry pointed out that an implementation can have the C locale
give meanings to the above-ASCII code points.  (I had thought this was
an error.)  Change label name and comments in these .t files
correspondingly.

re/charset.t: Make sure locale is loaded at compile time

Commit 569f7fc5d4ec06501b46a72075ff434fe1bf4332 caused this test file
to start failing on VMS.  The reason is that the test for locale became
run-time.  This patch restores the test to compile time, by using the
'if' module.

The reason 'if' wasn't used in the original patch was to reduce reliance
on possibly untested modules, but it makes things easier.  We hope to
get a resolution on p5p on which test groups in the suite can use things
like 'if', but since 5.17.9 is due out momentarily, I'm using 'if' for
now so that VMS will pass its tests again.

Update Win32 to CPAN version 0.46

  [DELTA]

    0.46    [2013-02-19]
            - add Win2012/Win8 detection (thanks to Michiel Beijen) [rt#82572]
              [perl#116352]

Add some text on the interoperability of Moose & Moo to perlootut

Add a note to the future that they should read the latest perlootut

Run podtidy on perlootut

Rename PL_interp_size_5_16_0 to PL_interp_size_5_18_0.

Re-order intrpvar.h to minimise holes in the interpreter struct.

Holes were created by commit f59909ab8dad6ceb (April 2012) which removed
PL_reginterp_cnt, commit 7dc8663964c66a69 (Nov 2012) which removed
PL_rehash_seed_set, and commit 8936b48a49448f4e (Dec 2012) which removed
PL_glob_index.

There is still an unavoidable U16 sized hole on the default threaded
configuration on x86_64. (U8 if PERL_SAWAMPERSAND is defined).

Fix [perl #116523] Configure is confused by '=' in output from uname -a

Previously, Configure used to try to post-process the output of uname -a.
This was to support Xenix, which gave output like this:
    sysname=XENIX
    nodename=whatever
    release=2.3.2 .. etc...

Configure attempted to parse that output and build up a $myuname variable
that resembled those generated on traditional Unix systems.
More recently, a user reported uname -a output on a CVS checkout
of NetBSD that also included information such as

       root=username@hostname:/dirname module=src branch=HEAD

The Configure code misparsed that and ended up throwing most of it away.
This patch takes the simplest approach and just skips the attempt to
process the Xenix output.

This is based on commit 05a827803ab3908a795df94f6ed3890e24a6df2 in the
metaconfig repository.

Silence encoding.pm deprecation warnings in the core test suite

Updated Changes for Module-CoreList

perldelta for #116322 and #112244

dist/IO/IO.xs: Silence compiler warning

This variable is unused, doesn't need to be declared.

Tests for [perl #116322]

PATCH: [perl #116322]: getc() and ungetc() with unicode failure

ungetc() had no knowledge of UTF-8.  This patch adds it.

The committer fleshed out the author's code to make a patch, making
a few small changes.

Some tests for [perl #112244]

Leon Timmermans is short of tuits right now, so I added these tests for
his patch.  Since I don't really know what I'm doing here, other tests
should eventually be added by someone who does know.

PATCH [perl #112244]

:crlf currently doesn't fall back on :pending the way :perlio
does when the unread data doesn't fit into its own buffer. Instead it
just rejects them. This patch resolves that.  Tests are coming in a
future commit

The committer added a cast to get it to compile on Win32, and silence a
gcc warning on Linux

start making entries for 5.19 releases

Document d8a821fe in perldelta

[perl #116809] Upstream status in corelist

First patch explains in corelist output that deprecation means that
module will be CPAN-only.
Second patch adds new option, -u (upstream information). Shows if
module is primarily maintained in perl core or on CPAN and bug tracker
URL. Module::CoreList already contains this data, only output will be
added.

Signed-off-by: Chris 'BinGOs' Williams <chris@bingosnet.co.uk>

Document 721fe537 in perldelta

Document 0f87678 in perldelta

Document encoding deprecation in perldelta

Update Encode to CPAN version 2.48

  [DELTA]

  $Revision: 2.48 $ $Date: 2013/02/18 02:23:56 $
  ! encoding.pm
    t/Mod_EUCJP.pm t/enc_data.t t/enc_eucjp.t t/enc_module.t t/enc_utf8.t
    t/encoding.t t/jperl.t
    [PATCH] Deprecate encoding.pm
    https://rt.cpan.org/Ticket/Display.html?id=81255
  ! Encode/Supported.pod
    Fixed: Pod errors
    https://rt.cpan.org/Ticket/Display.html?id=81426
  ! Encode.pm t/Encode.t
    [PATCH] Fix for shared hash key scalars
    https://rt.cpan.org/Ticket/Display.html?id=80608
  ! Encode.pm
    Fixed: Uninitialized value warning from Encode->encodings()
    https://rt.cpan.org/Ticket/Display.html?id=80181
  ! Makefile.PL
    Install to 'site' instead of 'perl' when perl version is 5.11+
    https://rt.cpan.org/Ticket/Display.html?id=78917
  ! Encode/Makefile_PL.e2x
    find enc2xs.bat if it works on windows.
    https://github.com/dankogai/p5-encode/pull/7
  ! t/piconv.t
    Fix finding piconv in t/piconv.t
    https://github.com/dankogai/p5-encode/pull/6

Remove two spaces after periods to keep the document consistent

Remove two superfluous commas in perlobj

Very small tweak to perlobj in dereferencing method call example

Replace Mouse with Moo in perlootut

utf8_heavy.pl: Add 'use re "/aa"'

This would speed up compilation and execution by a trivial amount, but
makes sure that no regular expression even tries to compile in Unicode,
which could cause recursive invocation of this.  Before this construct
was invented, special care had to be taken to keep that from happening;
those measures remain in effect, but this gives added insurance.

Add Kent Fredric to AUTHORS

mg.c : revert ENV{x} = undef behaviour to be empty string, not key deletion

pod/perldelta.pod: document reversion of ENV{foo} = undef behaviour  in delta
t/op/magic.t: add a test for ENV{foo} = undef

Added Lucas Holt to AUTHORS

[perl #116785] libc_r was removed from recent versions of MidnightBSD and older versions work better with -pthread

This corrects build errors with threading enabled on MidnightBSD 0.4-CURRENT

Signed-off-by: Chris 'BinGOs' Williams <chris@bingosnet.co.uk>

[perl #116587] Data::Dumper intermittently fails tests on 5.8.*

In a nutshell, we don't care enough about 5.8 to try really hard to fix
(obscure) vstring problems there. But we care enough not to make DD fail
its tests because of such a corner case. We don't want to screw all
users that are trapped in the last decade.

Thus, skip certain tests pre-5.10.

In warnings.pm, delete a hash slice, instead of using a loop.

Deleting a hash slice compiles 5 fewer ops, and executes 21 fewer than
looping over the keys to delete each in turn. Whilst this is arguably a
micro-optimisation, it does not increase obfuscation and is in code loaded
by nearly every Perl program, so feels worthwhile.

perldelta for 012528a99df1906d6fe.

Implement useversionedarchname in configure.com.

If this ever becomes the default, we should really remove the
versioned subdirectory from archlib that we've always had.
As of now, using this feature on VMS gives us redundancies such as:

archlib='perl_root:[lib.VMS_IA64-5_17_9.5_17_9]'

Allow VMS features to differ from Perl's defaults.

Since we now enable some features that are not the CRTL's defaults,
we should allow them to be explicitly disabled in the environment
if someone wants the old behavior.