This is a live mirror of the Perl 5 development currently hosted at
11 years agoA plethora of isarev tests
Father Chrysostomos [Sat, 30 Oct 2010 13:51:33 +0000 (06:51 -0700)]
A plethora of isarev tests

11 years agoCorrect reference to get_linear_isa in get_isarev’ description
Father Chrysostomos [Sat, 30 Oct 2010 12:48:41 +0000 (05:48 -0700)]
Correct reference to get_linear_isa in get_isarev’ description

11 years agoTo-do tests for isarev (more to come)
Father Chrysostomos [Sat, 30 Oct 2010 12:46:47 +0000 (05:46 -0700)]
To-do tests for isarev (more to come)

11 years agoIncrease mro’s version
Father Chrysostomos [Sat, 30 Oct 2010 12:15:36 +0000 (05:15 -0700)]
Increase mro’s version

11 years agoMerge the direct structure lookup for B::{PV,PVMG,PVLV} to the common routine.
Nicholas Clark [Sat, 30 Oct 2010 13:26:53 +0000 (14:26 +0100)]
Merge the direct structure lookup for B::{PV,PVMG,PVLV} to the common routine.

On this platform this reduces the object code size by about 5K.

11 years agoMerge the direct structure lookup for B::IV:: and B::NV:: into 1 routine.
Nicholas Clark [Sat, 30 Oct 2010 12:43:34 +0000 (13:43 +0100)]
Merge the direct structure lookup for B::IV:: and B::NV:: into 1 routine.

As for OPs and GPs, encode the type and offset in the XS alias integer.
Use different names for the type macros, to give them different constant
values, to allow the 3 accessor routines to have no gaps in the values used
for the set of types that they deal with. Otherwise, at least 1 switch
statement would compile to a jump table with "holes".

On this platform, this refactoring reduces the object code size by 4.5K.

11 years agoMerge the implementation of B::IV::{needs64bits,packiv} using ALIAS.
Nicholas Clark [Sat, 30 Oct 2010 11:09:26 +0000 (12:09 +0100)]
Merge the implementation of B::IV::{needs64bits,packiv} using ALIAS.

11 years ago257e0650 used the "wrong" croak() - correct B::GV:SV to use Perl_croak().
Nicholas Clark [Sat, 30 Oct 2010 10:40:08 +0000 (11:40 +0100)]
257e0650 used the "wrong" croak() - correct B::GV:SV to use Perl_croak().

Perl_croak() is a variable argument C function, hence can't have a macro to
permit the use of croak() with/without a context pointer. croak *is* defined
as a macro, substituting croak_nocontext. But we have a context to hand, so we
can use Perl_croak() directly.

11 years agoMerge B::GV::{file,name} using ALIAS.
Nicholas Clark [Sat, 30 Oct 2010 10:10:18 +0000 (11:10 +0100)]
Merge B::GV::{file,name} using ALIAS.

This saves about 700 bytes of object code on this platform.

11 years agoFor B's GP accessors, encode the type and offset in the XS alias integer.
Nicholas Clark [Sat, 30 Oct 2010 09:35:27 +0000 (10:35 +0100)]
For B's GP accessors, encode the type and offset in the XS alias integer.

This allows most of the code for simple structure lookups in the GP of a GV to
be merged into one XS routine. This saves over 5.5K of object code on this

11 years agoIn B.xs, move the direct COP access for stash and file into the common code.
Nicholas Clark [Sat, 30 Oct 2010 08:03:29 +0000 (09:03 +0100)]
In B.xs, move the direct COP access for stash and file into the common code.

Both pairs of accessors are provided for both ithreads and not, but for each,
one pair is direct structure access, and 1 pair "faked up" with a more comple
macro. The direct structure access pair uses the common code (B::OP::next)

This reduces the object code by about 1K for both threaded and unthreaded

11 years agoMacros are not called with aTHX_
Father Chrysostomos [Sat, 30 Oct 2010 06:11:51 +0000 (23:11 -0700)]
Macros are not called with aTHX_

11 years agoSwitch S_mro_get_linear_isa_c3 to HvENAME
Father Chrysostomos [Sat, 30 Oct 2010 05:55:07 +0000 (22:55 -0700)]
Switch S_mro_get_linear_isa_c3 to HvENAME

11 years agoRemove the #ifdef PERL_CORE surrounding the HvENAME* macros
Father Chrysostomos [Sat, 30 Oct 2010 05:51:13 +0000 (22:51 -0700)]
Remove the #ifdef PERL_CORE surrounding the HvENAME* macros

