perlbug did not previously generate a From: header. While some MTAs do
the "right" thing and insert a valid "From:", not all of them do,
potentially resulting in dropped mail.

Ensure ExtUtils::CBuilder's 04-base.t cleans up its temporary directories.

The tempdir() function has the rather confusing default of *not* cleaning up
("because of issues with backwards compatibility") so one needs the CLEANUP
flag. (The analogous newdir() method in the OO interface doesn't suffer this
gotcha.)

Document 'test_porting' and start a section on how committing to blead

Update Encode to CPAN version 2.42

  [DELTA]

  $Revision: 2.42 $ $Date: 2010/12/31 22:48:48 $
  ! Encode.xs
  ! Unicode/Unicode.xs
    Applied: RT#64371: Update for 5.14 API changes
    http://rt.cpan.org/Ticket/Display.html?id=64371

Avoid using OP_NAME() in unimplemented_op(), as it won't give useful strings.

Using OP_NAME() isn't going to be helpful here. Firstly, it doesn't cope
with out of range op numbers - it only "special" cases op_custom.  Secondly,
as the three ops we "panic" on are padmy, mapstart and custom, if we get
here for a custom op then that means that the custom op didn't have an
implementation. Given that OP_NAME() looks up the custom op by its pp_addr,
likely it will return NULL, unless someone (unhelpfully) registers
&PL_unimplemented_op as the address of their custom op.  NULL doesn't
generate a useful error message. "custom" does. */

Porting/Maintainers.pl: add (MY)META.json to ignored files

standardise amagic method naming

Some amagic-related macros take the full method enumeration name,
(e.g. "add_amg"); while others "helpfully" allow you to pass a shortened
version, ("add"), and do a CAT2(meth,_amg) behind the scenes.

Standardise on passing the full name; this makes it less confusing and
allows you to grep for the enumeration name in the source.

It updates two macros to accept full enumeration names: tryAMAGICunTARGET
(which isn't used outside the core apparently), and AMG_CALLun, which is
replaced by a new AMG_CALLunary (since AMG_CALLun is used outside the
core).

VMS nits in cpan/Test-Harness/t/source.t

1.)  Can't get the basename of a directory in native format.

2.)  Symlinks still a bit dodgy -- skip tests for now.

Submitted upstream at:

https://rt.cpan.org/Ticket/Display.html?id=64353

Skip ExtUtils-CBuilder base tests expected and known to fail on VMS.

Merge the opcode bodies for pp_bind and pp_connect.

Most socket ops weren't warning for unopened handles unless 'closed' was enabled

They were checking that category 'closed' was enabled for warnings before
calling report_evil_fh(), which in turn was (correctly) checking category
'unopened'.

Convert a pp_sys test to use warnings 'closed', instead of the broader 'io'.

It's actually only testing warnings enabled by 'closed', so test using the
tighter category.

pp_leavewrite wasn't warning for unopened handles unless 'closed' was enabled.

It was checking that category 'closed' was enabled for warnings before calling
report_evil_fh(), which in turn was (correctly) checking category 'unopened'.

Better handling of magic methods freeing the SV

This is a fix for RT #81230 (and more). Currently, mg_get() works around
the case where the called magic (e.g. FETCH) frees the magic SV. It does
this by unconditionally pushing the SV on the tmps stack before invoking
the method.

There are two issues with this. Firstly, it may artificially extend the
life of the SV. This was the root of the problem with #81230. There, the
DB_File code, under -T, created a tainted tied object. Accessing the
object (within FETCH as it happens), caused mg_get() to be invoked on the
object (due to the taint magic), and thus extend the life of the object.
This then caused c<untie %h if $h{k}> to give the warning
    untie attempted while 1 inner references still exist.
This only became noticeable after efaf36747029c85b4d8825318cb4d485a0bb350e,
which stopped wrapping magic method calls in SAVETMPS/FREETMPS.

The second issue issue that this protection only applies to mg_get();
functions like mg_set() can still segfault if the SV is deleted.

This commit fixes both problems as follows:

