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.

Pod formatting nits

Merge commit 'avar/avar/pod-perlrepository' into blead

consting in lex_stuff_pvn

t/mro/vulcan* fixes

It seems that Dylan Reference Book has been relocated, so included is
the small patch for the vulcan_* test files with the new link.

Deprecation warnings should always be mandatory since 5.12.0

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

New deprecation warning: Dot after %s literal is concatenation

Fixes and new functions for Module::CoreList

  Fixed functions will edge-case involving querying for Module::CoreList
    itself. Pointed out by Ilmari.

  Added removed_from() and removed_from_by_date() functions
    for querying which release a module was removed from core.

  Amended corelist utility to use new removed from functions when
    stating when a module entered core ( and when it left it ).

  Added tests to the testsuite to cover the edge-cases and new funcs.

time() resolution is full seconds

So any fudging in the timing needs to be at least 1 second to have any
effect. Upped the total $sleep value to 4 (on Windows) to make sure at
least 3 seconds have passed.  Amends commit 0ebb4f0.

time() resolution is full seconds

So any fudging in the timing needs to be at least 1 second to have any
effect. Upped the total $sleep value to 4 (on Windows) to make sure at
least 3 seconds have passed.  Amends commit 0ebb4f0.

Consistently use OP_DESC instead of OP_NAME in error messages

This solves bug: [perl #74572] chop dies with error about schop

This is not backwards-compatible, as some error messages can change,
as seen by the required change to t/op/sprintf2.t.

POD revision for Module::CoreList.

  Fully documented all the functions.

  Fully documented all the provided hash data structures.

FreeBSD 7+ no longer contains /usr/bin/objformat, so the following
(harmless) following error message is printed when hints are run:

./hints/freebsd.sh: /usr/bin/objformat: not found

The reason for this is outlined in this email from Feb 2007
http://lists.freebsd.org/pipermail/freebsd-ports/2007-February/038523.html

This patch just skips the objformat check for versions 7+ and
assumes ELF.  (The end result is unchanged, but the error
message will not be printed anymore.)

Fix for find_version() in Module::CoreList.

Add fudge to timing checks on Windows

sleep() time on Win32 may be rounded down to multiple of
the clock tick interval.

http://www.nntp.perl.org/group/perl.perl5.porters/2010/03/msg157878.html

Remove emacs/cperl-mode.el and helpers

http://www.nntp.perl.org/group/perl.perl5.porters/2010/03/msg158079.html

Update PerlStdIOGets() signature

Brings it in line with the rest of the PerlStdIO* functions that
accept parameters in the same order as the corresponding PerlSIO_*
macros.  See also commit ecc880cc.

Fix PerlSIO_fputc() and PerlSIO_fputs() signatures

They should have the same prototype as in stdio.

This "proper fix" replaces the 5.12.0 compatible fix in commit 634b482,
but cannot be integrated into 5.12.1 because it breaks binary compatibility.

See also http://rt.perl.org/rt3/Public/Bug/Display.html?id=72704

XSUB.h is supposed to redefine fgets under PERL_IMPLICIT_SYS, but doesn't.

See also http://rt.cpan.org/Public/Bug/Display.html?id=55049
with workaround in http://code.google.com/p/perl-devel-nytprof/source/detail?r=1168

Apply minimal patch for Perl bug 72704.

Arguments to fputc() and fputs() are reverted on Windows.

This is not the optimal patch, but will be backwards compatible
with what is in 5.12.0.  For the "correct" patch please see

http://rt.perl.org/rt3/Public/Bug/Display.html?id=72704

Remove rendundant CPAN key

follow up fix for fd69380d5d5b95ef16e2521cf4251b34ee0ce151

The original fix for tied elements losing magic had a bug that was masked
by a bool casting issue. Once the casting was fixed, the bug surfaced:
elements of @+ lost their values when returned from a sub. By removing the
TEMP flag from the regdatum PVLV, we force it to be copied when returned.

Ignore the /UU directory that exists during ./Configure

remove bool* cast from SAVEBOOL

SAVEBOOL() should only be used on variables of type bool; casting
pointers to variables of other types into bool* is just wrong.

Fix [perl #74542] 5.12.0 crash on diverse platforms

Filetest ops don't always expect an op on the stack, so we should use
TOPs only if we're sure that we're not stat'ing the _ filehandle.
This is indicated by OPf_KIDS (as checked in ck_ftst).

Don't ask the porters to fetch commits from a private URL

git@github.com:USERNAME/perl.git is only accessible to USERNAME (or
collaborators), git://github.com/USERNAME/perl.git is available to
everyone.

Consistently use a topic branch in the GitHub examples

Also use an example of an actual topic branch commit of that got into
Perl (<51dd1af81002161108rff277d7r6d0ccee6b45982a0@mail.gmail.com>).

Previously the docs referenced 3adac458cb which doesn't exist in the
repository, 0c24b290ae does.

Recommend git format-patch -M (detect renames) and origin..

The latter doesn't strictly matter, but it's easier to see what's
going on with those two dots.

Remove Locale-Codes internals from core

These (and the examples) don't have to be shipped with core perl. By not
shipping them, we can trivially make sure that they won't be erroneously
installed either.

Revert "GitHub's mirror is now at mirrors/perl, not github/perl"

This reverts commit f9bcfeacc119e37fd70816a2cb4678fded53e8d1.

add TODO test for #74484 trie leak

Also add leak_expr function to svleak.t to test an expression for leakage
rather than a whole sub

GitHub's mirror is now at mirrors/perl, not github/perl

# New Ticket Created by  (Ævar Arnfjörð Bjarmason)
# Please include the string:  [perl #74532]
# in the subject line of all future correspondence about this issue.
# <URL: http://rt.perl.org/rt3/Ticket/Display.html?id=74532 >

From: Ævar Arnfjörð Bjarmason <avar@cpan.org>

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