summaryrefslogtreecommitdiff
path: root/scope.c
Commit message (Collapse)AuthorAgeFilesLines
* Chainsaw DEBUG_S out, as suggested by Vincent Pit.Rafael Garcia-Suarez2008-02-261-4/+0
| | | p4raw-id: //depot/perl@33376
* assert() that every NN argument is not NULL. Otherwise we have theNicholas Clark2008-02-121-0/+83
| | | | | | | | | | | | ability to create landmines that will explode under someone in the future when they upgrade their compiler to one with better optimisation. We've already done this at least twice. (Yes, some of the assertions are after code that would already have SEGVd because it already deferences a pointer, but they are put in to make it easier to automate checking that each and every case is covered.) Add a tool, checkARGS_ASSERT.pl, to check that every case is covered. p4raw-id: //depot/perl@33291
* Investigation reveals that the work of restoring the iterator to theNicholas Clark2008-01-261-9/+12
| | | | | | | | | | | | | | | | | 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
* POPLOOP is actually doing all the work of Perl_save_padsv() already!Nicholas Clark2008-01-261-1/+1
| | | p4raw-id: //depot/perl@33078
* Re: struct context now 12.5% smaller than 5.10Benjamin Smith2008-01-261-6/+5
| | | | | | Message-ID: <20080124215537.GB10198@vtrl.co.uk> Date: Thu, 24 Jan 2008 21:55:37 +0000 p4raw-id: //depot/perl@33070
* In struct block_loop, merge itermax and iterlval into a unionNicholas Clark2008-01-241-1/+1
| | | | | | | | | 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 withNicholas Clark2008-01-241-1/+1
| | | | | 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, andNicholas Clark2008-01-241-0/+2
| | | | | | 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
* Split CXt_LOOP into CXt_LOOP_PLAIN and CXt_LOOP_FOR, eliminating theNicholas Clark2008-01-241-1/+2
| | | | | CXp_FOREACH flag added as part of given/when. p4raw-id: //depot/perl@33057
* In struct block_subst, access the member once via a macro CxONCE()Nicholas Clark2008-01-211-1/+1
| | | | | which will allow the storage location to be changed. p4raw-id: //depot/perl@33034
* In struct block_eval, access the members old_in_eval and old_op_typeNicholas Clark2008-01-211-3/+3
| | | | | | 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_sub and block_format, access the members hasargs andNicholas Clark2008-01-201-4/+3
| | | | | | lval via macros CxHASARGS() and CxLVAL(), which will allow the storage location to be changed. p4raw-id: //depot/perl@33017
* Split struct block_sub into struct block_sub and struct block_format.Nicholas Clark2008-01-201-10/+10
| | | | | | (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 struct block_loop access element label via the macro CxLABEL()Nicholas Clark2008-01-201-2/+1
| | | | | (for the places that aren't about to change) p4raw-id: //depot/perl@33010
* Removes the code that is supposed to restore magic on leaving theRafael Garcia-Suarez2007-06-291-15/+2
| | | | | | | | | | | | | | | localization of an array or a hash. This fixes some memory leaks. Basically we were overwriting the magic of the outer value (value from the outer scope) by the magic of the inner value (therefore making that outer magic leaking in memory). But that inner magic was created by mg_localize() by copying *some* of the outer magic. Consequently the outer value already has that magic. So just keep it and don't bother. That change might introduce obscure bugs. On the other hand, it might also cure obscure bugs, related to the inner value acquiring container magic during its lifetime, or to the outer value loosing non-container magic. No test in the test suite seems to test that. p4raw-id: //depot/perl@31505
* blead segfaults on local *@; eval {1} because ERRSV assumes thatNicholas Clark2007-05-311-0/+8
| | | | | | | GvSV(PL_errgv) is always non-NULL. That stopped being the case with change 25009 (ish) - when we stopped automatically creating a(n unused) SV at GV creation time. p4raw-id: //depot/perl@31313
* Re: localising hash element by variableBo Lindbergh2007-05-291-1/+1
| | | | | Message-Id: <BC2C451F-B286-4D38-923E-E3B473F7B5E1@hagernas.com> p4raw-id: //depot/perl@31301
* Re: mro status, etcBrandon Black2007-04-301-2/+3
| | | | | | From: "Brandon Black" <blblack@gmail.com> Message-ID: <84621a60704291527y1b39be37l221ef66e4c828f66@mail.gmail.com> p4raw-id: //depot/perl@31107
* Un-mathom Perl_save_I16Jerry D. Hedden2007-04-231-0/+10
| | | | | | From: "Jerry D. Hedden" <jdhedden@cpan.org> Message-ID: <1ff86f510704230610t13c62378s381eaceae3e7e9b1@mail.gmail.com> p4raw-id: //depot/perl@31033
* Re: new C3 MRO patchBrandon Black2007-04-191-2/+2
| | | | | | From: "Brandon Black" <blblack@gmail.com> Message-ID: <84621a60704171114k29b0460el5b08ce5185d55ed5@mail.gmail.com> p4raw-id: //depot/perl@30980
* Get further through the build with read-only optrees. Free()ing copsNicholas Clark2007-04-041-0/+8
| | | | | | | | | as currently implemented seems to want write activity to them. Also we save a field in PL_curcop at one point, and sometimes this can be in a read only COP. If so, the end of scope restoration is always to the same value, but it's easier to cheat and skip the restoration than to dig around and avoid the save. p4raw-id: //depot/perl@30843
* Remove the other 4 bits of MAD code designed to abort on local $^L.Nicholas Clark2007-03-171-17/+0
| | | p4raw-id: //depot/perl@30607
* Remove the abort() trap, which Larry said was actually only everNicholas Clark2007-03-161-6/+0
| | | | | intended as a way to enter gdb by using local $^L p4raw-id: //depot/perl@30603
* Update copyright years in .c filesRafael Garcia-Suarez2007-01-051-1/+1
| | | p4raw-id: //depot/perl@29696
* The first three patches from:Marcus Holland-Moritz2007-01-041-1/+2
| | | | | | | | Subject: [PATCH] Cleanup SVf arguments (2nd try) Message-ID: <20070101201613.4120d9ef@r2d2> Adds (void*) casts to %-p and %p printf arguments p4raw-id: //depot/perl@29686
* handle cloning of parsers on the save stackDave Mitchell2007-01-031-0/+4
| | | p4raw-id: //depot/perl@29678
* Perl_save_destructor is not a mathomJerry D. Hedden2006-12-071-0/+10
| | | | | | From: "Jerry D. Hedden" <jdhedden@yahoo.com> Message-ID: <20061206191213.35036.qmail@web30206.mail.mud.yahoo.com> p4raw-id: //depot/perl@29482
* save_I8 is not a mathom.Rafael Garcia-Suarez2006-12-061-0/+10
| | | p4raw-id: //depot/perl@29476
* [perl #40557] regexec.c saves context stack position improperly Dave Bailey2006-10-181-0/+4
| | | | | | From: Dave Bailey (via RT) <perlbug-followup@perl.org> Message-ID: <rt-3.5.HEAD-4979-1161103047-337.40557-75-0@perl.org> p4raw-id: //depot/perl@29033
* Store a pointer to the op in struct block_loop, rather than the Nicholas Clark2006-09-171-5/+3
| | | | | | | pointers to last and redo. (for ithreads also drop the next pointer, as pp_ctl.c doesn't re-write it for ithreads). This shrinks the struct to 40 bytes on ILP32 systems, and therefore struct block to 64 bytes. p4raw-id: //depot/perl@28858
* Revert changes 28772, 28773, 28774 as they won't gain us anything - INicholas Clark2006-09-161-3/+3
| | | | | | failed to realise that struct block_sub is not the largest of the 4 structures in the block.blk_u union. (It's actually block_loop) p4raw-id: //depot/perl@28853
* Access to cx->blk_sub.lval via macros (as it's about to move).Nicholas Clark2006-09-011-1/+1
| | | p4raw-id: //depot/perl@28773
* Access cx->blk_sub.hasargs via a pair of macros (as it's about toNicholas Clark2006-09-011-2/+2
| | | | | move) p4raw-id: //depot/perl@28772
* g++ large patchJarkko Hietaniemi2006-08-071-1/+1
| | | | | Message-ID: <44D2E203.5050201@iki.fi> p4raw-id: //depot/perl@28662
* Rename cop_hints to cop_hints_hashNicholas Clark2006-05-201-2/+2
| | | p4raw-id: //depot/perl@28252
* De-duplicate the code that creates new GPs into Perl_newGP().Nicholas Clark2006-05-021-12/+2
| | | p4raw-id: //depot/perl@28058
* [perl #39012] another REIFY bugDave Mitchell2006-04-271-1/+1
| | | p4raw-id: //depot/perl@27986
* Change SAVECOPWARNINGS(c) to SAVECOMPILEWARNINGS() - it's only used toNicholas Clark2006-04-141-8/+5
| | | | | | | save the warnings on PL_compiling, so constraining its use to what we can test seems to make sense. Particularly as testing Perl_ss_dup is tricky. p4raw-id: //depot/perl@27805
* Remove SAVEt_FREESHAREDPV, as nothing is using it, and it isn't cateredNicholas Clark2006-04-131-4/+0
| | | | | | for in Perl_ss_dup, hence nothing is testing it and nothing will alert us if it breaks. p4raw-id: //depot/perl@27792
* Change cop_warnings from an SV holding the warnings bitmask to aNicholas Clark2006-04-121-0/+15
| | | | | | | directly (shared) malloc()ed buffer holding the warnings bitmask. This avoids bugs/crashes when the interpreter that created an optree is freed but the optree remains in use by other interpreters. p4raw-id: //depot/perl@27779
* By swapping the order of pushes onto the save stack forNicholas Clark2006-04-081-2/+2
| | | | | SAVEt_GENERIC_PVREF, we can merge it with SAVEt_PPTR in Perl_ss_dup(). p4raw-id: //depot/perl@27740
* Move all the regexp state variables into a single structure.Nicholas Clark2006-04-061-37/+1
| | | | | | This allows it to be saved, restored and cloned with a single Copy() (but inevitably still some fixup) p4raw-id: //depot/perl@27732
* Save the regexp engine state as 1 block on the save stack.Nicholas Clark2006-04-051-0/+53
| | | p4raw-id: //depot/perl@27728
* Perl_save_alloc can use the new(er) SSGROW rather than looping.Nicholas Clark2006-04-051-3/+1
| | | p4raw-id: //depot/perl@27726
* Abstract all the accesses to cop_arybase (apart from ByteLoader)Nicholas Clark2006-04-021-0/+5
| | | p4raw-id: //depot/perl@27671
* Automatically set HINT_LOCALIZE_HH whenever %^H is modified.Nicholas Clark2006-04-011-1/+21
| | | p4raw-id: //depot/perl@27666
* Serialise changes to %^H onto the current COP. Return the compile timeNicholas Clark2006-03-311-0/+2
| | | | | | state of %^H as an eleventh value from caller. This allows users to write pragmas. p4raw-id: //depot/perl@27643
* Fix bug #38815 (localising keys which are UTF-8 encoded didn't deleteNicholas Clark2006-03-301-1/+1
| | | | | them correctly on scope exit) p4raw-id: //depot/perl@27637
* re-[PATCH] Re: [PATCH] Poison now in two different flavours!Jarkko Hietaniemi2006-03-291-2/+2
| | | | | Message-ID: <442680D4.3000809@gmail.com> p4raw-id: //depot/perl@27626
* Note the cause of another MAD test failure - an explicit abort trapNicholas Clark2006-03-091-0/+1
| | | | | for saving PL_formfeed is being hit. p4raw-id: //depot/perl@27431
View Lorry Controller Status