11 years agoAdd HvENAME_HEK
Father Chrysostomos [Sat, 30 Oct 2010 05:26:34 +0000 (22:26 -0700)]

11 years agoAdd docs for hv_ename_*
Father Chrysostomos [Sat, 30 Oct 2010 05:17:44 +0000 (22:17 -0700)]
Add docs for hv_ename_*

11 years agoperlmroapi: suggest HvNAME as a fallback
Father Chrysostomos [Sat, 30 Oct 2010 05:12:55 +0000 (22:12 -0700)]
perlmroapi: suggest HvNAME as a fallback

11 years agoAdd test for the crashes fixed in 00169e2
Father Chrysostomos [Sat, 30 Oct 2010 05:07:32 +0000 (22:07 -0700)]
Add test for the crashes fixed in 00169e2

11 years agoSwitch the core MRO code over to HvENAME
Father Chrysostomos [Sat, 30 Oct 2010 03:45:34 +0000 (20:45 -0700)]
Switch the core MRO code over to HvENAME

This has the side-effect of fixing these one-liners:

$ perl5.13.5 -le' my $glob = \*foo::ISA; delete $::{"foo::"}; *$glob = *a'
Bus error
$ perl5.13.5 -le' my $glob = \*foo::ISA; delete $::{"foo::"}; *$glob = []'
Bus error
$ perl5.13.6 -le'sub baz; my $glob = \*foo::bar; delete $::{"foo::"}; *$glob = *baz;'
Bus error
$ perl5.13.6 -le'sub foo::bar; my $glob = \*foo::bar; delete $::{"foo::"}; *$glob = *baz;'
Bus error

In the first two cases the crash was inadvertently fixed (isn’t it
nice when that happens?) in 5.13.6 (by 6f86b615fa7), but there was
still a fatal error:
Can't call mro_isa_changed_in() on anonymous symbol table at -e line 1.

Because sv_clear calls ->DESTROY, if an object’s stash has been
detached from the symbol table, mro_get_linear_isa can be called on a
hash with no HvENAME. So HvNAME is used as a fallback for those cases.

11 years agoAdd HvENAME
Father Chrysostomos [Fri, 29 Oct 2010 15:59:33 +0000 (08:59 -0700)]

Add HvENAME as an alias for HvENAME_get and make it public.

This is now the preferred name for use in isa caches.

11 years agoIn S_hv_delete_common, call mro_package_moved after the deletion
Father Chrysostomos [Fri, 29 Oct 2010 13:17:28 +0000 (06:17 -0700)]
In S_hv_delete_common, call mro_package_moved after the deletion

11 years agoFix Math::BigInt::FastCalc build under Win32 with croak_xs_usage().
George Greer [Sat, 30 Oct 2010 00:39:33 +0000 (20:39 -0400)]
Fix Math::BigInt::FastCalc build under Win32 with croak_xs_usage().

11 years agoMerge B::COP::{cop_seq,hints,line} into the common B::OP accessor XS code.
Nicholas Clark [Fri, 29 Oct 2010 20:40:18 +0000 (21:40 +0100)]
Merge B::COP::{cop_seq,hints,line} into the common B::OP accessor XS code.

On this platform, this saves almost 2.5K.

Keep IVp last in the switch values, as it's only used for ithreads. Otherwise
there would be an avoidable hole in the jump table for the non-ithreads object

11 years agoMerge B::PADOP::{gv,sv} using the ALIAS mechanism.
Nicholas Clark [Fri, 29 Oct 2010 20:01:08 +0000 (21:01 +0100)]
Merge B::PADOP::{gv,sv} using the ALIAS mechanism.

On this platform, this saves around 600 bytes.

11 years agoMerge B::PADOP::padix into the common B::OP accessor XS code.
Nicholas Clark [Fri, 29 Oct 2010 19:43:49 +0000 (20:43 +0100)]
Merge B::PADOP::padix into the common B::OP accessor XS code.

On this platform, this saves about 800 bytes.

11 years agoSort the MANIFEST (after change 1222f39eae03eee3)
Nicholas Clark [Fri, 29 Oct 2010 20:18:56 +0000 (22:18 +0200)]
Sort the MANIFEST (after change 1222f39eae03eee3)

