assert() that we can't reach two unreachable code paths
(in Perl_magic_get and Perl_pp_shift).

p4raw-id: //depot/perl@27154

The return value from S_hv_auxinit can be ignored, as it's also stored
within the passed in hv parameter.

p4raw-id: //depot/perl@27153

A better fix than 27148

Subject: Win32 assert (was Re: Smoke [5.9.4] 27145 FAIL(M) MSWin32 WinXP/.Net SP2 (x86/2 cpu))
From: Nicholas Clark <nick@ccl4.org>
Date: Fri, 10 Feb 2006 14:58:47 +0000
Message-ID: <20060210145847.GA85020@plum.flirble.org>

Subject: RE: Win32 assert (was Re: Smoke [5.9.4] 27145 FAIL(M) MSWin32 WinXP/.Net SP2 (x86/2 cpu))
From: "Jan Dubois" <jand@ActiveState.com>
Date: Fri, 10 Feb 2006 09:03:41 -0800
Message-ID: <024401c62e63$f1e53c10$6062a8c0@candy>

p4raw-id: //depot/perl@27152

With PERL_POISON defined, ensure freshly malloc()ed memory isn't zeros,
and when PERL_TRACK_MEMPOOL is also defined scribble on any extension
added by realloc().

p4raw-id: //depot/perl@27151

Fix the effects of 27136 on Win32 with DEBUGGING and PERL_IMPLICIT_SYS

p4raw-id: //depot/perl@27148

Change 27136 seems to have inadvertently upset some compilers,
presumably where NULL was defined as 0, not (void *)0

p4raw-id: //depot/perl@27145

Fix breakage in S_set_csh() from change #27136.
p4raw-link: @27136 on //depot/perl: 5f66b61cbe1f2ea985f00c0d9a66153e7a3b6ee1

p4raw-id: //depot/perl@27140

Document incompatible interpretation of chdir FOO.
Ref change 27125.

p4raw-id: //depot/perl@27139

Really regenerate headers

p4raw-id: //depot/perl@27138

Regenerated headers for change #27136.
p4raw-link: @27136 on //depot/perl: 5f66b61cbe1f2ea985f00c0d9a66153e7a3b6ee1

p4raw-id: //depot/perl@27137

Trying my "remove the pTHXes" patch again
Message-ID: <20060209154018.GA14610@petdance.com>

p4raw-id: //depot/perl@27136

Skip new chdir warning test if fchdir is not available

p4raw-id: //depot/perl@27135

Avoid calling all the scanning code in gv_fetchpvn_flags when saving
$1 etc, as we can perfom a hash lookup on %:: directly.

p4raw-id: //depot/perl@27134

lastparen in the regexp structure is never initialised, and so will
still be uninitialised after a failed match, yet various points in
mg.c read it come what may. Should fix bug 38461.

p4raw-id: //depot/perl@27133

Bareword file handles not supported for chown/chmod/utime.

p4raw-id: //depot/perl@27132

Wave hands about encoding::warnings

p4raw-id: //depot/perl@27131

