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

checkcfgvar.pl should also check Porting/config.sh "just in case".

p4raw-id: //depot/perl@33046

The return of USE_DTRACE

p4raw-id: //depot/perl@33045

Regen uconfig.h

p4raw-id: //depot/perl@33043

Upgrade to PathTools-3.27

p4raw-id: //depot/perl@33042

Get C++ compiles going on Solaris again.

p4raw-id: //depot/perl@33041

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

p4raw-id: //depot/perl@33040

Regen. Lots of under-the-hood changes accumulated.

p4raw-id: //depot/perl@33038

Remove code obsoleted by change 18.

p4raw-id: //depot/perl@33036

Eliminate the U8 sbu_once from struct subst, and shrink sbu_rflags from
I32 to U8, which reduces the size of the struct by at least 4 bytes.

p4raw-id: //depot/perl@33035

In struct block_subst, access the member once via a macro CxONCE()
which will allow the storage location to be changed.

p4raw-id: //depot/perl@33034

In pp_subst, rxtainted is not a boolean, as it stores 2 bits of values.

p4raw-id: //depot/perl@33033

Tidy up context type flags.

p4raw-id: //depot/perl@33032

Ignore build dir when installing perl
From: "Jerry D. Hedden" <jdhedden@cpan.org>
Message-ID: <1ff86f510801190919o6e3af1bboff1f885c76cf2bc7@mail.gmail.com>

p4raw-id: //depot/perl@33031

In struct block_eval, eliminate old_in_eval and old_op_type by storing
the data in blk_u16.

p4raw-id: //depot/perl@33030

In struct block_eval, access the members old_in_eval and old_op_type
via macros CxOLD_IN_EVAL() and CxOLD_OP_TYPE(), which will allow the
storage location to be changed.

p4raw-id: //depot/perl@33029

In struct block change blku_type from U8 to U16, and the "spare" U8 to
U16, with the lockstep changes in struct subst. Eliminate lval from
struct block_sub, and instead store it in the U16 in struct block.

p4raw-id: //depot/perl@33028

Squeeze the context type down to 4 bits, and move the private flags to
fit within the next 4 bits.

p4raw-id: //depot/perl@33027

When changing the op_ppaddr of an op, one must keep its op_type
in sync. That helps writers of alternate runloops.

p4raw-id: //depot/perl@33026

Give G_VOID, G_SCALAR and G_ARRAY the same numeric values as
OPf_WANT_VOID, OPf_WANT_SCALAR and OPf_WANT_LIST.

p4raw-id: //depot/perl@33025

More places that could be using G_WANT, not picked up by change 33021.

p4raw-id: //depot/perl@33024

