Fix another Win32/VC6 compiler warning caused by #33231
(missed by #33336--this one only occurs in debug builds)

p4raw-id: //depot/perl@33385

Change variable name to fix debug build with PERL_OLD_COPY_ON_WRITE

p4raw-id: //depot/perl@33384

Comment on why I don't think changing Perl_safesysmalloc_size() in av.c
analagous to the change in sv.c is a good idea. [It's not a language
design issue, so sadly I can't get a talk out of it. Or is that
fortunately? :-)]

p4raw-id: //depot/perl@33383

Fix preprocessor syntax

p4raw-id: //depot/perl@33380

If the C library provides malloc_size(), we can use that in the same
places as Perl's malloced_size(), except that we need to be careful of
any PERL_TRACK_MEMPOOL manipulations in force. Wrap both as
Perl_safesysmalloc_size(), to give a consistent name and interface.

p4raw-id: //depot/perl@33379

In Perl_sv_usepvn_flags(), with MYMALLOC, use the actual malloc()ed
size for SvLEN(), rather than an estimate.

p4raw-id: //depot/perl@33378

If we have malloced_size() available, then avoid rounding up the string
to the next (guessed) plausible alignment size, and instead find out
how much memory was actually allocated, so that we can set this in the
scalar's SvLEN(). This way, sv_grow() will be called far less often.

p4raw-id: //depot/perl@33377

Chainsaw DEBUG_S out, as suggested by Vincent Pit.

p4raw-id: //depot/perl@33376

Typo found by Roland Giersig

p4raw-id: //depot/perl@33375

Tidy up error messages

p4raw-id: //depot/perl@33374

Re: making it easier not to get feedback
From: "Vincent Pit" <perl@profvince.com>
Message-ID: <1232.90.46.219.25.1204016219.squirrel@90.46.219.25>

p4raw-id: //depot/perl@33373

Document perlthanks

p4raw-id: //depot/perl@33372

Also install perlbug as perlthanks, to make it easy for people to send
their appreciation to perl-thanks@perl.org.
It would be nice to get feedback other than bug reports.

p4raw-id: //depot/perl@33371

Silence Borland compiler warnings (except for warnings from zlib) here:
http://www.nntp.perl.org/group/perl.daily-build.reports/2008/02/msg53937.html

p4raw-id: //depot/perl@33370

Ensure that constant folding runs with IN_PERL_RUNTIME true, by copying
the current compiling cop to a different address. This ensures that
lexical hints are correctly honoured, and allows us to fold sprintf.

p4raw-id: //depot/perl@33369

Two break; statements that aren't (yet) needed, but may trip someone up
in the future.

p4raw-id: //depot/perl@33368

Avoid a segfault case in MRO code, based on :

