[rt #111640] TODO tests for warnings from -X on File::stat

Don’t stringify GV in numeric cx outside warnings scope

The GV is only stringified for the sake of the warning message, so
there is no point in wasting CPU cycle if it will be unused.

feature.pl: Make 5.even bundle imply 5.odd

Since the 5.18 feature bundle should be added long before 5.18, to
avoid last-minute blunders, and since it’s easy to forget to do it in
advance, have feature.pl do it automatically.

Don’t let method-BLOCK read beyond the stack

$ ./perl -Ilib -e 'use B::Deparse; warn for new{}'
Can't call method "new" on an undefined value at -e line 1.
$ ./perl -Ilib -e 'use B::Deparse; warn for "foo", new{}'
Can't call method "new" without a package or object reference at -e line 1.

Now, why did adding "foo" there change the error message?  Because
new{} looks one past the end of the stack.  Adding "foo" just caused
it to look at the next dropping left behind by B::Deparse, which just
happened to be some non-ref that was not recognised as a package name.

In fact, I can even do this to control what value it picks up:

$ ./perl -Ilib -e '@_ = ("foo"); new{}'
Can't locate object method "new" via package "foo" (perhaps you forgot to load "foo"?) at -e line 1.

And then it calls a method with literally no arguments in @_:

$ ./perl -Ilib -we 'use B::Deparse; @_ = "B::Deparse"; warn new{}'
Use of uninitialized value $class in bless at lib/B/Deparse.pm line 569.
Explicit blessing to '' (assuming package main) at lib/B/Deparse.pm line 569.
Can't locate object method "init" via package "main" at lib/B/Deparse.pm line 588.

And the ultimate:

$ ./perl -Ilib -we 'for(1..1000000) {eval " warn +(1)x$_, new{}"}'
Bus error
$ ./perl -Ilib -we 'for(866..1018) { eval { warn +(1)x$_, new{} }}'
Bus error

OK, that’s enough fun.

With this commit, I’m making it an error to call a method this way
with no arguments.  I’m using the ‘without a package or object refer-
ence’ error message, as opposed to ‘on an undefined value’, because
there isn’t any undefined value; there’s nothing at all.

Document hashref_locked() and hashref_unlocked().  Add tests for them, include debugging by Father C++.

Make lock_hash_recurse() unlock_hash_recurse() exportable; include them in
SYNOPSIS; write tests for them.

Revise 'carp test' test. In general, tests of error messages should be written
with like() rather than is().  Why?  Because we rarely want to test for the
complete error message if that requires us to exactly calculate strings such
as the line number at which an error occurred.

Correct comment typo in op.h

DosGlob.pm: Fix pod syntax

Remove obsolete comment from DosGlob.pm

This comment was added by commit 37248846, without explanation.

DosGlob.pm already had ‘use strict’ three days before that, added in
commit b75c8c73, three days earlier.

I can only assume that 37248846 was written before b75c8c73 was
applied, and that simply adding ‘use strict’ didn’t work, considering
that b75c8c73 changed quite a few things to make things strict-safe.

Increase $File::DosGlob::VERSION to 1.07

Make lvalue subs copy returned PADTMPs in rvalue cx

I was trying to write a JAPH, but did not get what I expected:

$ ./perl -Ilib -e '@UNIVERSAL::ISA = CORE; print "just another "->ucfirst, "perl hacker,\n"->ucfirst'
Perl hacker,
Perl hacker,

This happened because coresubs use leavesublv, to avoid copying the
return value wastefully.

But since this is exactly the same ucfirst op being called each time
(the one in &CORE::ucfirst’s op tree), and since ucfirst uses TARG, we
end up with the same scalar.

We have the same problem with lvalue subs:

$ ./perl -Ilib -e 'sub UNIVERSAL::ucfirst :lvalue { ucfirst $_[0] } print "just another "->ucfirst, "perl hacker,\n"->ucfirst'
Perl hacker,
Perl hacker,

(This is not a regression, as 5.14 gave ‘Can't modify ucfirst in
lvalue subroutine return’.)

So ‘fixing’ coresubs would not be a solution, but a workaround.

The solution therefore is for leavesublv to copy PADTMPs in
rvalue context.

Commit 80422e24c fixed this for potential lvalue list context (i.e.,
for(lvsub()) {...}), but it wasn’t sufficient.

scope.c: Simplify and clarify comment

This comment seems to imply that this code is just working around a
problem in gv.c, which we could simply correct there.  I’ve already
tried making the quoted code in gv.c handle *^H without a hash, but it
doesn’t actually solve the problem.  The real problem is that rv2hv
could also add a hash to *^H, via GvHVn, and that is simply not set up
to deal with autovivifying magic at all, and probably shouldn’t be.

Also, quoting a piece of code that occurs elsewhere is just asking for
it to drift apart.  By this time, the code in gv.c that is quoted in
scope.c is actually three times the length now, and looks completely
different.

Don't warn about "ambiguous without parens" for ctrl-glob

This fixes the following bogus warning [perl #112456]:

  $ perl -e 'undef *^H'
  Warning: Use of "undef" without parentheses is ambiguous at -e line 1.

Compare to the non-warning variant:

    $ perl -e 'undef *{^H}'

[perl #112418] Fix POD paragraph formatting

There was no empty line before a verbatim paragraph, making it
not verbatim.

Increase $Hash::Util::VERSION to 0.12

Add subroutines hash_locked() and hashref_locked() to Hash::Util.

Make @EXPORT_OK, synopsis, and list of functions tested with
can_ok() consistent with one another.  Rationalize the way
functions are grouped within @EXPORT_OK and the other locations.
Add tests for hash_locked(), hashref_locked(), hash_unlocked() and
hashref_unlocked().  Add descriptions for several unit tests which
lacked them.

For RT #112126.

File::Find: typo

Individual files may appear in list of directories to be searched.

Document that, then demonstrate that with additional tests.

For RT #59750.

Individual files may appear in list of directories to be searched.

Document that, then demonstrate that with additional tests.

For RT #59750.

Increase $File::Find::VERSION to 1.21

[perl #112358] Storable: Don’t create RV with no refcnt

Otherwise assigning to it will cause the referent to be freed, because
nothing but Storable knows that it has no reference count.

Storable.xs was creating a new RV without increasing the refe-
rence count on the referent.  It was then using it to call the
STORABLE_freeze method on the object.  Since Perl passes arguments
by reference, it was possible to unref the reference by assigning to
$_[0] within STORABLE_freeze.  That would cause the object’s reference
count to go down.

Increase $Storable::VERSION to 2.35

Remove todo for UTF8 source filters

Source filters don’t really make sense on character streams.  They are
designed for streams of bytes coming straight from a file.  Things
stop making sense if you have ‘use utf8’ in a UTF-8 scalar (does that
mean double-decode?).

It’s for this reason that evalbytes respects source filters, while
eval does not.  (It doesn’t outside the unicode_eval feature, because
it was never really thought about and the implementation didn’t take
it into account, resulting in strange behaviour.  It doesn’t with the
unicode_eval feature, because it was intentionally prohibited.)

[perl #112184] Handle $^N in Perl_magic_set

$^N is a magical variable, like $1 and $2, with the usual ‘sv’
magic.  So it is handled by Perl_magic_get and Perl_magic_set.  But
Perl_magic_set didn’t have a case for it, so it simply ignored it and
did nothing, like a tied variable with an empty STORE method.

Now assigning to $^N has the same affect as assigned to the numbered
variable to which it corresponds.  If there is no corresponding cap-
ture from the last match, or in the absence of regexp plugins, it
croaks with ‘Modification of a read-only value’.

perldata: Consistent spaces after dots

Copy call checker when cloning closure prototype

Otherwise cv_set_call_checker has no effect inside an attribute han-
dler for a closure.

[perl #111000] Let hv_store work on hint hashes

Magic attached to hash elements has its key stored differently depend-
ing on how it was supplied to hv_common.  hv_store passes a string/
length pair to hv_common, while hv_store_ent passes an SV.

magic_clearhint wasn’t able to handle string/length pairs, and only
worked with SVs, resulting in assertion failures or crashes.

This commit fixes magic_clearhint, so that XS code can use hv_store on
hint hashes.

mg.c:magic_clearhint: remove redundant PERL_UNUSED_ARG

XS-APItest/t/hash.t: comment typo

Increase $XS::APItest::VERSION to 0.39

pp_ctl.c:pp_goto: Don’t repeat yourself

No need to say DIE three times.

Produce the right error for goto "\0"

Since we have supported for embedded nulls in strings, we shouldn’t
be using if(*label) to see whether label has a non-zero length.

It’s probably not possible to get a null into a label, but we should
still say ‘can’t find’ rather than ‘must have’ in that case.

[perl #111794] Make goto "" like goto ${\""}

The logic was written in such a way that goto "" just happened to slip
past all the checks and cause pp_goto to return NULL for the next op,
which means the end of the program.

goto ${\""} dies with ‘goto must have label’, so goto ""
should as well.

This also adds other tests for that error, which was apparently
untested till now.

Teach B::Concise about UTF8 labels

Increase $B::Concise::VERSION to 0.90

Corrections to AUTHORS should go to perlbug

regen/opcodes: Rmv evalonce comment

This goes all the way back to when perl 5.0 was being polished up.
The idea behind evalonce is that eval "constant string" should be
optimisable by being compiled ahead of time, just like eval { ... }.
But this could never work properly, because BEGIN blocks would only
fire once.  Also, eval '$x .. $y' is an easy way to create two separ-
ate flip-flops.  There are undoubtedly many other reasons why this
could never work.

So there is no reason to keep this comment any longer, as it is not
(or should not be) a to-do item.

pp_hot.c:pp_entersub: Rmv comment about setting PL_compcv

PL_compcv is meant to point to the currently compiling sub, even dur-
ing an eval’s run time.  (See commit 676a678.)  Therefore, this com-
ment’s suggestion is incorrect.

Removed a redundant 'once'

Fix non-GCC compilation

I mistakenly thought XPUSHs(...) was an expression.  Now it is.

[Merge] Filetest refactoring

This changes the way filetests handle the stack, allowing for sig-
nificant simplification of the code.  It also fixes a bug with ‘-t
bareword’ writing one past the end of the stack.

pp_sys.c:S_try_amagic_ftest: Remove SPAGAIN

Overloading pushes a new stack to prevent the need for this.

pp_sys.c: Remove SPAGAIN after my_(l)stat_flags

As of the previous commit, these two functions no longer affect
the stack.

Move stack extension into FT_* macros

Every filetest operator with the OPf_REF flag set has to extend the
stack for the argument.  One operator was missing it until the previ-
ous commit.  Since we have special return macros for these operators,
put the stack extension there, to avoid a repeat of this type of mis-
take and reduce repetitiveness.

Make -t BAREWORD extend the stack

It appears always to have been this way.  It was writing past the end
of the stack.

pp.h: Missing macro parentheses

This wasn’t affecting anything, but was a bug waiting to happen.

Make filetest ops fiddle the stack less

See commits 8db8f6b697e and d6642e439 for the background.

Instead of sometimes popping items off the stack for filetest operat-
ors and sometimes not, just leave the item on the stack and use SETs
to return the value (which the overload code was doing already any-
way).  This simplifies the code and allows d6642e439 to be reverted.
It also makes things theoretically faster, simply because there is
less stuff happening.

mktables: Defer accepting Unicode's BELL character

As soon as 5.17 started, this created a failing test, which was to
remind us that the deprecation cycle was over, and that the name BELL
should be now used for U+1F514 instead of U+0007.  See commit messages
for 3ffed8c228f4a59b1527ae7ed584ca21386420f7 and
10914c783fe2ea3ee73a870599f30cedb7de96d0

This change puts off the failure unless we try to ship a 5.18 without
dealing with the issue.  I'm not quite ready to deal with it, but will
do so shortly, long before 5.18.

update the release schedule: 5.16 is 2012-2013

update the release schedule for 5.16, 5.17, 5.18

there are no longer any scheduled 5.12 releases

add the 5.17 feature bundle

uconfig regen needed after version bump!

bump version to 5.17.0

create the new perldelta for 5.17.0

include some more data in new-perldelta output

add the 5.16.0 epigraph

perlhist: add the perl-5.16.0 and -RC2 dates

remove the RC2; this will be the real perl 5.16.0

perldelta: fix the example of lvalue ref from substr

reported by Shlomi Fish

perldelta: update acknowledgements with new contributors

minor tag consistency fixes for perldelta.pod

perldelta: note the removal of the perl4 core .pl files

Mistakenly omitted (somehow) from a patch originally from Reini
Urban

[perl #112924] [PATCH] Clarify test instructions in INSTALL

While trying to track down a failed test, I found the instructions in
the INSTALL file to be less than optimal.

This patch re-orders the suggestions to put the ones most likely to work
first, clarifies which directory you should be in, and moves a "see-also"
type reference to where it is more likely to be useful.

The catalyst for this was that

    ./perl -MTestInit cpan/Archive-Extract/t/01_Archive-Extract.t

fails because it can't find strict.pm.  (See [perl #1122926].)  Since it's
probably not the only such case, I thought it useful and prudent to
advise using t/harness instead as a first recourse.

perldelta: fix bad references to "unicode_strings"

The documentation written for 2e2b2571 erroneously mentions
"unicode_semantics" instead of "unicode_strings".

prevent PERL_UNICODE from affecting t/mro/package_aliases_utf8.t

perldelta: known issue: t/op/filetest.t

note the gcc -O2 and link-time-optimization problem

our next release is RC2

perldelta: Americanise spellings

reflect Socket update in Module::CoreList

Update Socket to CPAN version 2.001

2.001   CHANGES:
         * Apply (modified) patch from ppisar@redhat.com to fix memory
           addressing bug with Zero() - RT76067
         * Document that inet_pton() doesn't work on hostnames, only textual
           addresses - RT76010
         * Ignore any existing-but-undefined hints hash members to
           getaddrinfo()

Done for the critical RT76067 fix.

perldelta typo fixes (from mauke)

Revert part of 34d9f36f9

I was going to apply this after code freeze, but I made a mistake
when switching branches locally and ended up combining it with
another commit.

AUTHORS: Shirataka -> Shirakata

perldelta: extraneous double spaces

v5.16 RC0 perldelta cleanup

Below is a patch with some simple typo and verbosity cleanup in
the current pod/perldelta.pod in blead as of ~30 minutes ago.

[perl #112944] perldelta: typo

Add Shirataka Kentaro to AUTHORS

add 5.16.0-RC0 and -RC1 to perlhist

minor grammar correction

thanks, Jim Keenan!

add Daniel Kahn Gillmor to AUTHORS

document the yet-explained Win32 test hanging

We will ship with this unfixed unless someone comes up with the
cure in the next week.

perldelta: fix a noun/verb number agreement

reported by mauke

skip t/win32/runenv.t unless -DPERL_IMPLICIT_SYS

this test fails without PERL_IMPLICIT_SYS, as reported by Steve
Hay in <CADED=K4EqXkJa2uC13wVYY_=uGDCx=uQ_rXu3Me4+3FvVM8D+g@mail.gmail.com>

Revert fixes for [rt.cpan.org #61577]

These changes introduced some test failures on AIX and other platforms,
and rather than dig around for more failing platforms during the RCx
period, we will revert this to reapply later when it is more tested.

This reverts commit 01b71c89216c9f447494638a5d108e13c45c3863.

This reverts commit b6903614db213f07401367249dc84c896eb099b7.

This reverts commit 271d04eee1933df0971f54f7bf9a5ca3575e7e6a.

next release will be RC1

perldelta: fix version named in acknowledgements

In the Linux hints, invoke gcc with LANG and LC_ALL set to "C".

The output of gcc -print-search-dirs is subject to localisation, which means
that the literal text "libraries" will not be present if the user has a
non-English locale, and we won't determine the correct path for libraries
such as -lm, breaking the build. Problem diagnosed by Alexander Hartmaier.

Don't test that errno is still 0 after POSIX::f?pathconf

I think the best we can do with respect to the f?pathconf tests is to
make sure that the perl call doesn't die, and that the system call
doesn't fail.  And it's arguable we should only be testing the former.
But since we've been testing more that this anyway, it's probably safe
to test both.

With respect to the sysconf call, I think we shouldn't test more than
that perl doesn't die.  Any further testing would require different
tests based the argument being passed in.  Before doing that, it's
probably worth considering the purpose of the tests.  I don't think we
really want to test that POSIX has been implemented correctly, only that
our layer over it is correctly implemented.

This fixes RT #112866.

perldelta: Remove duplicate paragraph

study as no-op is a bugfix, not performance enhancement

perldelta: Add ‘(5.14.2)’ markers

perldelta: Explain the ‘(5.14.1)’ markers

perldelta: Use single quotes in C<>

C<> renders as "..." in nroff, so C<... "..." ...> ends up looking weird.

perldelta: Use L<> to link to changed module pods

Spotted by Vincent Pit

perldelta: Reorder to avoid pronoun confusion

Spotted by Zsbán Ambrus

perldelta: typo

Spotted by Zsbán Ambrus

perldelta: Add future deprecation text about \Q

perldelta: misuse of commas

perldelta: typo