perldelta: Sort updated modules

perldelta for ff781254a3

Update ExtUtils-Manifest to CPAN version 1.69

    [DELTA]

    1.69  2014-11-16
      - also skip _build_params, generated by Module::Build::Tiny

perldelta for #123103 / 1d5686ec7b

perldelta for 96d7c888

perldelta for f4725fad1a

perldelta for #122782 / e660c409

perldelta for #122829 / 00e40766a

perldelta for #121374 / a2b5b206

perldelta for #123057 / e4e95921

perldelta for 03d05f6e34e

perldelta for c57eecc5

perldelta for 82209a5

perldelta for 25f5d5405

perldelta for ce409cc88

perldelta for #122695 / 51a82aa9fb

perldelta for #122680 / e1729dc6ad

perldelta for #123029

Inadvertently fixed by c0683843e92.

Follow-up commits specific to this bug:

c4a33ec [perl #123029]: add regression test
a4f1ca6 state.t: Improve test for #123029
9f53080 sv.c: Always #define COW threshold constants

perldelta for fa367bcbf7

Increase $I18N::Langinfo::VERSION to 0.12

$@ check in I18N::Langinfo documentation is reversed

misc optimizing in DynaLoader

dl_last_error is "#define dl_last_error (SvPVX(MY_CXT.x_dl_last_error))"
since the data is already in a SV *, use newSVsv instead of turning it into
a strlen-ed (eventually) char *.

-win32 dl_load_file never uses flags, so don't do a SvIV in void context
- dl_load_file cant use "flags=NULL" because of test
"calling DynaLoader::dl_load_file() with no argument" in DynaLoader.t
-OS_Error_String stop repeatedly computing address into MY_CXT and
 derefing after PerlProc_GetOSError

function sizes in machine code bytes on VC 2003
b4 0x69 OS_Error_String af 0x67
b4 0x9D _XS_DynaLoader_dl_error af 0x6B
b4 0x108 _XS_DynaLoader_dl_load_file af 0xD6

readd noreturn and silence "noreturn that returns" warning on MSVC

Based on commit 73758d77 (by me), in commit 117af67d629 more things got
noreturn removed on MSVC. See also ML post
"(Hugmeir) Re: [perl.git] branch blead, updated. v5.21.0-377-gdc3bf40"
This caused a measurable increase in machine code size in 117af67d629 .
In commit 73758d77 , the reason there was no increase is
Perl_magic_regdatum_set is called only through a magic vtable. Optimizing
this to noreturn is forbidden unless the struct member type specifies it
(and it obviously doesn't, since this is the magic vtable).

The other not-noreturn on MSVC function, Perl_screaminstr, has no core
usage (its only reference is in the export table) or CPAN grep usage so
therefore it is being removed. It was made fatal in commit 9e3f0d16db .

before .text section of perl521.dll on VC 2003 32b, 0xc66a3 bytes, after
0xc6453

perldiag: Correct ‘Can't use %s for loop variable’

It is no longer true that the variable can only be a simple scalar.
More to the point, this message should never be reached.  If you try
to use something prohibited as your loop variable, you get a syntax
error or ‘Missing $’.

op.c: cmmoent typo

op.c: Correct comment added by c427f4d2d

Fix double free with unterminated /(?{ <<END })/

If we are parsing from a stream (file), and we are inside a quote-like
operator, and we find a here-doc marker that tries to extract the
here-doc body from the stream (this is the last line of the quote-like
operator, or it only has one line), and the the here-doc terminator
cannot be found, then we end up trying to free a scalar twice:

$ ./miniperl -e '"${ print <<END"'
Can't find string terminator "END" anywhere before EOF at -e line 1.
Attempt to free temp prematurely: SV 0x7fcafb82fd98, Perl interpreter: 0x7fcafb803200 at -e line 1.
Attempt to free unreferenced scalar: SV 0x7fcafb82fd98, Perl interpreter: 0x7fcafb803200 at -e line 1.

I caused that in v5.17.3-187-g3328ab5.

The current line of code in the parser is usually stored in
PL_parser->linestr (aka PL_linestr) and gets freed when the parser
itself is freed.

The heredoc parser, when extracting the body from a stream, tempor-
arily sets aside PL_linestr, replacing it with another SV.  If it
doesn’t find the terminator, it frees the PL_linestr value that has
been set aside, under the assumption that parser_free will take care
of freeing the new value.

Inside quote-like operators that does not work, because PL_linestr has
been localised and set to a new value, already prospectively freed via
SAVEFREESV, in sublex_push.  So we can’t free that value again.

Update Test-Simple to alpha 074

Don’t always skip in-memory tests is t/base/rs.t

There is no PerlIO::via::scalar, as far as I can tell.  In-memory han-
dles just use PerlIO::scalar.

The code was added by cd1a9f552.  I’m not sure it was ever cor-
rect.  It was intended to skip tests under -Uuseperlio (see <20080614182005.yxuyk1mntwkog0o0@horde.wizbit.be>), but
skipped them on all builds.  I’m not sure that we even support
-Uuseperlio any more.

In the mean time, 1ab48e3a screwed up the test count.

t/base/rs.t: Suppress warning

Put PL_cop_seqmax++ code in one spot

[perl #115066] Fix wrongly nested ‘use’ deparsing

B::Deparse was incorrectly putting ‘use’ statements and BEGIN blocks
inside other ‘use’ statements containing do-blocks, even if they were
originally outside.

Subroutines (except for cloned closures, which we don’t have to worry
about here) have an OUTSIDE pointer, pointing to the outer sub con-
taining the declaration of the sub in question.  So we can check that
to make sure we are putting the sub declaration in the right place.

Not only does this fix the reported case, but it also will allow
sequence numbers in inner subs to be reused by statements in outer
subs, which I may need in a future commit.

Revert the ‘return’ part of 6edcbed6

The gotos were jumping over initializations, causing
C+++threads+debugging to fail.

t/loc_tools.pl: try code page 1252 locales

This makes sure this common locale on Windows doesn't get overlooked.

t/loc_tools.pl: Nits in comments, white-space only

Silence spurious locale warnings

When testing the locales on the system, we don't want to be warned that
particular ones may not work well with Perl, as the point of the test is
to find and list the incompatibilities.

When looking for locales suitable for some purpose, we try every one
possible until we find one that matches what we're looking for.  This
may entail briefly switching into locales that aren't fully compatible
with Perl.  Again we don't want to be warned about such a locale.  We're
not using it for anything except to determine suitability anyway.

Remove unused exit handler in vms/vms.c.

This code became obsolete in ff7adb5275976 and has just been
sitting there taking up space.

Spell behaviour consistently in perlsub

We had six instances without U, and two with it.

make debugging easier in memory allocator code in perl.c and util.c

-show intermediate values to make C debugging easier
-Perl_safesysfree overwrote var where with a different value, this caused
 alot of confusion for me of trying to hunt for a pointer from a stack
 trace with conditional breakpoints, so don't change var where in an
 unoptimized build
-in Perl_safesysrealloc and Perl_safesysmalloc provide 1 exit path, so
 the returned value is easily seen and BPed on unoptimized builds

Deparse.t: To-do test for #115066

Helper function for 'close enough with epsilon'.

The skip count was all wrong.

Though that shows that acosh must be quite widely supported.

The 0d71ffa7 was quite sloppy with the sub-skips.

Remove vmsish pragma from one-liners in Test::Simple's exit.t.

Awaiting upstream application at:

https://github.com/Test-More/test-more/pull/494

This was broken by a recent change that added double quotes around
the double quotes that were already there.

It also hasn't been working as intended for these tests for some
time as they manipulate the exit code directly in an END block,
and that sneaks by the effects of the pragma when run at compile
time.

This works:

  $ perl  -e "require vmsish; import vmsish 'hushed'; END {$? = 1};"

This (which is what the tests are doing) doesn't work:

  $ perl -Mvmsish=hushed -e "END {$? = 1};"
  %SYSTEM-F-ABORT, abort

So we'll just live with the noise from the exit messages.  They
don't seem to cause any trouble for the core test suite.

Make $/=-1 warning default like other dep warnings

Fold t/lib/warnings/irs into mg

It took me a while to find these tests.  Most of these tests are arranged
by which C file they occur in.  (These warning tests were originally
added to make sure all warnings in all C files get tested.)

t/lib/warnings/mg: Remove comment

The warning this refers to was removed (or moved to perl5db.pl) in
commit bbdae7b29367.

The miniperl skip in utf8cache.t was wrong.

But also test.pl needed fixing to be able to skip correctly.

quadmath: try finding if only available as gcc internal library.

C99 math: lgamma and tgamma emulations.

fdim NaN guard.

These are all emulated, so can be tested.

Emulate signbit() if necessary.

fpclassify is emulated if necessary.

checkcfgvar.pl needs autodie needs Fatal needs Scalar::Util needs lib.

We no more need to guard these with HAS_ACOSH.

Remove Cygwin cases.  Case.

Remove unneeded Win32 cases.

undef c99_foo ifndef HAS_FOO

cosh and log10 are already C89.

Remove now empty case.

Tru64 more undef hints.

Remove unneeded Tru64 cases.

VMS should now be handled.

We now have symbols for llrint and llround.

Android should be doing the Configure scans.

Remove trivial unneeded cases of not having C99 math.

Only #ifndef gcc cases remain.

Remove unneeded HP-UX cases.

quadmath has tgammaq, not tgammal.

Make the C99 *l test a little bit more accepting.

The NO_C99_LONG_DOUBLE_MATH is no more used.

AIX: Configure now detect this, leave the comment.

(Some AIX do not have long double math interfaces.)

VMS configure.com C99 math symbols

checkcfgvar.pl --regen --default=undef; make regen

Add Configure symbols to all the missing C99 math APIs.

Almost all - e.g. isless() will stand in for the other isfoo()
order comparison macros.

Also not going for all the *l() versions of each interface.

Sort perldiag

B.pm: Document COP_SEQ_RANGE_*

lib/locale.t: Display debug information under harness

This just changes any debugging (which is by default off) goes to
STDERR.

Reinstate "Raise warnings for poorly supported locales"

This reverts commit 1244bd171b8d1fd4b6179e537f7b95c38bd8f099,
thus reinstating commit 3d3a881c1b0eb9c855d257a2eea1f72666e30fbc.

handy.h: Add missing parentheses to macro #define

These being missing caused 3d3a881c1b0eb9c855d257a2eea1f72666e30fbc
to have to be reverted.  It only shows up on platforms that don't have
an isblank() libc function.

perlfunc: Fix typo

Add warning message for locale/Unicode intermixing

This is explained in the added perldiag entry.

tie.t: Don’t skip base.pm test unconditionally

but only if we are under miniperl and base has not been built yet.

I made a mistake in 19295df3 by omitting the ?.  Thanks to Karl
Williamson for noticing it.

Stop crypt($tied,...) from calling FETCH twice

It was doing so if FETCH returned a UTF8 string the first time.  It
then turned on the UTF8 flag on the result of the second FETCH, even
if it wasn’t UTF8:

$ ./miniperl -e 'sub TIESCALAR{bless[substr(chr 256,0,0), chr 255]} sub FETCH{shift @{$_[0]}}tie $t, ""; crypt $t, ""'
Wide character in crypt at -e line 1.

(There are no wide characters fed to crypt() there.)

Turn off UTF8 flag on crypt ret val

crypt was not turning off the UTF8 flag on its return value.  On some
systems, such as VMS, it returns a string of random bytes, not neces-
sarily ASCII.  If the UTF8 flag is already turned on on its target
(the SV used to return the value), then the return value is scrambled.

Normally crypt’s target is its own, and never gets the UTF8 flag
turned on.  However, $lexical = crypt(...) gets optimised, such that
$lexical becomes the target of crypt, and crypt writes to it directly.
So it very well may have the UTF8 flag on already.

There are already tests in lex_assign.t that fail
on VMS because of this (see the thread that includes
<CA+vYcVyvq=j-vdWcODSfCjzM9-cYFYfY4hE-B1aWRRP4-=3KNA@mail.gom>), but
those tests are fragile, and this should be tested more explicitly.

Revert "t/TEST: add -w to shebang, fix a used-once warning"

This reverts commit da6e3e61d3d3194e7f769b7ddf7e6d16904c24d3.

This causes problems for deparse mode, which emits pages of warnings.
If we are going to add -w, we ought first to fix deparse testing to
work with it.

‘%s found where op expected’ under fatal warnings

This is related to ticket #123195.

This code:

use warnings FATAL => 'all';
use strict;
$foo;
myfunc 1,2,3
__END__

was giving this:

Global symbol "$foo" requires explicit package name (did you forget to declare "my $foo"?) at - line 3.
(Do you need to predeclare myfunc?)
syntax error at - line 4, near "myfunc 1"
Execution of - aborted due to compilation errors.

How can we have ‘Do you need to predeclare myfunc?’ without any prior
mention of myfunc in the diagnostics?  Fatal warnings were modified in
594b6fac to queue compile-time warning and emit them as part of the
error message.  But the logic was not quite right for yywarn, which is
used for ‘foo found where operator expected’.  The warning was just
disappearing outside of an eval.  qerror was treating PL_in_eval as
a boolean, so we need to clear the flag we have just set, before
we call it.

perldelta for 890b5089ee

Alphabetise perldiag

[Merge] Constant inlining clean-up

This branch makes perl more consistent in how it turns sub(){...} into
a constant.

Previously, it did not behave as documented, or at least not all the
time.  And whether an anonymous sub was clonable made a difference to
the behaviour.  Clonable subs are usually closures, but they could
also be anonymous subs containing state declarations or string evals,
or any anonymous subs under the debugger.  ‘Closures’ below includes
any clonable subs.

• Explicit return never makes a constant (as promised in the docs).
• Statements optimised away are ignored (not just with closures).
• Lvalue and custom attributes now prevent constants with closures,
  as they have already done for non-closures.
• Method attributes no longer prevent constants with non-closures.
• Closing over a variable potentially modified elsewhere will con-
  tinue to make a constant as before, but is detected and deprecated.
• More complex cases than sub(){$x}, such as sub(){foo() if 0; $x}
  just follow the new behaviour, which is not to make a constant if
  $x is modifiable elsewhere.

And a few other bug fixes.

Document sub inlining changes

Most of the changes do not need to be documented, because the previous
behaviour did not match the documentation or was just plain buggy.

pad.c:cv_clone_pad: Avoid copying sv

When we capture the lexical variable in order to make sub () {$x}
constant, we don’t have to copy it if it is not modified or referenced
elsewhere.

Account for state vars when const-izing sub(){$x}

If the only lvalue use of a lexical ‘my’ variable is its declaration,
then it is fine to turn a sub that closes over it into a constant.

But with state variables we have to be a little more careful.

This is fine:

    state $x = something();
    sub () { $x }

because the variable is only assigned to once.  But this modifies the
same variable every time the enclosing sub is called:

    state $x++;
    sub () { $x }

So that closure must remain a closure, and not become a constant.

(However, for a simple lexical scalar in the sub like that, we still
make it a constant, but deprecate the usage.)

Account for string eval when const-izing sub(){$x}

If we have a string eval in the same scope as the variable, it is
potentially in value context.

Make op.c:op_const_sv static

It is no longer called from any other file.

Inline op_const_sv into cv_clone

op_const_sv is actually two functions in one.  This particular calling
convention (CvCONST) was used only by cv_clone.

Half the code was not even necessary for cv_clone’s use (the other
half only for its use), so this reduces the total number of lines.

Make sub () { 0; 3 } inlinable once more

It probably stopped being inlinable in commit beab0874143b.

Following op_next pointers to see whether a sub’s execution chain con-
sists of a constant followed by sub exit does not make sense if the
op_next pointers have not been set up yet.

So call LINKLIST earlier, so that we can merge the two calls to
op_const_sv in newATTRSUB (no need to search for constants twice).
That will allow sub () { 0; 3 } to be inlined once more, as it was in
perl 5.005 (I checked) and probably in 5.6, too (I didn’t check).

This also allows us to remove initial the OP_LINESEQ check, which
was added to help this function into the block when we have no
op_next pointers.

op_const_sv is now called only before the peephole optimiser and
finalize_op, which removes the need for the special explicit return
check (it hasn’t been optimised out of the execution chain yet) and
the need to account for constants that have been relocated to the pad
by finalize_op.

Allow sub():method{CONSTANT} to be inlined

This brings non-closure subs into conformity with closures.

First arg to op_const_sv is never null