Anton Berezin says that on FreeBSD we're wrong to be using -lc_r, and
should just be using -pthread on the linker line.
This stops the new taint tests looping infinitely. It also seems to
cure the TODO test in posix.t - masked SIGINT is now received.

p4raw-id: //depot/perl@27995

threads - miscellaneous
From: "Jerry D. Hedden" <jerry@hedden.us>
Message-ID: <20060426112405.fb30e530d17747c2b054d625b8945d88.4331e666e7.wbe@email.secureserver.net>

p4raw-id: //depot/perl@27994

t/op/sort.t using test.pl
Message-ID: <4450F094.2010907@landgren.net>

p4raw-id: //depot/perl@27993

Removing another context parm
Message-ID: <20060427153156.GA11445@petdance.com>

p4raw-id: //depot/perl@27992

t/op/loopctl.t using test.pl
Message-ID: <4450FBB8.8070203@landgren.net>

p4raw-id: //depot/perl@27991

dVAR (-DPERL_GLOBAL_STRUCT_PRIVATE) update
Message-ID: <44510AF6.6040603@gmail.com>

p4raw-id: //depot/perl@27990

Typo fix in perlretut by Simon Taylor

p4raw-id: //depot/perl@27989

Fix typo in function call

p4raw-id: //depot/perl@27988

Re: [PATCH] use snprintf/strlcpy/strlcat when useful
Message-ID: <444E3EFB.8020503@gmail.com>

p4raw-id: //depot/perl@27987

