Perl 5.12.1 now includes Safe.pm 2.27. Module::CoreList should say so

New release date for 5.12.1 in light of the new RC

Merge branch 'blead' of ssh://perl5.git.perl.org/gitroot/perl into blead

* 'blead' of ssh://perl5.git.perl.org/gitroot/perl:
  Return 0 (with a warning) for sprintf("%.0g") and sprintf("%.0f")
  Updating list of DOSish perls in perlport.pod.
  Add Porting/checkpodencoding.pl to MANIFEST
  separate the 0755 from the . $TAINT0. this silences the warning "Dot after octal literal is concatenation".
  Clarify that mail to perlbug also goes to p5p

Correcting the release date of 5.12.1

Return 0 (with a warning) for sprintf("%.0g") and sprintf("%.0f")

There is special case code in the sprintf implementation, for simple %f and %g
formats, conditionally compiled in only when NVs are doubles. Under long
doubles, these are handled by the general purpose code, which always returns
0 if the argument is missing. Note that sprintf(" %.0g"), ie a leading space,
sufficient to bypass the special case code, would return the string " 0".

The special case code used to return an empty string, meaning that the
behaviour of sprintf("%.0g") and sprintf("%.0f") was inconsistent between a
perl built with doubles, and a perl with long doubles, and the behaviour of
sprintf("%.0g") and sprintf(" %.0g") was inconsistent.

5b98cd54dff3b163 fixed #62874 - the special case code did not warn, but
changed behaviour to return 0. d347ad18ecf3da70 undid the behaviour change,
viewing it as a regression. However, the tests added in 5b98cd54dff3b163
expose the inconsistency in behaviour between doubles and long doubles.

There should be no inconsistency, hence the only logically consistent
conclusion is that the special case implementation was wrong - it cannot
give results inconsistent with the general code. Hence this commit changes
it to return 0 (with a warning). This is achieved by simply skipping the
special case code, if there are insufficient arguments.

Updating list of DOSish perls in perlport.pod.

Add Porting/checkpodencoding.pl to MANIFEST

separate the 0755 from the . $TAINT0. this silences the warning "Dot after octal literal is concatenation".

Clarify that mail to perlbug also goes to p5p

The wording added in 64a8e22bd3 didn't clarify that mail to perlbug is
also forwarded to perl5-porters. Make this section clearer so that it
now explains that perlbug is just perl5-porters + RT tracking.

See the discussion after Craig A. Berry's comment in RT #73914 for
reference.