11 years agoAdd test for [perl #78674]
Father Chrysostomos [Fri, 29 Oct 2010 19:55:22 +0000 (12:55 -0700)]
Add test for [perl #78674]

11 years agoMerge B::SVOP::{gv,sv} into the common B::OP accessor XS code.
Nicholas Clark [Fri, 29 Oct 2010 19:30:23 +0000 (20:30 +0100)]
Merge B::SVOP::{gv,sv} into the common B::OP accessor XS code.

On this platform, this saves about 700 bytes.

11 years agoIn B.xs, move conditionally defined PMOP macros to their points of use.
Nicholas Clark [Fri, 29 Oct 2010 18:55:49 +0000 (19:55 +0100)]
In B.xs, move conditionally defined PMOP macros to their points of use.

Remove the definition for PMOP_pmflags, now unused.

11 years ago[perl #78674] Fix stack pointer corruption in pp_concat() with 'use encoding'
Niko Tyni [Thu, 28 Oct 2010 20:52:17 +0000 (23:52 +0300)]
[perl #78674] Fix stack pointer corruption in pp_concat() with 'use encoding'

sv_utf8_upgrade_nomg() may reallocate the stack via sv_recode_to_utf8()
if 'use encoding' is in effect, causing stack pointer corruption.

11 years agoUse STDOUT in defout.t to suppress that ‘#foo’ under TEST/harness
Father Chrysostomos [Fri, 29 Oct 2010 16:28:57 +0000 (09:28 -0700)]
Use STDOUT in defout.t to suppress that ‘#foo’ under TEST/harness

11 years agoMerge B::PMOP::precomp and B::PMOP::reflags, using the ALIAS mechanism.
Nicholas Clark [Fri, 29 Oct 2010 16:34:00 +0000 (17:34 +0100)]
Merge B::PMOP::precomp and B::PMOP::reflags, using the ALIAS mechanism.

Like B::PMOP::pmoffset, this needs to be controlled by C pre-processor symbols,
so can't be done with an ALIAS keyword.

This saves just over .5K on this platform.

11 years agoMerge B::PMOP::pmoffset into the common B::OP accessor XS code.
Nicholas Clark [Fri, 29 Oct 2010 16:08:24 +0000 (17:08 +0100)]
Merge B::PMOP::pmoffset into the common B::OP accessor XS code.

11 years agoReorder parts of B.xs to reduce the amount of conditionally compiled code.
Nicholas Clark [Fri, 29 Oct 2010 15:42:52 +0000 (16:42 +0100)]
Reorder parts of B.xs to reduce the amount of conditionally compiled code.

Some code with #ifdef guards on PERL_VERSION can be merged.
B::COP::label can have a C<const char *> return type on all perl versions.

11 years agoMerge B::PMOP::pmflags into the common B::OP accessor XS code.
Nicholas Clark [Thu, 28 Oct 2010 16:34:01 +0000 (17:34 +0100)]
Merge B::PMOP::pmflags into the common B::OP accessor XS code.

This saves about 800 bytes on this platform.

11 years agoFor B's OP accessors, encode the type and offset in the XS alias integer.
Nicholas Clark [Thu, 28 Oct 2010 10:29:21 +0000 (11:29 +0100)]
For B's OP accessors, encode the type and offset in the XS alias integer.

This allows most of the code for simple structure lookups to be merged into
one XS routine. Start by merging U8 and PADOFFSET lookups in struct op with
the existing common OP* accessor code. This saves over 2K of object code on
this platform.

11 years agoMerge all B::OP::* accessors returning B::OP objects.
Nicholas Clark [Wed, 27 Oct 2010 15:27:27 +0000 (16:27 +0100)]
Merge all B::OP::* accessors returning B::OP objects.

Use the ALIAS value to store the member's offset within the structure, and
perform explicit pointer arithmetic to read it. This reduces the object size
by about 5K on this platform.

11 years agoMath::BigInt::FastCalc doesn't need to set $BASE or $BASE_LEN.
Nicholas Clark [Thu, 28 Oct 2010 19:17:29 +0000 (20:17 +0100)]
Math::BigInt::FastCalc doesn't need to set $BASE or $BASE_LEN.

The variables are neither documented nor exported, and nothing on CPAN uses
them. Hence there is no need to set them, which means that the return list from
Math::BigInt::Calc::_base_len() can be passed directly into FastCalc's XS

11 years agoMath::BigInt::FastCalc doesn't need to override methods in a BEGIN block.
Nicholas Clark [Thu, 28 Oct 2010 18:10:17 +0000 (19:10 +0100)]
Math::BigInt::FastCalc doesn't need to override methods in a BEGIN block.

The BEGIN context of the code C<use>ing it will be early enough for the
subroutine assignments to take full effect.

11 years agoRefactor FastCalc to initialise BASE and BASE_LEN as part of bootstrap.
Nicholas Clark [Thu, 28 Oct 2010 13:57:13 +0000 (14:57 +0100)]
Refactor FastCalc to initialise BASE and BASE_LEN as part of bootstrap.

This avoids the need for a separate, special, one-shot _set_XS_BASE routine.
We take advantage of the fact that all arguments to XSLoader::load() are passed
on to the module's bootstrap routine.

11 years agoReorder the list return of Math::BigInt::Calc::_base_len().
Nicholas Clark [Thu, 28 Oct 2010 13:15:01 +0000 (14:15 +0100)]
Reorder the list return of Math::BigInt::Calc::_base_len().

A search of CPAN shows that this private function is only used internally
between Math::BigInt::Calc, Math::BigInt::FastCalc and their test suites.

11 years agoFix NAMECOUNT output format in dump.c
Father Chrysostomos [Fri, 29 Oct 2010 05:34:10 +0000 (22:34 -0700)]
Fix NAMECOUNT output format in dump.c

Remove quotation marks and, as a bonus, eliminate a compiler warning.

11 years agohv_ename_delete should not delete the HvNAME
Father Chrysostomos [Fri, 29 Oct 2010 05:22:10 +0000 (22:22 -0700)]
hv_ename_delete should not delete the HvNAME

This is something that 78b79c7758 missed.

When xhv_name is a HEK *, it is both the regular name and the effect-
ive name at the same time. Only when xhv_name_count is negative is the
regular name not also the effective name.

hv_ename_delete needs to take the HEK that xhv_name points to and
put it in a new HEK * array in xhv_name. This array will just have
one element.

When xhv_name_count is negative, effective names start with the second
element. So we set xhv_name_count to -1 so there isn’t one.

11 years agoB::GV::is_empty and B::GV::isGV_with_GP can be merged using ALIAS.
Nicholas Clark [Fri, 29 Oct 2010 11:41:10 +0000 (12:41 +0100)]
B::GV::is_empty and B::GV::isGV_with_GP can be merged using ALIAS.

This saves about 700 bytes on this platform.

11 years agoUpdate CPANPLUS to CPAN version 0.9010
Chris 'BinGOs' Williams [Fri, 29 Oct 2010 11:04:32 +0000 (12:04 +0100)]
Update CPANPLUS to CPAN version 0.9010


  CHanges for 0.9010      Thu Oct 28 23:20:19 2010
  * Added PERL_MM_OPT to the CPANPLUS-Dist-MM test to hopefully
    resolve RT#57241 which relates to local::lib and cpanm

  Changes for 0.9009      Mon Oct 25 21:03:39 2010
  * Resolve issues with the SQLite source engine

  Changes for 0.9008      Mon Oct 25 17:01:34 2010
  * Normalise prereqs version strings before resolving prereqs

11 years agoB::PV::PV doesn't need a special case for pad names post 5.9.x
Nicholas Clark [Fri, 29 Oct 2010 11:01:11 +0000 (12:01 +0100)]
B::PV::PV doesn't need a special case for pad names post 5.9.x

The PV-related flags on SVs used to store names in pads were made sane by
commit 931b58fb28fa5ca7.

11 years agoUse the same XS body for B::IV::RV and B::PV::RV.
Nicholas Clark [Fri, 29 Oct 2010 10:29:38 +0000 (11:29 +0100)]
Use the same XS body for B::IV::RV and B::PV::RV.

This reduces the object code size by about 600 bytes on this platform.

The input typemap checking makes no distinction between different SV types,
so the XS body will generate the same C code, despite the different XS
"types". So there is no change in behaviour from doing newXS like this,
compared with the old approach of having a (near) duplicate XS body.
We should fix the typemap checking.

11 years agoA clearer implementation of B::SV::object_2svref()
Nicholas Clark [Fri, 29 Oct 2010 09:59:23 +0000 (10:59 +0100)]
A clearer implementation of B::SV::object_2svref()

The same number of lines of code, but it avoids using 2 #defines simply to
take advantage of typemap entries, which the reader may not be familiar with.

11 years agoB::SV{REFCNT,FLAGS,SvTYPE,POK,ROK,MAGICAL} can be implemented via ALIAS.
Nicholas Clark [Fri, 29 Oct 2010 09:28:53 +0000 (10:28 +0100)]

Maksing the flags directly here breaks a bit of the encapsulation of sv.h, but
saves about 4K on this platform.

11 years agoUpdate CPANPLUS-Dist-Build to CPAN version 0.50
Chris 'BinGOs' Williams [Fri, 29 Oct 2010 08:49:11 +0000 (09:49 +0100)]
Update CPANPLUS-Dist-Build to CPAN version 0.50


  0.50 Thu Oct 28 23:02:19 BST 2010
    - We clear the relevant env vars that effect Module::Build
      during the 'install' test. local::lib and cpanm set these.

11 years agoRecommend a more reliable way of identifying magic
Florian Ragwitz [Fri, 29 Oct 2010 07:03:23 +0000 (09:03 +0200)]
Recommend a more reliable way of identifying magic

Also illustrate it with some example code.

11 years agoTeach dump.c about all magic flags
Florian Ragwitz [Fri, 29 Oct 2010 06:42:03 +0000 (08:42 +0200)]
Teach dump.c about all magic flags

Also rearrange the flags according to their value and trim some trailing
whitespace in that area.

11 years agoTeach dump.c about ENAMEs
Father Chrysostomos [Fri, 29 Oct 2010 05:06:33 +0000 (22:06 -0700)]
Teach dump.c about ENAMEs

11 years agoadd SAVEFREECOPHH()
Zefram [Thu, 28 Oct 2010 21:01:11 +0000 (22:01 +0100)]

Add the facility for the save stack to free (decrement the refcount of)

11 years agoFile::CheckTree.t: @INC should be change for chdir
Karl Williamson [Thu, 28 Oct 2010 04:54:59 +0000 (22:54 -0600)]
File::CheckTree.t: @INC should be change for chdir

This test has a workaround for windows that isn't needed if it just sets
up the @INC initially to allow for finding things both before and after
the chdir.

And, it's unreasonable to expect that nothing it calls would in turn
never do their own module load, so the @INC needs to be correct anyway.

11 years agoIn B.xs, move the definition of PMOP_pmnext adjacent to its usage.
Nicholas Clark [Thu, 28 Oct 2010 15:36:56 +0000 (16:36 +0100)]
In B.xs, move the definition of PMOP_pmnext adjacent to its usage.

Now it lives within the #ifdef guarding the conditional compilation of the only
code that uses it.

11 years agoThe method B::PMOP::pmregexp never existed. Remove it from the documentation.
Nicholas Clark [Thu, 28 Oct 2010 15:03:49 +0000 (16:03 +0100)]
The method B::PMOP::pmregexp never existed. Remove it from the documentation.

A macro was added for it when B.xs was first written, but that macro was never
used for a method definition.

11 years agoIn S_scan_const(), use the return value of my_sprintf() to avoid a strlen().
Nicholas Clark [Thu, 28 Oct 2010 13:55:51 +0000 (14:55 +0100)]
In S_scan_const(), use the return value of my_sprintf() to avoid a strlen().

Brought to you by the campaign for the elimination of strlen().

11 years agoMerge the XS implementation of B::OP::name and B::OP::desc
Nicholas Clark [Thu, 28 Oct 2010 12:17:06 +0000 (13:17 +0100)]
Merge the XS implementation of B::OP::name and B::OP::desc

This saves 4 lines of source code, but over 1K of object code.

11 years agoperldelta up to 0b6a3b5adbe83
Father Chrysostomos [Thu, 28 Oct 2010 07:04:58 +0000 (00:04 -0700)]
perldelta up to 0b6a3b5adbe83

11 years agoIncrease Hash::Util::FieldHash’s version
Father Chrysostomos [Thu, 28 Oct 2010 06:44:06 +0000 (23:44 -0700)]
Increase Hash::Util::FieldHash’s version

11 years agoDefer to perlfunc on \G and pos()
brian d foy [Thu, 28 Oct 2010 06:49:57 +0000 (01:49 -0500)]
Defer to perlfunc on \G and pos()

11 years agoMerge branch 'briandfoy/pos' into blead
brian d foy [Thu, 28 Oct 2010 06:38:30 +0000 (01:38 -0500)]
Merge branch 'briandfoy/pos' into blead

11 years agoNote that pos() only affects the next match RT #78636
brian d foy [Thu, 28 Oct 2010 06:36:57 +0000 (01:36 -0500)]
Note that pos() only affects the next match RT #78636

11 years agoType- and think-o’s
Father Chrysostomos [Thu, 28 Oct 2010 06:33:31 +0000 (23:33 -0700)]
Type- and think-o’s

11 years agoperldelta for 2acc3314e31
Father Chrysostomos [Thu, 28 Oct 2010 06:31:08 +0000 (23:31 -0700)]
perldelta for 2acc3314e31

11 years agoperldelta up to e1be28b42dd83 plus corrections
Father Chrysostomos [Thu, 28 Oct 2010 06:12:01 +0000 (23:12 -0700)]
perldelta up to e1be28b42dd83 plus corrections

11 years agoIncrease Sys::Hostname’s version
Father Chrysostomos [Thu, 28 Oct 2010 05:56:37 +0000 (22:56 -0700)]
Increase Sys::Hostname’s version

11 years agoRevert "Bump charnames’ version"
Father Chrysostomos [Thu, 28 Oct 2010 05:55:13 +0000 (22:55 -0700)]
Revert "Bump charnames’ version"

This reverts commit fc7c69e2154d208e45cf3f5c98b74ed035d8f50c.

11 years agoRun 01SelfLoader.t also under taint mode
Father Chrysostomos [Thu, 28 Oct 2010 05:16:37 +0000 (22:16 -0700)]
Run 01SelfLoader.t also under taint mode

This is to test for [perl #72062].

11 years agoMark failing tests in t/op/stash.t as to-do
Father Chrysostomos [Thu, 28 Oct 2010 04:49:03 +0000 (21:49 -0700)]
Mark failing tests in t/op/stash.t as to-do

Commit 78b79c775, in adding test for the breakage introduced in
35759254, inadvertently triggered a bug that goes back at least to
perl 5.8.7, but which only occurs with threads.

11 years agofix pod formatting in perldelta.pod
David Golden [Wed, 27 Oct 2010 22:34:11 +0000 (18:34 -0400)]
fix pod formatting in perldelta.pod

11 years agoperldelta up to 157486ae134392
Father Chrysostomos [Wed, 27 Oct 2010 20:46:13 +0000 (13:46 -0700)]
perldelta up to 157486ae134392

11 years agoIncrease SDBM_File’s version
Father Chrysostomos [Wed, 27 Oct 2010 20:43:05 +0000 (13:43 -0700)]
Increase SDBM_File’s version

11 years agoIncrease Hash::Util’s version
Father Chrysostomos [Wed, 27 Oct 2010 20:41:02 +0000 (13:41 -0700)]
Increase Hash::Util’s version

11 years agoIncrease Fcntl’s version
Father Chrysostomos [Wed, 27 Oct 2010 20:28:38 +0000 (13:28 -0700)]
Increase Fcntl’s version

11 years agoperldelta up to 582b18310
Father Chrysostomos [Wed, 27 Oct 2010 19:59:24 +0000 (12:59 -0700)]
perldelta up to 582b18310

11 years agoBump ODBM_File’s version
Father Chrysostomos [Wed, 27 Oct 2010 19:52:56 +0000 (12:52 -0700)]
Bump ODBM_File’s version

11 years agoBump NDBM_File’s version
Father Chrysostomos [Wed, 27 Oct 2010 19:51:50 +0000 (12:51 -0700)]
Bump NDBM_File’s version

11 years agoRemove THREADSV_NAMES, part of 5005 threads that the chainsaw missed.
Nicholas Clark [Wed, 27 Oct 2010 20:25:54 +0000 (22:25 +0200)]
Remove THREADSV_NAMES, part of 5005 threads that the chainsaw missed.

Also remove the documentation of OPf_SPECIAL for OP_ENTERITER, as that was only
for 5.005 threads. Stop B::Deparse misinterpreting OPf_SPECIAL on OP_ENTERITER.

11 years agoB::Deparse should call B::threadsv_names, instead of hard coding a list.
Nicholas Clark [Wed, 27 Oct 2010 20:14:06 +0000 (22:14 +0200)]
B::Deparse should call B::threadsv_names, instead of hard coding a list.

These are only used under 5.005threads, which was removed in 5.10.

11 years agoGet B::Deparse to load on perls back to 5.8.1
Nicholas Clark [Wed, 27 Oct 2010 19:37:44 +0000 (21:37 +0200)]
Get B::Deparse to load on perls back to 5.8.1

But note - not all its tests are parsing, and some routines will fail at runtime
due to methods missing from older Bs.

11 years agoRenaming of stashes should not be visible from Perl
Father Chrysostomos [Wed, 27 Oct 2010 16:44:04 +0000 (09:44 -0700)]
Renaming of stashes should not be visible from Perl

Change 35759254 made stashes get renamed when moved around. This had
an unintended consequence: Typeglobs, ref() return values, stringifi-
cation of blessed references and __PACKAGE__ are all affected by this.

This commit makes a new distinction between stashes’ names and effect-
ive names. Stash names are now unaffected when the stashes move
around. Only the effective names are affected. (The apparent presence
of any puns in the previous sentence is purely incidental and most
likely the result of the reader’s inferential propensity.)

To this end a new HvENAME_get macro is introduced, returning the first effective name (what HvNAME_get was returning). (Only one effective
name needs to be in effect at a time.) hv_add_name and hv_delete_name
have been renamed hv_add_ename and hv_delete_ename. hv_name_set is
modified to leave the effective names in place unless the name is
being set to NULL.

These names are now stored in HvAUX as follows: When xhv_name_count is
0, xhv_name is a HEK pointer, containing the name which is also the
effective name. When xhv_name_count is not zero, then xhv_name is a
pointer to an array of HEK pointers. If xhv_name_count is positive,
the first HEK is the name *and* one of the effective names. When
xhv_name_count is negative, the first HEK is the name and subsequent
HEKs are the effective names.

11 years agoSimplify the conditionals in the XS code for B::COP::{io,warnings}
Nicholas Clark [Wed, 27 Oct 2010 16:16:27 +0000 (17:16 +0100)]
Simplify the conditionals in the XS code for B::COP::{io,warnings}

The same post-processed C code will generated as before for PERL_VERSION >= 9,
and the same sequence of perl API calls for PERL_VERSION < 9.

11 years agoUpdate UPSTREAM status for perlreftut and Tie::File in
Chris 'BinGOs' Williams [Wed, 27 Oct 2010 15:08:58 +0000 (16:08 +0100)]
Update UPSTREAM status for perlreftut and Tie::File in

  From IRC logs:

  [16:05] <yrlnry> Tie::File can be marked first-come.
  [16:06] <yrlnry> perlreftut can be marked 'blead' since there is no CPAN version.
  [16:06] <yrlnry> perlreftut has required only very rate changes, which are easily handled by sending patch files to the pumpkings.

11 years agoUsing ALIAS B::OP::{opt,spare,type} can be implemented as 1 XS function.
Nicholas Clark [Wed, 27 Oct 2010 14:21:40 +0000 (15:21 +0100)]
Using ALIAS B::OP::{opt,spare,type} can be implemented as 1 XS function.

This reduces the object code size by almost 2K on this platform.

11 years agoDon't fold constants in sprintf() if locales are used
Niko Tyni [Wed, 27 Oct 2010 07:56:24 +0000 (10:56 +0300)]
Don't fold constants in sprintf() if locales are used

Commit c427f4d2d4575fbc8a5190932fe321136c7597b3 in 5.10.1 made sprintf()
ignore LC_NUMERIC for numeric constants.

11 years agoLC_NUMERIC documentation updates + tests
Niko Tyni [Tue, 19 Oct 2010 18:55:14 +0000 (21:55 +0300)]
LC_NUMERIC documentation updates + tests

Most of the confusion around LC_NUMERIC was fixed with commits
7e4353e96785be675a69a6886d154405dbfdc124 and
but two errors remain:

- the early parts of perllocale.pod still say printf() uses LC_NUMERIC
  with just 'use locale' when actually a POSIX::setlocale() call is
  also needed

- format() hasn't used LC_NUMERIC unconditionally since 5.005_03
  (commit 097ee67dff1c60f201bc09435bc6eaeeafcd8123).

Update the documentation and test the claims in t/run/locale.t.

11 years agoRefactor LC_NUMERIC test out of t/run/fresh_perl.t
Niko Tyni [Wed, 27 Oct 2010 06:49:38 +0000 (09:49 +0300)]
Refactor LC_NUMERIC test out of t/run/fresh_perl.t

Neither lib/locale.t nor t/run/fresh_perl.t should be used for new tests,
so take the locale related tests and the setup code from fresh_perl.t
to make ground for more.

11 years agoUpdate Archive-Extract to CPAN version 0.46
Chris 'BinGOs' Williams [Wed, 27 Oct 2010 11:09:43 +0000 (12:09 +0100)]
Update Archive-Extract to CPAN version 0.46


  Changes for 0.46    Tue Oct 26 23:45:52 2010
  * Correct speeling errors in docs, spotted by
  * Only use unzip from /usr/pkg/bin when on
    NetBSD which will be the Info-ZIP version

11 years agonew API functions op_scope and op_lvalue
Zefram [Tue, 26 Oct 2010 22:58:42 +0000 (23:58 +0100)]
new API functions op_scope and op_lvalue

The function scope() goes into the API as op_scope(), and mod() goes
into the API as op_lvalue().  Both marked experimental, because their
behaviour is a little quirky and not trivially dequirkable.

11 years agoANYOF regnode often overallocates space
Karl Williamson [Tue, 26 Oct 2010 19:14:12 +0000 (13:14 -0600)]
ANYOF regnode often overallocates space

The ANYOF regnode is used for [bracketed character classes].  It is
normally a struct regnode_charclass, but sometimes it is a struct
regnode_charclass_class, which adds an extra 4 bytes at the end to store
which of things like \d, \w, [:posix:] within the outer bracketed class
are to be matched at run time, like [\d\s[:upper:]].

However these are currently only stored the if the match is to be locale
based.  I haven't looked in the history, but the comments seem to
indicate that in the past those things were stored there all the time.

But, the space is allocated even if nothing gets stored there.  This
patch only allocates the space if there is one of these classes
occurring in a [bracketed class] and something is stored there, namely
only if the regex is being compiled under 'use locale'.

The previous code was not noting that these classes matched code points
(in the 'stored' variable), but was relying on ANYOF_LARGE getting set
which thus avoided it thinking it was a single character class that
could be optimized to an EXACT node.  Thus, this patch has to explicitly
set 'stored' to > 1, as the function already does elsewhere for similar

11 years agoCleanup output from t/io/defout.t RT#78572
Jerry D. Hedden [Tue, 26 Oct 2010 22:58:55 +0000 (15:58 -0700)]
Cleanup output from t/io/defout.t RT#78572

t/io/defout.t pollutes the output of 'make test':

  # write

  The attached patch fixes this.

11 years agoRevert "[PATCH] Cleanup output from t/io/defout.t RT#78572"
Father Chrysostomos [Tue, 26 Oct 2010 22:57:06 +0000 (15:57 -0700)]
Revert "[PATCH] Cleanup output from t/io/defout.t RT#78572"

This reverts commit 4d2eb54d4d30c9c5b093de9a57c324d5551c58fe.

11 years ago[PATCH] Cleanup output from t/io/defout.t RT#78572
Jerry D. Hedden [Tue, 26 Oct 2010 21:11:32 +0000 (22:11 +0100)]
[PATCH] Cleanup output from t/io/defout.t RT#78572

  t/io/defout.t pollutes the output of 'make test':

  # write

  The attached patch fixes this.

Signed-off-by: Chris 'BinGOs' Williams <>
11 years agofix thinko in code comment
David Mitchell [Tue, 26 Oct 2010 16:07:57 +0000 (17:07 +0100)]
fix thinko in code comment

11 years agoUpdate CPAN to CPAN version 1.94_62
Chris 'BinGOs' Williams [Tue, 26 Oct 2010 13:41:47 +0000 (14:41 +0100)]
Update CPAN to CPAN version 1.94_62


  2010-10-26  Andreas J. Koenig  <>

        * release 1.94_62

        * address RT #62064: build_requires_install_policy set to "no" did not
        work correctly (reported by Xavier Bergade)

        * address RT ##55091: don't ask the proxy credentials if proxy_user
        empty (fixed by Robert Bohne)

        * address RT #55093: no_proxy doesn't work with more then one entries
        (fixed by Robert Bohne)

11 years agodetect the failure introduced in 304474c3 and fixed in cefd5c7c
Tony Cook [Tue, 26 Oct 2010 12:09:45 +0000 (23:09 +1100)]
detect the failure introduced in 304474c3 and fixed in cefd5c7c

11 years agoIn I18N::Langinfo::langinfo(), use newSVpv(), as it will call strlen()
Nicholas Clark [Tue, 26 Oct 2010 04:50:14 +0000 (05:50 +0100)]
In I18N::Langinfo::langinfo(), use newSVpv(), as it will call strlen()

newSVpv() will return an undef if the passed pointer is NULL, which allows
langinfo() to be simplified further.

11 years ago-$zero should not modify $zero
Father Chrysostomos [Tue, 26 Oct 2010 04:29:24 +0000 (21:29 -0700)]
-$zero should not modify $zero

With change a5b92898, negation started modifying numeric arguments,
causing problems for modules like Data::Float.