Remove redundant check

p4raw-id: //depot/perl@33248

feature bundles are the same across all subversions
Message-ID: <20080113214537.13226.qmail@lists.develooper.com>

p4raw-id: //depot/perl@33247

Re: [PATCH] Re: Perl @ 32984 (also relevant to [perl #33849])
Message-ID: <Pine.LNX.4.64.0802011239170.1831@fractal.phys.lafayette.edu>

p4raw-id: //depot/perl@33246

Re: [PATCH] sv.c: consting
Message-ID: <20080204144419.GB20276@refcnt.homeunix.org>

p4raw-id: //depot/perl@33245

corelist changes:
- Add a new -d option to find first perl version by date
  and not by version number
- Better handling of perl versions that end with a 0
- use version.pm only for version numbers that have multiple dots

p4raw-id: //depot/perl@33244

Cope with differing prototypes for ECONNREFUSED etc on Win32 between
5.8.x and 5.10.x

p4raw-id: //depot/perl@33242

New blade

p4raw-id: //depot/perl@33241

in unpack, () groups in scalar context were still returning a list,
resulting in garbage on the stack, which could manifest as a SEGV
(Bug 50256)

p4raw-id: //depot/perl@33239

Adapt Safe innards to older (XS) versions of version.pm

p4raw-id: //depot/perl@33238

Add a new test for Safe

p4raw-id: //depot/perl@33237

Fix CPAN bug #32896: make version.pm loadable in a Safe compartment

p4raw-id: //depot/perl@33236

Removed mention of a book that was never published.

p4raw-id: //depot/perl@33235

Remove duplication from encoding.pm POD
From: "Robert May" <robertmay@cpan.org>
Message-ID: <54bdc7510801311136p7ee53dbet2afd5239e1ad03eb@mail.gmail.com>

p4raw-id: //depot/perl@33234

[patch] B portability macros
Message-ID: <47A3BC9F.7080200@gmail.com>

p4raw-id: //depot/perl@33233

Micro-optimise the order of the context types. [Because I can :-)]
Here saves 72 bytes in pp_ctl.o. Small, but in the right direction.

p4raw-id: //depot/perl@33232

sv.c: consting
Message-ID: <20080203185315.GA20276@refcnt.homeunix.org>

p4raw-id: //depot/perl@33231

fix B::Debug pmnext
From: "Reini Urban" <rurban@x-ray.at>
Message-ID: <6910a60802020733u7817a55dm6f5e213625063204@mail.gmail.com>

p4raw-id: //depot/perl@33230

Fix op/reg_email_thr.t when PERLIO=stdio

p4raw-id: //depot/perl@33229

taint.c: consting
Message-ID: <20080129124723.GA13056@refcnt.homeunix.org>
Date: Tue, 29 Jan 2008 13:47:23 +0100

p4raw-id: //depot/perl@33228

Recode all the old variable-like substitutions with the generic line
removal mechanism of change 33223.

p4raw-id: //depot/perl@33226

Produce a more generic expectation management system, and use it to
encode the 5.8.x vs 5.10/5.11 differences.
Provide general purpose TODOs for both the intial tests and Gerard's
"is it unchanged on repeat?" test.

p4raw-id: //depot/perl@33223

Upgrade to Math::Complex 1.48 and Math::Trig 1.13

p4raw-id: //depot/perl@33222

5.8.x and earler have a SUBPROCESS line in the dump of a PVIO.

p4raw-id: //depot/perl@33208

Change 27244 wasn't quite correct. XSUB is dumped as 0x0, not 0.

p4raw-id: //depot/perl@33206

Remove the _ prototype, as Maintainers is used by makemeta, and in turn
that has to be run with the newly built perl. Which can be 5.8.x.

p4raw-id: //depot/perl@33193

In XS_PerlIO_get_layers() take advantage of the implementation of
PerlIO_get_layers(), by co-opting the new SVs it creates, rather than
copying them.

p4raw-id: //depot/perl@33182

In XS_re_regexp_pattern(), use newSVpvn_flags() to avoid calls to
sv_2mortal().

p4raw-id: //depot/perl@33181

In pp_split(), eliminate most (all?) of the conditional calls to
sv_2mortal() by conditionally passing SVs_TEMP to newSVpvn_flags().

p4raw-id: //depot/perl@33178

Break apart the list of functions defined in universal.c by perl
version (from 5.8.8 upwards)

p4raw-id: //depot/perl@33170

The penultimate deparse test needs a name, else things can warn.

p4raw-id: //depot/perl@33163

threads::shared TODO test for outstanding bug
From: "Jerry D. Hedden" <jdhedden@cpan.org>
Message-ID: <1ff86f510801300615t1f9e8c68u5ff8caaeffb1a835@mail.gmail.com>

p4raw-id: //depot/perl@33154

Typo fix in change #33058

Subject: Re: [PATCH t/cmd/for.t] Regression tests for 'for reverse ..'
Message-ID: <20080130200922.GA20450@fury.crisman.org>
p4raw-link: @33058 on //depot/perl: 4b70616db4b88e90e04bbf6a612b803f6a4d8dc1

p4raw-id: //depot/perl@33153

don't forbid brace groups with g++ 4.2.2
From: "Robin Barker" <Robin.Barker@npl.co.uk>
Message-ID: <46A0F33545E63740BC7563DE59CA9C6D093A17@exchsvr2.npl.ad.local>

p4raw-id: //depot/perl@33152

fix for regression to File/DosGlob.pm
From: "Davies, Alex" <adavies@ptc.com>
Message-ID: <A69AA663CE9BBC44AE1DA72483DE15DE07E02181@HQ-MAIL3.ptcnet.ptc.com>

Fixes File::DosGlob's handling of drive relative glob patterns
(e.g. "D:*pl")

p4raw-id: //depot/perl@33144

Re: [perl #50322] CGITempFile causes "Insecure dependency in sprintf" in perl 5.10.0
From: "Steffen Mueller via RT" <perlbug-followup@perl.org>
Message-ID: <rt-3.6.HEAD-4355-1201526176-323.50322-94-0@perl.org>

Fixes [perl #50322]

p4raw-id: //depot/perl@33143

Fix compiler warning introduced in DEBUGGING builds by #33103

p4raw-id: //depot/perl@33142

Assorted POD nits from the Debian bug list.

p4raw-id: //depot/perl@33129

Formatting for C<<{ bydepth => 1 }>> doesn't seem to be working
right in the manpage.

p4raw-id: //depot/perl@33127

Need to substitute out the placeholder '$RV' for earlier perls too.

p4raw-id: //depot/perl@33125

Bump Safe's version

p4raw-id: //depot/perl@33124

Nicer fix than #33106 (thanks to Nicholas Clark)

p4raw-id: //depot/perl@33109

Fix Win32 compiler warnings introduced by #33081 and #33085

p4raw-id: //depot/perl@33106

Document why ok() is being used instead of like(), to stop someone
undoing it.

p4raw-id: //depot/perl@33105

More wording nits about perl's time() versus the libc's time()

p4raw-id: //depot/perl@33104

toke.c/universal.c: consting
Message-ID: <20080128125741.GA27390@refcnt.homeunix.org>

p4raw-id: //depot/perl@33103

Make this test pass with the Test::More that comes with perl 5.6.2

p4raw-id: //depot/perl@33102

mistaken variable in pod
From: "Yitzchak Scott-Thoennes" <sthoenna@efn.org>
Message-ID: <40783.64.81.167.122.1201556369.squirrel@webmail.efn.org>

p4raw-id: //depot/perl@33101

Upgrade to Tie-RefHash-1.38

p4raw-id: //depot/perl@33099

Upgrade to Archive-Extract-0.26

p4raw-id: //depot/perl@33098

Upgrade to SelfLoader-1.15

p4raw-id: //depot/perl@33097

Bump the version of Safe

p4raw-id: //depot/perl@33096

Minor rewording of the localtime() documentation,
based on :

Subject: [perl #49914] gmtime()/localtime() do not use time()
From: Abe Timmerman (via RT) <perlbug-followup@perl.org>
Message-ID: <rt-3.6.HEAD-18434-1200621906-376.49914-75-0@perl.org>

p4raw-id: //depot/perl@33095

Fix uc/lc warnings in CGI.pm
From: "Jerry D. Hedden" <jdhedden@cpan.org>
Message-ID: <1ff86f510801280719w4d56807esd8318bc6d0eadcdd@mail.gmail.com>

p4raw-id: //depot/perl@33094

In Safe, load Carp::Heavy only if it exists (to remain
compatible with older perls)

p4raw-id: //depot/perl@33093

Warning cleanup, and avoid a double call to uc

p4raw-id: //depot/perl@33092

Upgrade to Test::Harnes 3.07

p4raw-id: //depot/perl@33091

TAP::* modules are part of the Test::Harness distribution

p4raw-id: //depot/perl@33090

also report forced tokens when using -DT
Message-ID: <20080117173652.GB4969@ostwald>

p4raw-id: //depot/perl@33089

Make lc/uc/lcfirst/ucfirst warn when passed undef.
Naive implementation.

p4raw-id: //depot/perl@33088

Test that lc, uc, etc on undef return ""

p4raw-id: //depot/perl@33087

The third argument of PUSHEVAL was never used by anything. Ever.

p4raw-id: //depot/perl@33086

FW: [PATCH] RE: [PATCH] volatile, avoid clobbered
From: "Robin Barker" <Robin.Barker@npl.co.uk>
Message-ID: <46A0F33545E63740BC7563DE59CA9C6D093A06@exchsvr2.npl.ad.local>

p4raw-id: //depot/perl@33085

ext/IPC/SysV/SysV.xs [Re: [Fwd: Smoke [5.11.0] 33016 FAIL(XM)
Message-ID: <479D338E.9000402@iki.fi>

p4raw-id: //depot/perl@33084

util.c: some consting
Message-ID: <20080125001052.GA29980@refcnt.homeunix.org>

p4raw-id: //depot/perl@33083

fix misleading comment in op.h

p4raw-id: //depot/perl@33082

The layout for struct block_loop under ithreads can be simplified.
Instead of wedging the pad offset into a void* iterdata, and always
storing PL_comppad even when it isn't used, instead do this:

PAD        *oldcomppad; /* Also used for the GV, if targoffset is 0 */
/* This is also accesible via cx->blk_loop.my_op->op_targ */
PADOFFSET  targoffset;

and store the GV pointer in oldcompad. Pointers to pointers seems
cleaner. This also allows us to eliminate the flag bit CXp_PADVAR.

p4raw-id: //depot/perl@33081

Investigation reveals that the work of restoring the iterator to the
pad is shared between POPLOOP, using itersave, and the end of scope
restore action requested by Perl_save_padsv(). In fact, the only user
of SAVEt_PADSV is pp_enteriter, and it already provides enough
information to allow it to perform the sv_2mortal() in POPLOOP.
So make it do so. Rather than creating a new routine, use the existing
routine because nothing else (at least nothing else known to Google's
codesearch) uses it. But rename it just in case something we can't see
is being naughty and using our private functions - they will get
link errors against 5.12.

All this means that itersave is now redundant. So remove it.
This makes struct context 48 bytes on ILP32 platforms with 32bit IVs,
down from 64 bytes in 5.10. 33% more context stack in the same memory.

p4raw-id: //depot/perl@33080

Change 33072 missed embed.h. I wasn't aware that it was affected by
opcode.pl. You live and learn (and should run p4 diff -se ...).

p4raw-id: //depot/perl@33079

POPLOOP is actually doing all the work of Perl_save_padsv() already!

p4raw-id: //depot/perl@33078

Standardise the conditional compilation protection of ({}) from
#if defined(__GNUC__) && !defined(__STRICT_ANSI__) && !defined(PERL_GCC_PEDANTIC)
to
#if defined(__GNUC__) && !defined(PERL_GCC_BRACE_GROUPS_FORBIDDEN)
because the ({}) construction can be used under __STRICT_ANSI__
(and should be, because it avoids temporary use of PL_Sv).

p4raw-id: //depot/perl@33077

As itersave points to the initial CxITERVAR(), and the state of
SvPADMY() does not change over the duration of the scope, we can
perform conditional actions at loop push time. For the non-pad case,
a reference to the initial CxITERVAR() is already held on the scope
stack thanks to SAVEGENERICSV(*svp); in pp_enteriter. So there is no
need to save another reference to it in itersave - it's not going away.

p4raw-id: //depot/perl@33076

Restore the else block accidently eaten by change 33074.

p4raw-id: //depot/perl@33075

In POPLOOP, if CxITERVAR(cx) is non-NULL, then so is itersave, and
itersave is a less complex expression for the C compiler.

p4raw-id: //depot/perl@33074

Some code can be removed following change 33070.

p4raw-id: //depot/perl@33073

Eliminate the OP_SETSTATE, which had been disabled by change 4309.

p4raw-id: //depot/perl@33072

Change 33069 missed updating dump.c to dump PVIOs correctly.

p4raw-id: //depot/perl@33071

Re: struct context now 12.5% smaller than 5.10
Message-ID: <20080124215537.GB10198@vtrl.co.uk>
Date: Thu, 24 Jan 2008 21:55:37 +0000

p4raw-id: //depot/perl@33070

PVIOs don't need space for SvNVX allocated.

p4raw-id: //depot/perl@33069

Using PL_sv_no in place of any !SvOK() maximum removes a little bit of
hot code in pp_iter.

p4raw-id: //depot/perl@33068

Silence "possible data loss" warning.

p4raw-id: //depot/perl@33067

socketpair() *is* available on Win32
From: "Jan Dubois" <jand@activestate.com>
Message-ID: <032201c85ed6$e56edf40$b04c9dc0$@com>

p4raw-id: //depot/perl@33066

Don't warn about imprecision when decrementing IV_MIN.
Based on a patch by Jerry D. Hedden, but only instead only disable
warnings for the specific operations that we know will warn.

p4raw-id: //depot/perl@33065

... and the rest; was RE: unnecessary define
From: "Robin Barker" <Robin.Barker@npl.co.uk>
Message-ID: <46A0F33545E63740BC7563DE59CA9C6D093A08@exchsvr2.npl.ad.local>
Date: Thu, 24 Jan 2008 13:28:33 -0000

p4raw-id: //depot/perl@33064

In struct block_loop, merge itermax and iterlval into a union
lval_max_u, as CXt_LOOP_LAZYIV doesn't use iterlval and the other
LOOP types don't use itermax. This reduces struct block_loop by 1 IV.
As it's the largest component of the unions making up struct context,
this reduces struct context. On ILP32 it will now be 56 bytes, down
from the 64 of 5.10.x, as I've already removed the element 'label'.

p4raw-id: //depot/perl@33063

Merge CXt_LOOP_STACK's use of itermax for the reverse minimum with
iterary, as the two structure members are not used simultaneously.

p4raw-id: //depot/perl@33062

Change the context type of for ($a .. $b) to CXt_LOOP_LAZYIV, and
assert that it isn't using  cx->blk_loop.iterlval.
Fix a casting bug when assigning a sentinal to cx->blk_loop.iterary.

p4raw-id: //depot/perl@33061

Avoid using cx->blk_loop.itermax when reverse iterating an array.

p4raw-id: //depot/perl@33060

Split out foreach iterations of temporary lists on the stack to
CXt_LOOP_STACK. Don't use cx->blk_loop.iterary to store PL_curstack.

p4raw-id: //depot/perl@33059

Regression tests for 'for reverse ..'
Message-ID: <20080123225325.GA25959@abigail.be>
Date: Wed, 23 Jan 2008 23:53:25 +0100

p4raw-id: //depot/perl@33058

Split CXt_LOOP into CXt_LOOP_PLAIN and CXt_LOOP_FOR, eliminating the
CXp_FOREACH flag added as part of given/when.

p4raw-id: //depot/perl@33057

RE: [PATCH ext/Devel/DProf/Makefile.PL] unnecessary define
From: "Robin Barker" <Robin.Barker@npl.co.uk>
Message-ID: <46A0F33545E63740BC7563DE59CA9C6D093A01@exchsvr2.npl.ad.local>
Date: Wed, 23 Jan 2008 14:13:30 -0000

p4raw-id: //depot/perl@33056

Update uconfig.h to add USE_DTRACE back in.

p4raw-id: //depot/perl@33055

Oops, change 33049 really did need to update all the Win?? headers
as the symbol isn't used in a conditional compile.

p4raw-id: //depot/perl@33054

Change 33052 missed one Nullch. Oops.
Exterminate! Exterminate! Exterminate!

p4raw-id: //depot/perl@33053

Purge all use of Nullch in non-dual life modules.
Exterminate! Exterminate! Exterminate!

p4raw-id: //depot/perl@33052

Deprecate (and remove core use of ) Nullav, Nullcv, Nullgv, Nullhe,
Nullhek and Nullhv. Nullop is going to be a bit less simple.

p4raw-id: //depot/perl@33051

Comment change from 33009 that was actually in POD, so needs to be
propagated to perlintern.pod

p4raw-id: //depot/perl@33050

Fix the misplaced warnings and failing tests caused by the precision
loss warning on ++ and -- by moving the check to Configure time,
creating a new config.sh variable nv_overflows_integers_at which
contains an constant expression for the value of the NV which can't
be incremented by 1.0

p4raw-id: //depot/perl@33049

Teach checkcfgvar.pl that : is also a comment character in shell scripts

p4raw-id: //depot/perl@33048

Re-generated sample files

p4raw-id: //depot/perl@33047