PATCH [perl #74978] dot after } breaks \N{}

The problem is that a dot can come between the braces in \N{foo.bar},
but when searching for it, I didn't stop looking at the right brace, so
it generated an error inappropriately.

This is essentially a minimum patch; efficiency could be improved
slightly with a little more work.

Note in comment that many \N{...} tests won't work here

Comment where to find file's format

perldelta item on PL_restartjmpenv

perldelta item on $[ usage

perldelta item on B::Deparse package+label fix

perldelta item on reliable exception handling

perlunifaq, uniintro: fix for 80 col display

perlunicode: fix for 80 col display

perlreref: missing info, 80 col display

The \p{Posix...} classes had not gotten added yet to the ref pod; there
were some reformattings to make things display properly in an 80 column
window.

perlrecharclass: Clarify \p{Punct}, fix for 80 col

While not strictly wrong, the hre was missing info for what \p{Punct}
does.

perlrebackslash: fix for 80 col display

perlre: fix for 80 col display

perlport: fix 80 col display; broken link

Fix to display ok in 80 columns

Fix perlebcdic for for 80 column tty; fix C<>s

Two C<>'s were unclosed

Add =encoding utf8 to all core non-ASCII POD files + test script

It's now possible to run:

    perl Porting/checkpodencoding.pl

To check if the core contains any naughty POD that uses non-ASCII
without declaring an encoding.

With this patch all the POD in core (except POD tests we're ignoring)
has a correct =encoding directive. The script also flags problems in
lib/* and cpan/* which aren't being fixed as part of this commit.

RT #34604 didn't honour tied overloaded values

A tied hash lookup could return an overloaded object but sort wouldn't
notice that it was overloaded because it checked for overload before doing
mg_get().

add SV_SKIP_OVERLOAD flag to sv_2*v_flags fns

While trying to coerce an SV into a string or whatever, stop if you
suddenly discover it's overloaded (this may not happen until after you've
called it's get magic)

add flags arg to sv_2nv (as sv_2nv_flags)

Modify Module::CoreList docs to state the availability of the removed_from() functions.

Backport patch from CPANPLUS-0.9004 to cpanp-run-perl

  This resolves RT #55964 and RT #57106 relating to the failure to
  install distributions that use Module::Install::DSL

  The addition of a BEGIN {} block to cpanp-run-perl resolves the
  issue.

Bump Module::Corelist for cherrypicking to maint-5.12

Make t/op/sprintf.t test multiple warnings

Stop returning 0 for sprintf("%.0g")

And also from throwing two "Missing argument" warnings for this construct.

This was a regression introduced by 5b98cd54dff3b16344eab33ce6b09fb6fb1b89c2.

Really test special sprintf formats

Make sv_vcatpvfn() complain when special formats "%s" and "%.0f" can't find their argument

This fixes [RT #62874] : printf does not print a warning when a lone %s conversion is used

Change perlgpl.pod to GPL 1 to match README

The README and Copying files are specific that Perl is licensed
under Artistic or GPL 1 (or later).  For whatever reason, perlgpl.pod
was created with GPL 2.  This commit changes it back to GPL 1 for
consistency with the actual license.

It also standardizes the introduction to both perlgpl and perlartistic
for consistent language explaining the dual license.

Move code to after the variable declarations.

Currently we only compile the top level code with "extreme warnings", so code
in ext/ isn't subject to -Wdeclaration-after-statement, hence why this was
missed. Now that dual life modules have been evicted from ext/, we could
probably consider notching up the warnings for all code in ext/.

Merge branch 'blead' of ssh://perl5.git.perl.org/gitroot/perl into blead

* 'blead' of ssh://perl5.git.perl.org/gitroot/perl:
  bump PerlIO::scalar version
  RT 43789: "in memory" files don't call STORE

bump PerlIO::scalar version

RT 43789: "in memory" files don't call STORE

The code in PerlIO-scalar that implements the open $fh, '>' \$buffer
feature did not, apart from accidentally, support get/set magic and thus
tied buffers. This patch remedies that: mostly by just blindly sprinkling
SvGETMAGIC/SvSETMAGIC about, rather than doing any deep analysis and
understanding of the code. One main change I did was to add a
PerlIOScalar_read() function, rather than rely on the default behaviour
(which implements it in terms of PerlIOScalar_get_ptr() etc), since that
approach had a tendency to call FETCH multiple times

Revert "New deprecation warning: Dot after %s literal is concatenation"

This reverts commit 6fb472bab4fadd0ae2ca9624b74596afab4fb8cb.

Zefram asked me to revert this as he's going to be doing something more
pluggable

Revert "Fix tests and add one more test for the deprecation warning added in last change"

This reverts commit c4deb7365787eb01845a9d0e371e343169530659.

Zefram asked me to revert this as he's going to be doing something more
pluggable

Revert "Deprecation warnings should always be mandatory since 5.12.0"

This reverts commit a7e260e62a5e47961e908363da32ef16f41301b2.

Zefram asked me to revert this as he's going to be doing something more
pluggable

Revert "tweak "0x123.456" deprecation"

This reverts commit 1183a10042af0734ee65e252f15bd820b7bbe686.

Zefram asked me to revert this as he's going to be doing something more
pluggable

When saving ints, if the value is small enough save it with the type.

This uses a new type, SAVEt_INT_SMALL.

When saving I32s, if the value is small enough save it with the type.

This uses a new type, SAVEt_I32_SMALL.

If we're going to introduce an @@ array, we'll want to be able to parse $#@ too

Give an error message if bison is not found at all

For SAVEt_I16, save the value with the type.

note in perldelta that local @tied not tied now

make 'local $tied' untied

When localising a tied scalar, don't make the scalar tied

make local @tied, %tied, untied

Fix for RT #7938, #7939: when localising an array or hash, don't make the
new aggregate tied.

The old behaviour of { local @tied; ... } was equivalent to:

    {
my $saved = \@tied;
*tied = [];
tied(@tied) = tied(@$saved)  # if tied() were an lvalue function
...
*tied = $saved;
    }

This patch simply removes the 'tied(@tied) = ...' step

die() no longer propagates outside of a Safe compartment

so mark the test for that as a TODO. We'll decide later what behaviour
is desirable here. Note that it warns instead.

Fix parameter name for die_unwind() in embed.fnc

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)