[perl #106288] Tests for \scalar @array

utf8.c: white-space, comment clarification only

utf8.c: foldEQ_utf8_flags() use specific flag, not just any

The test here was if any flag was set, not the particular desired one.
This doesn't cause any bugs as things are currently structured, but
could in the future.

The reason it doesn't cause any bugs currently are that the other
flags are tested first, and only if they are both 0 does this flag get
tested.

Remove some commits from the to-doc list that have been covered

perldelta updates for VMS changes.

Stop seek($glob_copy...) from clearing PL_last_in_gv

seek had the same bug as tell.  Here is the commit message from
8dc99089, which fixed tell:

----------------------------------------------------------------------

Stop tell($glob_copy) from clearing PL_last_in_gv

This bug is a side effect of rv2gv’s starting to return an incoercible
mortal copy of a coercible glob in 5.14:

$ perl5.12.4 -le 'open FH, "t/test.pl"; $fh=*FH; tell $fh; print tell'
0
$ perl5.14.0 -le 'open FH, "t/test.pl"; $fh=*FH; tell $fh; print tell'
-1

In the first case, tell without arguments is returning the position of
the filehandle.

In the second case, tell with an explicit argument that happens to
be a coercible glob (tell has an implicit rv2gv, so tell $fh is actu-
ally tell *$fh) sets PL_last_in_gv to a mortal copy thereof, which is
freed at the end of the statement, setting PL_last_in_gv to null.  So
there is no ‘last used’ handle by the time we get to the tell without
arguments.

This commit adds a new rv2gv flag that tells it not to copy the glob.

By doing it unconditionally on the kidop, this allows tell(*$fh) to
work the same way.

Let’s hope nobody does tell(*{*$fh}), which will unset PL_last_in_gv
because the inner * returns a mortal copy.

This whole area is really icky.  PL_last_in_gv should be refcounted,
but that would cause handles to leak out of scope, breaking programs
that rely on the auto-closing ‘feature’.

Stop eof($glob_copy) from clearing PL_last_in_gv

eof had the same bug as tell.  Here is the commit message from
8dc99089, which fixed tell:

----------------------------------------------------------------------

Stop tell($glob_copy) from clearing PL_last_in_gv

This bug is a side effect of rv2gv’s starting to return an incoercible
mortal copy of a coercible glob in 5.14:

$ perl5.12.4 -le 'open FH, "t/test.pl"; $fh=*FH; tell $fh; print tell'
0
$ perl5.14.0 -le 'open FH, "t/test.pl"; $fh=*FH; tell $fh; print tell'
-1

In the first case, tell without arguments is returning the position of
the filehandle.

In the second case, tell with an explicit argument that happens to
be a coercible glob (tell has an implicit rv2gv, so tell $fh is actu-
ally tell *$fh) sets PL_last_in_gv to a mortal copy thereof, which is
freed at the end of the statement, setting PL_last_in_gv to null.  So
there is no ‘last used’ handle by the time we get to the tell without
arguments.

This commit adds a new rv2gv flag that tells it not to copy the glob.

By doing it unconditionally on the kidop, this allows tell(*$fh) to
work the same way.

Let’s hope nobody does tell(*{*$fh}), which will unset PL_last_in_gv
because the inner * returns a mortal copy.

This whole area is really icky.  PL_last_in_gv should be refcounted,
but that would cause handles to leak out of scope, breaking programs
that rely on the auto-closing ‘feature’.

Oops.  You can’t do ->op_first on a baseop.

This was the fault of commit 8dc99089.

Minor spelling/grammar/context fixes

Stop tell($glob_copy) from clearing PL_last_in_gv

This bug is a side effect of rv2gv’s starting to return an incoercible
mortal copy of a coercible glob in 5.14:

$ perl5.12.4 -le 'open FH, "t/test.pl"; $fh=*FH; tell $fh; print tell'
0
$ perl5.14.0 -le 'open FH, "t/test.pl"; $fh=*FH; tell $fh; print tell'
-1

In the first case, tell without arguments is returning the position of
the filehandle.

In the second case, tell with an explicit argument that happens to
be a coercible glob (tell has an implicit rv2gv, so tell $fh is actu-
ally tell *$fh) sets PL_last_in_gv to a mortal copy thereof, which is
freed at the end of the statement, setting PL_last_in_gv to null.  So
there is no ‘last used’ handle by the time we get to the tell without
arguments.

This commit adds a new rv2gv flag that tells it not to copy the glob.

By doing it unconditionally on the kidop, this allows tell(*$fh) to
work the same way.

Let’s hope nobody does tell(*{*$fh}), which will unset PL_last_in_gv
because the inner * returns a mortal copy.

This whole area is really icky.  PL_last_in_gv should be refcounted,
but that would cause handles to leak out of scope, breaking programs
that rely on the auto-closing ‘feature’.

Avoid an unused temp scalar in sv.c:S_sv_unglob

This is something I missed in 804b5ed7.

[perl #97988] Nullify PL_last_in_gv when unglobbed

Code like this can cause PL_last_in_gv to point to a coercible glob:

    $f{g} = *STDOUT;
    readline $f{g};

If $f{g} is then modified such that it is no longer a glob,
PL_last_in_gv ends up pointing to a non-glob:

    $f{g} = 3;

If $f{g} is freed now, the PL_last_in_gv-nulling code in sv_clear will
be skipped, as it only applies to globs.

    undef %f; # now PL_last_in_gv points to a freed scalar

The resulting freed scalar can be reused by another handle,

    *{"foom"} = *other;

causing tell() with no arguments to return the position on *other,
even though *other was no the last handle read from.

This commit fixes it by nulling PL_last_in_gv when a coercible glob
is coerced.

Cwd pod consistency tweaks

perldiag: Consistent style of diag categories

Fix the one remaining long line in perldiag

bisect-runner.pl should only dump out a failed patch once by default.

Only show the "converted to a context diff" version if it was actually
converted to one before (attempted) application.

Teach bisect-runner.pl how to work around various 5.005_5x build problems.

This area of history is particularly hairy because of all the back-and-forth
merging between //depot/perl and //depot/cfgperl, which confuses bisect.
It's also potentially interesting, as it's the start of the 5.6.0 development
series, so many new features arrive in quick succession, making it a likely
spot for regressions to appear.

In particular, we can now build perl for this commit:

  commit 4592e6caefc41a75573f112714d170071892a537
  Author: Nick Ing-Simmons <nik@tiuk.ti.com>
  Date:   Sat May 8 14:16:30 1999 +0000

      Implement OPEN, EOF, SEEK, TELL, BINMODE and FILENO as TIEHANDLE methods.
      Provide Tie::StdHandle
      Basic update of docs.

      p4raw-id: //depot/perl@3330

bisect-runner.pl must not be fooled by a t/test that symlinks to ../miniperl

It had merely been testing that the target file was +x
For t/test it must also check that t/perl wasn't a symlink to miniperl
created as a side effect of make miniperl failing. To add to the fun, for
some revisions the symlink is to '.././perl', not '../perl'.

In bisect-runner.pl, clarify the variables relating to the make target.

The 4 things in play are
* the (standard) target name used on the command line
* the actual target we ask make to build
* the file we expect it to generate as a result
* whether that file was generated correctly

The previous variable names were not that clear about this.

[rt.cpan.org #45885] File::Spec: Don’t use tainted tmpdir in 5.6

Perl 5.6 doesn’t have ${^TAINT}, so the taint check in
File::Spec::Unix->_tmpdir was skipped.  This commit adds a taint check
using eval { eval $safe_substring } for 5.6.

AUTHORS: better email address

Patch attached includes better address for the long-term.

Mention evalbytes in perlvar as a (_) function

Remove use of local($_) from dumpvar.pl

I know we should really be fixing local($_), but that code
in scope.c is rather twisted, and I want to see this fix
in 5.16.

perlvar: Don’t mention 5.9

perlvar: Consistent use of spaces after dots

This commit changes it to use two spaces, for several reasons.  It was
already inconsistent.  Although it used one space in most cases, the
rest of Perl’s documentation uses two.  pod2man expects two spaces, as
it adds two spaces for sentences ending at the end of a line in the
middle of a paragraph, as it was designed to work with Perl’s origi-
nal pod docs, which used two spaces.  If this document used one space
throughout, one would have to go through it and make sure no sentences
end at the ends of lines except at the end of a paragraph.

perldiag: Rewrap ‘Code point 0x%X is not Unicode’

to make splain output look better.

Increase $diagnostics::VERSION to 1.26

Teach splain about %X

perldiag: Separate two paragraphs

otherwise they flow together as one paragraph in formatted pod.

perldiag: gmtime(nan) too small does not occur

perldiag: Fix some jagged lines

Reflow some entries to make the right margin less jagged in
splain output.

Update Timothe Litt’s e-mail address

Update Socket to CPAN version 1.97

  [DELTA]

  1.97    CHANGES:
         * Rewritten Makefile.PL configure-time logic to use only core's
           ExtUtils::CBuilder rather than CPAN's ExtUtils::CChecker
         * Fix implementation of synthesized newSVpvn_flags() to also work on
           threaded 5.10.0
         * Set INSTALLDIRS=perl on perl before 5.11; required as it's
           replacing a core module

The t/gol-compat.t file is explicitly excluded from Getopt::Long in core

Correct &CORE::foo entry in perldiag

perldiag: missing C<>

I know there are dozens and dozens of these that I have not fixed,
but I happened to notice these two first and wanted to remove them
from my notes.

perldiag: extraneous backslash

perldiag: use F<> for file names

perldiag: wrap long lines

perldiag: Retitle ‘Cannot copy’

Without the ‘in %s’ it covers both forms of the error message.

perldiag: Consistent use of spaces after dots

Where the changes made splain output look worse, I reflowed the text.

toke.c, perldiag: Note delay of some things to 5.18

We are mostly allowing two release cycles before removing a deprecated
feature; therefore several things have been retained for 5.16, that are
documented as being removed for it; Update the docs.

Small perldelta language tweaks for rindolf's changes

Document the "perl -d" bug fixes commit.

In pod/perldelta.pod.

Doc 5d5d9ea ("c 3" debugger command fix) in perldelta

perldiag: double spaces after dots

perldiag: rewrap an entry

so that splain output looks better.

perldiag: rewrap an entry

to fit comfortably in an 80-column terminal.

Increase $Locale::Maketext::VERSION to 1.21

Add Thomas Sibley to AUTHORS

Document Locale::Maketext's numerate method

Previously this was only implicitly documented by a passing reference to
it a subsequent section.

Doc: numerify was renamed to numf very early on

[perl #91514] Use correct error msg for ｄｅｆａｕｌｔ

This commit not only mentions ｄｅｆａｕｌｔ (as opposed to ｗｈｅｎ)
in the error message about it being outside a topicalizer, but
also normalises those error messages, making them consistent with
ｃｏｎｔｉｎｕｅ and other loop controls.  It also makes the perldiag
entry for ｗｈｅｎ actually match the error message.

Really update perldelta with some older changes that hadn't been included yet

This time without reverting a week or two worth of other change documentation
(version control is hard).

Revert "Updated perldelta for some older changes that hadn't been included yet"

This reverts commit c15f899f18827e563db510796dac34fdff0db1d8.

Updated perldelta for some older changes that hadn't been included yet

Put tlhackque’s real name in AUTHORS

Teach win32/Makefile 'test_porting'

Teach win32/makefile.mk 'test_porting'

perldelta up to 5dca8ed9d28

Add tlhackque to AUTHORS

Carp.pm localise $SIG{__DIE__} before eval

Otherwise we end up calling other code’s $SIG{__DIE__}.

Increase $Carp::VERSION to 1.24

Carp.t: Add tests for PL_last_in_gv

Add missing <fh> line # phrase to Carp messages

Carp::croak is documented in the camel book as "working like die does,
except misdirecting blame to the caller."

Perldoc says "croak - die of errors (from perspective of caller)".

This is almost true, however careful inspection reveals that croak
does not include file position information ( ", <fh> line $." ) as die
does (when it can).

It turns out that it *is* possible to determine the current file han-
dle (although I wish it were easier).

Carp should do this and add it to the traceback, as it really is use-
ful information.

pp_hot.c: First letter of latin-1 classnames wasn't being checked correctly.

Previously the first letter for latin-1 classnames was being mischecked, only
allowing ASCII, which caused an instance of the Unicode Bug for downgradable
classnames.

perldelta up to 5e5a1632

plus the array_base feature

Name anon handles __ANONIO__

rather than $__ANONIO__

That dollar sign *has* to have been a mistake.  In ck_fun, the
name was set to __ANONIO__, but it seems the change that added it
(afd1915d43) did not account for the fact that a little later on the
same function checks to makes sure it begins with a dollar sign, as it
could only be a variable name.

rv2gv’s use of $__ANONIO__ (added recently by yours truly) was just
copying was ck_fun was doing.

Document the array_base feature

Disable $[ under 5.16

This adds the array_base feature to feature.pm

Perl_feature_is_enabled has been modified to use PL_curcop, rather
than PL_hintgv, so it can work with run-time hints as well.
(PL_curcop holds the current state op at run time, and &PL_compiling
at compile time, so it works for both.)  The hints in $^H are not
stored in the same place at compile time and run time, so the FEATURE_IS_ENABLED macro has been modified to check first whether
PL_curop == &PL_compiling.

Since array_base is on by default with no hint for it in %^H, it is
a ‘negative’ feature, whose entry in %^H turns it off.  feature.pm
has been modified to support such negative features.  The new FEATURE_IS_ENABLED_d can check whether such default features
are enabled.

This does make things less efficient, as every version declaration
now loads feature.pm to disable all features (including turning off
array_base, which entails adding an entry to %^H) before loading the
new bundle.  I have plans to make this more efficient.

feature.pm: Consistent use of spaces after full stops

perlvar: document what $[ will do under use v5.16

Bring perldiag/‘Assigning non-zero to $[’ closer to reality

Or closer to what will imminently be its reality

Restore ‘Assigning non-zero to $[’ to perldiag

This was removed by b82b06b8ca32.  The message is coming back soon, at
least under ‘use 5.016’....

perldelta_template: revise text about incompatible changes

utf8.c: Change prototypes of two functions

_to_uni_fold_flags() and _to_fold_latin1() now have their flags
parameter be a boolean.  The name 'flags' is retained in case the usage
ever expands instead of calling it by the name of the only use this
currently has.

This is as a result of confusion between this and _to_ut8_fold_flags()
which does have more than one flag possibility.

scope.c: Allow successful saving of PL_tainted

leave_scope() saves and restores PL_tainted upon entry and exit.  This
means that any attempt to save this variable on the stack will fail, as
its unstacked value will overwrite the popped one.

To counteract this, we update our saved version with the popped value.

pp.c: Changing case of utf8 strings under locale uses locale for < 255

As proposed on p5p and approved, this changes the functions uc(), lc(),
ucfirst(), and lcfirst() to respect locale for code points < 255; and
use Unicode semantics for those above 255.  This results in better, but
not perfect results, as noted in the changed pods, and brings these
functions into line with how regular expression pattern matching already
works.

utf8.c: White-space changes only

This indents previous lines that are now within new blocks

utf8.c: Allow Changed behavior of utf8 under locale

This changes the 4 case changing functions to take extra parameters to
specify if the utf8 string is to be processed under locale rules when
the code points are < 256.  The current functions are changed to macros
that call the new versions so that current behavior is unchanged.

An additional, static, function is created that makes sure that the
255/256 boundary is not crossed during the case change.

utf8.c: Add comment

locale.t: Prepare for adding more tests

This custom TAP output .t has hard-coded test numbers.  Rather than
disturb those, I plan to add tests that come after all the current ones,
and this paves the way for that.

locale.t: Add ability to output a message with 'ok'

locale.t uses its own custom TAP messages.  This changes the 'ok' sub
to accept another parameter, a test description to add to the TAP line.
And it changes the taint functions to produce a generic one of these

Revise feature.pm documentation

core-cpan-diff: clarify phrasing of missing file warning

Fix Maintainers.pl and META.yml for removal of lib/version.t

Merge CPAN version.pm tests into CORE.

It has become increasingly difficult to maintain the CPAN tests
and the CORE tests, so include the [subtly edited] CPAN tests
into the CORE code instead.  NOTE: this also bumps $VERSION to
match the forthcoming CPAN release.

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

Update threads to CPAN version 1.86

  [DELTA]

  1.86 Wed Dec 14 21:12:51 2011
    - Fix for mingw-w64 build due to setjmp difference

Sync Module-CoreList MANIFEST file with what is on CPAN

Fix Module::Build entry in Maintainers.pl

Corrected path for CUSTOMIZED and updated the documentation of
CUSTOMIZED as a reminder for others.

Update Maintainers.pl to quiet DAGOLDEN core-cpan-diff warnings

Reformatted Maintainers.pl for whitespace and layout consistency

(12:16:39) autarch: can I just run perltidy on the whole file?
(12:16:59) rjbs: Yes.
(12:17:14) ***rjbs will take the heat, if there is any, unless you actually BREAK something. :-)
(12:17:30) rjbs: where something is not "my custom Maintainers.pl parser, written in 1995"

More files to exclude from the core-cpan-diff

perlfunc: Document kill without list of processes

perlsec: Fix example to localize $@.

Not doing this can cause hard-to-find bugs.

op/taint.t: localize $@

This is so the subroutine's calling 'eval' won't affect things outside
it.

locale.t: Don't destroy caller's $@

This bug was very hard for me to find.  Leave the outside $@ untouched.

perllocale: Don't mention /l modifier

The /l modifier really shouldn't be used explicitly.  So remove this
mention of it.  It is documented in perlre, with appropriate cautions
against its explicit use.

perllocale: Nits

This adds a clarification, a couple grammar and typo fixes, and outdents
some verbatim sections to fit within 79 columns.

charnames: Add %Carp::Internal entry

This will be useful in a future commit.  charnames is about to be split
into two modules, one called by the other, and warnings::warn() would
otherwise return if warnings are enabled in the parent module.