Merge remote branch 'zefram/zefram/reliable_exception' into blead

Conflicts:
pp_ctl.c

Make tests pass with open pragma in effect

Silence a couple of false positive "may be used uninitialized" gcc warnings

For SAVEt_I8, save the value with the type.

fix a couple of var types

these errors were introduced in my trie-allocation patch,
2e64971a6530d2645969bc489f564bfd3ce64993

Update AUTHORS/checkAUTHORS

shift; optimization

For SAVEt_BOOL, save the value with the type.

POD tweak to 'require' in perlfunc relating to putting Perl code in @INC

tweak "0x123.456" deprecation

Some improvements to the deprecation added in commit
6fb472bab4fadd0ae2ca9624b74596afab4fb8cb:

- warning message includes the word "deprecated"
- warning is in "syntax" category as well as "deprecated"
- more systematic tests
- dot detected more efficiently by incorporation into existing switch
- small doc rewording
- avoid the warning in t/op/taint.t

Reinstate some documentation about quotemeta

(that was removed in 8bdbc703cb0af3faf2163ebce618944c51f180a0)

tries: don't allocate memory at runtime

This is an indirect fix for
    [perl #74484] Regex causing exponential runtime+mem usage

The trie runtime code was doing more SAVETMPS than FREETMPS and was thus
growing a large tmps stack on heavy backtracking. Rather than fixing this
directly, I rewrote part of the trie code so that it no longer needs to
allocate memory in S_regmatch (it still does in find_byclass()).

The basic issue is that multiple branches in the trie may trigger an
accept state; for example:

    "abcd" =~ /xyz/abcd.*X|ab.*Y|/

here, words (branches) 2 and 3 are accept states. The original approach
was, at run time, to create a list of accepted word numbers and the
character positions of the end of each of those words. Then run the rest
of the pattern for each word in the list in turn (in word index order).
This requires memory for the list to be allocated and freed.

The new approach involves creating extra info at compile time; in
particular, for each word, a pointer to the previous accepted word (if
any) in the state tree. For example for the above pattern, part of the
state tree may be

      q    b    c    d
    1 -> 2 -> 3 -> 4 -> 5
            (#3)       (#2)

(e.g. at state 1, if the next char is 'a', we transition to state 2).
Here, state 3 is an accept state with word #3, and 5 is an accept state
with word #2. So we build a table indexed by word number, which has
wordinfo[2] = 3, wordinfo[3] = 0, thus building the word chain 2->3->0.

At run time we run the trie to completion, and remember the word
associated with the longest accept state (word #2 above). Then by following
back the chain of .prev fields, we can produce a list of all accepting
words. We then iteratively find the smallest-numbered (ie LH-most) word in
the chain, and run with it. On failure and backtrack, we find the
next-smallest and so on.

Since we are no longer recording the end-position of each word in the
string, we have to recalculate this for each backtrack. We initially
record the end-position of the shortest accepting word, and given that we
know the length of each word, we can calculate the new position each time
as an offset from that first word. Depending on unicode and folding, that
calculation can be cheap or expensive.

This algorithm is optimised for the typical case where there are a small
number (<= 2) accepting states.

This patch creates a new compile-time array, trie->wordinfo[], indexed by
word number, which contains relevant info about each word. This also
supersedes the old trie->newword[] array, whose function of recording
"overspills" of multiple words per accept state, is now handled as part of
the wordinfo[].prev chain.

[perl #74856] Fix POD syntax in perlapi

Please don't use any of your git aliases in perlrepository.pod

For SAVEt_ALLOC, store the number of save stack entries used with the type.

PATCH: Make perluniprops.pod platform neutral

# New Ticket Created by  karl williamson
# Please include the string:  [perl #74830]
# in the subject line of all future correspondence about this issue.
# <URL: http://rt.perl.org/rt3/Ticket/Display.html?id=74830 >

This is accomplished by changing mktables which generates it

>From d114f6f25c763ecbd00d6271462e5d5c272457e7 Mon Sep 17 00:00:00 2001
From: Karl Williamson <khw@khw-desktop.(none)>
Date: Sat, 1 May 2010 09:22:32 -0600
Subject: [PATCH] Add missing comma to generated pod

Signed-off-by: H.Merijn Brand <h.m.brand@xs4all.nl>

For SAVEt_REGCONTEXT, store the number of save stack entries used with the type.

Permit array assignment to steal temps and copy shared hash key scalars.

Scalar assignment to array elements already does this. (As does all other
scalar assignment, and list assignment to hashes.) Prior to 4c8f17b905f2
(change 7867) list assignment to arrays did steal temps.

Better fix for RT #2140 (list assignment with duplicated temporaries)

4c8f17b905f2 (change 7867) took the approach of a special case in sv_setsv()
when PL_op indicated that the current OP was OP_AASSIGN. The problem is in one
part of pp_aassign, where it was using sv_mortalcopy() on values that were
correctly marked as temporaries, but also still needed later. Hence a more
targetted solution is to avoid that call, and to instead use API calls that
will not steal temporaries.

Fix c6bf6a65 - 64 bit big endian builds were broken.

Bump version of Carp

[perl #74808] Carp.pm POD error

The documentation incorrectly states that the $Carp::Verbose
variable makes cluck generate stack backtraces.  cluck
already generates stack backtraces.  It should say that
the variable makes croak generate stack backtraces.

Reduce Errno memory usage by around 55%.

Use Proxy Constant Subroutines rather than full-fat subroutines, and simplify
the implementation of the tied hash methods.

Remove unused %errno and $AUTOLOAD from the generated Errno.pm

For SVt_CLEAR, store the pad offset with the type.

This saves 1 slot on the save stack for each lexical encountered at run time.

Make Term::ReadLine::findConsole fall back to STDIN if /dev/tty can't be opened

Remove the lexical $len and associated calculation, which is never used.

On the save stack, store the save type as the bottom 6 bits of a UV.

This makes the other 26 (or 58) bits available for save data.

Untangle REGCP_FRAME_ELEMS from REGCP_OTHER_ELEMS.

Update the Change log in Module::CoreList to include recent commits

For threads, we ignore all files in the distribution's examples directory.

Upgrade to IPC::Cmd 0.58

RT#73814 - unpack() didn't handle scalar context correctly for %32H and %32u

split() would crash because the third item on the stack wasn't the
regular expression it expected.  unpack("%2H", ...) would return both
the unpacked result and the checksum on the stack, similarly for
unpack("%2u", ...).

Update title of "Supported Platforms" in perlport to avoid
breaking pod links at each perl release (spotted by Karl Williamson)

Re-run bump-perl-version to update a few missing spots

Teach bump-perl-version about "libperl511.a"

Don't use a C++ keyword as a variable name ("new").

put package declaration before label in deparsing

When deparsing a nextstate op that has both a change of package (relative
to the previous nextstate) and a label, the package declaration must be
emitted first, because it is syntactically impermissible for a label to
prefix a package declaration.

Merge branch 'dual/Safe' into blead

Bump Safe's version to 2.27 and update Changes and META.yml

Mention that Safe::reval() no wraps returned coderefs

Deprecate Perl_ptr_table_clear(). Nothing outside sv.c uses it.

Inline the necessary parts of Perl_ptr_table_clear() into Perl_ptr_table_free().
No need to reset memory to zero that is about to be freed anyway.

Regression tests for the ptr_table_* API.

Revert "Un-TODO warning test"

This reverts commit efbe327085cc15510d8c261772e9ac21be3635de.

Wrap by default coderefs returned by rdo and reval

(suggested by Tim Bunce)

Add &version::vxs::stringify to the default share

import Pod-Simple 3.14 for C<< >> fix

Correct POD link to perlrun.pod to be more accurate

Now liking to the specific section in perlrun that discusses
sitecustomize.pl

Signed-off-by: Ævar Arnfjörð Bjarmason <avar@cpan.org>

Add POD index entries for sitecustomize and sitecustomize.pl

This covers all non-perldelta occurrences of documentation that
discusses sitecustomize.pl

Signed-off-by: Ævar Arnfjörð Bjarmason <avar@cpan.org>

Improve documentation about sitecustomize.pl.

This documentation patch is more a rewording of the content of
http://www.nntp.perl.org/group/perl.perl5.porters/2007/10/msg129930.html
than anything original.

The actual authors are therefore Michael G Schwern, Jan Dubois, Peter
Dintelmann, and H.Merijn Brand.

Signed-off-by: Philippe Bruhat (BooK) <book@cpan.org>
Signed-off-by: Ævar Arnfjörð Bjarmason <avar@cpan.org>

don't use $[ in library code

Remove all uses of $[, both reads and writes, from library code.
Test code (which must test behaviour of $[) is unchanged, as is the
actual implementation of $[.  Uses in CPAN libraries are also untouched:
I've opened tickets at rt.cpan.org regarding them.

Add miyagawa to the list of release engineering suckers.

Convert Perl_magic_methcall() to varargs.

This means removing its macro wrapper, as there's no portable way to do varargs
macros.

For Perl_magic_methcall() add G_UNDEF_FILL to fill the stack with &PL_sv_undef.

This replaces the previous special case of using a negative argument count to
signify this, allowing the argument count to become unsigned. Rename it from n
to argc.

Fix possible undefined behaviour introduced by b9e00b79e4947c49d5520633f9efd2a8e39ec14f

Add a note about perlport in the release managers' guide

Fill blank with the release date of 5.12 in perlport

Save the popped cx->blk_eval.old_namesv before calling LEAVE

It's fine to still refer to cx members between POPEVAL and LEAVE, but there
are a few places where the namesv is read after LEAVE happens. This is bad
because LEAVE can call arbitrary code ; in particular, it can call a destructor
that does call_sv(cv, G_EVAL), in which case the old eval context cx gets
overwritten by the new one and cx->blk_eval.old_namesv points to garbage.

Fix h2ph and test

avoid use of operator name in macro

Fix Module::Build::Platform::cygwin runtime error

This addresses runtime errors in Module::Build::Platform::cygwin
as reported by Jesse Vincent.  The changes are backported from
the Module::Build repository and the version number has been
slightly incremented to indicate a change from the version on CPAN.

This hopefully addresses test errors in t/actions/installdeps.t, but
the error seems specific to building perl on cygwin and has not been
reported when Module::Build was tested on cygwin by CPAN Testers.

Globs that are in symbol table can be un-globbed

If a symbol table entry is undefined when a glob is assigned into it, it
gets a FAKE flag which makes it possible to be downgraded when non-glob
is subsequently assigned into it. It doesn't really matter, until we
decide to localize it -- it wouldn't be possible to restore its GP upon
context return if it changed type, therefore we must not do that.

This patch turns off FAKE flag when localizing a GV and restores it when
the context is left. A test case is included.

remove Perl_pmflag

regen header after last patches

Note in delta an incompatibility for \cX

X in "\cX" must now be ASCII.  Previously, it could be anything, but led
to a malformed utf8 error if wasn't Latin1.  This construct was never
intended to be used for anything but (a subset of) ASCII.

(moved to the delta for 5.13.1)

Deal with "\c{", and its kin

make regen is needed

This patch forbids non-ascii following the "\c".  It also terminates for
"\c{" with a message to contact p5p if there is need for continuing its
current definition.  And if the character following the "\c" causes the
result to not be a control character, a warning is issued.  This is
currently 'deprecated', which by default is turned on.  This can easily
be changed later.

This patch is the initial patch.  It does not do any fancy showing the
context where the problematic construct occurs.  This can be added
later.

It gathers the 3 occurrences of evaluating \c and puts them in one
common routine.

Make sure isCNTRL and isASCII work on signed chars

Prior to this patch, there is a potential bug in these two macros, in
which, if they are called with a signed character outside the ASCII
range, it will be negative and they always returned true for negative.
Casting the parameter to an unsigned should fix that by having it be
interpreted as a number above the ASCII range.

Minor edition nit in perlfunc

perlebcdic.pod nits plus improve controls docs

The controls all now have names, and the part about \c\ has been
corrected.  The table widths have been changed; all recipes have been
tested on the new tables.

Clarify \c in perlop.pod.

And structure the table containing \c better.

Edits to perlrecharclass.pod

A number of clarification and wording edits have been made, fixing some
broken links, and details especially on \d in the Unicode range.  Fixed
an incorrect character ordinal

Nits in perlre.pod, x-referencing, broken links

Fix broken links

perlfunc.pod: case-change cleanup; mention packtut

Specifies completely the behavior of the case-changing functions, and
mentions in the existence of the pack tutorial for the packing ones.

Nits in perlunicode.pod

Clarify \c usage in perlrebackslash.pod

Nits in perlunifaq.pod

Nits in perluniintro.pod

Make accurate the advice about eighth-bit set characters, and a few
editing improvements.

Remove false statement about Unicode strings

It is simply not true that all text strings are Unicode strings in Perl.

Don't use Test::More in t/op/* tests

add tests for version::is_strict() and version::is_lax()

fix version::is_strict/is_lax exporting

These were being exported with a wrapper that treated them as method
calls, which causes them to fail.  They are just functions, are
documented as such, and should never be subclassed, so this patch
just exports them directly as functions without the wrapper.

Fix utf8::is_utf8 to respect GMAGIC (e.g. $1)

avoid multiple FETCHes

The fix 2d961f6deff7 for RT #5475 included a mechanism for the early
calling of get magic on something like

    $tied[0];

so that even though the element is used in void context, we still call
FETCH. Some people seem to rely on this.

However, the call to mg_get() didn't distinguish between a tiedelem
member retrieved from a tied array/hash, and a tiedscalar element
retrieved from a plain array/hash. In the latter case, the S_GSKIP
protection mechanism doesn't apply and a simple $foo = $h{tiedelem}
generated two calls to FETCH.

Fix this by only calling mg_get() on the element if it came from a *tied*
array/hash.

A side-effect of this fix is that the following no longer calls FETCH:

    my @plain_array;
    tie $plain_array[0], ....; # element 0 is now a tied scalar
    $plain_array[0]; # void context:  no longer calls FETCH.

This required one test in op/tie.t to be fixed up, but in general I think
this is a reasonable compromise.

Bump versions of charnames and Unicode::UCD after last patches

Adapt plan after last patch

PATCH [perl #72624] charnames::viacode(0) returns undef

The viacode() function contained the code from the _getcode() function from
Unicode::UCD, unchanged.  However, the rest of viacode() requires that
the result be specially formatted to do a string match with leading
zeros inserted to bring the length up to 4 if less than that.  The
original function only needs to get the number right, as a numerical
comparison is done, so it doesn't do this.  This showed up with calling
viacode with 0, but the bug also affected any input that looked like a
hex number, or a U+ number, such as 'BEE' or 'U+EF'.  These need to be
massaged into '0BEE' and '00EF' for the pattern match later in the
routine to succeed.

The patch also adds a test case to Unicode::UCD to verify that it really
does work ok on 0.

PATCH: memory leak introduced in 5.12.0

There is a small possibility of a memory leak in toke.c when there is a
deprecated character in the name in a \N{...} construct, and the Perl is
embedded or something like that so that memory isn't freed up when it
exits.  This patch avoids the creation of a new scalar, and gives a
better error message besides.

5.12.0 test failures in lib/blib.t and lib/locale.t: Darwin/PPC

Change the flags argument to magic_methcall/magic_methcall1 from I32 to U32.

unwinding target nominated by separate global

When unwinding due to die, the new global PL_restartjmpenv points
to the JMP_ENV at which longjmping should stop and control should
be transferred to PL_restartop.  This replaces the previous
use of cxstack[cxstack_ix+1].blk_eval.cur_top_env, located in a
nominally-discarded context frame.

Don't allocate pointer table entries from arenas.

Instead, allocate a private arena chain per pointer table, and free that chain
when its pointer table is freed. Patch from RT #72598.

More defensive definition of memEQs().

In S_magic_methcall1(), tweak the initialisation of arg1 to reduce code size.

add Perl_magic_methcall

Add a new function that wraps the setup needed to call a magic method like
FETCH (the existing S_magic_methcall function has been renamed
S_magic_methcall1).

There is one functional change, done mainly to allow for a single clean
wrapper function, and that is that the method calls are no longer wrapped
with SAVETMPS/FREETMPS. Previously only about half of them had this, so
some relied on the caller to free, some didn't. At least we're consistent
now. Doing it this way is necessary because otherwise magic_methcall()
can't return an SV (eg for POP) because it'll be a temp and get freed by
FREETMPS before it gets returned. So you'd have to copy everything, which
would slow things down.

In Socket.xs, convert 3 croak()s to use %s and a constant string for the name.

This actually saves space, because xsubpp has to use each XS function's full
name as a constant string to pass to newXS(), so any re-use of it is free.

Fix broken -Uuseperlio build on VMS.

We were checking a variable that doesn't exist in the non-default
case of disabling perlio.  Now we only look at it when it exists.

Fix -Uuseperlio command-line option in configure.com.

Formerly it only worked if you went through all the questions
interactively and explicitly answered no.

Update Module::Load::Conditional to CPAN version 0.38

  Addresses a serious problem with CPANPLUS and behaviour of
  version-0.82. Checking for out of date modules with 'o'
  in CPANPLUS blows up if any modules that are installed
  have non-numeric version strings.

  Extract from Changelog:

  Changes for 0.38    Fri Apr 23 15:52:38 BST 2010
  =================================================
  * New release of version raises exceptions on
    parse errors. Use eval to guard ourselves
    against this.

  Changes for 0.36    Tue Feb  9 14:16:21 GMT 2010
  =================================================
  * Apply patch from Pavel Shaydo RT #53546 to improve
    the performance of _parse_version()

[perl #73776] "???? - please notify IZ"

Change 27536 (45f4726) started using -1 as a special len value for utf8
magic. I believe this marker indicates that the utf8 length cache needs
to be calculated, in which case dump.c can ignore this case.