In S_parse_body(), sv hasn't been needed since -P bit the dust.
(And probably should have been removed with change 32958,
2f3529075cdaceeaf619b8bd308885878ec9828a)

Merge branch 'blead' of ssh://stevep@perl5.git.perl.org/gitroot/perl into blead

fileno() was failing when passed a NULL.  I'm guessing that we don't really need to increase the refcount on a NULL fd.

Fix unused variable warning

Attached patch fixes the following build warnings:

perlmini.c: In function `S_incpush':
perlmini.c:4358: warning: unused variable `addoldvers'

perl.c: In function `S_incpush':
perl.c:4358: warning: unused variable `addoldvers'

From 87b6dbeb18691177a2b2e8dc971da242ac831c03 Mon Sep 17 00:00:00 2001
From: Jerry D. Hedden <jdhedden@cpan.org>
Date: Mon, 23 Feb 2009 10:58:14 -0500
Subject: [PATCH] Fix unused variable warning

Re-apply cd5cc49dbc0e5ee748252c2da8b435855908e6d2 (Module::Build on VMS, #42724)

Update to Module::Build 0.31_04 (with a tweak to MBTest.pm)

Temporarily reverse out change cd5cc49dbc0e5ee748252c2da8b435855908e6d2.

Module::Build Changes from 0.31012, which hadn't been updated.

Add deprecate.pm. Deprecate shipping Switch.pm in the core distribution.

vms.c - Remove .DIR; in UNIX mode.

When doing a readdir() or glob() with DECC$FILENAME_UNIX_REPORT active,
the ".DIR" suffix needs to be removed for directories.

VMS-specific nit in S_init_perllib after 2cace6acdfad87b2d298bff18b91b339c18fd1e3.

time_t is unsigned 32-bit on VMS; pick sGMTIME_max and sLOCALTIME_max accordingly.

For S_incpush(), dir is never NULL, and len is always > 0.

Optimise S_incpush() by avoiding repeatedly copying libdir to subdir.
Specifically, copy it once with newSVsv(), then pass libdir to
S_incpush_if_exists(), and if that creates a new SV, use newSVsv() there to
re-do the copy. Otherwise reset the length of the passed-in SV (which is
subdir), back to the length of libdir, effectively truncating it back to be
equal to libdir. This avoids repeated copying of the same bytes over the same
memory that already holds those bytes.

In S_incpush, move the declaration of subdir inside the only block that uses it.

Tidy up incpush. Re-order the macros, and re-indent some code.

Move unversioned directories in PERL_OTHERLIBDIRS later in @INC.
Split INCPUSH_ADD_SUB_DIRS into INCPUSH_ADD_VERSIONED_SUB_DIRS and
INCPUSH_ADD_ARCHONLY_SUB_DIRS, and tweak the flags used for the two halves of
PERL_OTHERLIBDIRS so that .../$archname and .../ come after .../$old_version

7fc731078209e8c26ed8bba477edfd90724c7146 missed one (addsubdirs || addoldvers).

In S_init_perllib(), push the bare VENDORLIB_STEM onto @INC after its subdirs.

In S_init_perllib(), push the bare SITELIB_STEM onto @INC after its subdirs.

In S_incpush(), move push @INC, ".../$archname" if -d ".../$archname"; later.

This effects this change:

--- Start       2009-02-21 14:04:40.000000000 +0100
+++ /dev/fd/63  2009-02-22 16:41:18.000000000 +0100
@@ -1,14 +1,14 @@
 /Users/nick/Sandpit/511v/hashbangI/5.11.0/darwin-thread-multi-64int-2level
 /Users/nick/Sandpit/511v/hashbangI/5.11.0
-/Users/nick/Sandpit/511v/hashbangI/darwin-thread-multi-64int-2level
 /Users/nick/Sandpit/511v/hashbangI/5.11.-2/darwin-thread-multi-64int-2level
 /Users/nick/Sandpit/511v/hashbangI/5.11.-2
+/Users/nick/Sandpit/511v/hashbangI/darwin-thread-multi-64int-2level
 /Users/nick/Sandpit/511v/hashbangI
 /Users/nick/Sandpit/511v/cliI/5.11.0/darwin-thread-multi-64int-2level
 /Users/nick/Sandpit/511v/cliI/5.11.0
-/Users/nick/Sandpit/511v/cliI/darwin-thread-multi-64int-2level
 /Users/nick/Sandpit/511v/cliI/5.11.-1/darwin-thread-multi-64int-2level
 /Users/nick/Sandpit/511v/cliI/5.11.-1
+/Users/nick/Sandpit/511v/cliI/darwin-thread-multi-64int-2level
 /Users/nick/Sandpit/511v/cliI
 /Users/nick/Sandpit/511v/perl5lib0/5.11.0/darwin-thread-multi-64int-2level
 /Users/nick/Sandpit/511v/perl5lib0/5.11.0

In S_init_perllib(), initialise perl5lib to NULL, to keep trace flow happy.

In S_incpush(), replace (addsubdirs || addoldvers) with a constant variable.

In S_init_perllib(), s can be const char*

In S_init_perllib(), only call PerlEnv_getenv("PERL5LIB") once.

Unwind the implicit loop in S_init_perllib(), by writing the code out longhand.
Call it only once, remove the old_vers parameter, and all the related
conditional code.

Change the value passed into S_init_perllib() to a boolean flag.

NetWare has PRIVLIB_EXP as something other than a string constant. Cope with it.

Use "system" gmtime in time64 on VMS so C<vmsish 'time'> pragma works.

It's not really the system gmtime; it's the homegrown one in vms/vms.c
that checks hints and does shifting between UTC and local time when
requested.

In S_init_perllib(), correct errors in the MACOS_TRADITIONAL conditional code.
(Errors that originated in a26c0e281cb6068a8d148933281d8186f1eb4206 and
50d61629dc7fe34f077b9f66c50287d839e06378)

Re-apply changes 4adc95e616bac7ee and 200cbd6aa595a074 (RT #61492)
(ExtUtils::MM_Win32 should not generate "mt" command when CRT is statically
linked)(We'd failed to send this one upstream. Bad us.)

Upgrade to ExtUtils::MakeMaker 6.49_01

Skip time64 when there is no 64-bit integer type available.

The y2038 code currently requires that a 64-bit integer type be available,
so the build falls down hard if we try to use it where there isn't one.  A
better fix might be to try NV for time_t and a 32-bit integer for year.

N.B.  We test for the definedness of Quad_t rather than HAS_QUAD since the
latter is explicitly undef'd in the case where 64-bit types are available
but are not being used for Perl's IV and UV.

Avoid VMS long filename support on older systems that don't have it.

Move redefinition of lstat above its first use in vms/vms.c.

More smartmatch tests

This includes a TODO for overloaded objects.
Tests will need to be augmented and better organized.

Add a parameter to win32_get_{priv,site,vendor}lib(), to return the length,
as we already know it, and use it in S_init_perllib() to save a strlen() in
S_incpush_use_sep().

Text::Wrap maintained via the CPAN release

Check for thread failure in prime number example

Silence a casting warning with memchr()

Wonky example in perlthrtut

Fix at least the syntax of the example.
See : Message-ID: <F00A81D78228413FBED223C3919BE75B@Amelie>

Merge branch 'blead' of ssh://perl5.git.perl.org/gitroot/perl into blead

Remove the -x from the cherry pick and show how to merge the whole branch (suggested by rgs)

Reorder and complete tests for smart matching with objects

Fix {%hash} ~~ %hash test

This was not parsed as a smart match, because {%hash} was taken
as a block, not a an anonymous hash.

Also, avoid to construct an anonymous hash with an odd number of
elements.

Better diagnostics for the ~~ test

Read from DATA line per line, so warnings are reported from the correct
line. Make test names and error reports more readable.

Remove an extra space

Add a section on merging from a branch via GitHub

Fixed missing word in Text::Wrap POD
(cherry picked from commit 3adac458cb1c1d41af47fc66e67b49c8dec2323f)

[PATCH] DB_File 1.819

Small change to t/db-recno.t [RT# 43288]

Message-ID: <00ad01c99214$89f5fa50$9de1eef0$@Marquess@ntlworld.com>

Remove the stub nw_get_sitelib(), and undefine SITELIB_EXP for NetWare.
(nw_get_sitelib() was hard coded to return NULL, which is a slower equivalent
to simply not defining SITELIB_EXP in the first place. I suspect a cargo cult
of win32/ without actually taking the time to understand it.)

Pass the length of the string to S_incpush_use_sep(), where known.

Tests for S_incpush_use_sep() splitting on : (or platform equivalent)

In runenv.t, break apart running perl and testing the output into two functions.

Get $ENV{PERL5LIB} into a known consistent state under TEST - delete it!
(It doesn't strike me as good that t/TEST and t/harness are inconsistent -
the former runs every test with -MTestInit; the latter sets $ENV{PERL5LIB} to
'../lib'; and on the gripping hand, we have boilerplate in every test to set
@INC. No choice seems ideal.

In S_incpush(), avoid "possible loss of data" warnings from the C compiler.

Move PERL_VERSION_STRING from patchlevel.h to perl.h
(and revert a89d24b9fe54d44395806c16ac1c19f3c678cbbb)
It's getting too complex to work around some versions of awk which don't want
to implement the regular expressions that they are documented to implement.

Ignore #defines in in patchlevel.h that end the line with backslash.
(Otherwise config.sh ends up with part of PERL_VERSION_STRING, and the build
chokes to a stop.)

As WIN32 is already using different flags to incpush_use_sep for SITELIB_EXP,
VENDORLIB_EXP and PRIVLIB_EXP, requiring conditional compilation, it doesn't
feel like extra complexity to make explict calls in S_init_perllib() to the 3
win32_*() functions they expand to, which allows a NULL pointer check.

By spliting PERL_ARCH_FMT into PERL_ARCH_FMT_PREFIX and SUFFIX, we can replace
two more Perl_sv_setpvf()s with sv_setsv()/sv_catpvs() pairs.

As PERL_ARCH_FMT_PATH is a compile time constant, can simplify one sv_setpvf()
to sv_setsv() and sv_catpvs().

PERL_FS_VER_FMT is only used with the current perl version, which is known at
compile time, so replace it with PERL_FS_VERSION, a compile time constant.

Abolish PERL_AV_SUFFIX_FMT, which was rendered obsolescent 8 years ago with
084592ab0b76f3cbd4d089afb08ccea7ba1c9dd8.

Replace *printf "%d.%d.%d", PERL_REVISION, PERL_VERSION, PERL_SUBVERSION
with PERL_VERSION_STRING, which can be determined at compile time, as a
constant string.

Update copyright years. 2008 sneaks in because of 46807d8e809cc127621bf85d9e9

add missing perforce-era entry to Changes files

Use tempfile() in tests

Don't add PRIVLIB_EXP to @INC twice.

[perl #63234] [DOC PATCH] fix some missing parts of IO::Handle pod

This documents the previously undocumented fcntl and ioctl methods, and
adds a see also to the IO::File module where the documentation refers
to one of its methods.

Upgrade to Encode 2.31

Extra regex tests

Message-ID: <20090215185207.gsnhhqdegckws0co@horde.wizbit.be>
Message-ID: <20090215210634.u02f15b284ogc4s4@horde.wizbit.be>

Yves states that patches to ExtUtils::Install should go to blead first

Ensure that the pointer to S_incpush_use_sep() is never NULL.

In @INC, move ARCHLIB and PRIVLIB after the current version's site_perl and
vendor_perl.

Loop in S_init_perllib(), only calling S_incpush*() with INCPUSH_ADD_OLD_VERS
the second time (and only for those entries at had it). Implement the loop by
calling init_perllib() twice, to avoid a rats nest of re-indenting. Add a new
flag to S_incpush() INCPUSH_NOT_BASEDIR, to supress pushing the base directory
a second time on the secnod call.

With this change, re-ordering of @INC from version-orientated to prefix-
orientated is partly complete. ARCHLIB and PRIVLIB remain at their old place in
the @INC order.

Refactor the separator splitting loop of S_incpush() into a S_incpush_use_sep().
Add a parameter to S_incpush() to optionally pass in the length. As S_incpush()
treats the directory parameter as const char, remove some malloc()s elsewhere
that were copying data on the assumption that it was not const safe.

In S_incpush(), rename the parameter from dir to p, rather than copying it.

ExtUtils::Install VMS extended character set support

Preview from https://rt.cpan.org/Ticket/Display.html?id=42149

Refactor S_incpush() to take 1 flags parameter, instead of 5 positional booleans
(which are impossible to remember).

For -I, need to also unshift version and architecture libs onto @INC (RT#6665)
(20189146be79a0596543441fa369c6bf7f85103f only added the given directory.)

Clear Module::Build smoke on VMS while waiting for upstream.

Except for the version bump, this is the same as:

https://rt.cpan.org/Public/Bug/Display.html?id=42724

Add git_version.h to private so that make distclean deletes it.
(It was already in a different target that caused make clean to delete it.)

Ignore the OS X GNUmakefile*s, and t/test_state.

Ignore lib/NDBM_File.pm

Make git-find-p4-change only find changes done under perforce.
That way, later cherry-picking that happens to copy the 'p4raw-id:'
text won't be located by mistake.

Forbid to use "foreach" as an attribute

(like all other control flow statements)

More tests for coderefs and smart match

Remove code that has never been used in any test

Handle uninitialized interpreter when performing vmsish pragma checks.

$object ~~ undef should always test for definedness

(even if $object hasn't overloaded the ~~ operator)

Add tests for undef smart-matching

Document error "Smart matching a non-overloaded object breaks encapsulation"

That error was added in e67b97bd974194ad616acbb7813c5631aacd6be7

Remove obsolete comment; document test format

More tests for array-ref smart match

Remove one of the two definitions of the a_const contant sub

More gitignore adjustments

Ignore lib/Attribute, now that Attribute::Handlers is under ext/.

Ignore generated Makefile.PL files for Safe and Attribute-Handlers.
I preferred putting those in specific .gitignore files instead of
ignoring Makefile.PLs in the whole ext/ hierarchy, in order to
ensure a less confusing git behaviour for future generations.

Remove double slash in MANIFEST

(so Porting/manicheck does not complain)

Move Attribute::Handlers from lib to ext. The layout in ext is the same as the
CPAN distribution, which simplifies dual life module maintenance.

make_ext.pl now generates a Makefile.PL if needed.
Remove ext/Safe/Makefile.PL as a proof of concept.

Find extensions on VMS even when they don't have a Makefile.PL.