First, the protection mechanism is moved out of mg_get() and into
save_magic() / restore_magic(), so that it protects more things.
Secondly, the protection is now:

* in save_magic(), SvREFCNT_inc() the SV, thus protecting it from being
  freed during FETCH (or whatever)

* in restore_magic(), SvREFCNT_dec() the SV, undoing the protection
  without extending the life of the SV, *except* if the refcount is
  1 (ie FETCH tried to free it), then push it on the mortals stack
  to extend it life a bit so our callers wont choke on it.

Update Renee Baeckers mail address

Signed-off-by: David Golden <dagolden@cpan.org>

properly clean up work files from testing GDBM_File

close the produced test file so win32 can unlink it

Previously runs would leave a GLOB(0x...) file in t/

skip ExtUtils-CBuilder base tests expected and known to fail on Win32

As report_evil_fh() checks WARN_{CLOSED,UNOPENED}, don't duplicate this.

This trades reduced code size for an extra function call in the error path with
warnings disabled. (And removes a duplicated check for the case of taking the
error path *with* warnings enabled.)

Removing the check from Perl_do_close() does not change behaviour, as io is
NULL there, hence Perl_report_evil_fh() will always be checking WARN_UNOPENED
and setting vile to "unopened".

As report_wrongway_fh() checks ckWARN(WARN_IO) internally, don't duplicate this.

This trades reduced code size for an extra function call in the error path with
warnings disabled. (And removes a duplicated check for the case of taking the
error path *with* warnings enabled.)

Remove OP_phoney_{IN,OUT}PUT_ONLY, as they are no longer used or supported.

Argument op to report_evil_fh() is always PL_op->op_type, so need not be passed

The io argument to report_evil_fh() is always GvIO(gv), so need not be passed.

Extract the OP_phoney_* code from report_evil_fh() into report_wrongway_fh()

Previously Perl_report_evil_fh()'s body was just an if/else at the top level -
a good sign that it is actually implementing two disjoint functions.

In pp_sockpair, remove duplication of code to close the supplied file handles.

It's not necessary to (also) test gv1 and gv2 before returning undef as an
error, because io1 will automatically be NULL if gv1 is NULL, and similarly
io2 if gv2 is.

Fix typo in warning code added to pp_sockpair in c289d2f7288798f8.