Subject: [perl #51092] [PATCH] Segfault when calling ->next::method on non-existing package
From: ilmari@vesla.ilmari.org (via RT) <perlbug-followup@perl.org>
Message-ID: <rt-3.6.HEAD-15287-1203654581-377.51092-75-0@perl.org>

p4raw-id: //depot/perl@33367

Re: [PATCH] POD fixes
Message-ID: <47BFFDCB.60107@profvince.com>

p4raw-id: //depot/perl@33366

Regenerate embed.h

p4raw-id: //depot/perl@33365

Re: [patch] optimize OP_IS_(FILETEST|SOCKET) macros
From: "Jim Cromie" <jim.cromie@gmail.com>
Message-ID: <cfe85dfa0802101152n4e1b9e07pc7fb7ad9241a9794@mail.gmail.com>

p4raw-id: //depot/perl@33364

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

p4raw-id: //depot/perl@33363

Thread::Queue 2.06
From: "Jerry D. Hedden" <jdhedden@cpan.org>
Message-ID: <1ff86f510802221410r2ceda3deg6cd503f0037b0805@mail.gmail.com>

p4raw-id: //depot/perl@33362

Thread::Semaphore 2.07
From: "Jerry D. Hedden" <jdhedden@cpan.org>
Message-ID: <1ff86f510802221405w15277004u53e7e0a2d2603049@mail.gmail.com>

p4raw-id: //depot/perl@33361

threads::shared 1.17
From: "Jerry D. Hedden" <jdhedden@cpan.org>
Message-ID: <1ff86f510802221331u48f85e8cg884c02a145dbee10@mail.gmail.com>

p4raw-id: //depot/perl@33360

threads 1.69
From: "Jerry D. Hedden" <jdhedden@cpan.org>
Message-ID: <1ff86f510802221326ib73c7e5kf26ad87086ee113d@mail.gmail.com>

p4raw-id: //depot/perl@33359

Revert change 33326
From: "Jerry D. Hedden" <jdhedden@cpan.org>
Message-ID: <1ff86f510802221321r8af47eh9a8c2772b9aa33c8@mail.gmail.com>

p4raw-id: //depot/perl@33358

cygwin hints
From: "Reini Urban" <rurban@x-ray.at>
Message-ID: <6910a60802210538p4fdb84abwf5128e87399dddf4@mail.gmail.com>

p4raw-id: //depot/perl@33357

Re: [PATCH] Splitting OP_CONST (Was: pp_const, not, that, hot?)
Message-ID: <47B60D72.50708@profvince.com>
Date: Fri, 15 Feb 2008 23:08:50 +0100

p4raw-id: //depot/perl@33356

As best as I (and my minion, gcc -Os) can tell, PL_curcop and
PL_in_eval do not need to be volatile. This improves the generated code
measurably - for example toke.o is 1.5% smaller. Every little helps.

p4raw-id: //depot/perl@33355

Update to embed.h somehow missed from change 33343.

p4raw-id: //depot/perl@33354

Take advantage of the fact that we can use indent as a stdin/stdout
filter to reduce its workload (and ours) by only sending it the 3 or
so lines that we are interested in printing, not the preceding
bucket loads.

p4raw-id: //depot/perl@33353

[patch] Porting/expand-macros.pl gets 'indent'ing
Message-ID: <47AB9C4A.3080107@gmail.com>
Date: Thu, 07 Feb 2008 17:03:22 -0700

p4raw-id: //depot/perl@33352

Regenerate the win32/config_H.* files using the makefile targets
fixed-up by #33349.

There shouldn't be any real changes here. If there are then I've
screwed up.

p4raw-id: //depot/perl@33350

Fix-up Win32's regen_config_h targets for semi-automatic updating of
the win32/config_H.* files

p4raw-id: //depot/perl@33349

Correct large files / lseek size & type settings in Win32 canned configs.

Change 25208 switched off USE_LARGE_FILES in win32/config_H.* but left
LSEEKSIZE/Off_t_size and Off_t as 8 and __int64 (or long long)
respectively. Similarly change 25215 switched off uselargefiles in
win32/config.* but left lseeksize and lseektype as 8 and __int64 (or
long long) respectively. Change 25216 fixed the Borland settings in
win32/config.bc on the basis that Borland should always be using 4 and
long, but really all the other files should be using 4 and long for
their default values as well to match the default values of
USE_LARGE_FILES and uselargefiles. Having done that, we must then
reverse the logic for fiddling with these values in win32/config_sh.PL:
they are now changed to 8 and __int64 (or long long) if uselargefiles
*is* defined (except for Borland, which always wants 4 and long).

p4raw-id: //depot/perl@33347

Missed a file from regen.pl

p4raw-id: //depot/perl@33346

Sync win32/config.* with Porting/config.sh

p4raw-id: //depot/perl@33345

Silence warning from VC8 when building without USE_LARGE_FILES

p4raw-id: //depot/perl@33344

Following 33291, on VMS we need to use the same prototypes other platforms
use in order to get the do_spawn-related assert macros defined.  Based
on suggestions by John Malmberg.

p4raw-id: //depot/perl@33343

fix variable names in 'ununit var' warnings in evals

p4raw-id: //depot/perl@33342

Fix [perl #50430].
Not 100% sure this is right, but it fixes the bug without breaking
any tests so hopefully it's at least better than it was before.

p4raw-id: //depot/perl@33341

Fix Module-Build test that has been failing on Win32
since the upgrade of Test-Harness at #32659

p4raw-id: //depot/perl@33340

Fix CBuilder test that has been failing on Win32
since the upgrade at #33263

p4raw-id: //depot/perl@33339

Eliminate ck_lengthconst.

p4raw-id: //depot/perl@33338

Setting the f flag on length causes the op to be constant folded.

p4raw-id: //depot/perl@33337

Fix Win32/VC6 compiler warnings caused by #33231

p4raw-id: //depot/perl@33336

Visual C++ 2008 Express Edition is now out: update README.win32

p4raw-id: //depot/perl@33335

Enable caching of strxfrm() results also for readonly SVs as
returned e.g. by "keys %hash". This speeds up sorting of lots
of hash keys significantly. See also:

Subject: Slowdown of "sort keys %hash" under "use locale"
Message-ID: <20080215170015.72f6160d@r2d2>

p4raw-id: //depot/perl@33334

was RE: [PATCH] re patch 33127: formatting for C<<{ bydepth => 1 }>>
From: "Robin Barker" <Robin.Barker@npl.co.uk>
Message-ID: <46A0F33545E63740BC7563DE59CA9C6D093A46@exchsvr2.npl.ad.local>

plus another POD fix found by podchecker

p4raw-id: //depot/perl@33333

consting for .c files in tests
From: "Robin Barker" <Robin.Barker@npl.co.uk>
Message-ID: <46A0F33545E63740BC7563DE59CA9C6D093A3F@exchsvr2.npl.ad.local>

p4raw-id: //depot/perl@33332

Thread::Queue 2.03
From: "Jerry D. Hedden" <jdhedden@cpan.org>
Message-ID: <1ff86f510802151102s41bebc4xab19aa6e464dbf04@mail.gmail.com>

p4raw-id: //depot/perl@33331

[perl #50946] modified hints for Darwin x86  64bit
From: "Daniel M. Quinlan" (via RT) <perlbug-followup@perl.org>
Message-ID: <rt-3.6.HEAD-24181-1203286856-952.50946-75-0@perl.org>

p4raw-id: //depot/perl@33330

Thread::Semaphore 2.04
From: "Jerry D. Hedden" <jdhedden@cpan.org>
Message-ID: <1ff86f510802150812r3facd53cs1913dd82c3070ac0@mail.gmail.com>

p4raw-id: //depot/perl@33329

Modulo operator and floating point numbers
From: "Ken Williams" <kenahoo@gmail.com>
Message-ID: <6a7ee8cc0802162122r4e59b93boee18b1f045b8954d@mail.gmail.com>

p4raw-id: //depot/perl@33328

[ patch ] silence 2 possibly uninitialized vars
Message-ID: <47B7879B.5040302@gmail.com>

p4raw-id: //depot/perl@33327

Don't install threads files on non-threaded Perls
From: "Jerry D. Hedden" <jdhedden@cpan.org>
Message-ID: <1ff86f510802150802g5bf52a52nc9543eb236063b86@mail.gmail.com>

p4raw-id: //depot/perl@33326

Fix bug 50496 -- regcomp.c=~s/lastcloseparen/lastparen/g
-- lastcloseparen is literally the index of the last paren closed
-- lastparen is index of the highest index paren that has been closed.
In nested parens, they will be completely different.
'ab'=~/(a(b))/ will have: lastparen = 2, lastcloseparen = 1
'ab'=~/(a)(b)/ will have: lastparen = lastcloseparen = 2

p4raw-id: //depot/perl@33325

Fix perlbug 50114 and document what the code does a bit better

p4raw-id: //depot/perl@33324

Fix macro name in comment

p4raw-id: //depot/perl@33323

Correctly reference count the hints hash

p4raw-id: //depot/perl@33322

Resync with metaconfig. Escape the last ~.

p4raw-id: //depot/perl@33321

Shell patterns are subject to tilde expansion. So the case statement
in the filexp script (written out by Configure) for expanding ~ and
~/*  has never worked. It's always been relying on the /bin/csh glob
for the ~* case. Few people ever noticed because one has to be
building to one's home directory on a system with no csh installed.
This is a bug from Perl 1.0's Configure!

p4raw-id: //depot/perl@33318

Typo catch, by Abigail

p4raw-id: //depot/perl@33317

Avoid utf8 warnings when printing diagnostics

p4raw-id: //depot/perl@33316

Silence compiler warnings on Win32/VC6

Subject: RE: Bit-fields patch causes warnings on Win32/VC6
From: "Jan Dubois" <jand@activestate.com>
Message-ID: <002f01c86f3f$27e23ca0$77a6b5e0$@com>

p4raw-id: //depot/perl@33315

sprintf "%+d"
Message-ID: <20080214164314.GA31694@abigail.be>

p4raw-id: //depot/perl@33314

Re: [perl #50496] Bug Report: 'keys %+' does not return the correct keys.
Message-ID: <20080207165330.GA25617@abigail.be>

with tweaks (one more test marked TODO)

p4raw-id: //depot/perl@33313

Add missing file to MANIFEST, to go with change #33311
p4raw-link: @33311 on //depot/perl: 27eaf14c55a541e1e4f2529a8084e15a8cc43d41

p4raw-id: //depot/perl@33312

Re: [perl #50706] %^H affecting outside file scopes
Message-ID: <20080212190522.GB16896@bort.ca>

p4raw-id: //depot/perl@33311

doio.c:Perl_my_lstat:/* XXX Do really need to be calling SvPV() all these times? */
From: "Vincent Pit" <perl@profvince.com>
Message-ID: <37048.147.210.17.175.1202998889.squirrel@147.210.17.175>

p4raw-id: //depot/perl@33310

Make the new warning report undef constants as undef

p4raw-id: //depot/perl@33309

Fix leak when issuing new warning implemented by #33305

p4raw-id: //depot/perl@33308

Revert change #33302. This change was wrong, since it was
using OP_ENTERSUB as a LISTOP, whereas it's a UNOP.
p4raw-link: @33302 on //depot/perl: 781e3f050073176aa21ffc243184a1cb57f4ec92

p4raw-id: //depot/perl@33307

Fix linker error introduced by #33305

p4raw-id: //depot/perl@33306

Show constant in "Useless use of a constant in void context"
Message-ID: <20080214042916.GA12678@bort.ca>

p4raw-id: //depot/perl@33305

use svtype
From: "Robin Barker" <Robin.Barker@npl.co.uk>
Message-ID: <46A0F33545E63740BC7563DE59CA9C6D093A35@exchsvr2.npl.ad.local>

p4raw-id: //depot/perl@33304

Jerry D. Hedden now maintains Thread::Semaphore on CPAN

p4raw-id: //depot/perl@33303

[perl #47047] Use of inherited AUTOLOAD for non-method is deprecated
From: Rick Delaney (via RT) <perlbug-followup@perl.org>
Message-ID: <rt-3.6.HEAD-24634-1193800780-55.47047-75-0@perl.org>

p4raw-id: //depot/perl@33302

PERL_MAGIC_uvar_elem should be 'u' in dump.c
From: "Vincent Pit" <perl@profvince.com>
Message-ID: <36116.147.210.17.175.1202897411.squirrel@147.210.17.175>

p4raw-id: //depot/perl@33301

pp_system explicitly passes NULL to do_aspawn, so NULL must be "OK"
*Now* my Win32 DEBUGGING build works :-)

p4raw-id: //depot/perl@33300

Drat. #33298 doesn't fix it: need this too.
And even then I now get an assertion from win32.c line 628. Sigh...

p4raw-id: //depot/perl@33299

Correct another variable name in embed.fnc
(Fixes my DEBUGGING builds on Win32)

p4raw-id: //depot/perl@33298

Read-only variable tests
From: "Jerry D. Hedden" <jdhedden@cpan.org>
Message-ID: <1ff86f510802081106l737e4dcdibb73c8eacb4c39d8@mail.gmail.com>

with one less TODO test

p4raw-id: //depot/perl@33297

Unused var in perlio.c (revised)
From: "Jerry D. Hedden" <jdhedden@cpan.org>
Message-ID: <1ff86f510802120700q689fb457ya5939bb440626157@mail.gmail.com>

p4raw-id: //depot/perl@33296

Make sure we only find the macro we were looking for,
not something that only starts with the same string.

p4raw-id: //depot/perl@33295

Upgrade to PathTools-3.2701

p4raw-id: //depot/perl@33294

Upgrade to Math-Complex-1.52

p4raw-id: //depot/perl@33293

Fix bit-fields for VC [was RE: [perl #50386] GIMME_V broken with 5.10.0/GCC and XS?]
From: "Jan Dubois" <jand@activestate.com>
Message-ID: <02ee01c8651b$17ef72f0$47ce58d0$@com>

p4raw-id: //depot/perl@33292

assert() that every NN argument is not NULL. Otherwise we have the
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

Correct the paramter to Perl_op_xmldump(). The one that got away from
change 33289.

p4raw-id: //depot/perl@33290

Correct (some) variable names in embed.fnc where they differ from the
actual function declaration.

p4raw-id: //depot/perl@33289

In Perl_load_module_nocontext(), ver can actually be NULL.
In Perl_hv_copy_hints_hv(), ohv can actually be NULL.
In Perl_sortsv(), Perl_sortsv_flags() and S_qsortsvu(), array can be
NULL (if the number of elements to sort is <= 1).
In Perl_save_nogv(), gv can not be NULL.
In Perl_sv_cmp() and Perl_sv_cmp_locale(), both SVs can be NULL.
In Perl_ptr_table_fetch(), the sv can be NULL.
In PerlIO_set_ptrcnt(), ptr can be NULL.

p4raw-id: //depot/perl@33287

Teach Math::Complex the maximum NV for a 16 byte float. (At least, a
16 byte float on x86_64. This feels fragile.)

p4raw-id: //depot/perl@33286

Perl_sv_2iv_flags(), Perl_sv_2uv_flags, Perl_sv_2nv and
Perl_sv_2pv_flags() all return 0/0/0.0/"" without warning given a
NULL SV pointer so they ought to be marked as NULLOK in embed.fnc.

p4raw-id: //depot/perl@33285

Perl_ck_retarget() doesn't even exist, so little point in mentioning
it.

p4raw-id: //depot/perl@33284

In Perl_sv_catpv(), Perl_sv_catpv_mg() the ptr can be not NULL.
In Perl_sv_inc() and Perl_sv_dec(), the sv can be not NULL.
In Perl_parser_dup() the proto parser can be NULL.
In Perl_ptr_table_find(), the sought-for pointer can be NULL.
In Perl_save_set_svflags(), the saved SV can't be NULL.

p4raw-id: //depot/perl@33283

Upgrade to Math-Complex-1.51

p4raw-id: //depot/perl@33282

Upgrade to Test-Harness-3.09

p4raw-id: //depot/perl@33281

Re: Unwanted warnings from "PerlIO::scalar"
Message-ID: <20080208135008.GA3885@osiris.mauzo.dyndns.org>

p4raw-id: //depot/perl@33280

In Perl_regfree_internal(), rx is actually not NULL.
In Perl_regnext(), p can actually be NULL.

p4raw-id: //depot/perl@33279