[perl #39012] another REIFY bug

p4raw-id: //depot/perl@27986

Upgrade to ExtUtils-Install-1.39

p4raw-id: //depot/perl@27983

Upgrade to Encode-2.15

p4raw-id: //depot/perl@27982

Removing unused context
Message-ID: <20060427053215.GA31445@petdance.com>

p4raw-id: //depot/perl@27976

Change type of temporary padoff_du used in dUNDERBAR
from I32 to PADOFFSET, which is more correct (and fixes
a warning in Devel::PPPort).

p4raw-id: //depot/perl@27968

Re: [PATCH] cleanup 212 warnings emitted by gcc-4.2
Message-ID: <20060424232038.7550f9b6@r2d2>

p4raw-id: //depot/perl@27962

threads - consolidate XS functions
From: "Jerry D. Hedden" <jerry@hedden.us>
Message-ID: <20060425075005.fb30e530d17747c2b054d625b8945d88.e23f2fed42.wbe@email.secureserver.net>

p4raw-id: //depot/perl@27961

Make Module-Build aware of new Test-Harness output

See also:
http://www.xray.mpe.mpg.de/mailing-lists/perl5-porters/2006-04/msg00755.html

p4raw-id: //depot/perl@27960

Coverity has started to notice another inconsistency in our NULL
pointer checking.

p4raw-id: //depot/perl@27959

use NOOP macro
Message-ID: <20060424184451.GA1479@petdance.com>

p4raw-id: //depot/perl@27958

Continuing threads sync
From: "Jerry D. Hedden" <jerry@hedden.us>
Message-ID: <20060424130023.fb30e530d17747c2b054d625b8945d88.969ed54130.wbe@email.secureserver.net>

p4raw-id: //depot/perl@27957

Perl_unpack_str() is a mathom!

p4raw-id: //depot/perl@27956

Really bump the version of File::Find.

p4raw-id: //depot/perl@27955

Reverting change #27953, as it appears that File::Find has
localized the C<$_> usable by the wanted function since
version 1.04 (change #18501, to be exact).  Update the
docs to express this change.  Finally, bump the version.
p4raw-link: @27953 on //depot/perl: 668a78bfe3bf9a6d16f5b059341fb64b1e120158
p4raw-link: @18501 on //depot/perl: 17ab9c147afa44f9f620bca432b3948e42711043

p4raw-id: //depot/perl@27954

Re: [perl #38965] File::Find documentation - is "Don't modify thesevariables" still valid?
Message-ID: <444CB02A.2000604@landgren.net>

p4raw-id: //depot/perl@27953

Re: recursion now removed from the regex engine
Message-ID: <9b18b3110604231210w5deff59el3a1a79c98f34e0fd@mail.gmail.com>
Date: Sun, 23 Apr 2006 21:10:34 +0200

p4raw-id: //depot/perl@27952

[perl #38709] Opening '|-' triggers unjustified taint check
From: mhasch@cpan.org (via RT) <perlbug-followup@perl.org>
Message-ID: <rt-3.0.11-38709-130993.2.89182805885773@perl.org>

p4raw-id: //depot/perl@27951

Add regression test for bug #38475

p4raw-id: //depot/perl@27950

Change 27947 forgot to remove one now-unneeded cast.

p4raw-id: //depot/perl@27949

[perl #38475] attribute multiline fix (in tokenizer)
From: bas@quarantainenet.nl (via RT) <perlbug-followup@perl.org>
Message-ID: <rt-3.0.11-38475-129844.11.5904582303176@perl.org>

p4raw-id: //depot/perl@27948

Following change 27942 it's clear that the return type of
glob_2inpuv_number() should change. Rename it and glob_2inpuv() to
reflect what they do.

p4raw-id: //depot/perl@27947

Cleanup some gcc warnings
Third and fourth patches from:
Subject: Re: [PATCH] cleanup 212 warnings emitted by gcc-4.2
Message-ID: <20060423044704.6a383ee8@r2d2>

p4raw-id: //depot/perl@27946

A couple of casts to PADOFFSET.
2nd patch from :
Subject: Re: [PATCH] cleanup 212 warnings emitted by gcc-4.2
Message-ID: <20060423044704.6a383ee8@r2d2>

p4raw-id: //depot/perl@27945

Ensure GvNAME doesn't return NULL
First patch from :
Subject: Re: [PATCH] cleanup 212 warnings emitted by gcc-4.2
Message-ID: <20060423044704.6a383ee8@r2d2>

p4raw-id: //depot/perl@27944

doop.c consting, take 2
Message-ID: <20060424014509.GA29642@petdance.com>

p4raw-id: //depot/perl@27943

Refactor S_glob_2inpuv
Message-ID: <20060424021239.GA5449@petdance.com>

p4raw-id: //depot/perl@27942

Inlining static funcs in perl.c
Message-ID: <20060424025027.GA25998@petdance.com>

p4raw-id: //depot/perl@27941

Assimilate Test::Harness 2.57_06

p4raw-id: //depot/perl@27940

Update to AUTHORS file
Message-ID: <20060422183911.GA19435@rodd.us>

p4raw-id: //depot/perl@27939

"Don't Repeat Yourself" on the "We *really* need to overwrite these
symbols" section.
Minor tweaks to make it possible to use the code unchanged with 5.8.x

p4raw-id: //depot/perl@27938

Remove the enable_debugging member from the structure, and instead
decree that state & 1 indicates debugging. This is simpler.

p4raw-id: //depot/perl@27937

Use a struct to hold the regexp engine state information, to make
swapping possible between more two alternatives.

p4raw-id: //depot/perl@27936

Merge the install and uninstall routines.

p4raw-id: //depot/perl@27935

Refactor re.pm slightly. This has no change to documented behaviour,
but anyone mad enought to set ${^RE_DEBUG_FLAGS} and then do
no re 'Debug'; will now find that it will load the debuging version of
the regexp engine.

p4raw-id: //depot/perl@27934

Rework threads destruct call
From: "Jerry D. Hedden" <jerry@hedden.us>
Message-ID: <20060420135320.fb30e530d17747c2b054d625b8945d88.ef565d84db.wbe@email.secureserver.net>

p4raw-id: //depot/perl@27933

Re: [perl #38904] Text::ParseWords doesn't always handle backslashes correctly
Message-ID: <Pine.NEB.4.62.0604201539380.20332@otaku.freeshell.org>

p4raw-id: //depot/perl@27932

Why hide functions (and duplicate them) when you can just avoid
compiling them in the first place?

p4raw-id: //depot/perl@27931

Given that use re 'debug' can turn on DEBUGGING even on a non-DEBUGGING
build, we cannot assume that PL_reg_starttry isn't needed. So copy it
in Perl_ss_dup.

p4raw-id: //depot/perl@27930

Remove the last traces of explicitly setting HINT_LOCALIZE_HH from
perl modules.

p4raw-id: //depot/perl@27927

Re: TODO tests and test::harness
Message-ID: <9b18b3110604201336k5a974f28h732a2819853b995@mail.gmail.com>

p4raw-id: //depot/perl@27925

Add test boilerplate and use test.pl instead of Test::More

p4raw-id: //depot/perl@27923

Add new tests for bug #32840 provided by David Landgren,
as a new file rxcode.t (they mostly test $^R for now)

p4raw-id: //depot/perl@27922

Tweak change 23256 to continue passing on 5.8.x

p4raw-id: //depot/perl@27919

Convert two ok()s to better constructions.

p4raw-id: //depot/perl@27918

[REPOST] [PATCH] Remove superfluous counter in threads
From: "Jerry D. Hedden" <jerry@hedden.us>
Message-ID: <20060419053214.fb30e530d17747c2b054d625b8945d88.4606123135.wbe@email.secureserver.net>

p4raw-id: //depot/perl@27917

Fix typo in the description of change 21323.

p4raw-id: //depot/perl@27913

New test for the "Assignment to both a list and a scalar" error,
suggested by Stephen McCamant

p4raw-id: //depot/perl@27912

regmatch(): replace magic value -1000 with symbolic CHRTEST_VOID

p4raw-id: //depot/perl@27911

op_type shrinking
Message-ID: <20060420041653.GA28523@petdance.com>

p4raw-id: //depot/perl@27910

regexec consting & localizing
Message-ID: <20060419173543.GA8268@petdance.com>

p4raw-id: //depot/perl@27909

Localizing vars
Message-ID: <20060420044401.GA2792@petdance.com>

p4raw-id: //depot/perl@27908

regmatch: merge the greedy and non-greedy branches of CURLYM

p4raw-id: //depot/perl@27907

[perl #38892] PATCH: remove spurious warning from Benchmark.pm
From: Linda Walsh (via RT) <perlbug-followup@perl.org>
Message-ID: <rt-3.0.11-38892-132143.0.516271519365361@perl.org>

p4raw-id: //depot/perl@27906

Changes to perlfaq8 "How do I find out if I'm running interactively
or not?" suggested by Slaven Rezic in RT #38901: perlfaq8 correction

p4raw-id: //depot/perl@27905

Add a TODO test for RT #1085: "what should be output of perl -we
 'print do { foreach (1, 2) { 1; } }'"

p4raw-id: //depot/perl@27904

regmatch(): make IFMATCH use PUSH_STACK rather than fake recursion

p4raw-id: //depot/perl@27903

Upgrade to Test::Harness 2.57_05

p4raw-id: //depot/perl@27902

newCONSTSUB needs to be robust in case CopFILE is NULL.

p4raw-id: //depot/perl@27901

Oops. Need a macro to convert cv_ckproto() to cv_ckproto_len().

p4raw-id: //depot/perl@27900

There's a macro to get ${^RE_DEBUG_FLAGS} when debugging,
so use it (and this fixes a valgrind memory access error)

p4raw-id: //depot/perl@27899

Avoid temporarily writing over the prototype when reporting an error.
(And beef up the relevant tests to really check that it all works).

p4raw-id: //depot/perl@27898

Replace some Copy() by Move() calls, because valgrind reports
we can have overlapping memory areas here

p4raw-id: //depot/perl@27897

Calling cv_undef() on the CV created by newCONSTSUB() would leak like
a Jumblie's preferred maritime craft. To free CvFILE for this case,
take advantage of the 0 length prototype that will also be there,
and hang it from the prototype. To do this properly means changing
code to actually pay attention to SvCUR() on prototypes. It turns out
that we always know the length of the prototype string, so this may
be faster. Certainly, it's a memory saving (even ignoring the leak).

p4raw-id: //depot/perl@27896

Fix compilation of microperl, which doesn't have INT_MAX.

p4raw-id: //depot/perl@27895

Revert 27869 at Jarkko's request.

p4raw-id: //depot/perl@27894

regmatch: fix a || vs | typo in CACHEsayYES and add a test from Hugo.
Also defer altering the cache bit until CACHEsayYES/NO is called

p4raw-id: //depot/perl@27893

Add a TODO regarding merging the win32 and wince codebases.

p4raw-id: //depot/perl@27892

When checking whether perl is built with -g in optimisation
flags, use a more tolerant regexp, because we can now have
things like '-O2 -g'.

p4raw-id: //depot/perl@27891

Expand "am I hot or not" into a more general "Profile Perl" section.

p4raw-id: //depot/perl@27890

A fix to change #27845 to get it to compile with -DPERL_MAD.
p4raw-link: @27845 on //depot/perl: 4199688e3d699f53e17448d3bad6e93e57d49dcc

p4raw-id: //depot/perl@27889

win32/Makefile changes for Windows 64 bit
From: "Jan Dubois" <jand@ActiveState.com>
Message-ID: <036401c661fb$bc7a1200$2217a8c0@candy>

and make similar changes to win32/makefile.mk
and delete a spurious extra line from that file

p4raw-id: //depot/perl@27888

Addition to README.riscos suggested by Nicholas

p4raw-id: //depot/perl@27887

Note that Rafael has done the first cut of state variables.
Document that UTF-8 cleansing the tokeniser and the pads is TODO.

p4raw-id: //depot/perl@27886

Relocatable @INC and parallel builds pretty much work now.

p4raw-id: //depot/perl@27885

Support compiling for RISC OS
Message-ID: <2498b2184e.ajw498@caramel.cp15.org>

p4raw-id: //depot/perl@27884

Coverity still thinks that there is a route through do_vop that can
leak resources. I believe that it's spotted that you can skip all the
cases in the switch. Plug that hole.

p4raw-id: //depot/perl@27883

Coverity insists that if we deference a pointer without checking, and
subsequently check that same pointer before deferencing it, something
in our code or our logic is bogus. So assert() that it's safe the
first time and remove the second check.

p4raw-id: //depot/perl@27882

Fix an error, spotted by Tim Bunce.

p4raw-id: //depot/perl@27881

Coderefs in @INC are done and documented.

p4raw-id: //depot/perl@27880

Document all the return values that code references in @INC can return.

p4raw-id: //depot/perl@27879

Patch to fix RT #31227: perldoc -f, -q ignores extra arguments with no notice.
Never made it to the mailing list though, so, was never found.  Patch
previously provided to the Pod-Perldoc CPAN RT queue.

p4raw-id: //depot/perl@27878

Coverity is upset about a signed int in ext/Filter/Util/Call/Call.xs,
but the real problem is down in Perl_filter_read() in toke.c.

p4raw-id: //depot/perl@27877

Coverity is a persistent beast. Hot on the tails of fixing one leak,
it notices a slightly earlier leak.
(Which also suggests that marking bugs as RESOLVED isn't a great plan,
as it's not clear whether that calls off the dogs of war. I'd much
prefer it to be forced to scan again, and give a positve "all clear"
(or more accurately "nowt wrong I can see with that"))

p4raw-id: //depot/perl@27876

Turn on UTF8 cache assertions with -Ca

p4raw-id: //depot/perl@27875

Oops. Change 27826 had an error in the definition of BGET_PADOFFSET().

p4raw-id: //depot/perl@27874

Upgrade to CGI.pm-3.19.

p4raw-id: //depot/perl@27873

Note that PVIOs probably aren't worth shrinking. Note 3 possible
subflags for -C if the two Unicode TODOs are done.

p4raw-id: //depot/perl@27872

Re: [PATCH] update to pp_complement() via Coverity
Message-Id: <20060416184044.4ADB.BQW10602@nifty.com>
Date: Sun, 16 Apr 2006 18:45:37 +0900

Subject: Re: [PATCH] update to pp_complement() via Coverity
From: SADAHIRO Tomoyuki <bqw10602@nifty.com>
Message-Id: <20060417230614.20A5.BQW10602@nifty.com>
Date: Mon, 17 Apr 2006 23:06:21 +0900

p4raw-id: //depot/perl@27871

In Change 27764 I forgot to at PL_hints_mutex to the list of symbols to
exclude when not compiling with ithreads. It would be good to automate
the processing of this.

p4raw-id: //depot/perl@27870

(tentative) regcomp.c: try convincing Coverity that data ptr in study_chunk is non-NULL when needed
Message-Id: <20060416092648.812D36CF2D@aprikoosi.hut.fi>
Date: Sun, 16 Apr 2006 12:26:48 +0300 (EEST)

p4raw-id: //depot/perl@27869

Note in av_store() that S_regclass relies on being able to pass NULL.

p4raw-id: //depot/perl@27868