[perl #107366] Don’t clone GVs during thread join

unless they are orphaned.

This commit stops globs that still reside in their stashes from being
cloned during a join.

That way, a sub like sub{$::x++}, when cloned into a subthread and
returned from it, will still point to the same $::x.

This commit takes the conservative approach of copying on those globs
that can be found under their names in the original thread.

While this doesn’t work for all cases, it’s probably not possible to
make it work all the time.

Simplify gv:S_maybe_add_coresub

It was working around the fact that newATTRSUB expects to be able to
look up the GV by name.

And for speed, it was going through hoops to try to avoid creating
extra SVs holding the name unnecessarily.

By tweaking newATTRSUB to accept a GV instead of a name, we can sim-
plify not only S_maybe_add_coresub but all its callers, too.

sigsystem.t depends on fork().

So don't run it if the function doesn't exist.

Suppress void warning in t/op/sprintf.t

Correct comment in op.c

‘Backward compatibility mode’ is poppycock, I’m afraid.  When part of
an argument list a sub call it may or may not be an lvalue.  That can-
not be determined until the value passed is (or is not) assigned to.
It has little to do with backward compatibility.

[perl #103492] Make %n printf format work with Unicode

It was using the internal byte count instead of the number of charac-
ters.  The iatter is documented.  The former is useless, even for C
code calling this, as later arguments could cause the current buffer
to be upgraded to utf8, throwing off any offsets returned.

[perl #103492] Give lvalue cx to (s)printf args

Or potential lvalue context, like function calls.

The %n format code’s existence renders these two very much like func-
tion calls, as they can modify their arguments.

This allows sprintf("...%n", substr ...) to work.

[perl #103492] Correct docs for %n format in perlfunc

hv.c: Consistent spaces after dots in apidocs

Remove method cache reset from hv.c:hv_free_ent

It hasn’t been necessary since commit f50383f5.  Before that it wasn’t
sufficient.  See commit 5743f2a for details.

If a hash element is being deleted, S_hv_delete_common takes care of
this.  If a hash is being freed or cleared, hv_undef or hv_clear takes
care of it.

squash some code in hv.c:S_hv_delete_common

Changes four commits ago made this possible.

[perl #100340] Free hash entries before values on delete

When a hash element is deleted in void context, if the value is freed
before the hash entry, it is possible for a destructor to see the hash
in an inconsistent state--inconsistent in that it contains entries
that are about to be freed, with nothing to indicate that.  So the
destructor itself could free the very same hash entry (e.g., by
freeing the hash), resulting in a double free, panic, or other
unpleasantness.

mro.c: Consistent use of spaces after dots in apidocs

mro.c: Use F<> for file names

Update method caches for non-void stash elem deletions

Commit 71be2cb (inseparable changes from patch from perl5.003_13 to
perl5.003_14) added code to hv_free_ent to reset method caches if
necessary.

Later, a bug fix in the deletion code (f50383f5) made it necessary for
the value in the HE to be set to &PL_sv_placeholder, so it wouldn’t
free the sv just yet.  So the method cache code (which by then had
changed from using PL_sub_generation to using mro_method_changed_in)
got repeated in S_hv_delete_common.

The only problem with all this is that hv_free_ent was the wrong place
to put it to begin with.  If delete is called in non-void context,
the sv in it is not freed just yet, but returned; so hv_free_ent was
already being called with a HE pointing to &PL_sv_placeholder.

Commit f50383f5 only added the mro code for the void case, to avoid
changing existing behaviour when rearranging the code.

It turns out it needs to be done in S_hv_delete_common uncon-
ditionally.

Changing a test in t/op/method.t to use ()=delete instead of delete
makes it fail, because method caches end up going stale.  See the test
in the diff.

'S' case already exists with ALTERNATE_SHEBANG.

Follow-up to b7e077d0225eac833ce2eb6fe9e1807f77d0f848.

Current VMS can handle filenames with spaces.

If they are escaped and on an ODS-5 volume, that is. We
then have to clean up all versions of the files created
by a test we weren't running before.

sigsystem.t: put miniperl skip earlier

Under minitest, extensions like constant.pm and Data::Dumper
haven’t been processed yet, so they won’t be available.

Correct skip count in sigsystem.t

Allow Data::Dumper to load on miniperl

For a long time it has been doing eval { require XSLoader }, but
XSLoader will load on miniperl, it’s just XSLoader::load() that fails.

Increase $Data::Dumper::VERSION to 2.135_03

I’m about to make a change, and 2.135_02 has already been used by a
CPAN dev release.

Block SIGCHLD during system() call (per POSIX)

Added tests for SIGCHLD blocking during system()

strlcpy fix-ups in vms/vms.c following a35dcc95dd24.

In three places I was using the intended string length rather than
the buffer length as the length argument, and in one place I was
assuming the return value was the number of non-null bytes copied,
which is not true when you are intentionally copying only part of
the source string.  If strlcpy is supposed to be idiot-proof, they
clearly didn't anticipate the superior form of idiot I've proven
myself to be.

Update Version-Requirements to CPAN version 0.101021

  [DELTA]

  0.101021  2011-12-30 15:39:56 America/New_York
          DEPRECATED DEPRECATED DEPRECATED DEPRECATED

          Version::Requirements is now DEPRECATED

          Use CPAN::Meta::Requirements, which is a drop-in replacement.

perldelta up to a65c7ff

perldelta for deparsing "foo::]"->...

perldelta for tying empty %^H (cb1f05e8f)

perldelta for deparsing "string"->[0]

perldelta for deparsing open("string")

perldelta for safe %^H cloning

Test ‘Too late for -[Mm]’

[perl #104288] Die with ‘Unrecognized switch’ on #! line

If a switch such as -x that cannot occur on the shebang line is used
there, perl dies with ‘Can’t emulate...’.

If an unrecognised switch is used on the command line, perl dies with
‘Unrecognized switch...’.

It just happens to use the former error message for unrecognized
switches on the shebang line, which can be confusing:

$ perl -e '#!perl -G'
Can't emulate -G on #! line at -e line 1.

This commit changes it to output the same message as command-line
switches for those that are unrecognised wherever they are used.

Fix bad reference in sv.h’s docs

SvPVx should be referring to SvPV as the faster version, not SvPVX.

Update Maintainers.pl for the CGI.pm update.

Update CPAN-Meta to CPAN version 2.113640

  [DELTA]

  2.113640  2011-12-30 15:19:46 America/New_York

  - Version::Requirements has now been merged as CPAN::Meta::Requirements,
    rendering Version::Requirements itself redundant

[perl #107296] Document File::Glob’s default flags

Increase $File::Glob::VERSION to 1.16

include a package POSIX::SigSet

this is mostly for the benefit of the PAUSE indexer, as described
in the code comment

mktables: Fix bug for early Unicode versions

mktables should be able to be run on any Unicode version.  Prior to this
patch, it was trying to do something unconditionally with the Name_Alias
property, whereas it should only do that on versions that have this
property.

mktables: Change function name

This is in preparation for Unicode 6.1 for which the new name will make
more sense.

mktables: Unicode 6.1 table will be empty

So, change mktables so it is expecting that.

mktables: Add comment

Unicode:UCD: Add comments

Unicode::UCD: Extend prop_invmap() to handle multi-entries per code point

This is in preparation for Unicode 6.1, in which the Name_Alias property
has more than one entry in its table per code point.

Unicode::UCD: white-space only

Unicode::UCD: Allow prop_invlist() to work on non-compact tables

This accumulates adjacent ranges in case the table is not stored
compactly.

Unicode::UCD: Special case prop_aliases() 'indic'

This is in preparation for Unicode 6.1 which adds a property whose name
begins with 'indic', as in Indic languages.  This creates a conflict
with Perl's longstanding use of 'in' as a prefix in a property name.
"\p{infoo}" is a synonym for "\p{blk=foo}".  The conflict is resolved
here with a special case.

Unicode::UCD: Allow prop_invmap() to work on non-compact binary properties

It may be that the tables input to prop_invmap() are not in the most
compact form.  There was a problem coping with these on binary
properties; which this commit fixes by realizing that a binary property
omits its 'Y' property value.

The .t does not pass these tests when the input is not compact, though.

Unicode::UCD: Correct pod

The example is wrong; now the full script names are output, not the
short ones.

Add a test for "n EXPR()".

perl -d: add a test for s EXPR().

This patch involves addding another "perl -d" input script that contains a
subroutine that isn't called by default.

perl -d: add a test to the 's' command.

Conflicts:

lib/perl5db.t

Add a test for s.

perl -d: test for 'T'.

perl -d: add test for the x command.

Add a test for the debugger p command.

Update CGI to CPAN version 3.59

  [DELTA]

  Version 3.59 Dec 29th, 2011

  [BUG FIXES]
  - We no longer read from STDIN when the Content-Length is not set, preventing
    requests with no Content-Length from freezing in some cases. This is consistent
    with the CGI RFC 3875, and is also consistent with CGI::Simple. However, the old
    behavior may have been expected by some command-line uses of CGI.pm.
    Thanks to Philip Potter and Yanick Champoux. See RT#52469 for details:
    https://rt.cpan.org/Public/Bug/Display.html?id=52469

  [INTERNALS]
  - remove tmpdirs more aggressively. Thanks to rjbs (RT#73288)
  - use Text::ParseWords instead of ancient shellwords.pl. Thanks to AlexBio.
  - remove use of define(@arr). Thanks to rjbs.
  - spelling fixes. Thanks to Gregor Herrmann and Alessandro Ghedini.
  - fix test count and warning in t/fast.t. Thanks to Yanick.

Clarify that \Q, \U, \L don't require \E

Signed-off-by: Abigail <abigail@abigail.be>

Note POSIX changes in perldelta.

Provide the correct POSIX return value for POSIX::dup2() on Win32.

Microsoft, in their wisdom, chose to ignore the POSIX spec when implementing
their dup2(), and have theirs return 0 on success, instead of the file
descriptor. It seems that no other vendor is this, um, "special", so code
the exception directly, as we don't run Configure on Win32, so there's little
point probing for this.

This resolves RT #98912.

Convert POSIX::sleep to an XS wrapper for PerlProc_sleep().

Previously it was a Perl wrapper for CORE::sleep, converting CORE::sleep's
return value of elapsed time slept into the POSIX return value of seconds
remaining. However, that approach could sometimes return a negative result
if CORE::sleep had slept for more than a second longer than the requested
time.

perldelta for Unicode::UCD change

Unicode::UCD: Use 6.1 format for Name_Alias

The Name_Alias property is changing significantly in Unicode 6.1.  It
will have an extra value for each alias (and there will be a lot more
aliases).  That extra value is considered informative, and gives the
type of alias this is.  prop_invmap() should return both the alias and
its type.  This commit changes it so that even in 6.0, it will return
what it is going to return in 6.1 (for the 6.0 aliases).

This function has not been released in a stable Perl version.  Thus,
this gets the ultimate format into Perl before anyone has come to depend
on the earlier one.

squish

regcomp.c: Silence valgrind warning

This happens only in doing debug output.  Initialize these two debugging
variables

mktables: Add comment

mktables: Arrange to handle 6.1 ScriptExtensions.txt

Unlike 6.0, this property is listed in 6.1's PropValueAliases.txt, so is
automatically created in 6.1.  Thus this needs to be checked for and the
initialization moved out of the constructor.

mktables: Accommodate new @missings in Unicode 6.1

The Unicode 6.1 release candidate includes a new @missings line for the
ScriptExtenstions.txt file which we need to know about and deal with.

mktables: Make protected method available

This is needed in 6.1 for a special case.  It still should be used only
in limited circumstances so is still protected (meaning only that its
name begins with an underscore).

mktables: Add capability to add dup FIFO

This is in preparation for Unicode 6.1.  This adds the capability for a
duplicate code point to be added to a table after all existing
duplicates.  Previously, new duplicates were added before any existing
ones.

mktables: Rename variable

This is in preparation for Unicode 6.1.  The variable means LIFO, and in
6.1, a FIFO mode will also be required, so indicate that the current
variable means LIFO.

charnames.t: Skip null name test

In versions of Unicode earlier than 6.1, there was no possibility of a
name being empty here; but 6.1 will make that happen, so guard against
it.

charnames.t: Add test names

This adds test names to nearly all the ones that were missing.  Most
were done via a global substituted in a text editor.

charnames.t: Fix test that passed whether or not it should

This test was calling grep, then the comma operator, then the non-empty
string after the comment operator caused it to always succeed.

mktables: Comments only

Unicode/UCD.t: Avoid early prototype messages

Add an ampersand before the calls to silence these warnings

mktables: Ignore @missing lines in certain source files

This is in preparation for Unicode 6.1.  The release candidate version of these files
changes the syntax of this line in several of its files, so that
mktables sees them as a problem.  And, they aren't used anyway.

perlunicode: nit

mktables: Account for 2 provisional files

This is in preparation for Unicode 6.1.  These two provisional
properties are available starting in 6.0, but the rest of the data base
did not refer to them, so we could safely ignore them.  But in the beta
6.1, they are referred to elsewhere, which makes mktables generate hooks
for them, and then complains when those hooks are unused.

This patch should work no matter how 6.1 final comes out, and allows
someone who really wants to use these provisional properties to
recompile Perl without having to delve too far into the mktables code to
get it to work.

mktables: Need to have two classes of unused files

The current code works only when skipped files aren't actually present.
Otherwise, it deletes them from a hash, which causes a wrong warning to
be output.  By creating a separate hash for skipped files, we avoid
that.

mktables: Consistently store lc of key in hash

In most places, the keys to this hash are stored in lowercase, but not
formerly this one.  This enables not having to lc in other places.

mktables: Can skip processing suppressed properties

mktables: White-space only

To make vertically line up based on an added test

mktables: Don't test suppressed table validity

This omits the test for should a table be full for suppressed tables.
Other actions in the code may or may not bypass filling them, knowing
they are suppressed.

mktables: reorder two tests

This simply swaps two blocks so that the fate of a table is fully
figured out before deciding if it should match everything or not.

Expand tabs in Artistic license to spaces

This addresses [rt.perl.org #74046] and Debian #572245
( http://bugs.debian.org/572245 )

Allow diag.t to pass tests for multi-line warnings.

It's been sending the full warning text to t/test.pl's pass()
function.  If there are newlines in the the warning, t/TEST (though
not t/harness) will spew "Unexpected output at ..." messages and
fail the test.  In this context, the warning is just a message
printed by the test infrastructure, so we might as well use the
version of the warning that's already had newlines replaced with
spaces.

Increase $POSIX::VERSION to 1.28

Signal handlers must run before sigsuspend returns

The whole point of sigsuspend and pause is to wait until a signal has
arrived, and then return *after* it has been triggered. Currently
delayed/"safe" signals prevent that from happening, which might cause
race conditions.

This patch prevents that (as far as possible) by running the signal
handlers ASAP.

ExtUtils::ParseXS: Bump versions to 3.11

This is necessary for a CPAN release that fixes a versioning problem in
version 3.09 (CPAN RT #73555).

No functional changes.

Export PerlIOBase_open

perl -d: display lines inside subroutines.

This is another fix to the perl debugger while making sure that lines
inside subroutines will be printed in the run-time display of the perl
debugger: https://rt.perl.org/rt3/Ticket/Display.html?id=104820 .

Croak when setting a tied @DB::args

In commit f8c105437 I stopped the setting of a tied @DB::args from
crashing, by untying the array.  Untying an array automatically with-
out saying anything seems a little too much like action at a distance.
For now, let’s just croak, as we can always change it later once we
decide what should happen.  At least that’s better than crashing.

[Merge] perldiag/diag.t clean-up

This is a random sequence of a about one and a half dozen commits that
all happen to be in the same area.

At least one of these changes the wording of a message, so it should
probably go in perldelta.

Speed up diag.t slightly

By using strings instead of creating qr// things that get flattened,
we can reduce the run time by 5%.

perldiag: Delete redundant entry

This was made redundant three commits ago when I added more
descriptive entries for the two variations of this message.

diag.t: Delete documented items from to-do

Regen pod issues

podcheck.t doesn’t like links with formatting codes, even though the
spec says to do it that way.

Document various messages in perldiag

diag_listed_as galore

In two instances, I actually modified to code to avoid %s for a
constant string, as it should be faster that way.