[perl #38464] chdir() on closed handles
From: Peter Dintelmann (via RT) <perlbug-followup@perl.org>
Message-ID: <rt-3.0.11-38464-129771.3.46114092925276@perl.org>

Plus a regression test.

p4raw-id: //depot/perl@27130

Handle unused args
Message-ID: <20060207223656.GA5177@petdance.com>

p4raw-id: //depot/perl@27129

[patch] several broken make *.valgrind targets
Message-ID: <43E91B2A.4060700@gmail.com>

p4raw-id: //depot/perl@27128

Borland's C compiler warns that the & is unnecessary.

p4raw-id: //depot/perl@27127

Merging pp_bit_or and pp_bit_xor shrinks the object code by about .7K.
The overloading tests are not free.

p4raw-id: //depot/perl@27126

Allow bareword file handle as argument to chdir().
This copies the mechanism used by truncate().
Fixes bug #38457.

p4raw-id: //depot/perl@27125

All the trancendental unary operators can be merged into PP_sin
(cos, exp, log, sqrt)

p4raw-id: //depot/perl@27124

Avoid gcc warning about possibly uninitialised variables.

p4raw-id: //depot/perl@27123

Create new macros AMG_CALLun_var, AMG_CALLun_var and tryAMAGICun_var
which don't do the pre-processor string manipulation internally.

p4raw-id: //depot/perl@27122

pp_pop can be implemented by pp_shift.

p4raw-id: //depot/perl@27121

Re: [PATCH] Documentation patch for overload
Message-ID: <43E71DD4.5010108@sneakemail.com>

p4raw-id: //depot/perl@27120

pp_hex can be implemented trivially by pp_oct, making pp_hex a mathom.

p4raw-id: //depot/perl@27119

Merge pp_index and pp_rindex - we have another mathom.

p4raw-id: //depot/perl@27118

Some refactoring to converge pp_index and pp_rindex

p4raw-id: //depot/perl@27117

Fix bug 38454 (rindex corrects for $[ on bytes rather than UTF-8)

p4raw-id: //depot/perl@27116

Re: Configure.com issue, more quoting needed.
Message-id:  <OF3536E719.A8B8D167-ON8525710D.0065011E-8525710D.00655ED0@factset.com>

p4raw-id: //depot/perl@27115

patch@27082 Allow fatal exceptions to bring up VMS debugger
From: "John E. Malmberg" <wb8tyw@qsl.net>
Message-id: <43E516E0.1000508@qsl.net>

p4raw-id: //depot/perl@27114

Optimise index so that if the big string is ISO-8859-1 but the little
string is UTF-8, it tries to downgrade the little string, rather than
upgrade the big string. For half-meg big strings this is a fourfold
speed gain.

p4raw-id: //depot/perl@27113

Given that the memory allocated in Perl_bytes_from_utf8 and
Perl_bytes_to_utf8 will immediately be written to, I see no need to
allocate it zeroed.

p4raw-id: //depot/perl@27112

Rename S_glob_assign to S_glob_assign_glob
Rename S_pvgv_assign to S_glob_assign_ref

I don't think that they share enough common code to merge them.

p4raw-id: //depot/perl@27111

I think that "merge Perl_sv_2[inpu]v" and "reduce duplication in
sv_setsv_flags" are about as done as they can be.

p4raw-id: //depot/perl@27110

In Perl_sv_upgrade, the assignment to new_type_details only needs to
be done once.

p4raw-id: //depot/perl@27109

Document the limitations of calling exit() (instead of _exit()) from
a child process on Solaris.

p4raw-id: //depot/perl@27108

Remove a duplicate flag copy line from Perl_sv_setsv_flags.

p4raw-id: //depot/perl@27107

Move Perl_save_long, Perl_save_I16, Perl_save_I8, Perl_save_iv,
Perl_save_nogv, Perl_save_list, Perl_save_destructor to mathoms.c
Perl_save_svref can't actually move because it calls a static function.

p4raw-id: //depot/perl@27106

Expunge USE_5005THREADS from perl.c

p4raw-id: //depot/perl@27105

Fix bug #38439 - reference to typeglob assignment needs to be based
on SvROK(sstr) rather than SvTYPE(sstr) == SVt_RV.

p4raw-id: //depot/perl@27104

5.8.8: Unicos test skip (and for the record: config.sh and harness results)
Message-ID: <43E5D085.90400@iki.fi>

p4raw-id: //depot/perl@27103

Cleaning up shadowed variables
Message-Id: <B375E848-FA07-479D-9571-FA3925106D16@petdance.com>

p4raw-id: //depot/perl@27102

Merge the PVCV case into the common code in S_pvgv_assign.

p4raw-id: //depot/perl@27101

Merge the PVAV and PVHV clauses into the common code of S_pvgv_assign.

p4raw-id: //depot/perl@27100

Merge the default case (GvSV) with the common code in S_pvgv_assign.

p4raw-id: //depot/perl@27099

Start to merge the 6 arms of S_pvgv_assign into one.

p4raw-id: //depot/perl@27098

[patch] rework arenas - repair arenasets
Message-ID: <43E5F012.1080203@gmail.com>
Date: Sun, 05 Feb 2006 05:31:14 -0700

p4raw-id: //depot/perl@27097

Documentation patch for overload
Message-ID: <43E5E0E9.1070707@sneakemail.com>

p4raw-id: //depot/perl@27096

The message "Can't upgrade that kind of scalar" can be far more
informative.

p4raw-id: //depot/perl@27095

new_body_type doesn't need to subtract the offset, that's what
new_body_allocated is for.

p4raw-id: //depot/perl@27094

Yes, I broke Win32 with change 27089 by having a { in the wrong place.

p4raw-id: //depot/perl@27093

Remove "Can't upgrade to undef" from perldiag, as the corresponding
code is now gone from sv_upgrade.

p4raw-id: //depot/perl@27092

Merge and refactor the PVAV and PVHV code in Perl_sv_upgrade, which
reduces the object size by about 80 bytes.

p4raw-id: //depot/perl@27091

Eliminate a croak we can't get to.

p4raw-id: //depot/perl@27090

Without this Windows and Netware compilers should be complaining
about perl.c having code before a declaration, *but only if compiling
with PERL_TRACK_MEMPOOL*. So, will it fix the smoke failures? Place
bets now.

p4raw-id: //depot/perl@27089

Don't export PL_memory_debug_header to Win32 (and others) unless
PERL_TRACK_MEMPOOL is defined.

p4raw-id: //depot/perl@27088

Fixed embed.fnc entry for S_forbid_setid() and picked up a change in
pod/perlapi.pod for good measure when regenerating files.

p4raw-id: //depot/perl@27087

bcc was issuing a warning about this line in Perl_sv_2cv, so tweak it
to keep it happy.

p4raw-id: //depot/perl@27086

Add regression tests to lib/charnames.t to ensure that unicore/Names.pl
is 100% 7 bit ASCII.

p4raw-id: //depot/perl@27085

Enhance PERL_TRACK_MEMPOOL so that it also emulates the PerlHost
behaviour of freeing up all memory at thread exit. With this and
tools such as valgrind you will now get warnings as soon as you
read from the deallocated memory, rather than just a warning much
later about freeing to the wrong pool.

p4raw-id: //depot/perl@27084

The TODO 'Tidy up global variables' can go, replaced with a second
paragraph on 'Ordering of "global" variables'.

p4raw-id: //depot/perl@27083

I believe that mathoms have been completely TODOne.

p4raw-id: //depot/perl@27082

Rejig the definition of ARENAS_PER_SET to maximise the size of the
array whilst keeping the structure within PERL_ARENA_SIZE.

p4raw-id: //depot/perl@27081

Tweak arena sets to avoid assignment between different pointers.

p4raw-id: //depot/perl@27080

[patch] arena rework - arena sets
Message-ID: <43DF4F66.4080808@gmail.com>
Date: Tue, 31 Jan 2006 04:52:06 -0700

p4raw-id: //depot/perl@27079

warnings.h does some evil(*) pointer arithmetic on (SV *)0, so a
simple replace of Nullsv with NULL turns out to be a bad idea.
* Technically undefined behaviour, I believe.

p4raw-id: //depot/perl@27078

Remove obsolete comment.

p4raw-id: //depot/perl@27077

S_find_beginning is only called from one place, so by hoisting the
forbid_setid check for -x into the caller, we can eliminate the
suidscript parameter from S_find_beginning.

p4raw-id: //depot/perl@27076

And as we've now got to the point where all calls to Perl_moreswitches
have -1 as the second argument, we can remove the second argument,
which gets us back to where we started, only with the elimination of
a per-thread variable.

p4raw-id: //depot/perl@27075

The MSDOS-only call to moreswitches will never reach forbid_setid, so
it can have a parameter of -1 without changing any behaviour.
I see no reason to forbid "command line" switches when they are
actually coming from the #! line of the script itself (as read by perl)
as they must have been written by the owner of the script, rather than
being crafted by the user manipulating the command line, and therefore
cannot be subverted. (Or alternatively, can be subverted as easily as
the rest of the script, and we're permitting *that* to be run)

p4raw-id: //depot/perl@27074

Pull the variables fdscript and suidscript into a much smaller scope.
Replace suidscript with literal -1 where it could only have been -1.
(Re-indent the other code inside the new block creating the scope.
 Rejig the indentation of the conditionals)

p4raw-id: //depot/perl@27073

Change 27070 failed to initialise a variable. (But despite that still
passed all tests locally.)

p4raw-id: //depot/perl@27072

Silence a warning about "statement not reached" from the Sun C compiler.
It was accurate - the conditional code used to give 2 return statements.

p4raw-id: //depot/perl@27071

Eliminating PL_suidscript is more tricky, and requires changing the
prototype of Perl_moreswitches.

p4raw-id: //depot/perl@27070

Passing the flag to S_forbit_setid as a single char rather than a
string is a slight space optimisation.

p4raw-id: //depot/perl@27069

Oops. I *thought* that I had checked that all changed files were open.
Clearly not. (Fixes change 27066)

p4raw-id: //depot/perl@27068

Oops. When changing warnings.pl one should re-run it. Well spotted
Rafael.

p4raw-id: //depot/perl@27067

It's actually easier to get rid of PL_fdscript than we thought.

p4raw-id: //depot/perl@27066

Re: [PATCH] s/Null(gv|hv|sv)/NULL/g
Message-ID: <20060203152449.GI12591@accognoscere.homeunix.org>
Date: Fri, 3 Feb 2006 16:24:49 +0100

p4raw-id: //depot/perl@27065

Fix file where internal XS functions are defined

p4raw-id: //depot/perl@27064

Help to compile on VMS with UNLINK_ALL_VERSIONS
See :
Subject: bug@26347 embed.fnc/embed.h with unlnk macro
From: "John E. Malmberg" <wb8tyw@qsl.net>
Message-ID: <439F8F20.4050207@qsl.net>

p4raw-id: //depot/perl@27063

[PAUSE] CPAN Upload: A/AN/ANDK/CPAN-1.83_64.tar.gz
Message-ID: <87acda0z8y.fsf@k75.linux.bogus>

p4raw-id: //depot/perl@27062

L<PerlIO> and Pod::Html
Message-Id: <20060109125220.3BB0.BQW10602@nifty.com>

p4raw-id: //depot/perl@27061

s/Nullcv/NULL/g
Message-ID: <20060202150241.GF12591@accognoscere.homeunix.org>
Date: Thu, 2 Feb 2006 16:02:41 +0100

p4raw-id: //depot/perl@27060

Change PL_perlio_fd_refcnt from a fixed size static array to a pointer
to a dynamic array.

p4raw-id: //depot/perl@27059

It would be nice to have tests in B for the functions getting changed.

p4raw-id: //depot/perl@27057

Death to magic characters!  No pun intended.

p4raw-id: //depot/perl@27056

Re: [PATCH] s/Null(av|ch)/NULL/g
Message-ID: <20060202093849.GD12591@accognoscere.homeunix.org>

p4raw-id: //depot/perl@27054

gv_fetchpvn_flags ranks highly in the profile, and the ::/' scanning
loop is iterated over millions of times. Add a flag GV_NOTQUAL purely
as an optimisation, when the caller is passing in a string that is
known not to contain any package separators.

p4raw-id: //depot/perl@27053

Update to Module::CoreList 3.04.
Cleanup corelist manpage.
Remove -w in corelist shebang to avoid spurious warnings.

p4raw-id: //depot/perl@27052

perl-5.8.8 also contained change 27040.

p4raw-id: //depot/perl@27051

Yet more instances of gv_fetchpv... that should be GV_ADD rather than
TRUE.
Convert two gv_fetchpvn_flags to gv_fetchpvs.

p4raw-id: //depot/perl@27050

And another assumption.

p4raw-id: //depot/perl@27049

Since PERL_COPY_ON_WRITE became PERL_OLD_COPY_ON_WRITE it seems that
no-one has been testing it, as there was a const too far.

p4raw-id: //depot/perl@27048

Doc nit found by Stephen Gordon.

p4raw-id: //depot/perl@27047

doubleplusoops. Apart from the documented NUL termination assumption.
Now removed.

p4raw-id: //depot/perl@27046

References can't have SVp_IOK or SVp_NOK set, so no need for all the
conditional code to copy IVX and NVX if true. "You cannot get here".

p4raw-id: //depot/perl@27045