Add Zefram as our release manager victim for 20 December

Add two missing break in Perl_magic_set's big switch

This fixes $^A being reset when $1..$2 are localized before any regexp match
happened.

Remove unneeded use of INT2PTR() in the XS for Tie::Hash::NamedCapture.

Start perl5135delta

Module::CoreList updates for 5.12.2

Avoid needless use of UTF8=1 format [RT#56336]

Some literals (e.g. q'abc') don't set the UTF8 flag for pure ASCII literals.
Others (e.g. -abc) do. This should be consistent.

TODO tests for avoid needless use of UTF8=1 format [RT#56336]

Some literals (e.g. q'abc') don't set the UTF8 flag for pure ASCII literals.
Others (e.g. -abc) do. This should be consistent.

add more tests to lib/mauve.t so it tests also that mauve::reftype can return "LVALUE"

use more efficient sv_reftype_len() interface

fix the AIX skip to only skip on AIX

rt77432: sigaction would crash/assert with a replaced %SIG

mauve.t needs access to %Config, make sure it's available

create the "mauve" temporary namespace for things like reftype

Scalar::Util::reftype(), refaddr() and blessed() are all a bit
less useful than they could be as they all return C<undef> when
their argument is not a reference. While this is logical, it also
means that using these routines has to be guarded, and normally
guarded in such a way that the internal logic is called twice.

Additionally these routines are quite commonly used and having to
load an additional DLL and XS code every program is inefficient.

Therefore we introduce the "mauve" namespace for to hold the "fixed"
equivalents, this namespace is /always/ loaded (like the 're' or 'utf8'
namespaces), and thus these routines can be accessed easily at any time.
We also provide a new module wrapper in t/lib which allows these
routines to be exported into other namespaces if the user so chooses.

At Jesse's request I have included weaken() and as it seemed logical to
do so I have also added isweak().

Once we have a good name for the namespace we can s/mauve/whatever/g

add sv_reftype_len() and make sv_reftype() be a wrapper for it

sv_reftype() mostly returns strings whose length is known at compile
time, so we can avoid a strlen() call if we return the length.

Additionally, the non-length interface is potentially buggy in the
face of class names which contain "\0", therefore providing a way
to obtain the true length allows us to avoid any trickyness.

when building under git, test that MANIFEST and git ls-files match up properly

Update Unicode-Collate to CPAN version 0.58

  [DELTA]

  0.58  Sun Aug 29 19:56:50 2010
    - U::C::Locale newly supports locales: af, cy, da, fo, haw, is, kl, sw.

Fix a typo in the PL_keyword_plugin docs

Make PerlIOUnix_open honor default permissions on VMS.

When perlio became the default and unixio became the default bottom
layer, the most common path for creating files from Perl became
PerlIOUnix_open, which has always explicitly used 0666 as the
permission mask.  This has the following undesireable effects on
VMS:

1.) The execute bit is lost regardless of whether it's in the default
    permissions.

2.) Delete permission (which doesn't exist in the Unix permission mask) is
    copied from write permission, so granting write permission also grants
    delete even if it's not in the default permission mask.  This can result
    in an inadvertent widening of permissions.

3.) System permissions (which don't exist in the Unix permission mask) are
    copied from owner permissions, so any distinction between system and
    owner is lost.

4.) ACLs are not inherited.  For example, setting a default_protection ACE
    on a directory such that all world access is disallowed will be ignored;
    world will have the intersection of RWD (the final 6 in 0666) and whatever
    the default permissions are regardless of what the ACL says.  Thus not
    inheriting ACLs can result in the inadvertent widening of permissions.

The way to avoid all of this is to pass 0777 as the permissions to open().
In the VMS CRTL, 0777 has a special meaning over and above intersecting
with the current umask; specifically, it allows Unix syscalls to preserve
native default permissions.  Details currently documented at:

http://h71000.www7.hp.com/doc/732final/5763/5763pro_060.html#umask_routine

Remove OA_RETINTEGER, unused since 2002 (commit e7311069df54baa6)

This returns us to 8 flag bits, and restores OCSHIFT and OASHIFT to 8 and 12
Previously these were 9 and 13, and effectively PL_opargs[] was using 33 of
32 bits, relying on the ugly hack that no 5 argument builtin had an optional
5th argument, hence the (13 + 5 * 4)th bit was always zero.
is effectively 33 bits.

silence APItest deprecation warning

mktables: Silence common build warning

The first time mktables is run on a clean directory, mktables.lst is
non-existent.  Rather than warn on this common occurrence, just assume
it is ok, and continue.  The point of the warning was to point out a
potential problem.  But it occurs so often that people have gotten
inured (habituated) to it, and it has lost all effectiveness.  Better to
not output it, and have any problems show up as repeated rebuilds that
will annoy enough to be dealt with.

