Revert "[perl #95550] Remove prototypes from (l)stat"

It turns out this problem is more knotty than I initially realised.
stat had a prototype of (*), even though it could be called with no
arguments.  Since the (*) prototype does not parse its argument the
same way that stat parses *its* argument, I thought that changing (*)
to (;*) would be no better, since it’s still not correct.  So I simply
set the prototype to undef.

My thinking was faulty, for two reasons:

• The prototype serves to indicate the precedence, not just the types
  of arguments.  An undefined prototype on an overridable prefix func-
  tion implies that it takes a list.  So this causes problems for the
  imminent &CORE::subs feature, as there is not yet a clean mechanism
  for CVs to parse their arguments.
• The (*) prototype character does not parse the same way as *any*
  built-in function with that character in its prototype.  So stat is
  no worse than any other built-in.  It doesn’t make sense to remove
  the prototype from stat without removing it from about 40 other
  built-in functions; and that’s where pedantry conflicts with
  usefulness.

This commit restores the (*) prototype to stat and lstat.  The next
commit will give it a (;*) prototype, as that matches it more closely.
It’s not perfect, but it’s no worse that other built-ins with a * in
their prototypes.

Better st_ino_size probe in configure.com.

Follow-up to adf2bd2837.  This is being used for compile-time
comparisons; sizeof may be compile-time, but is apparently not
available before macro substitution, leading to problems like:

....^
%CC-I-IGNOREEXTRA, Spurious token(s) ignored on preprocessor directive line.
at line number 2837 in file MDA0:[SMOKE.blead]pp_sys.c;1

