This is a live mirror of the Perl 5 development currently hosted at https://github.com/perl/perl5
perl5.git
15 years agoIn Change 27764 I forgot to at PL_hints_mutex to the list of symbols to
Nicholas Clark [Mon, 17 Apr 2006 14:48:13 +0000 (14:48 +0000)]
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

15 years ago(tentative) regcomp.c: try convincing Coverity that data ptr in study_chunk is non...
Jarkko Hietaniemi [Sun, 16 Apr 2006 12:26:48 +0000 (15:26 +0300)]
(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

15 years agoNote in av_store() that S_regclass relies on being able to pass NULL.
Nicholas Clark [Mon, 17 Apr 2006 13:15:34 +0000 (13:15 +0000)]
Note in av_store() that S_regclass relies on being able to pass NULL.

p4raw-id: //depot/perl@27868

15 years agoAdd assertions to cover cases where the Coverity scanner thinks we
Nicholas Clark [Mon, 17 Apr 2006 13:14:55 +0000 (13:14 +0000)]
Add assertions to cover cases where the Coverity scanner thinks we
could are deferencing a NULL pointer, but we think it missed something
it should have been capable of figuring out.

p4raw-id: //depot/perl@27867

15 years agoAdd Dave's explainations of why certain pointers are always non-NULL,
Nicholas Clark [Mon, 17 Apr 2006 12:48:43 +0000 (12:48 +0000)]
Add Dave's explainations of why certain pointers are always non-NULL,
and assert() that it is true.

p4raw-id: //depot/perl@27866

15 years agofix cygwin build
Yitzchak Scott-Thoennes [Sun, 16 Apr 2006 18:05:20 +0000 (11:05 -0700)]
fix cygwin build
Message-ID: <20060417010520.GA2888@efn.org>

p4raw-id: //depot/perl@27865

15 years agoSupport for -DDEBUGGING and its alias -DEBUGGING
H.Merijn Brand [Mon, 17 Apr 2006 12:18:07 +0000 (12:18 +0000)]
Support for -DDEBUGGING and its alias -DEBUGGING

p4raw-id: //depot/perl@27863

15 years agoRevert change 27818, as change 27853 fixes the real bug.
Nicholas Clark [Mon, 17 Apr 2006 12:07:23 +0000 (12:07 +0000)]
Revert change 27818, as change 27853 fixes the real bug.

p4raw-id: //depot/perl@27862

15 years agoCompress::Zlib
Paul Marquess [Mon, 17 Apr 2006 10:58:30 +0000 (11:58 +0100)]
Compress::Zlib
From: "Paul Marquess" <Paul.Marquess@ntlworld.com>
Message-ID: <019b01c66205$7dc7ea50$6601a8c0@myopwv.com>

p4raw-id: //depot/perl@27861

15 years agopp_ctl.c caching
Andy Lester [Sun, 16 Apr 2006 23:40:35 +0000 (18:40 -0500)]
pp_ctl.c caching
Message-ID: <20060417044035.GA28818@petdance.com>
Date: Sun, 16 Apr 2006 23:40:35 -0500

p4raw-id: //depot/perl@27860

15 years agodooop.c: the strong asserts in Sv* macros could cause memory leakage -- move the...
Jarkko Hietaniemi [Mon, 17 Apr 2006 10:19:37 +0000 (13:19 +0300)]
dooop.c: the strong asserts in Sv* macros could cause memory leakage -- move the macro calls earlier (Coverity CID 84)
Message-Id: <20060417071937.C13346CF2D@aprikoosi.hut.fi>
Date: Mon, 17 Apr 2006 10:19:37 +0300 (EEST)

p4raw-id: //depot/perl@27859

15 years agoRevert 27856.
Nicholas Clark [Mon, 17 Apr 2006 10:28:40 +0000 (10:28 +0000)]
Revert 27856.

p4raw-id: //depot/perl@27857

15 years agoThe danger of piping an mbox to patch is that it contains more than
Jarkko Hietaniemi [Sun, 16 Apr 2006 11:19:25 +0000 (14:19 +0300)]
The danger of piping an mbox to patch is that it contains more than
one message. So both:

Subject: [PATCH] doop.c: one more code path where memory could leak (Coverity)
Message-Id: <20060416081925.680336CF2D@aprikoosi.hut.fi>
Date: Sun, 16 Apr 2006 11:19:25 +0300 (EEST)

and

Subject: [PATCH] doop.c: one more code path where memory could leak (Coverity)
From: jhi@cc.hut.fi (Jarkko Hietaniemi)
Message-Id: <20060416081925.680336CF2D@aprikoosi.hut.fi>
Date: Sun, 16 Apr 2006 11:19:25 +0300 (EEST)

p4raw-id: //depot/perl@27856

15 years agogeneralize the 'continue on success' mechanism of deferred REs
Dave Mitchell [Sun, 16 Apr 2006 23:30:36 +0000 (23:30 +0000)]
generalize the 'continue on success' mechanism of deferred REs

p4raw-id: //depot/perl@27855

15 years agoabstract regmatch slab access with SLAB_FIRST and SLAB_LAST macros
Dave Mitchell [Sun, 16 Apr 2006 23:12:31 +0000 (23:12 +0000)]
abstract regmatch slab access with SLAB_FIRST and SLAB_LAST macros

p4raw-id: //depot/perl@27854

15 years agostop PL_op and PL_curpad referring to different pads at start of doeval
Dave Mitchell [Sun, 16 Apr 2006 23:07:13 +0000 (23:07 +0000)]
stop PL_op and PL_curpad referring to different pads at start of doeval
(this could make find_uninit_var do Bad Things)

p4raw-id: //depot/perl@27853

15 years agoFix bug introduced with change 27841. (len + 1, not just len).
Nicholas Clark [Sun, 16 Apr 2006 21:44:59 +0000 (21:44 +0000)]
Fix bug introduced with change 27841. (len + 1, not just len).

p4raw-id: //depot/perl@27852

15 years agoCoverity is flagging a potential problem because it sees a check for
Nicholas Clark [Sun, 16 Apr 2006 19:06:19 +0000 (19:06 +0000)]
Coverity is flagging a potential problem because it sees a check for
NULL and assumes that this means that the variable host could be
NULL. It can't, and the check added in change 13291 was a little bit
more than the minimal solution needed for the bug report
[ID 20011126.148]
Hence remove the NULL check.

p4raw-id: //depot/perl@27851

15 years agoop.c patch, 2nd version
Andy Lester [Fri, 14 Apr 2006 23:29:24 +0000 (18:29 -0500)]
op.c patch, 2nd version
Message-ID: <20060415042924.GA1786@petdance.com>
Date: Fri, 14 Apr 2006 23:29:24 -0500

p4raw-id: //depot/perl@27850

15 years agoA scalar reference returned from a coderef in @INC is treated as the
Nicholas Clark [Sun, 16 Apr 2006 15:04:36 +0000 (15:04 +0000)]
A scalar reference returned from a coderef in @INC is treated as the
initial "content" of the file. When it is exhausted input is taken
from a real file handle, or a generator sub, if either exists.

p4raw-id: //depot/perl@27849

15 years agoremove some cruft from makedef.pl
Jan Dubois [Sun, 16 Apr 2006 02:05:00 +0000 (19:05 -0700)]
remove some cruft from makedef.pl
From: "Jan Dubois" <jand@ActiveState.com>
Message-ID: <022e01c66134$d7a8dec0$2217a8c0@candy>

p4raw-id: //depot/perl@27848

15 years agotestcase for change #27832
Jan Dubois [Sun, 16 Apr 2006 00:57:00 +0000 (17:57 -0700)]
testcase for change #27832
From: "Jan Dubois" <jand@ActiveState.com>
Message-ID: <022a01c6612b$575cb7e0$2217a8c0@candy>

p4raw-id: //depot/perl@27847

15 years agoStray pod/perlapi.pod change found after a regen.
Steve Peters [Sun, 16 Apr 2006 13:44:47 +0000 (13:44 +0000)]
Stray pod/perlapi.pod change found after a regen.

p4raw-id: //depot/perl@27846

15 years agodump.c patch redux
Andy Lester [Sun, 16 Apr 2006 00:29:36 +0000 (19:29 -0500)]
dump.c patch redux
Message-ID: <20060416052936.GA19143@petdance.com>

p4raw-id: //depot/perl@27845

15 years agoPerl_do_vop can profitably use Perl_sv_usepvn_flags, as it has
Nicholas Clark [Sun, 16 Apr 2006 13:25:18 +0000 (13:25 +0000)]
Perl_do_vop can profitably use Perl_sv_usepvn_flags, as it has
allocated space for an initialised a trailing NUL.

p4raw-id: //depot/perl@27843

15 years agoIn Perl_sv_usepvn_flags, assert() that the caller has RTFM.
Nicholas Clark [Sun, 16 Apr 2006 12:06:57 +0000 (12:06 +0000)]
In Perl_sv_usepvn_flags, assert() that the caller has RTFM.

p4raw-id: //depot/perl@27842

15 years agoProvide a new flag for sv_usepvn_flags, SV_HAS_TRAILING_NUL, which
Nicholas Clark [Sun, 16 Apr 2006 11:35:38 +0000 (11:35 +0000)]
Provide a new flag for sv_usepvn_flags, SV_HAS_TRAILING_NUL, which
signals that the buffer passed in is already well-formed for use as
SvPVX (i.e. it is followed by a trailing NUL), and hence the realloc()
to add a NUL can be skipped.

p4raw-id: //depot/perl@27841

15 years agoMerge sv_usepvn and sv_usepvn_mg into sv_usepvn_flags. "Promote" the
Nicholas Clark [Sun, 16 Apr 2006 11:03:28 +0000 (11:03 +0000)]
Merge sv_usepvn and sv_usepvn_mg into sv_usepvn_flags. "Promote" the
other two to mathoms.c

p4raw-id: //depot/perl@27840

15 years agoIn Perl_sv_usepvn(), force a move under DEBUGGING to shake out caller
Nicholas Clark [Sun, 16 Apr 2006 10:37:46 +0000 (10:37 +0000)]
In Perl_sv_usepvn(), force a move under DEBUGGING to shake out caller
bugs.

p4raw-id: //depot/perl@27839

15 years agoFix htmllink.t to cater for the case where perladmin isn't an empty
Nicholas Clark [Sun, 16 Apr 2006 10:37:04 +0000 (10:37 +0000)]
Fix htmllink.t to cater for the case where perladmin isn't an empty
string.

p4raw-id: //depot/perl@27838

15 years agoCoverity notes that we might be dereferencing fgv before a check that
Nicholas Clark [Sun, 16 Apr 2006 09:36:18 +0000 (09:36 +0000)]
Coverity notes that we might be dereferencing fgv before a check that
it's not NULL. In fact, the code ordering meant that one "if (fgv)"
would always be true. So fix this.

p4raw-id: //depot/perl@27837

15 years agoupdate to pp_complement() via Coverity
Andy Lester [Sun, 16 Apr 2006 00:14:23 +0000 (19:14 -0500)]
update to pp_complement() via Coverity
Message-ID: <20060416051423.GA17063@petdance.com>

p4raw-id: //depot/perl@27836

15 years agoclean up hv_assert()
Andy Lester [Sat, 15 Apr 2006 23:18:27 +0000 (18:18 -0500)]
clean up hv_assert()
Message-ID: <20060416041827.GA3031@petdance.com>

p4raw-id: //depot/perl@27835

15 years agoRemoved unused var in a macro
Andy Lester [Sat, 15 Apr 2006 23:32:53 +0000 (18:32 -0500)]
Removed unused var in a macro
Message-ID: <20060416043253.GA3438@petdance.com>

p4raw-id: //depot/perl@27834

15 years agoChange anchor generation in Pod::Html for "=item item 2"
Jan Dubois [Fri, 14 Apr 2006 13:13:20 +0000 (06:13 -0700)]
Change anchor generation in Pod::Html for "=item item 2"
From: "Jan Dubois" <jand@ActiveState.com>
Message-ID: <017a01c65fff$dfd0f370$2217a8c0@candy>

p4raw-id: //depot/perl@27833

15 years agoPod::Html::depod() doesn't work correctly for multi-line strings
Jan Dubois [Mon, 10 Apr 2006 15:32:23 +0000 (08:32 -0700)]
Pod::Html::depod() doesn't work correctly for multi-line strings
From: "Jan Dubois" <jand@ActiveState.com>
Message-ID: <017b01c65cee$a2f5fe00$6062a8c0@candy>

p4raw-id: //depot/perl@27832

15 years agoMissing part of change 25186
Jan Dubois [Thu, 13 Apr 2006 18:26:29 +0000 (11:26 -0700)]
Missing part of change 25186
From: "Jan Dubois" <jand@ActiveState.com>
Message-ID: <009401c65f62$74fded70$2217a8c0@candy>

p4raw-id: //depot/perl@27831

15 years agoMissing part of change 25676
Jan Dubois [Thu, 13 Apr 2006 18:19:26 +0000 (11:19 -0700)]
Missing part of change 25676
From: "Jan Dubois" <jand@ActiveState.com>
Message-ID: <009001c65f61$78866d60$2217a8c0@candy>

p4raw-id: //depot/perl@27830

15 years agoHandle null io in leavewrite (not possible?) (Coverity)
bah@longitude.com [Sat, 15 Apr 2006 14:13:41 +0000 (10:13 -0400)]
Handle null io in leavewrite (not possible?) (Coverity)
Message-ID: <1661.38.96.138.153.1145124821.squirrel@mail.longitude.com>
Date: Sat, 15 Apr 2006 14:13:41 -0400 (EDT)

p4raw-id: //depot/perl@27829

15 years agoPotential (?) memory leak in Storable.xs (Coverity)
bah@longitude.com [Sat, 15 Apr 2006 13:54:10 +0000 (09:54 -0400)]
Potential (?) memory leak in Storable.xs (Coverity)
Message-ID: <1561.38.96.138.153.1145123650.squirrel@mail.longitude.com>
Date: Sat, 15 Apr 2006 13:54:10 -0400 (EDT)

p4raw-id: //depot/perl@27828

15 years agoTeach Perl_do_sv_dump how to cope with IOs that have bogus things in
Nicholas Clark [Sat, 15 Apr 2006 22:44:30 +0000 (22:44 +0000)]
Teach Perl_do_sv_dump how to cope with IOs that have bogus things in
GV slots. (Which source filters are prone to do to them).

p4raw-id: //depot/perl@27827

15 years agoCoverity was getting upset about an assignment from a function
Nicholas Clark [Sat, 15 Apr 2006 21:50:12 +0000 (21:50 +0000)]
Coverity was getting upset about an assignment from a function
returning int to a U8 variable. Curiously it was only getting upset
about the first of many.
However, we are doing NO ERROR CHECKING on whether we read in the
requested number of bytes of bytecode. All except BGET_op_tr_array
will now croak on a short read.

p4raw-id: //depot/perl@27826

15 years agoCoverity reports that Storable can potentially cause a NULL pointer
Nicholas Clark [Sat, 15 Apr 2006 21:43:13 +0000 (21:43 +0000)]
Coverity reports that Storable can potentially cause a NULL pointer
dereference while iterating a hash. This should never happen unless
the hash lies about how many keys it has, so croak if a fib is spotted.

p4raw-id: //depot/perl@27825

15 years agoCoverity notes that resource are leaked in the unfinished and never
Nicholas Clark [Sat, 15 Apr 2006 21:11:44 +0000 (21:11 +0000)]
Coverity notes that resource are leaked in the unfinished and never
called function _utf8_to_bytes.

p4raw-id: //depot/perl@27824

15 years agoCoverity correctly reports that gv might (just) be NULL. So don't
Nicholas Clark [Sat, 15 Apr 2006 20:55:20 +0000 (20:55 +0000)]
Coverity correctly reports that gv might (just) be NULL. So don't
derefernece it if it is.

p4raw-id: //depot/perl@27823

15 years agoUpdate the assertion to a croak "Out of range special SV number"
Nicholas Clark [Sat, 15 Apr 2006 20:54:44 +0000 (20:54 +0000)]
Update the assertion to a croak "Out of range special SV number"
to give ByteLoader more meaningful failure diagnostics.

p4raw-id: //depot/perl@27822

15 years agoI believe that it makes little sense for the simple user filters
Nicholas Clark [Sat, 15 Apr 2006 20:19:22 +0000 (20:19 +0000)]
I believe that it makes little sense for the simple user filters
provided by coderefs to be run in block mode. Hence always read from
the upstream filter in line mode, and always call the user's filter
with 1 line.

p4raw-id: //depot/perl@27821

15 years agoFix a bug - the introduction of the caching code means that we should
Nicholas Clark [Sat, 15 Apr 2006 20:01:58 +0000 (20:01 +0000)]
Fix a bug - the introduction of the caching code means that we should
initialise the SV upstream after it.
assert that filter_sub is always non-null now.

p4raw-id: //depot/perl@27820

15 years agoAugment the user filter caching code so that if the user filter returns
Nicholas Clark [Sat, 15 Apr 2006 19:37:32 +0000 (19:37 +0000)]
Augment the user filter caching code so that if the user filter returns
multiple lines, only one line at a time is returned.
Rename the variable len to status, as it is the status value.

p4raw-id: //depot/perl@27819

15 years agoDeep in the bowels of creating new and exciting bugs, I managed to
Nicholas Clark [Sat, 15 Apr 2006 19:17:51 +0000 (19:17 +0000)]
Deep in the bowels of creating new and exciting bugs, I managed to
segfault the uninitialised value reporting code. This protects against
the segfault.

p4raw-id: //depot/perl@27818

15 years ago$foo .= $bar doesn't warn if $foo is undefined, so simplify code.
Nicholas Clark [Sat, 15 Apr 2006 18:08:04 +0000 (18:08 +0000)]
$foo .= $bar doesn't warn if $foo is undefined, so simplify code.

p4raw-id: //depot/perl@27817

15 years agoIf the downstream caller wants block mode, and we're in line mode,
Nicholas Clark [Sat, 15 Apr 2006 18:05:12 +0000 (18:05 +0000)]
If the downstream caller wants block mode, and we're in line mode,
then don't return more bytes than they asked for. Hold bytes over
until next time if necessary.

p4raw-id: //depot/perl@27816

15 years agoEliminate the filter_child_proc hack from pp_require.
Nicholas Clark [Sat, 15 Apr 2006 17:00:54 +0000 (17:00 +0000)]
Eliminate the filter_child_proc hack from pp_require.

p4raw-id: //depot/perl@27815

15 years agoIf you return void from a filter subroutine, the caller's status is
Nicholas Clark [Sat, 15 Apr 2006 16:43:22 +0000 (16:43 +0000)]
If you return void from a filter subroutine, the caller's status is
used. If not, your last statement is the return value, and if it keeps
being +ve you'll never hit EOF.
The probably should be a sanity check for this in toke.c, but I'm not
sure how.

p4raw-id: //depot/perl@27814

15 years agoTests for having multiple pipes open simultaneously.
Nicholas Clark [Sat, 15 Apr 2006 16:42:23 +0000 (16:42 +0000)]
Tests for having multiple pipes open simultaneously.
(See comment added in change 3771 about this apparently being buggy)

p4raw-id: //depot/perl@27813

15 years agoAdd more tests for the builtin source filter implementation, and fix
Nicholas Clark [Sat, 15 Apr 2006 13:45:13 +0000 (13:45 +0000)]
Add more tests for the builtin source filter implementation, and fix
two bugs - it would loop infinitely if data were already in the read
buffer, and it would process those data twice.

p4raw-id: //depot/perl@27812

15 years agoAdd a test for source filters returned from code references in @INC.
Nicholas Clark [Sat, 15 Apr 2006 11:59:26 +0000 (11:59 +0000)]
Add a test for source filters returned from code references in @INC.

p4raw-id: //depot/perl@27811

15 years agoBump the expect file count in readdir.t. We keep writing more tests.
Nicholas Clark [Sat, 15 Apr 2006 11:22:29 +0000 (11:22 +0000)]
Bump the expect file count in readdir.t. We keep writing more tests.

p4raw-id: //depot/perl@27810

15 years agoperlio.c: layer data might be allocated and unused (Coverity)
Jarkko Hietaniemi [Sat, 15 Apr 2006 11:24:17 +0000 (14:24 +0300)]
perlio.c: layer data might be allocated and unused (Coverity)
Message-Id: <20060415082417.24F0A6D08C@ugli.hut.fi>

(with a correction)

p4raw-id: //depot/perl@27809

15 years agoFix bug in prime_env_iter where the terms of a comparison were reversed
juna [Fri, 24 Mar 2006 13:52:41 +0000 (05:52 -0800)]
Fix bug in prime_env_iter where the terms of a comparison were reversed
so we thought we didn't have a logical name table when we did.  Reported as:
Subject: LWP::Simple crashes on VMSperl
From: "juna" <ggl.20.jj...@spamgourmet.com>
Newsgroups: comp.os.vms,comp.lang.perl.misc
Message-ID: <1143237161.931669.48370@t31g2000cwb.googlegroups.com>

p4raw-id: //depot/perl@27808

15 years agoHonor READALL privilege in cando_by_name (from Scott
Craig A. Berry [Sat, 15 Apr 2006 00:45:26 +0000 (00:45 +0000)]
Honor READALL privilege in cando_by_name (from Scott
Lepage of HP OpenVMS Engineering)

p4raw-id: //depot/perl@27807

15 years agoDon't die after SS$_NOPRIV in Perl_vmssetenv, courtesy
Craig A. Berry [Sat, 15 Apr 2006 00:42:41 +0000 (00:42 +0000)]
Don't die after SS$_NOPRIV in Perl_vmssetenv, courtesy
of Scott Lepage at HP.

p4raw-id: //depot/perl@27806

15 years agoChange SAVECOPWARNINGS(c) to SAVECOMPILEWARNINGS() - it's only used to
Nicholas Clark [Fri, 14 Apr 2006 20:44:27 +0000 (20:44 +0000)]
Change SAVECOPWARNINGS(c) to SAVECOMPILEWARNINGS() - it's only used to
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

15 years agoIn the description of require, clarify the file handle return and why
Nicholas Clark [Fri, 14 Apr 2006 19:44:34 +0000 (19:44 +0000)]
In the description of require, clarify the file handle return and why
C<INC> must be qualified.

p4raw-id: //depot/perl@27804

15 years ago S_glob_2inpuv() did not check if lenp was NULL. Oops.
Nicholas Clark [Fri, 14 Apr 2006 16:43:03 +0000 (16:43 +0000)]
 S_glob_2inpuv() did not check if lenp was NULL. Oops.

p4raw-id: //depot/perl@27803

15 years agoReplace explict "/dev/null" with BIT_BUCKET, which varies by platform.
Nicholas Clark [Fri, 14 Apr 2006 16:18:30 +0000 (16:18 +0000)]
Replace explict "/dev/null" with BIT_BUCKET, which varies by platform.

p4raw-id: //depot/perl@27802

15 years agodoop.c: (Coverity) found a bug but not quite what Coverity thought it did (try valgri...
Jarkko Hietaniemi [Thu, 13 Apr 2006 19:20:46 +0000 (22:20 +0300)]
doop.c: (Coverity) found a bug but not quite what Coverity thought it did (try valgrind on the new bop.t without the doop.c patch)
Message-Id: <20060413162046.5F9636D08C@ugli.hut.fi>

p4raw-id: //depot/perl@27801

15 years agoMark new_warnings_bitfield as XE, and add it to __DATA__ in makedef.pl
Nicholas Clark [Fri, 14 Apr 2006 14:45:04 +0000 (14:45 +0000)]
Mark new_warnings_bitfield as XE, and add it to __DATA__ in makedef.pl
Remove a reference to the deleted function Perl_refcounted_he_dup.

p4raw-id: //depot/perl@27800

15 years agoIO::Compress::*
Paul Marquess [Fri, 14 Apr 2006 09:05:39 +0000 (10:05 +0100)]
IO::Compress::*
Message-ID: <004f01c65f9a$3871eb30$2405140a@myopwv.com>

p4raw-id: //depot/perl@27799

15 years agoperlio.c: (Coverity) eliminate temp ptr that confuses Coverity into thinking there...
Jarkko Hietaniemi [Fri, 14 Apr 2006 11:56:44 +0000 (14:56 +0300)]
perlio.c: (Coverity) eliminate temp ptr that confuses Coverity into thinking there is a resource leak
Message-Id: <20060414085644.94A076D08C@ugli.hut.fi>
Date: Fri, 14 Apr 2006 11:56:44 +0300 (EEST)

p4raw-id: //depot/perl@27798

15 years agoReverting change #27791 which may cause corruption addition corruption
Steve Peters [Fri, 14 Apr 2006 11:26:59 +0000 (11:26 +0000)]
Reverting change #27791 which may cause corruption addition corruption
within a regular expression.
p4raw-link: @27791 on //depot/perl: 4ae2f9814731932908f61ce4a79c6da6063584e7

p4raw-id: //depot/perl@27797

15 years agoFix implementation of threads->_handle on unixes.
Jerry D. Hedden [Tue, 4 Apr 2006 09:25:20 +0000 (02:25 -0700)]
Fix implementation of threads->_handle on unixes.
See :
Subject: RE: [PATCH] 2nd patch to sync blead 'threads' with CPAN [REPOST]
From: "Jerry D. Hedden" <jerry@hedden.us>
Message-ID: <20060404092520.fb30e530d17747c2b054d625b8945d88.4a8a6b16c9.wbe@email.email.secureserver.net>

p4raw-id: //depot/perl@27796

15 years ago(Hopefully) fix coredumps in ByteLoader, which were a side effect of
Nicholas Clark [Thu, 13 Apr 2006 21:59:52 +0000 (21:59 +0000)]
(Hopefully) fix coredumps in ByteLoader, which were a side effect of
fixing a bug in B, and non-robust autogenerated code. When
autogenerating initialiser statements for an array, it's a bad plan to
have the size of that array hard coded.

p4raw-id: //depot/perl@27795

15 years agoeliminate PL_regbol PL_regtill PL_regnarrate PL_reg_ganch PL_reg_sv
Dave Mitchell [Thu, 13 Apr 2006 18:31:54 +0000 (18:31 +0000)]
eliminate PL_regbol PL_regtill PL_regnarrate PL_reg_ganch PL_reg_sv
(only another 434 to go ...)

p4raw-id: //depot/perl@27794

15 years agodisable 'split /(?{ split "" })/' test until recursive split is fixed
Dave Mitchell [Thu, 13 Apr 2006 17:16:41 +0000 (17:16 +0000)]
disable 'split /(?{ split "" })/' test until recursive split is fixed

p4raw-id: //depot/perl@27793

15 years agoRemove SAVEt_FREESHAREDPV, as nothing is using it, and it isn't catered
Nicholas Clark [Thu, 13 Apr 2006 16:14:23 +0000 (16:14 +0000)]
Remove SAVEt_FREESHAREDPV, as nothing is using it, and it isn't catered
for in Perl_ss_dup, hence nothing is testing it and nothing will alert
us if it breaks.

p4raw-id: //depot/perl@27792

15 years agoAdd another NULL check thanks to Coverity.
Steve Peters [Thu, 13 Apr 2006 15:08:04 +0000 (15:08 +0000)]
Add another NULL check thanks to Coverity.

p4raw-id: //depot/perl@27791

15 years agoFor duping SAVEHINTS it rather helps if you pay attention to what is
Nicholas Clark [Thu, 13 Apr 2006 14:14:57 +0000 (14:14 +0000)]
For duping SAVEHINTS it rather helps if you pay attention to what is
actually being pushed onto the save stack.

p4raw-id: //depot/perl@27790

15 years agoregexec.c: (Coverity) add NULL check
Jarkko Hietaniemi [Thu, 13 Apr 2006 15:52:15 +0000 (18:52 +0300)]
regexec.c: (Coverity) add NULL check
Message-Id: <200604131252.k3DCqFUL497781@kosh.hut.fi>
Date: Thu, 13 Apr 2006 15:52:15 +0300 (EEST)

p4raw-id: //depot/perl@27789

15 years agoregexec.c: (Coverity) move NULL check even earlier
Jarkko Hietaniemi [Thu, 13 Apr 2006 15:39:33 +0000 (18:39 +0300)]
regexec.c: (Coverity) move NULL check even earlier
Message-Id: <200604131239.k3DCdXhe124107@kosh.hut.fi>
Date: Thu, 13 Apr 2006 15:39:33 +0300 (EEST)

p4raw-id: //depot/perl@27788

15 years agoAdd a break; to the end of the SAVEt_COP_WARNINGS case, and don't try
Nicholas Clark [Thu, 13 Apr 2006 13:02:30 +0000 (13:02 +0000)]
Add a break; to the end of the SAVEt_COP_WARNINGS case, and don't try
to dereference cop_hints if it's NULL.

p4raw-id: //depot/perl@27787

15 years agoFix B and ByteLoader to cope with cop_warnings no longer being an SV.
Nicholas Clark [Thu, 13 Apr 2006 12:40:24 +0000 (12:40 +0000)]
Fix B and ByteLoader to cope with cop_warnings no longer being an SV.

p4raw-id: //depot/perl@27786

15 years agoAdd SAVEt_COP_WARNINGS to ss_dup.
Nicholas Clark [Thu, 13 Apr 2006 10:46:32 +0000 (10:46 +0000)]
Add SAVEt_COP_WARNINGS to ss_dup.

p4raw-id: //depot/perl@27785

15 years agoRe: [PATCH] add pTHX_ parameter to new_warnings_bitfield()
Steve Hay [Thu, 13 Apr 2006 10:27:52 +0000 (11:27 +0100)]
Re: [PATCH] add pTHX_ parameter to new_warnings_bitfield()
Message-ID: <443E1998.9090909@uk.radan.com>
Date: Thu, 13 Apr 2006 10:27:52 +0100

p4raw-id: //depot/perl@27784

15 years agoeliminate PL_reg_re
Dave Mitchell [Thu, 13 Apr 2006 02:20:33 +0000 (02:20 +0000)]
eliminate PL_reg_re
(only another 439 to go ...)

p4raw-id: //depot/perl@27783

15 years ago[perl #38906] Regex engine does not release MINMOD
Animator [Wed, 12 Apr 2006 15:27:41 +0000 (08:27 -0700)]
[perl #38906] Regex engine does not release MINMOD
From: "Animator via RT" <perlbug-followup@perl.org>
Message-ID: <rt-3.0.11-38906-132222.3.24032491323436@perl.org>

zero-length minimum matches that failed could leak into a following
maximal match

p4raw-id: //depot/perl@27782

15 years agoD'oh! Don't try to do #ifdefs inside "functions" that are actually
Nicholas Clark [Wed, 12 Apr 2006 23:17:18 +0000 (23:17 +0000)]
D'oh! Don't try to do #ifdefs inside "functions" that are actually
macros.

p4raw-id: //depot/perl@27781

15 years agoAdd tests for all the other types that %^H serialisation is supposed
Nicholas Clark [Wed, 12 Apr 2006 22:49:58 +0000 (22:49 +0000)]
Add tests for all the other types that %^H serialisation is supposed
to support. In the process, found and corrected a bug.
Tests are good, m'kay.

p4raw-id: //depot/perl@27780

15 years agoChange cop_warnings from an SV holding the warnings bitmask to a
Nicholas Clark [Wed, 12 Apr 2006 22:45:12 +0000 (22:45 +0000)]
Change cop_warnings from an SV holding the warnings bitmask to a
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

15 years agoRemove the final recursion path from S_regmatch: (??{...})
Dave Mitchell [Wed, 12 Apr 2006 22:43:03 +0000 (22:43 +0000)]
Remove the final recursion path from S_regmatch:  (??{...})
Also put PUSH/POP_STATE infastructure in place
Also eliminate PL_reg_call_cc
       (only another 440 global vars to go ...)

p4raw-id: //depot/perl@27778

15 years agoTest that we can do embedded 0 bytes in hints.
Nicholas Clark [Wed, 12 Apr 2006 16:18:30 +0000 (16:18 +0000)]
Test that we can do embedded 0 bytes in hints.

p4raw-id: //depot/perl@27777

15 years agoStore cop_hints as a shared structure without using any SVs (on
Nicholas Clark [Wed, 12 Apr 2006 16:01:08 +0000 (16:01 +0000)]
Store cop_hints as a shared structure without using any SVs (on
ithreads).

p4raw-id: //depot/perl@27776

15 years agoRemove Perl_refcounted_he_copy and Perl_refcounted_he_dup.
Nicholas Clark [Wed, 12 Apr 2006 14:14:50 +0000 (14:14 +0000)]
Remove Perl_refcounted_he_copy and Perl_refcounted_he_dup.

p4raw-id: //depot/perl@27775

15 years agoNewxz() can use calloc() rather than malloc(), as the implementation
Nicholas Clark [Wed, 12 Apr 2006 12:31:29 +0000 (12:31 +0000)]
Newxz() can use calloc() rather than malloc(), as the implementation
of calloc() may know that it has fresh pages from the OS, and so
doesn't need to zero them itself. Plus our object code should be
slightly smaller.

p4raw-id: //depot/perl@27774

15 years agoAs OPs are shared between threads, they should be using
Nicholas Clark [Wed, 12 Apr 2006 12:01:38 +0000 (12:01 +0000)]
As OPs are shared between threads, they should be using
PerlMemShared_*alloc functions, rather than per-thread *alloc()s.

p4raw-id: //depot/perl@27773

15 years agoregexec.c: move NULL check earlier to make Coverity happier
Jarkko Hietaniemi [Tue, 11 Apr 2006 23:04:48 +0000 (02:04 +0300)]
regexec.c: move NULL check earlier to make Coverity happier
Message-Id: <200604112004.k3BK4mV3097400@kosh.hut.fi>
Date: Tue, 11 Apr 2006 23:04:48 +0300 (EEST)

p4raw-id: //depot/perl@27772

15 years agoregcomp.c (try II): use &PL_sv_undef for listsv as suggested by Nick
Jarkko Hietaniemi [Tue, 11 Apr 2006 23:53:25 +0000 (02:53 +0300)]
regcomp.c (try II): use &PL_sv_undef for listsv as suggested by Nick
Message-Id: <200604112053.k3BKrPgk310001@kosh.hut.fi>
Date: Tue, 11 Apr 2006 23:53:25 +0300 (EEST)

p4raw-id: //depot/perl@27771

15 years agoperlio.c: false Coverity finding (it cannot see that two pointers are the same):...
Jarkko Hietaniemi [Tue, 11 Apr 2006 23:39:32 +0000 (02:39 +0300)]
perlio.c: false Coverity finding (it cannot see that two pointers are the same): but help Coverity by removing the temp ptr
Message-Id: <200604112039.k3BKdWix334525@kosh.hut.fi>
Date: Tue, 11 Apr 2006 23:39:32 +0300 (EEST)

p4raw-id: //depot/perl@27770

15 years agolocale.c: more Safefree() (Coverity finding)
Jarkko Hietaniemi [Tue, 11 Apr 2006 22:08:40 +0000 (01:08 +0300)]
locale.c: more Safefree() (Coverity finding)
Message-Id: <200604111908.k3BJ8ewn030950@kosh.hut.fi>
Date: Tue, 11 Apr 2006 22:08:40 +0300 (EEST)

p4raw-id: //depot/perl@27769

15 years agoFirst stab at using the pad to store the serialised hints data.
Nicholas Clark [Tue, 11 Apr 2006 19:26:48 +0000 (19:26 +0000)]
First stab at using the pad to store the serialised hints data.
This approach is almost certainly not only wrong, but also doomed.
But it's needed to build a correct solution.

p4raw-id: //depot/perl@27768