Remove hardcoded cop.h constants from APItest.pm :-(
Add G_WANT. Make call.t use G_WANT.

p4raw-id: //depot/perl@33023

Change the wantarray result from caller from IV to bool for the SCALAR/
ARRAY case. This doesn't contradict the documentation, as there isn't
any. Oops.

p4raw-id: //depot/perl@33022

Add a flag G_WANT, as a mask for the bits G_SCALAR, G_ARRAY and G_VOID.

p4raw-id: //depot/perl@33021

Encode the G_* to OPf_WANT_* transform, the reverse of OP_GIMME(), in
a macro OP_GIMME_REVERSE() [so that it could be changed].

p4raw-id: //depot/perl@33020

Change the hasargs return value from caller from IV to bool, as it's
actually only a boolean, and sv_yes/sv_no are cheaper to "create".
The documentation says 'C<$hasargs> is true if ...' so this is fine.

p4raw-id: //depot/perl@33019

Eliminate hasargs from structs block_sub and block_format by storing
it with a private flag CXp_HASARGS in cx_type. (It's only a boolean.)

p4raw-id: //depot/perl@33018

In struct block_sub and block_format, access the members hasargs and
lval via macros CxHASARGS() and CxLVAL(), which will allow the storage
location to be changed.

p4raw-id: //depot/perl@33017

Move retop first in structs block_sub, block_format and block_eval.
This simplifies some code in Perl_deb_stack_all().

p4raw-id: //depot/perl@33016

Eliminate label from struct block_loop, as it can be accessed via
oldcop.

p4raw-id: //depot/perl@33015

Split struct block_sub into struct block_sub and struct block_format.
(CXt_SUB and CXt_FORMAT were using some comon members, but some members
were only for one or the other.)

p4raw-id: //depot/perl@33014

In Perl_cx_dup(), block copy the entire context stack, then walk it
to duplicate/fixup only the things that aren't simple binary copies.

p4raw-id: //depot/perl@33013

Extend PUSHFORMAT() to take a second parameter to set retop, to save
NULLing it and then reassigning.

p4raw-id: //depot/perl@33012

Fix (probable) bug-by-inspection -  CxREALEVAL(), CxTRYBLOCK(),
CxFOREACH() and CxFOREACHDEF() should all be masking against
CXTYPEMASK rather than their own type number.

p4raw-id: //depot/perl@33011

In struct block_loop access element label via the macro CxLABEL()
(for the places that aren't about to change)

p4raw-id: //depot/perl@33010

[perl #49522] state variable not available
Svf_PADSTALE means something different for state vars. Make sure
we always handle it correctly

p4raw-id: //depot/perl@33009

Avoid an unused argument in S_sv_2iuv_non_preserve() by using
conditional compilation to only pass it in if it's needed.

p4raw-id: //depot/perl@33004

Nick's a muppet - "all" needs to remain the first real target in the
Makefile. Do not be distracted by suffix rules above.

p4raw-id: //depot/perl@33003

Don't create local variables validarg and fdscript if they aren't going
to be used.

p4raw-id: //depot/perl@33002

The suidscript argument to S_forbid_setid(), S_open_script() and
S_validate_suid() is actually just a boolean. So make it so.

p4raw-id: //depot/perl@33001

Eliminate all PERL_UNUSED_ARG()s in S_validate_suid() by changing its
prototype depending on the compile time options. In turn, this finds
things that are unused in its callers.

p4raw-id: //depot/perl@33000

A target for sperl.i

p4raw-id: //depot/perl@32999

Change 32997 missed one conditionally unused argument.

p4raw-id: //depot/perl@32998

Collect all the S_validate_suid() unused argument logic in one place.

p4raw-id: //depot/perl@32997

Remove commented out declaration from S_validate_suid()

p4raw-id: //depot/perl@32996

New tests for new ops. Most of them are TODO

p4raw-id: //depot/perl@32995

Remove commented out code from S_validate_suid()

p4raw-id: //depot/perl@32994

Add Weed out needless PERL_UNUSED_ARG to perltodo. It's a good
"cage cleaner" task.

p4raw-id: //depot/perl@32993

warn if ++ or -- are unable to change the value because it's beyond
the limit of representation in NVs, using a new warnings category
"imprecision".

p4raw-id: //depot/perl@32990

Upgrade to Math-Complex-1.47

p4raw-id: //depot/perl@32989

Generate the warnings masks programatically.
Get the correct line number when reporting errors from &check_bits.

p4raw-id: //depot/perl@32988

Add a diagram for the 5.11 SV class structure, including B::REGEXP.

p4raw-id: //depot/perl@32987

Don't ask about a module when using -ok
Message-ID: <20080116174617.GA30446@abigail.be>
Date: Wed, 16 Jan 2008 18:46:17 +0100

p4raw-id: //depot/perl@32986

A couple of POD fixes by Steven Schubiger

p4raw-id: //depot/perl@32982

Thanks to Sébstien Aperghis-Tramoni's suggestion,  updated version
numbers in a few extra files and regenerated uconfig.h.

p4raw-id: //depot/perl@32981

Boolean priority bug, found by Mashrab Kuvatov:

Subject: Re: [perl #49646] perlbug AutoReply: open ':locale' does not work under locale with the modifier
Message-Id: <200801151517.46296.kmashrab@uni-bremen.de>

p4raw-id: //depot/perl@32980

Fix range operator
From: "Jerry D. Hedden" <jdhedden@cpan.org>
Message-ID: <1ff86f510801141656i325ac69ev8a0af47f9fe72a1e@mail.gmail.com>

p4raw-id: //depot/perl@32979

consting IO.xs
From: "Robin Barker" <Robin.Barker@npl.co.uk>
Message-ID: <46A0F33545E63740BC7563DE59CA9C6D0939A1@exchsvr2.npl.ad.local>

p4raw-id: //depot/perl@32978

When parsing LC_ALL or LANG to get the locale's encoding, ignore
whatever is after the @, since that's a modifier, not an encoding.

p4raw-id: //depot/perl@32977

consting
From: "Robin Barker" <Robin.Barker@npl.co.uk>
Message-ID: <46A0F33545E63740BC7563DE59CA9C6D0939CA@exchsvr2.npl.ad.local>

p4raw-id: //depot/perl@32976

Intermediate update/regen in cleanup process, dtrace support
is now backported

p4raw-id: //depot/perl@32974

Re-order so that the !SvOK() case is last (which should be rare)
Remove the FIXME comment as I had already fixed it.

p4raw-id: //depot/perl@32971

Upgrade to Math-Complex-1.44

p4raw-id: //depot/perl@32970

For 5.12: saner behaviour for `length`
(Make C<length undef> return undef).
Patch mostly by Rafael, with some fine tuning by me.

p4raw-id: //depot/perl@32969

Fix bug whereby length on a tied scalar that returned a UTF-8 value
would not be correct the first time. (And for the more pathological
case, would be incorrect if the UTF-8-ness of the returned value
changed.)

p4raw-id: //depot/perl@32968