That should be gv2, not gv1. However, I believe that it's impossible to reach
the warning code, given the structure of the optree that the perl 5
implementation produces, as gv1 and gv2 will never be NULL, and GvIOn() will
always return non-NULL. (Or croak, but that won't return).

avoid undefined behaviour in fetch and modify of SP from 0a0e3cc51e

pp_sys.c: In function ‘Perl_pp_send’:
pp_sys.c:1845: warning: operation on ‘sp’ may be undefined

Add CPAN::Meta::YAML as a dual-life core module

Simplify return handling for tied handle OPEN/PRINTF/READ/WRITE.

Their code used to explicitly move the return value from its current position
on the stack, to the position that was top of top of the stack just before the
call was made. However, the POPBLOCK() in pp_leavesub will restore the stack
pointer, and passing G_SCALAR to call_method() will force exactly one return
value, so all of this is needless defensiveness.

A perldelta entry describing the refactoring of the chop/chomp implementation.

Move do_chomp() from pp.c to doop.c, and make it static.

It was never part of the public API, and only ever used by pp_{s,}cho{,m}p.

Merge the opcode bodies for chop/chomp and schop/schomp.

Merge Perl_do_chop() and Perl_do_chomp().

They share code for dealing with PVAVs, PVHVs, read only values and handling
PL_encoding. They are not part of the public API, and Google codesearch shows
no users outside the core.

Convert Perl_do_chomp() to the same prototype as Perl_do_chop().

Pass in an SV to hold the count, rather than returning the count.

Refactor pp_chomp to process arguments in the same order as pp_chop.

Previously list chomp worked from last to first, whilst list chop worked from
first to last.

Fix links (and hence the podchecker test) after cce04bebd8af026c.

[perl #81016] Document ord("")

The following patch clarifies what ord("") does in pod/perlfunc.pod.

From the current documentation, it's not documented what it does, and one
could guess several different things: returns zero, returns undef, returns
zero with a warning, returns undef with a warning, dies.  (There's precedent
for dying: some BASIC implementations give an error if you ask for ASC("")
-- here ASC is their equivalent of our ord function.)

Upgrade to Thread::Queue 2.12

Upgrade to Thread::Semaphore 2.12

Reorganize perlhack.pod

Following on an IRC conversation, I've attempted to reorganize
perlhack for greater clarity.  I have only cut and paste blocks
of text and amended section titles and levels.  (I have not addressed
any of the numerous factual issues which remain.)

The resulting guide should be clearer for those trying to skim the
table of contents to understand what is covered in perlhack and
whether it is worth an in-depth read.

I see this change as the first step towards future improvements.

[perl #81218] Note that perlbug to p5p is subject to moderation

Calc.pm: Fix _modpow() part of RT#63237

Fix RT 62764: Math::BigFloat->bcmp() fails.

Why: The Math::BigFloat->bcmp() method returns the wrong result when the
exponent is too large to be represented exactly as a Perl numerical
scalar. In such cases, bcmp() returns 0 because it fails to distinguish
between the two exponents.

How: With this fix, bcmp() does not convert the exponents to Perl
numerical scalars, but keeps them as arbitrary precision integers, thus
returning the correct result regardsless of the number of digits in the
exponent.

Test: Two tests added. These tests fail with the old code.

Files:

- lib/Math/BigFloat.pm: New version of bcmp().

- t/bigfltpm.inc: Add two tests confirming desired behaviour.

- t/bare_mbf.t: Increment test count.

- t/bigfltpm.t: Increment test count.

- t/sub_mbf.t: Increment test count.

- t/with_sub.t Increment test count.

Fix RT 25274: Math::BigInt::Calc->_num() overflow.

Why: Math::BigInt::Calc->_num() converts a big integer (in the internal
format) to a Perl scalar. If the big integer is too large to be
represented as a Perl scalar, it might return a Perl scalar numeric
"nan", rather than "inf". The reason is that the current algorithm might
multiply "inf" by "0", giving a "nan" which propagates. The following
example illustrates the bug:

    perl -MMath::BigInt=lib,Calc -wle \
        'print Math::BigInt->new("1e999999")->numify()'

How: This fix computes the output in a different way, never multiply
"inf" by "0".

Test: It is not obvious to me how to test this automatically in a
portable way, since Perl has no standard way of stringifying a scalar
numeric infinity. However the desired behaviour is verified manually and
no existing tests fail with the new code.

Update perldelta for threads and threads::shared

[PATCH] Upgrade to threads::shared 1.36

Signed-off-by: Chris 'BinGOs' Williams <chris@bingosnet.co.uk>

[PATCH] Upgrade to threads 1.82

Signed-off-by: Chris 'BinGOs' Williams <chris@bingosnet.co.uk>

Quote 'threads' in test.pl

If unquoted, it may throw a warning when threads.pm can't be loaded.

Update Encode to CPAN version 2.41

  [DELTA]

  $Revision: 2.41 $ $Date: 2010/12/23 11:05:58 $
  lib/Encode/MIME/Header.pm
    Applied: RT#63387 encode of MIME-Header inserts too much whitespace
    http://rt.cpan.org/Ticket/Display.html?id=63387
  t/Aliases.t lib/Encode/Alias.pm
    Applied: RT#63286: Various Encode::Alias improvements
    http://rt.cpan.org/Ticket/Display.html?id=63286

update RMG discussion of post-release actions

use.perl.org is dead.  www.cpan.org is maintained by Ask, not Jarkko
any more.  Separate CPAN/src and www.cpan.org issues into distinct steps.

Fix IS_UTF8_CHAR() to recognise start bytes 0xF5, 0xF6, 0xF7.

The refactoring of 3b0fc154d4e77cfb inadvertently introduced a bug
in Perl_is_utf8_char() and its callers, such as Perl_is_utf8_string(),
whereby the beyond-Unicode characters 0x140000 to 0x1fffff were no longer
recognised as valid.

Convert lib/utf8.t to strict and warnings throughout.

Re-sort MANIFEST after a7dcd8d74e57d2a8.

No reference to $Id: in generated files

They might have served a purpose in the original files, but Nicholas
and Zefram expressed their concern that in the generated files, these
tags are misleading and unneeded.

Move metaconfig control comments into its own files

Restore a reference to $perlpath. Installation needs it

cbeaa1895 removed the reference to $perlpath

sin6_scope_id is supported by Winsock2.

The field is missing in the headers included with VC6, but
commit 1ab9ebc11 adds it in our win32/include/sys/socket.h.

avoid emitting control characters in test comments

multiple watchdog()s in the one test script are meaningless

d_sin6_scope_id configuration probe for VMS.

It's present on recent versions, but not all versions.  Follow-up
to f53580fec42f3b12264ee27b756dec257c0bb77a.

regexec.c: white-space only

Commit 9e2c615305806d76433db342e5659ffeccc3746a didn't adjust the white
space for the changes that it introduced.  This patch does that.

regexec:c Remove unreached code

The new name of ANYOF_LOC_NONBITMAP_FOLD makes it clear that the only
way folding can be aplicable here is if it is under locale.

Change name of regex intrnl macro to new meaning

ANYOF_FOLD is now used only under fewer conditions.  Otherwise the
bitmap of character 0-255 is fully calculated with the folds, and the
flag is not set.  One condition is under locale, where the folds aren't
known at compile time; the other is for things accessible through a
swash.

By changing the name to its new meaning, certain optimizations become more
obvious.

Add sin6_scope_id probe (LeoNerd)

Tweak IPC-Open3.t to cope with \r\n line endings on Win32.

The refactoring of bd29e8c290c68f4f to use Test::More broke the tests on Win32,
because it didn't realise that the previous code was relying on the test
harness being line ending agnostic.

stdint.h has yet to appear on VMS.

Despite the compiler's claiming C99 compliance.  What we probably
really need here is I_STDINT, or perhaps HAVE_INTMAX and
HAVE_UINTMAX, determined by Configure and friends.  But in any case
07208e09d4435b4e72743076e0bc290ef4b34911 broke the build, so this
gets it going again for now.

utf8.c: add to comment

regexec.c: Remove unnecessary statements

These variables are set to other values just a couple of lines below

perltodo: Revise utf8 todo

utf8.c, .h: Clarify pod and comment

mktables: fix typo in comment

Change regexes to debug dump non-ASCII as hex.

instead of the less familiar octal for larger values.  Perhaps they
should actually print the actual character, but this is far easier than
the previous to understand.

pv_escape: Add option to dump all non-ascii as hex

This patch adds an option to pv_escape() to dump all characters above ASCII
in hex.  Before, you could get all chars as hex or the Latin1 non-ASCII
as octal, whereas the typical values for these that people think in are
given in hex.

dump.c: correct pod statement

update TOC for perl5138delta and perl5139delta

create perldelta for 5.13.9

epigraph for 5.13.8

replace gratuitous Unicode dashes

tweak later segments of release procedure

Slightly more clue about the indexer failures.

Move some post-release steps into a new "the day after" section.

fix -tree test for non-threaded perl

add autodie tag :2.1001 for blead-updated version

add new release to perlhist

update Module::CoreList for 5.13.8

update META.yml

acknowledgements in perldelta

perldelta wording nit

update references to earlier versions

link perldelta entries to perlapi sections

bump perl version number for 5.13.8

perldelta details of module updates

near-finalisation of perldelta

Put perldelta into mostly-final form, and copyedit it.

perldelta for updated modules

bump version of many modules

Core-only modules that have changed from v5.13.7, and dual-life modules
that have changed from v5.13.7 and didn't show up in earlier passes.

bump Unicode-Normalize version for blead changes

bump Unicode-Collate version for blead changes

bump threads-shared version for blead XS changes

bump ExtUtils-CBuilder version for blead change

bump autodie version for blead test fix

perldelta up to ca88a729

perldelta for 8e88cfee26d866223a6b3bfffce6270271de00db