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>

Update Locale::Codes to 3.12

We're skipping inclusion of some files such as the unused Build.PL,
INSTALL, etc. cpan/Locale-Codes/internal/* should probably also not be
included in perl core.

Update Steffen Mueller's email address in AUTHORS

Add contact information for Sullivan Beck

... and mark him responsible for Locale::Codes, which is now
"upstream => cpan"

Make the Module::CoreList function API consistent

  Some functions could only be called with class method type syntax.
  Other functions could only be called with normal sub syntax.

  This makes the API consistent, functions can be called with
  either syntax.

Correct spaces in MANIFEST.

Fix MANIFEST by adding new file

387b6f8d0cc8df27b7a577cf8a47f25b089895e4 added new file to
cpan/ExtUtils-ParseXS without adding it to MANIFEST.

Upgrade Scalar-List-Utils to 1.23 from CPAN

Document the INCLUDE_COMMAND XS directive

ExtUtils::ParseXS has recognized the INCLUDE_COMMAND directive as a
smarter replacement for "INCLUDE: perl -e ... |" since version 2.21_01.

Upgrade ExtUtils::ParseXS to 0.2205

Remove Class::ISA from the core distribution. Get it from CPAN now.

Remove Pod::Plainer from the core distribution. Get it from CPAN now.

Remove Shell from the core distribution. Get it from CPAN now.

Remove Switch from the core distribution. Get it from CPAN now.

create perl5131delta

Add 5.13.0 to perlhist

Mention L<perlpolicy> and link to it in SEE ALSO

Only all-upper case "special" POD sections

"GETTING ACCESS TO THE REPOSITORY" is a bit too loud compared to
"Getting access to the repository". The POD standard itself doesn't
have anything to say about this, but most of our long =head1 sections
in pod/*.pod don't use all-caps.

Update to final content for release

Commit messages should start with a ~50 char summary

The first sentence is stolen from Git's own
Documentation/SubmittingPatches. We have a lot of commit messages that
overflow `git log --pretty=oneline', but it wouldn't hurt if we change
that sooner rather than later.

POD: Use F<> for F<utils/perldoc> and F<utils/perldoc.PL>

Don't tell the reader to `git checkout origin/*'

This results in a branchless checkout (detached HEAD). Making changes
like this is a valid way to use Git, but IMO not something we should
be recommending.

Branches are cheap and it's harder to lose your commits if they're on
a named branch than on a detached HEAD.

Update Module::CoreList for 5.13.0

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

[rt.cpan.org #56740] Format Perl versions >= 5.6.0 as X.Y.Z

Decimal style Perl version numbers are increasingly awkward.  5.01 or
5.010000 is hard to equate with 5.10.0.  It would be easier to read if
Module::CoreList formatted version numbers from 5.6 onward as dotted
versions to match how they are presented elsewhere.

Don't show aliased perl version numbers twice in corelist -a output

(bug [rt.cpan.org #56739])

Typo fix in version number alias 5.013

Module::Pluggable has upstream => cpan

According to Simon Wistow.

Replace mention of Switch.pm with given/when

Add another alias for Robin Barker

Spaces not tabs

Bump version number EU::MM is "first-come" so we have forked it

Bump version due to d97c33b505dd45c052a8bf42014917701b4502a5

Bump version due to 255fdf19250671082618dae106a246f12379dbe6

Bump version number EU::MM is "first-come" so we have forked it

Document prctl() via $0 assignment (7636ea95c5) in perldelta

Add =head2 to the perldelta change in da76b8593e

clarification of multi-bracket format

Possible undefined behaviour, spotted by gcc 4.5.0 and HP's updated compiler.

Revert "Documentation tweaks to Text::Tabs"

This change was reported upstream in <https://rt.cpan.org/Ticket/Display.html?id=56699>.
Committing it directly to blead was a mistake.

This reverts commit f6e2320c520b8d610a864cd42aece1f0df2db895.

Let the cmdVERSION usage message expand $0

It makes more sense than literally printing "usage: $0 ..."

Documentation tweaks to Text::Tabs

Bring Porting/Maintainers in sync with reality regarding Scalar-List-Utils

Update to MIME-Base64 3.09

Fixes issue where the Quoted-Printable encoder would sometimes output
lines that were 77 characters long.  The max line length should be 76.

This reverts revert 9ef40646096384d9f65acffab1b9a2df07d03a0c.

Escape patch strings before embedding them in patchlevel.h