Silence g++ compatibility warning

remove much dead code from regen.pl

avoid unnecessarily changing timestamp on perly.h

Make regen_perly.pl use rename_if_different() to avoid overwriting files
that aren't actually changing.

Peephole optimise adjacent pairs of nextstate ops.

Previously, in code such as

    use constant DEBUG=>0;

    sub GAK {
        warn if DEBUG;
        print "stuff\n";
    }

the ops for C<warn if DEBUG;> would be folded to a null op (ex-const), but
the nextstate op would remain, resulting in a runtime op dispatch of nextstate,
nextstate, ...

The execution of a sequence of nexstate ops is indistinguishable from just the
last nextstate op, so teach the peephole optimiser to eliminate the first of a
pair of nextstate ops. (Except where the first carries a label, as labels
mustn't be eliminated by the optimiser, and label usage isn't conclusively
known at compile time.)

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

Typo fix in perlfork

Remove a useless hardcoding of the current Perl version in README.AIX

First pass of CoreList update for 5.12.2

In mktables, disable string overloading when de-duping lists.

As the unique() function is called with either a list of strings or a list of
objects, and all objects stringify to distinct values, there is no behaviour
difference between using object stringification and object address for a
"seen" hash. There is, however, an 8% run time difference :-)

Devel::NYTProf strikes again.

Convert mktables to assign lexicals via do { no overloading; ... }; blocks.

This avoids repeating the lexical's name, whilst retaining the tight scope of
no overloading.

Change mktables to use pack 'J', $addr; to key its inside-out objects.

Using pack 'J', $addr; instead of 0 + $addr; reduces run time by about 1%,
according to my benchmarking.

In lib/unicore/mktables tweaks to tidy the file writing code.

In write(), don't concatenate all the lines to one scalar to print them, as
it takes fewer ops and less memory to pass print a list of values.
die if the print or close return errors, and don't print the success message
until the file is successfully closed.

Refactor make_property_test_script() to use write().

In lib/unicore/mktables minor tidying of the numifciation code.

In the inside-out object lookup code, strive for consistency, by always writing
0 + $ref with no spaces. No need to use 0+$ref as the arguments to == as it
will force numeric context.

make recursive part of peephole optimiser hookable

New variable PL_rpeepp makes it possible for extensions to hook
the per-op-chain part of the peephole optimiser (which recurses into
side chains).  The existing variable PL_peepp still allows hooking the
per-sub part of the peephole optimiser, maintaining perfect backward
compatibility.

eliminate unneeded code, and explain why the code was not needed

The commit message for v5.13.4-47-gd1c771f contained some misleading language
which I only noticed after I pushed. This change puts the comment in the
code and hopefully clarifies things properly.

In simple words: VERBS should *never* be included in the JUMPABLE condition.

VERB nodes in the regex engine should NOT be marked as JUMPABLE.

JUMPABLE nodes can be ignored during certain phases of regex execution,
including ones where backtracking is affected. This change disables this
behviour so that the VERBS can perform their desired results.

Committer has taken the liberty of modifying the patch so that all
VERBS are jumped, thus making the JUMPABLE expression a little simpler.
I have left Bram's change to JUMPABLE intact, but inside of a comment
for now.