Move bareword checking from the peephole optimizer to finalize_optree. Fixes [perl #95998]

The bareword checking is moved from the peephole optimizer to finalize_optree.
newRANGE needs additional bareword checking because the constants may
be optimized away by 'gen_constant_list'.
The OPpCONST_STRICT flag is removed after giving an error about a
bareword to prevent giving multiple errors about the same bareword.

Move context propagation and finalize_optree from do_eval to newPROG

Aborting after errors found by finalize_optree in do_eval wasn't done
properly and would cause memory problems.
This patch moves the context propagation and finalize_optree to
newPROG such that the normal error handling is done.
The eval context blk_gimme is used to communicate the context.

API to explicitly select to export XSUB symbols or not

This adds a few additional macros to XSUB.h:
XS_INTERNAL and XS_EXTERNAL are versions of the XS macro
that explicitly use internal (static) linking or not.
XSPROTO_INTERNAL and XSPROTO_EXTERNAL are the obvious equivalents
for XSPROTO (which is apparently not public yet we support its
use in SWIG...).

The XS and XSPROTO macros themselves are not defined to not
use STATIC, but this may (should?) be changed in the future.

ExtUtils::ParseXS: Check that an XSUB with CODE&RETVAL has an OUTPUT

If an XS paragraph/function definition that has a CODE section using
RETVAL, then we need an OUTPUT section or else things will go sour.

This adds a check for that condition and produces a friendly error
message.

See CPAN RT #69536.

Simplify embedvar.h, removing a level of macro indirection for PL_* variables.

For the default (non-multiplicity) configuration, PERLVAR*() macros now
directly expand their arguments to tokens such as C<PL_defgv>, instead of
expanding to C<PL_Idefgv>. This removes over 350 lines from F<embedvar.h>,
which defined macros to map from C<PL_Idefgv> to C<PL_defgv> and so forth.

Synchronise Maintainers.pl for Storable and version

workaround a type handling bug in SystemTap Dtrace 1.2

It uses a construct similar to:

  __typeof__((func)) x = func;

but hek_key is a char[1] so the above becomes:

  char arg1[1] = func;

which is invalid.

Using a temporary allows an implicit conversion to const char * and
avoids possible bugs that might be hidden by an explicit cast.

ExtUtils::ParseXS version bump: release 3.03

Rewrap some verbatim pod in Math::BigInt

I started to make Math::BigInt’s pod fit within 79 columns, but
soon petered out.  At least this is better than nothing.

Keep verbatim pod in various dist/* pods within 79 cols

Keep verbatim pod in I18N:LT and IO.pm within 80 cols

Keep verbatim pod in ExtUtils::Typemaps::OutputMap within 80 cols

Add perldelta for /aa and \b commit

This adds an entry for commit f2e96b5ddb6396417bc2fb71915c5025215060d6

Fix trivial test failure from dbef30a62b2ca0b186

For the record: This was my (Steffen's) fault, not Mike's!

Add Mike Sheldrake to AUTHORS

ExtUtils::ParseXS: Test All @INC Derived typemap Locations

The ExtUtils::ParseXS tests would sometime fail to locate some
core Perl typemaps in @INC. This fixes CPAN RT #70047.

Patch slightly modified by Steffen Mueller.
Includes the mandatory version bump to a development version.

autodoc.pl: output to fit in 79 columns

podcheck.t: Change verbatim line length max to 79

Some pagers reserve a column for their own use.  This patch has
podcheck.t warn if a verbatim line will wrap when such a pager is used
in an 80 column terminal window.  This effectively restores the
situation to what it was before commit
2cd46bfd6d03cd8826c5e968efdd36416191bc8f, but the message is now honest
about the situation, i.e., that 79 colummns are effectively the max
without a warning.

The point of this is to give a number to shoot for, 79, that will work
almost always (I believe though that there are pagers that reserve 2
columns in some cases), but of course the data file is continued to be
used to silence the warning if it isn't desired to get down to 79.

Keep verbatim pod in ExtUtils::Typemaps within 80 cols

Ignore another editor droppink

Keep verbatim pod in ExtUtils::Installed within 80 cols

Keep verbatim pod in Data::Dumper within 80 cols

Keep verbatim pod in File::VMS/Win32 within 80 cols

Keep verbatim pod in File::Mac/Unix within 80 cols

Keep verbatim pod in File::Spec within 80 cols

Update Encode to CPAN version 2.44

  [DELTA]

  $Revision: 2.44 $ $Date: 2011/08/09 07:49:44 $
  ! Unicode/Unicode.xs
    Addressed the following:
      Date: Fri, 22 Jul 2011 13:58:43 +0200
      From: Robert Zacek <zacek@avast.com>
      To: perl5-security-report@perl.org
      Subject: Unicode.xs!decode_xs n-byte heap-overflow
  ! Encode.pm encoding.pm
  ! lib/Encode/Alias.pm lib/Encode/Encoder.pm lib/Encode/Guess.pm
    Applied: RT#69735: patch for use constant DEBUG =>
    https://rt.cpan.org/Ticket/Update.html?id=69735

Refactor Porting/checkcfgvar.pl, simplifying the control flow.

In checkcfgvar.pl, eliminate the file-scope lexical %lst.

In checkcfgvar.pl, eliminate the file-scope lexical %MASTER_CFG.

In checkcfgvar.pl, inline and eliminate read_file() and config_h_SH_reader().

config_h_SH_reader() was called once in one place, to generate a closure
passed to read_file(). In turn, read_file()'s use of a callback to process
lines of the file that it opened concealed the near-linear flow of control.

use autodie; to simplify the error handling.

In checkcfgvar.pl, inline check_cfg() into its only caller.

work around a bug in dtrace - it strips const from function parameters

With usedtrace this caused:

- build failures under g++ (OS X)

- const qualification warnings with gcc (OS X)

The workaround simply adds const to char * parameters if not already
present.

The bug in dtrace was apparently reported to Sun in 2006:
  http://opensolaris.org/jive/thread.jspa?threadID=53810

Mozilla ran into the same problem:
  https://bugzilla.mozilla.org/show_bug.cgi?id=593483

Remove 5.8.x VOS specific code from Porting/checkcfgvar.pl.

Commit 3fd80bd61943d0f8 in 2002 removed vos/config.{alpha,gf}.def from
blead, but they were still present in maint-5.8, so the logic to process
them was retained, to minimise the differences in the Porting/* scripts.
maint-5.8 is EOL now, so these can be removed.

Add defaults for st_ino size/sign probes to uconfig64.sh

These were not added in 668fdbe135fd76c7 because Porting/checkcfgvar.pl was
not aware that uconfig64.sh needed testing.

Add uconfig64.sh to the list of files that Porting/checkcfgvar.pl checks.

Make prototype("CORE::CORE") die once more.

Commit b8c38f0a2a65 accidentally made prototype("CORE::CORE") return
undef instead of dying. This is the only case that reached the line
with the ‘Should not happen...’ comment.

This commit changes it to be handled earlier and also adds an asser-
tion to make sure that unreachable code really is unreachable (which
it should be now, I think, I hope...).

Only warn about "Statement unlikely to be reached" after an exec if exec is a statement.

Moving the OP_EXEC check to finalize_optree changed that it would also
warn if 'exec' is part of an expression. This patch changes the
behaviour to only warn if 'exec' is the whole statement, which
appears to be the behaviour before moving the check to finalize_optree.

Change aassign_common_vars to check using the optree without using the linked list.

Besides no longer depending on the op chain this also solves a bug
where the common vars where not detected because of logical operators.

Move aassign common var detection to a separate function.

AASSIGN_COMMON var detection bug with logical operators, TODO test

Upgrade bignum to version 0.29

Fix porting check for dots in directory names.

The existing check conflated two different potential problems,
multiple dots in a filename and dots in a directory name.

Multiple dots in a filename can cause various kinds of trouble,
depending on the VMS filesystem in use, but there are workarounds
for older filesystems and this problem is unlikely to cause a
build failure, so it's probably too stringent to make a test
fail when we find them (and there are lots of examples already
in the repository).

*Any* dots in a directory name are problematic because dot is the
directory delimiter.  Newer (ODS-5) filesystems can handle these
if escaping is done but older (ODS-2) filesystems can't and the
manifest checker in configure.com can't.  In other words, the
build falls down hard in this case.

The existing test was checking for multiple dots in a directory
name but not checking the filename at all.  I've changed it to
check for any dots in a directory name.

ExtUtils::Manifest 1.59 just made it to CPAN

Update Params-Check to CPAN version 0.32

  [DELTA]

  Changes for 0.32    Mon Aug  8 10:53:36 2011
  ============================================
  * Removed unnecessary use Data::Dumper

GNU gcc-4.6.1 revealed a broken system header on HP-UX 11.00

[perl #96228] perlop misdocuments ${ qr/x/ } as undef

In embed.pl, inline hide() and bincompat_var() into their only call point.

hide() has only been used by bincompat_var() since commit acfe0abcedaf592f
in 2001, and bincompat_var() only used in one place at the top level since
commit 87b9e16005b9e39b in 2010.

In embed.pl, refactor readvars() to return a sorted list.

Previously it would assign to the hash reference passed to it. However, the
rest of the code only ever used the keys of that hash, in sorted order, so it
is simpler to change readvars() to return that list directly.

Also move the declaration of the lexical $sym down to its first point of use.

In embed.pl, remove unused parameter $keep_pre from readvars().

$keep_pre was added by commit c6af7a1aaf1f569e in 1998, and the last user
removed by commit 51371543ca1a75ed in 1999.

Update CPAN to CPAN version 1.9800

  [DELTA]

  2011-08-07  Andreas J. Koenig  <andk@cpan.org>

  * release 1.9800

  * RT #69463: fix memory leak in CacheMgr (Serguei Trouchelle)

  2011-06-27  Andreas J. Koenig  <andk@cpan.org>

  * release 1.97_51

  * address #68835: Changed read_meta to ignore dynamic_config (David Golden)

  * bugfix: refuse to store_persistent if the own build_dir is not
  available (Andreas Koenig)

  * cosmetics: remove "Going to" from the beginning of user-visible
  strings (Jesse Vincent)

  * flock adjustments for Win32 from activestate (Christian Walde)

Update DB_File to CPAN version 1.824

  [DELTA]

  1.824 6 Aug 2011

   * Amendments to tests to work in blead
     [RT #70108]

Update DB_File to CPAN version 1.823

  [DELTA]

  1.823 6 Aug 2011

   * croak if attempt to freeze/thaw DB_File object
     [RT #69985]

prevent mixing non-native and native newlines in manifest files

Remove comment from ck_glob that is no longer relevant

This code was made more failsafe by commit 4984aa345.

Update ExtUtils-MakeMaker to CPAN version 6.59

  [DELTA]

  6.59  Fri Aug  5 12:09:00 BST
    No changes since 6.58_01

  6.58_01  Wed Aug  3 21:12:00 BST
    Doc Fixes
    * Corrected the repository location [chorny]

    Bug Fixes
    * Removed the %INC check from _has_cpan_meta and updated the CPAN::Meta
      bundled with EUMM to one that doesn't set $VERSION in a begin block
      so it can't appear to be loaded, thus causing us to mistakenly try
      and call methods on CPAN::Meta modules - this is due to older versions
      in site_perl shadowing our installation of other bundled modules.
      [rt.cpan.org 69465] [rt.cpan.org 69900] [MSTROUT]

RT #96354: \h \H \v and \V didn't check for EOL

The HORIZWS and similar regexp ops didn't check that the end of the string
had been reached; therefore they would blithely compare against the \0 at
the end of the string, or beyond.

Don't use [.lib.pods] staging directory in VMS builds.

It's arduous to maintain all the dependency rules, and since
7eb47696de, which removed the only reference to $(pod), we don't
even *run* any of those rules, i.e., the directory does not even
get populated during the build.  Its existence appears to be a
carryover from a time before installperl copied [.pods] in the
build directory to [.lib.pods] in the install directory.

Reducing the size of vms/descrip_mms.template by 44% probably
won't cause too many tears to be shed, either.

Don't install perldoc.com twice.

It gets built and installed in [.utils], so there's no reason to
copy it into [.lib.pods], from whence it gets installed a second
time.  Putting it in [.lib.pods] was introduced way back in
97abc6adff, well before installperl was even working on VMS.

No dots in directory names, please.

Dot is the directory delimiter on VMS, so it's awkward at best to use
it in directory names.  4eb81ef279, "Update Module-Metadata to CPAN
version 1.000005" introduced two new directories with dots in the
name, which confused the manifest checker and broke the build on VMS.

Fix typos in pod/perlgit.pod

Another ExtUtils::ParseXS version upgrade necessary

This is ExtUtils::ParseXS 3.02. Nothing to see here.

Promote ExtUtils::ParseXS to a stable release

Skip the hanging eintr.t test on GNU/kFreeBSD too

As we're still using a list of hanging operating systems,
include 'gnukfreebsd' by matching for 'freebsd' anywhere in $^O.

Support gcc-4.x on HP-UX PA-RISC/64

Correct the socketsize. Probably due to big-endian versus
little-endian, this has always worked on Intel CPUs. This
is a very very old problem,  and it has been the reason I
never used gcc-4 on PA-RISC,  because perl would not pass
the test suite in 64bitall.

Noticeable effects of wrong "$socksizetype" are return codes
of 0 (pass) from functions like getpeername (),  but invalid
(or none) data in the returned structures or a return length
of 0 or 256. In the latter case, the length is stored in the
second part of the 64bit long and the 32bit first part is 0.
This might be true on Intel-like machines too, but the tests
will pass, as the significant part of the returned length is
in the first 32bits and a pointer to int will still see that
correct.  Mind that in that case the 32bits after that might
have been overridden => can of worms.

Update Module-Metadata to CPAN version 1.000005

  [DELTA]

  1.0.5 2011-08-02 09:45:00

  - Localize $package::VERSION during version discovery (MIYAGAWA)
  - Fix references to Module::Build::ModuleInfo [RT #66133] (DAGOLDEN)
  - Added 'new_from_handle()' method [RT #68875] (DAGOLDEN)
  - Improved documentation (SYNOPSIS, broke out class/object method, and
    other minor edits) (DAGOLDEN)

Update CPAN-Meta to CPAN version 2.112150

  [DELTA]

  2.112150  2011-08-02 22:25:41 America/New_York

  [BUGFIX]

  - Stringify any objects encountered during conversion. (RT #67295)

  - Fixed some broken URLs in documentation and for LGPL in legacy
    conversion (RT #68738)

  - Fixed invalid private Pod in CPAN::Meta::Validator (RT #65925)

  [SPEC]

  - Clarified that file paths in the 'provides' section must be in
    Unix-style (i.e. forward slashes) (RT #69045)

  - Replaced examples using Module::Build::ModuleInfo with Module::Metadata
    (RT #66135)

Add the correct PL_sig_ignoring skip for VMS in makedef.pl.

This only exists when FAKE_PERSISTENT_SIGNAL_HANDLERS is defined,
and that's only defined when HAS_SIGACTION is false.

The problem was exposed by ed76ea85d1, which fixed a typo introduced
in aadb217dbe, but that typo introduced code that was a noop on all
platforms, including VMS, where $^O is 'VMS', not 'vms'.

TODO: The existence of PL_sig_defaulting and PL_sig_handlers_initted
depends on the definedness of FAKE_DEFAULT_SIGNAL_HANDLERS, which
depends on KILL_BY_SIGPRC, which is written directly to config.h
without being recorded in config.sh and is thus not visible in Config.
So there's no good way to know when these symbols should be skipped.

KILL_BY_SIGPRC currently gets defined on all VMS versions and platforms,
so it's a breakage-in-waiting but does not actually fall down (yet).

Remove Perl_mod

After mod was renamed to op_lvalue, this stub was added temporarily
to provide a smoother transition for the compilers.  The compiler
maintainer is happy with its extirpation at this stage.  See
ticket #78908.

Increase $diagnostics::VERSION from 1.23 to 1.24

RT #94988 - Support for %u in diagnostics.pm's printf()-format parser

Fix typos in pod/perlexperiment.pod

A few tweaks to the new perlexperiment.pod

• The :unique attribute has been deprecated already.
• The fields pragma has never been experimental and still exists.
• Grammar

Fix typos in several pod/perl*.pod files

Make verbatim pod in bignum.pm fit in 80 columns

Add perlexperiment to detail experimental features

nearly all this work merged from brian d foy's briandfoy/perlexperiment branch,
tidied up and squished by rjbs

SvREFCNT_dec already checks if the SV is non-NULL

Update Module-Load to version 0.20

  [DELTA]

  Changes for 0.20    Mon Aug  1 19:13:44 2011
  ============================================
  * Resolve [rt #69886] reported by Laurent Dami,
    trailing '::' silently ignored

Note the stat() change in perldelta

[rt #84590] try to preserve the inode number, even if it's large

This matches the checks done for other stat() fields.

Merge the refactored makedef.pl into blead.

This adds a new script, Porting/exercise_makedef.pl, to help test makedef.pl.

Note in perldelta that makedef.pl has been refactored.

In makedef.pl, refactor the code that reads *.sym files.

Hoist the test that chooses the prefix out of the loop. Tweak the regex to
avoid needing an explicit chomp/ Use 3-arg open and a lexical for the file
handle.

In makedef.pl, correct a typo in a comment.

In makedef.pl, avoid creating %PLATFORMS just for one existence check.

A single linear search of @PLATFORMS is simpler and possibly even faster
than building a hash from @PLATFORMS, making one lookup, then discarding the
hash.

Add a block to limit the scope of @PLATFORMS to the code that uses it.

In makedef.pl, use %ARGS instead of 4 variables for command line arguments.

This simplifies processing @ARGV.

In makedef.pl, consolidate all the C compiler flags parsing code.

This isn't quite a "pure" refactoring, as -Dfoo=bar read from %Config will
now be processed as $Config{foo} = 'bar'; instead of $Config{foo} = 1;
However, it won't change any behaviour, as the only time makedef.pl uses a
value of %define for anything other than a truth test is $define{PERL_DLL},
and that is passed in on the command-line by the Makefile invoking
makedef.pl

In makedef.pl, read ccflags and optimize from %Config.

Previously they were read by opening and parsing config.sh. Remove all code
related to reading config.sh, as it is no longer needed.

In makedef.pl, read 4 configuration values direct from %Config.

makedef.pl has been using Config since commit 9d6c7f2eccef26a6 in 2009.
Hence we can read them direct from %Config, instead of parsing config.sh
and storing the values in %define.

In makedef.pl, merge the code that reads config.sh

For now, retain compatibility (bug compatibility?) by not processing ccflags
and optimize on Win32, WinCE and NetWare.

In makedef.pl, export boot_* for static modules only on Win32 and WinCE.

The code as-is will only be run on Win32 and WinCE, because currently the
code to populate $static_ext is only run on Win32 and WinCE. As-is this code
can't be run on all platforms, because currently *nix (ie AIX) links static
extensions into the perl executable, rather than into the shared perl
library. Hence boot_* functions for static extensions aren't in the shared
perl library, so it would be an error to attempt to export them from it.

In makedef.pl, tidy up the MYMALLOC logic.

No need to call try_symbols() for PL_malloc_mutex, as it is found in
perlvars.h. Unlike PL_perlio_mutex, PL_malloc_mutex isn't listed in the
main !USE_ITHREADS skip list - perhaps it should be for consistency.

In makedef.pl, tidy up the USE_PERLIO logic.

No need to call try_symbols() for PL_perlio_mutex, as it is found in
perlvars.h, and no need to duplicate the skip for it if USE_ITHREADS is not
defined.

We do need to skip PL_perlio_mutex if if USE_PERLIO is not defined but
USE_ITHREADS is [not that this is a configuration that Configure will let
you select, until you edit it :-)]

Add a comment describing what the "Oddities from PerlIO" are.

Remove PerlIO_sv_dup from the list, as it's also in @layer_syms, and
inclusion in @layer_syms makes the former redundant. It was added to both in
commit 8437356be6fd345c in 2001.

In makedef.pl, consolidate all the code that tries platform-specific symbols.

Previously the code for OS/2 and VMS was run much earlier. Fortunately none
of the symbols tried were conditionally added to the skip list, else much
confusion would have ensued.

Move the OS/2 code that reads miniperl.map after the code that tries
platform-specific symbols.

In makedef.pl, remove duplicate symbols passed to try_symbols() for OS/2.

Code to try the 4 thread-specific symbols was added in 2003 by commit
5c728af092d5feba. However, all 4 symbols were already listed in makedef.pl,
added in 1999 by commit 3cfae81b38bc8edd.

(Unlike other platforms, on OS/2 adding symbols to %export that aren't
actually compiled in does not cause errors, as there is code later in
makedef.pl to read in miniperl.map, and strip from %export all symbols not
listed there.)

In makedef.pl, replace calls to try_symbol() with try_symbols().

Following the simplification of the previous commit, the two are now
equivalent.

In makedef.pl, change readvar() to add items to a passed-in hash reference.

Previously, readvar() returned a list of symbols, extracting the names of
the variables declared there, then transforming them either via a passed in
function, or prepending 'PL_'. The list was then either added to %skip, or
added to %export by try_symbols(), which had a kludge to normalise the
previous transformation to 'PL_'.

Instead, passing a reference to the target hash into readvar() allows that
kludge to be removed, considerably simplifying try_symbols().

In makedef.pl, no need to loop over a 1 item list.

In makedef.pl, inline output_symbol() into its only caller.

Remove code commented out by commit 51371543ca1a75ed in 1999 with the note
"binary compatibility between compilers is an exercise in frustration".