See discussion in thread for [perl #71942] *COMMIT bypasses optimisation
for futher details.

http://rt.perl.org/rt3/Ticket/Display.html?id=71942

There appears to be room for futher optimisation here
by moving the JUMPABLE logic to regex-compile time. Currently
it is arguable that the "optimisation" this patch seeks to avoid
is actually not an optimisation at all, as it happens OVER AND OVER
during execution of a match, thus the extra effort might actually
outweight the benefit, especially on large strings.

fix description of -DJ in perlrun

The description of the J debugging flag was logically the reverse of what
it should be; i.e. -DJ enables, rather than suppresses, output in debugger
packages

prevent Devel::Peek::Dump from lieing to us about evil class names

While one certainly can argue the merits of using a class name like "\0", it is legal
so lets avoid it confusing our primary debugging tool.

Fix escaping in opcode.h generation

The op list has no escapes, so there's no need check for already-escaped
characters, and the existing regex breaks any run of consecutive
backslashes, escaped or not.

perlunicode.pod: Fix misleading info, expand

There was some misleading, or uncharitably, wrong text in this pod about
user-defined casing.  And, it jumped the gun, presuming that 5.14 would
fix something for which there has not been a patch submitted yet.

And, I realized there was a way around having to figure out the utf8 for
a character.

mktables: Clarify comment output in files

I never understood this comment until it bit me.  Based on my new-found
experience, I've expanded it.

Add tests for commit 1b0124a7c19a5499714e57a00a3d30b2f2647b1f

[perl #77392]

Make sure that the stack is not popped when there is nothing on the stack.

Fix -Wwrite-string warnings in cygwin.c

[perl #77376]
Fix the following build warnings that are now being
generated because -Wwrite-string has been turned on:

cygwin.c: In function `do_aspawn':
cygwin.c:69: warning: assignment discards qualifiers from pointer target type
cygwin.c: In function `do_spawn':
cygwin.c:91: warning: initialization discards qualifiers from pointer
target type
cygwin.c: In function `init_os_extras':
cygwin.c:544: warning: initialization discards qualifiers from pointer
target type

Tests for bug 77414

Mark regcurly as experimental

Even though it isn't in the public API, some worry that we shouldn't
change it, as it's currently public, but may be made into an in-line
function known only to the two .c files that should call it.

update AUTHORS

Config.pm: report perl path in version conflict.

When Config.pm dies with a perl/library version mismatch error, include
the path of the perl executable in the error message.

This helps avoid confusion during perl build when a cc wrapper written in
perl is used (such as colorgcc)

exclude autodoc.pl from pod checks

This script contains snippets of pod in here-doc sections which
gives false positives

perlpod.api: improve the prose

Tweak the text of the previous commit that describes the status of
undocumented functions in the public API.

autodoc.pl: Clarify use of undocumented fcns

perlapi.pod was changed to list all the undocumented functions.  The
introductory text was not updated at that time.  Now every public API
function is at least listed in this pod.  Clarify that functions not
listed are not public and should not be used, and add a plea for
documenting the ones which are missing theirs.

don't taint $DB::sub

[perl #76872] showed a case where code like the following, run under -d,
would cause $DB::sub to get set:

    $tainted_expression && func()

The tainted expression sets PL_tainted, so calling func() under -d, which
sets $DB::sub, causes it to get tainted.

Consequently any further sub calls would set PL_tainted while getting the
old value of $DB::sub (and cause the new value to be tainted too), and if
the sub was XS, then its code would be executed with PL_tainted set.
It isn't an issue with perl subs as the first nextstate op resets
PL_tainted.

detaint paths added by blib.pm

Update ExtUtils-MakeMaker to CPAN version 6.57_01

  [DELTA]

  6.57_01  Tue Aug 24 01:36:20 PDT 2010
    Improvements
    * parse_abstract() is more robust supporting "Package.pm" and multiple
      dashes and spaces. [rt.perl.org 74438]

    Bug Fixes
    * Recognize .so files in AIX. [rt.cpan.org 41360] (Jens Rehsack)
    * Remove manual image-base generation on Win32/gcc [rt.cpan.org 47138]
      (Yasuhiro Matsumoto)
    * Use the bundled versions of our dependencies if they're not installed.
    * Eliminate use of foreach qw() which will be deprecated in 5.14.
      [rt.cpan.org 57124] (Zefram)

    Test Fixes
    * Guard against old versions of YAML::Tiny that worked differently.
      [rt.cpan.org 55500]

Ignore more editor droppings

Update Unicode-Collate to CPAN version 0.57

  [DELTA]

  0.57  Sun Aug 22 22:39:58 2010
    - Unicode::Collate::Locale newly supports locales: ca, et, fi, lv, sk, sl.

In bodies_by_type in sv.c, no need to store for values for HEs.

HEs use 0th root for their arenas, as SVt_NULL doesn't need an arena. But no
code in sv.c needs to know the size details for HEs, because values are all
passed in explicitly from S_new_he() in hv.c

typo in perlport

fix rt75680 - when working with utf8 strings one must always use s+=UTF8SKIP(s) to move to the next char

Most of the regex code where do the two types of increments are wrapped up in macros.

Unfortunately these macros arent suitable in this case because we use goto to jump
inside the loop under some situations, and since this is a one-off case I figured the
modest C&P associated was better than creating a new macro just for this case.

There is still a possible bug here marked by an XXX, which will need to be fixed
once I find out the correct way to simulate strptr--. Additionally I havent found
a test case that actually exposes this form of the bug.

Moral of the story, utf8 makes string scanning awkward... And slow...

document several optree construction functions

In makedef.pl, remove a redundant check for MULTIPLICITY.

Originally the else block was taken when MULTIPLICITY was defined, the if block
being taken for PERL_OBJECT. 6f4183fe04888927 (change 4878) made the if true
for PERL_OBJECT or MULTIPLICITY, rendering this check for MULTIPLICITY
vestigial.

Add vaproto and charbits entries to win32/config.gc64(nox)?

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

Make win32/config.gc64(nox)? match win32/config.gc.

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

Enlighten Porting/checkcfgvar.pl about win32/config.gc64(nox)? files

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

Add missing space in perltrap

It looks as though a space was inadvertently removed. The example makes no sense
without it.

add lex_stuff_pvs()

New macro lex_stuff_pvs(), wrapping lex_stuff_pvn() for literal strings.

Stop installing manpages for XS::APItest*

We don't install the modules anyway, so there's no need to have confusing docs
installed for them.

Remove CALL_FPTR and CPERLscope.

These are left from PERL_OBJECT, which was an implementation of
multiplicity using C++ objects. PERL_OBJECT was removed in 5.8, but the
macros seem to have been cargo-culted all over the core (including in
places where they would have been inappropriate originally). Since they
now do exactly nothing, it's cleaner to remove them.

I have left the definitions in perl.h, under #ifndef PERL_CORE, since
some CPAN XS code uses them (also often incorrectly). I have also left
STATIC alone, since it seems potentially more useful and is much more
ingrained.

The only appearance of these macros this patch doesn't touch is in
Devel-PPPort, because that's a CPAN module.

Fix my $x = 3; $x = length(undef);.

Assignment of length() to a lexical is optimized by passing the
assigned-to variable as TARG, avoiding a pp_padsv and pp_sassign.
9f621b which changed length(undef) to return undef didn't take this into
account, and used SETs (which doesn't set TARG), so the code above left
$x == 3.

Move the declaration for more_bodies outside of an #if defined(PERL_IN_SV_C)

It used to be a static function, hence the location, and I failed to spot that
I needed to move it, when I edited it to be non-static.

Problem spotted by Jerry D. Hedden.

while_readdir.t needs a private directory to run in.

It assumes that it can read the contents twice and get the same result.
It's running in parallel with tests which do sometimes write in t/ so if it
runs at that level, it will sometimes fail due to race conditions.

In sv.c, tidy up body allocation code.

Move the definition of del_body() closer to where it is used.
Inline del_body_allocated() into its only remaining user, del_XPVGV().
Coalesce adjacent C comments.

Inline S_get_arena() into Perl_more_bodies(), its only caller.

Expose more_bodies(), and use it to replace S_more_he().

Convert get_arena() to be static, as now its only user is Perl_more_bodies().

Perl_get_arena() was not in the public API, and neither Google codesearch
nor an upacked CPAN show anything to be using it.

In sv.c, pass in values to S_more_bodies, instead of using bodies_by_type.

Also fix one value passed to the debugging *_printf() in the #else block.

Refactor the loop of S_more_bodies() to be (hopefully) clearer.

Correct the size calculations for FIT_ARENA for SVt_REGEXP.

This should have been done as part of commit 601dfd0af0604fa7. Its omission
did not cause bugs; it merely resulted in slightly less effective arena sizing.

 Bump the version of Module-CoreList in Maintainers.pl

Mention Florian under THE KEEPERS OF THE PUMPKIN

update TOC and perldelta references

retitle perl5134delta.pod

create perl5135delta.pod

Add 5.13.4 epigraph

Disable regeneration of regcharclass.h on win32

There's really no need to do this on the user side, as regcharclass.h is a real
file in the repository. Having this rule, while maybe handy for regexp
developers working on win32, otherwise only causes trouble if regcharclass.pl
changes in a way that doesn't require regcharclass.h to change.

Add 5.13.4 to perlhist

Update Module::Corelist for 5.13.4

Update Changes for 5.13.4

Bump the perl version to 5.13.4

Improve perldelta formatting

Finish up perl5134delta

Fix stack corruption by unsupported filetests

Commit c410dd6ad7 indiscriminately pops elements from the stack
even when nothing has been pushed: file tests without arguments
(testing $_) and stacked filetests don't have anything on the
stack that needs to be removed.

The general idea is that we need to have the same side effects
as if we had called my_stat_flags(), so we shall only call POPs
under the same conditions as the code in my_stat_flags().

We break Switch

HTML::Parser is fixed already

perldelta t/re/overload.t

Add t/re/overload.t to MANIFEST

*.t and test.pl don't need to be executable

Test for +x bits for all the files in exec-bit.txt

The globbing only works when done in the right directory.

Make everything exec-bit.txt lists executable

All these files used to be executable in the release tarballs. Apparently things
also work without that in the repository, but I'd rather add this possibly
unecessary change to blead instead of breaking the upcoming release. This should
probably be looked into again afterwards.

Silence a warning in time64.c.  Need to downgrade a const char *
to a char *.

Moved prototype check to XS code, allowing the .pm file change from yesterday
to be removed.  Thanks to Rafael for pointing out prototypes in XS.