Front page | perl.perl5.changes |
Postings from May 2008
Change 33945: Integrate:
From:
Dave Mitchell
Date:
May 28, 2008 14:00
Subject:
Change 33945: Integrate:
Change 33945 by davem@davem-pigeon on 2008/05/28 20:51:40
Integrate:
[ 33553]
Subject: [PATCH] apidoc mismatch for Perl_magic_clearhint
From: Vincent Pit <perl@profvince.com>
Date: Mon, 24 Mar 2008 21:30:10 +0100
Message-ID: <47E80F52.4030805@profvince.com>
[ 33554]
Subject: [PATCH] IO::Socket::INET unnecessarily resolves "udp"
From: Niko Tyni <ntyni@debian.org>
Date: Mon, 24 Mar 2008 23:32:24 +0200
Message-Id: <1206394344-30835-1-git-send-email-ntyni@debian.org>
[ 33556]
Subject: [PATCH] borg parent.pm
From: "Yitzchak Scott-Thoennes" <sthoenna@efn.org>
Date: Wed, 5 Mar 2008 17:19:32 -0800 (PST)
Message-ID: <57512.71.32.86.11.1204766372.squirrel@webmail.efn.org>
Plus bump base.pm's version to a non-alpha number
[ 33557]
Subject: Re: [PATCH] Double warning with perl -we 'my $a; substr $a, 0, 10,
From: Vincent Pit <perl@profvince.com>
Date: Sat, 22 Mar 2008 13:37:42 +0100
Message-ID: <47E4FD96.6080304@profvince.com>
[ 33560]
Use sv_setpvs() like a few lines before since change #33557
[ 33584]
Subject: [PATCH] MAD dump xml escape regex
From: Gerard Goossen <gerard@tty.nl>
Date: Thu, 27 Mar 2008 13:55:31 +0100
Message-ID: <20080327125531.GN4409@ostwald>
[ 33594]
Subject: [PATCH] Re: Tests failed on PPC64
From: Dominic Dunlop <shouldbedomo@mac.com>
Message-Id: <53E6407E-B87C-4D6A-A6E7-D842BDF30292@mac.com>
Date: Fri, 14 Mar 2008 14:45:39 +0100
[ 33608]
Subject: [PATCH] add -v to regen.pl and friends
From: "Robin Barker" <Robin.Barker@npl.co.uk>
Date: Wed, 19 Mar 2008 10:55:59 -0000
Message-ID: <46A0F33545E63740BC7563DE59CA9C6D093AA6@exchsvr2.npl.ad.local>
Affected files ...
... //depot/maint-5.10/perl/MANIFEST#28 integrate
... //depot/maint-5.10/perl/Makefile.SH#6 integrate
... //depot/maint-5.10/perl/Porting/Maintainers.pl#9 integrate
... //depot/maint-5.10/perl/dump.c#7 integrate
... //depot/maint-5.10/perl/ext/IO/lib/IO/Socket/INET.pm#2 integrate
... //depot/maint-5.10/perl/lib/base.pm#2 integrate
... //depot/maint-5.10/perl/lib/parent.pm#1 branch
... //depot/maint-5.10/perl/lib/parent/t/compile-time-file.t#1 branch
... //depot/maint-5.10/perl/lib/parent/t/compile-time.t#1 branch
... //depot/maint-5.10/perl/lib/parent/t/lib/Dummy.pm#1 branch
... //depot/maint-5.10/perl/lib/parent/t/lib/Dummy/Outside.pm#1 branch
... //depot/maint-5.10/perl/lib/parent/t/lib/Dummy2.plugin#1 branch
... //depot/maint-5.10/perl/lib/parent/t/lib/FileThatOnlyExistsAsPMC.pmc#1 branch
... //depot/maint-5.10/perl/lib/parent/t/lib/ReturnsFalse.pm#1 branch
... //depot/maint-5.10/perl/lib/parent/t/parent-classfromclassfile.t#1 branch
... //depot/maint-5.10/perl/lib/parent/t/parent-classfromfile.t#1 branch
... //depot/maint-5.10/perl/lib/parent/t/parent-pmc.t#1 branch
... //depot/maint-5.10/perl/lib/parent/t/parent-returns-false.t#1 branch
... //depot/maint-5.10/perl/lib/parent/t/parent.t#1 branch
... //depot/maint-5.10/perl/mad/t/p55.t#3 integrate
... //depot/maint-5.10/perl/mg.c#9 integrate
... //depot/maint-5.10/perl/pod/perlintern.pod#4 integrate
... //depot/maint-5.10/perl/pp.c#7 integrate
... //depot/maint-5.10/perl/regen.pl#4 integrate
... //depot/maint-5.10/perl/regen_lib.pl#3 integrate
... //depot/maint-5.10/perl/t/lib/warnings/9uninit#6 integrate
... //depot/maint-5.10/perl/t/op/reg_namedcapture.t#2 integrate
... //depot/maint-5.10/perl/t/op/switch.t#3 integrate
Differences ...
==== //depot/maint-5.10/perl/MANIFEST#28 (text) ====
Index: perl/MANIFEST
--- perl/MANIFEST#27~33943~ 2008-05-28 08:54:22.000000000 -0700
+++ perl/MANIFEST 2008-05-28 13:51:40.000000000 -0700
@@ -2327,6 +2327,19 @@
lib/Package/Constants/t/01_list.t Package::Constants tests
lib/Params/Check.pm Params::Check
lib/Params/Check/t/01_Params-Check.t Params::Check tests
+lib/parent.pm Establish an ISA relationship with base classes at compile time
+lib/parent/t/compile-time-file.t tests for parent.pm
+lib/parent/t/compile-time.t tests for parent.pm
+lib/parent/t/lib/Dummy2.plugin test files for parent.pm
+lib/parent/t/lib/Dummy.pm test files for parent.pm
+lib/parent/t/lib/Dummy/Outside.pm test files for parent.pm
+lib/parent/t/lib/FileThatOnlyExistsAsPMC.pmc test files for parent.pm
+lib/parent/t/lib/ReturnsFalse.pm test files for parent.pm
+lib/parent/t/parent-classfromclassfile.t tests for parent.pm
+lib/parent/t/parent-classfromfile.t tests for parent.pm
+lib/parent/t/parent-pmc.t tests for parent.pm
+lib/parent/t/parent-returns-false.t tests for parent.pm
+lib/parent/t/parent.t tests for parent.pm
lib/perl5db.pl Perl debugging routines
lib/perl5db.t Tests for the Perl debugger
lib/perl5db/t/eval-line-bug Tests for the Perl debugger
==== //depot/maint-5.10/perl/Makefile.SH#6 (text) ====
Index: perl/Makefile.SH
--- perl/Makefile.SH#5~33904~ 2008-05-21 09:37:54.000000000 -0700
+++ perl/Makefile.SH 2008-05-28 13:51:40.000000000 -0700
@@ -998,9 +998,12 @@
.PHONY: regen_headers regen_pods regen_all
-regen regen_headers: FORCE
+regen: FORCE
-perl regen.pl
+regen_headers: FORCE
+ -perl regen.pl -v
+
regen_pods: FORCE
-cd pod; $(LDLIBPTH) $(MAKE) regen_pods
==== //depot/maint-5.10/perl/Porting/Maintainers.pl#9 (text) ====
Index: perl/Porting/Maintainers.pl
--- perl/Porting/Maintainers.pl#8~33915~ 2008-05-23 07:57:46.000000000 -0700
+++ perl/Porting/Maintainers.pl 2008-05-28 13:51:40.000000000 -0700
@@ -17,6 +17,7 @@
'arandal' => 'Allison Randal <allison@perl.org>',
'audreyt' => 'Audrey Tang <cpan@audreyt.org>',
'avar' => 'Ævar Arnfjörð Bjarmason <avar@cpan.org>',
+ 'corion' => 'Max Maischein <corion@corion.net>',
'craig' => 'Craig Berry <craigberry@mac.com>',
'dankogai' => 'Dan Kogai <dankogai@cpan.org>',
'dconway' => 'Damian Conway <dconway@cpan.org>',
@@ -652,6 +653,13 @@
'CPAN' => 1,
},
+ 'parent' =>
+ {
+ 'MAINTAINER' => 'corion',
+ 'FILES' => q[lib/parent lib/parent.pm],
+ 'CPAN' => 1,
+ },
+
'perlebcdic' =>
{
'MAINTAINER' => 'pvhp',
==== //depot/maint-5.10/perl/ext/IO/lib/IO/Socket/INET.pm#2 (text) ====
Index: perl/ext/IO/lib/IO/Socket/INET.pm
--- perl/ext/IO/lib/IO/Socket/INET.pm#1~32694~ 2007-12-22 01:23:09.000000000 -0800
+++ perl/ext/IO/lib/IO/Socket/INET.pm 2008-05-28 13:51:40.000000000 -0700
@@ -27,7 +27,7 @@
);
my %proto_number;
$proto_number{tcp} = Socket::IPPROTO_TCP() if defined &Socket::IPPROTO_TCP;
-$proto_number{upd} = Socket::IPPROTO_UDP() if defined &Socket::IPPROTO_UDP;
+$proto_number{udp} = Socket::IPPROTO_UDP() if defined &Socket::IPPROTO_UDP;
$proto_number{icmp} = Socket::IPPROTO_ICMP() if defined &Socket::IPPROTO_ICMP;
my %proto_name = reverse %proto_number;
==== //depot/maint-5.10/perl/lib/base.pm#2 (text) ====
Index: perl/lib/base.pm
--- perl/lib/base.pm#1~32694~ 2007-12-22 01:23:09.000000000 -0800
+++ perl/lib/base.pm 2008-05-28 13:51:40.000000000 -0700
@@ -2,7 +2,8 @@
use strict 'vars';
use vars qw($VERSION);
-$VERSION = '2.13';
+$VERSION = '2.14';
+$VERSION = eval $VERSION;
# constant.pm is slow
sub SUCCESS () { 1 }
@@ -192,6 +193,9 @@
=head1 DESCRIPTION
+Unless you are using the C<fields> pragma, consider this module discouraged
+in favor of the lighter-weight C<parent>.
+
Allows you to both load one or more modules, while setting up inheritance from
those modules at the same time. Roughly similar in effect to
==== //depot/maint-5.10/perl/lib/parent.pm#1 (text) ====
Index: perl/lib/parent.pm
--- /dev/null 2008-05-07 15:08:24.549929899 -0700
+++ perl/lib/parent.pm 2008-05-28 13:51:40.000000000 -0700
@@ -0,0 +1,136 @@
+package parent;
+use strict;
+use vars qw($VERSION);
+$VERSION = '0.221';
+
+sub import {
+ my $class = shift;
+
+ my $inheritor = caller(0);
+
+ if ( @_ and $_[0] eq '-norequire' ) {
+ shift @_;
+ } else {
+ for ( my @filename = @_ ) {
+ if ( $_ eq $inheritor ) {
+ warn "Class '$inheritor' tried to inherit from itself\n";
+ };
+
+ s{::|'}{/}g;
+ require "$_.pm"; # dies if the file is not found
+ }
+ }
+
+ {
+ no strict 'refs';
+ # This is more efficient than push for the new MRO
+ # at least until the new MRO is fixed
+ @{"$inheritor\::ISA"} = (@{"$inheritor\::ISA"} , @_);
+ };
+};
+
+"All your base are belong to us"
+
+__END__
+
+=head1 NAME
+
+parent - Establish an ISA relationship with base classes at compile time
+
+=head1 SYNOPSIS
+
+ package Baz;
+ use parent qw(Foo Bar);
+
+=head1 DESCRIPTION
+
+Allows you to both load one or more modules, while setting up inheritance from
+those modules at the same time. Mostly similar in effect to
+
+ package Baz;
+ BEGIN {
+ require Foo;
+ require Bar;
+ push @ISA, qw(Foo Bar);
+ }
+
+By default, every base class needs to live in a file of its own.
+If you want to have a subclass and its parent class in the same file, you
+can tell C<parent> not to load any modules by using the C<-norequire> switch:
+
+ package Foo;
+ sub exclaim { "I CAN HAS PERL" }
+
+ package DoesNotLoadFooBar;
+ use parent -norequire, 'Foo', 'Bar';
+ # will not go looking for Foo.pm or Bar.pm
+
+This is equivalent to the following code:
+
+ package Foo;
+ sub exclaim { "I CAN HAS PERL" }
+
+ package DoesNotLoadFooBar;
+ push @DoesNotLoadFooBar::ISA, 'Foo';
+
+This is also helpful for the case where a package lives within
+a differently named file:
+
+ package MyHash;
+ use Tie::Hash;
+ use parent -norequire, 'Tie::StdHash';
+
+This is equivalent to the following code:
+
+ package MyHash;
+ require Tie::Hash;
+ push @ISA, 'Tie::StdHash';
+
+If you want to load a subclass from a file that C<require> would
+not consider an eligible filename (that is, it does not end in
+either C<.pm> or C<.pmc>), use the following code:
+
+ package MySecondPlugin;
+ require './plugins/custom.plugin'; # contains Plugin::Custom
+ use parent -norequire, 'Plugin::Custom';
+
+=head1 DIAGNOSTICS
+
+=over 4
+
+=item Class 'Foo' tried to inherit from itself
+
+Attempting to inherit from yourself generates a warning.
+
+ use Foo;
+ use parent 'Foo';
+
+=back
+
+=head1 HISTORY
+
+This module was forked from L<base> to remove the cruft
+that had accumulated in it.
+
+=head1 CAVEATS
+
+=head1 SEE ALSO
+
+L<base>
+
+=head1 AUTHORS AND CONTRIBUTORS
+
+Rafaël Garcia-Suarez, Bart Lateur, Max Maischein, Anno Siegel, Michael Schwern
+
+=head1 MAINTAINER
+
+Max Maischein C< corion@cpan.org >
+
+Copyright (c) 2007 Max Maischein C<< <corion@cpan.org> >>
+Based on the idea of C<base.pm>, which was introduced with Perl 5.004_04.
+
+=head1 LICENSE
+
+This module is released under the same terms as Perl itself.
+
+=cut
==== //depot/maint-5.10/perl/lib/parent/t/compile-time-file.t#1 (text) ====
Index: perl/lib/parent/t/compile-time-file.t
--- /dev/null 2008-05-07 15:08:24.549929899 -0700
+++ perl/lib/parent/t/compile-time-file.t 2008-05-28 13:51:40.000000000 -0700
@@ -0,0 +1,47 @@
+#!/usr/bin/perl -w
+BEGIN {
+ if( $ENV{PERL_CORE} ) {
+ chdir 't' if -d 't';
+ chdir '../lib/parent';
+ @INC = '..';
+ }
+}
+
+use strict;
+use Test::More tests => 9;
+use lib 't/lib';
+
+{
+ package Child;
+ use parent 'Dummy';
+}
+
+{
+ package Child2;
+ require Dummy;
+ use parent -norequire, 'Dummy::InlineChild';
+}
+
+{
+ package Child3;
+ use parent "Dummy'Outside";
+}
+
+my $obj = {};
+bless $obj, 'Child';
+isa_ok $obj, 'Dummy';
+can_ok $obj, 'exclaim';
+is $obj->exclaim, "I CAN FROM Dummy", 'Inheritance is set up correctly';
+
+$obj = {};
+bless $obj, 'Child2';
+isa_ok $obj, 'Dummy::InlineChild';
+can_ok $obj, 'exclaim';
+is $obj->exclaim, "I CAN FROM Dummy::InlineChild", 'Inheritance is set up correctly for inlined classes';
+
+$obj = {};
+bless $obj, 'Child3';
+isa_ok $obj, 'Dummy::Outside';
+can_ok $obj, 'exclaim';
+is $obj->exclaim, "I CAN FROM Dummy::Outside", "Inheritance is set up correctly for classes inherited from via '";
+
==== //depot/maint-5.10/perl/lib/parent/t/compile-time.t#1 (text) ====
Index: perl/lib/parent/t/compile-time.t
--- /dev/null 2008-05-07 15:08:24.549929899 -0700
+++ perl/lib/parent/t/compile-time.t 2008-05-28 13:51:40.000000000 -0700
@@ -0,0 +1,21 @@
+#!/usr/bin/perl -w
+
+use strict;
+use Test::More tests => 3;
+
+{
+ package MyParent;
+ sub exclaim { "I CAN HAS PERL?" }
+}
+
+{
+ package Child;
+ use parent -norequire, 'MyParent';
+}
+
+my $obj = {};
+bless $obj, 'Child';
+isa_ok $obj, 'MyParent', 'Inheritance';
+can_ok $obj, 'exclaim';
+is $obj->exclaim, "I CAN HAS PERL?", 'Inheritance is set up correctly';
+
==== //depot/maint-5.10/perl/lib/parent/t/lib/Dummy.pm#1 (text) ====
Index: perl/lib/parent/t/lib/Dummy.pm
--- /dev/null 2008-05-07 15:08:24.549929899 -0700
+++ perl/lib/parent/t/lib/Dummy.pm 2008-05-28 13:51:40.000000000 -0700
@@ -0,0 +1,12 @@
+package Dummy;
+
+# Attempt to emulate a bug with finding the version in Exporter.
+$VERSION = '5.562';
+
+sub exclaim { "I CAN FROM " . __PACKAGE__ }
+
+package Dummy::InlineChild;
+
+sub exclaim { "I CAN FROM " . __PACKAGE__ }
+
+1;
==== //depot/maint-5.10/perl/lib/parent/t/lib/Dummy/Outside.pm#1 (text) ====
Index: perl/lib/parent/t/lib/Dummy/Outside.pm
--- /dev/null 2008-05-07 15:08:24.549929899 -0700
+++ perl/lib/parent/t/lib/Dummy/Outside.pm 2008-05-28 13:51:40.000000000 -0700
@@ -0,0 +1,6 @@
+package Dummy::Outside;
+
+sub exclaim { "I CAN FROM " . __PACKAGE__ }
+
+1;
+
==== //depot/maint-5.10/perl/lib/parent/t/lib/Dummy2.plugin#1 (text) ====
Index: perl/lib/parent/t/lib/Dummy2.plugin
--- /dev/null 2008-05-07 15:08:24.549929899 -0700
+++ perl/lib/parent/t/lib/Dummy2.plugin 2008-05-28 13:51:40.000000000 -0700
@@ -0,0 +1,7 @@
+package Dummy2;
+sub exclaim { "I CAN FROM " . __PACKAGE__ }
+
+package Dummy2::InlineChild;
+sub exclaim { "I CAN FROM " . __PACKAGE__ }
+
+1;
==== //depot/maint-5.10/perl/lib/parent/t/lib/FileThatOnlyExistsAsPMC.pmc#1 (text) ====
Index: perl/lib/parent/t/lib/FileThatOnlyExistsAsPMC.pmc
--- /dev/null 2008-05-07 15:08:24.549929899 -0700
+++ perl/lib/parent/t/lib/FileThatOnlyExistsAsPMC.pmc 2008-05-28 13:51:40.000000000 -0700
@@ -0,0 +1,5 @@
+package FileThatOnlyExistsAsPMC;
+
+sub exclaim { "I CAN FROM " . __PACKAGE__ }
+
+1;
==== //depot/maint-5.10/perl/lib/parent/t/lib/ReturnsFalse.pm#1 (text) ====
Index: perl/lib/parent/t/lib/ReturnsFalse.pm
--- /dev/null 2008-05-07 15:08:24.549929899 -0700
+++ perl/lib/parent/t/lib/ReturnsFalse.pm 2008-05-28 13:51:40.000000000 -0700
@@ -0,0 +1,5 @@
+package ReturnsFalse;
+
+sub exclaim { "I CAN FROM " . __PACKAGE__ }
+
+0;
==== //depot/maint-5.10/perl/lib/parent/t/parent-classfromclassfile.t#1 (text) ====
Index: perl/lib/parent/t/parent-classfromclassfile.t
--- /dev/null 2008-05-07 15:08:24.549929899 -0700
+++ perl/lib/parent/t/parent-classfromclassfile.t 2008-05-28 13:51:40.000000000 -0700
@@ -0,0 +1,21 @@
+#!/usr/bin/perl -w
+
+BEGIN {
+ if( $ENV{PERL_CORE} ) {
+ chdir 't' if -d 't';
+ chdir '../lib/parent';
+ @INC = '..';
+ }
+}
+
+use strict;
+use Test::More tests => 3;
+use lib 't/lib';
+
+use_ok('parent');
+
+# Tests that a bare (non-double-colon) class still loads
+# and does not get treated as a file:
+eval q{package Test1; require Dummy; use parent -norequire, 'Dummy::InlineChild'; };
+is $@, '', "Loading an unadorned class works";
+isn't $INC{"Dummy.pm"}, undef, 'We loaded Dummy.pm';
==== //depot/maint-5.10/perl/lib/parent/t/parent-classfromfile.t#1 (text) ====
Index: perl/lib/parent/t/parent-classfromfile.t
--- /dev/null 2008-05-07 15:08:24.549929899 -0700
+++ perl/lib/parent/t/parent-classfromfile.t 2008-05-28 13:51:40.000000000 -0700
@@ -0,0 +1,25 @@
+#!/usr/bin/perl -w
+
+BEGIN {
+ if( $ENV{PERL_CORE} ) {
+ chdir 't' if -d 't';
+ chdir '../lib/parent';
+ @INC = '..';
+ }
+}
+
+use strict;
+use Test::More tests => 4;
+use lib 't/lib';
+
+use_ok('parent');
+
+my $base = './t';
+
+# Tests that a bare (non-double-colon) class still loads
+# and does not get treated as a file:
+eval sprintf q{package Test2; require '%s/lib/Dummy2.plugin'; use parent -norequire, 'Dummy2::InlineChild' }, $base;
+is $@, '', "Loading a class from a file works";
+isn't $INC{"$base/lib/Dummy2.plugin"}, undef, "We loaded the plugin file";
+my $o = bless {}, 'Test2';
+isa_ok $o, 'Dummy2::InlineChild';
==== //depot/maint-5.10/perl/lib/parent/t/parent-pmc.t#1 (text) ====
Index: perl/lib/parent/t/parent-pmc.t
--- /dev/null 2008-05-07 15:08:24.549929899 -0700
+++ perl/lib/parent/t/parent-pmc.t 2008-05-28 13:51:40.000000000 -0700
@@ -0,0 +1,32 @@
+#!/usr/bin/perl -w
+BEGIN {
+ if( $ENV{PERL_CORE} ) {
+ chdir 't' if -d 't';
+ chdir '../lib/parent';
+ @INC = '..';
+ }
+}
+
+use strict;
+use Test::More;
+use lib 't/lib';
+
+plan skip_all => ".pmc are only available with 5.6 and later" if $] < 5.006;
+plan tests => 3;
+
+use vars qw($got_here);
+
+my $res = eval q{
+ package MyTest;
+
+ use parent 'FileThatOnlyExistsAsPMC';
+
+ 1
+};
+my $error = $@;
+
+is $res, 1, "Block ran until the end";
+is $error, '', "No error";
+
+my $obj = bless {}, 'FileThatOnlyExistsAsPMC';
+can_ok $obj, 'exclaim';
==== //depot/maint-5.10/perl/lib/parent/t/parent-returns-false.t#1 (text) ====
Index: perl/lib/parent/t/parent-returns-false.t
--- /dev/null 2008-05-07 15:08:24.549929899 -0700
+++ perl/lib/parent/t/parent-returns-false.t 2008-05-28 13:51:40.000000000 -0700
@@ -0,0 +1,26 @@
+#!/usr/bin/perl -w
+BEGIN {
+ if( $ENV{PERL_CORE} ) {
+ chdir 't' if -d 't';
+ chdir '../lib/parent';
+ @INC = '..';
+ }
+}
+
+use strict;
+use Test::More tests => 2;
+use lib 't/lib';
+
+use vars qw($got_here);
+
+my $res = eval q{
+ package MyTest;
+
+ use parent 'ReturnsFalse';
+
+ $main::got_here++
+};
+my $error = $@;
+
+is $got_here, undef, "The block did not run to its end.";
+like $error, q{/^ReturnsFalse.pm did not return a true value at /}, "A module that returns a false value raises an error";
==== //depot/maint-5.10/perl/lib/parent/t/parent.t#1 (text) ====
Index: perl/lib/parent/t/parent.t
--- /dev/null 2008-05-07 15:08:24.549929899 -0700
+++ perl/lib/parent/t/parent.t 2008-05-28 13:51:40.000000000 -0700
@@ -0,0 +1,81 @@
+#!/usr/bin/perl -w
+
+BEGIN {
+ if( $ENV{PERL_CORE} ) {
+ chdir 't' if -d 't';
+ chdir '../lib/parent';
+ @INC = '..';
+ }
+}
+
+use strict;
+use Test::More tests => 10;
+
+use_ok('parent');
+
+
+package No::Version;
+
+use vars qw($Foo);
+sub VERSION { 42 }
+
+package Test::Version;
+
+use parent -norequire, 'No::Version';
+::is( $No::Version::VERSION, undef, '$VERSION gets left alone' );
+
+# Test Inverse: parent.pm should not clobber existing $VERSION
+package Has::Version;
+
+BEGIN { $Has::Version::VERSION = '42' };
+
+package Test::Version2;
+
+use parent -norequire, 'Has::Version';
+::is( $Has::Version::VERSION, 42 );
+
+package main;
+
+my $eval1 = q{
+ {
+ package Eval1;
+ {
+ package Eval2;
+ use parent -norequire, 'Eval1';
+ $Eval2::VERSION = "1.02";
+ }
+ $Eval1::VERSION = "1.01";
+ }
+};
+
+eval $eval1;
+is( $@, '' );
+
+# String comparisons, just to be safe from floating-point errors
+is( $Eval1::VERSION, '1.01' );
+
+is( $Eval2::VERSION, '1.02' );
+
+
+eval q{use parent 'reallyReAlLyNotexists'};
+like( $@, q{/^Can't locate reallyReAlLyNotexists.pm in \@INC \(\@INC contains:/}, 'baseclass that does not exist');
+
+eval q{use parent 'reallyReAlLyNotexists'};
+like( $@, q{/^Can't locate reallyReAlLyNotexists.pm in \@INC \(\@INC contains:/}, ' still failing on 2nd load');
+{
+ my $warning;
+ local $SIG{__WARN__} = sub { $warning = shift };
+ eval q{package HomoGenous; use parent 'HomoGenous';};
+ like($warning, q{/^Class 'HomoGenous' tried to inherit from itself/},
+ ' self-inheriting');
+}
+
+{
+ BEGIN { $Has::Version_0::VERSION = 0 }
+
+ package Test::Version3;
+
+ use parent -norequire, 'Has::Version_0';
+ ::is( $Has::Version_0::VERSION, 0, '$VERSION==0 preserved' );
+}
+
==== //depot/maint-5.10/perl/mad/t/p55.t#3 (text) ====
Index: perl/mad/t/p55.t
--- perl/mad/t/p55.t#2~33943~ 2008-05-28 08:54:22.000000000 -0700
+++ perl/mad/t/p55.t 2008-05-28 13:51:40.000000000 -0700
@@ -75,30 +75,10 @@
../t/op/exec.t
../t/io/say.t
-../t/io/open.t
-../t/op/gv.t
-../t/op/re.t
-../t/op/tr.t
-../t/op/die.t
-../t/op/pat.t
-../t/op/reg_namedcapture.t
-../t/op/reg_email.t
-../t/op/reg_nc_tie.t
-../t/op/utf8decode.t
../t/op/state.t
-../t/op/subst.t
-../t/op/goto.t
../t/op/tiehandle.t
-../t/op/pack.t
../t/op/each_array.t
-../t/op/sprintf.t
-../t/op/attrs.t
-../t/op/universal.t
-../t/op/regexp.t
../t/lib/cygwin.t
-../t/run/switchd.t
-../t/comp/proto.t
-../t/win32/system.t
|;
my @files;
==== //depot/maint-5.10/perl/mg.c#9 (text) ====
Index: perl/mg.c
--- perl/mg.c#8~33943~ 2008-05-28 08:54:22.000000000 -0700
+++ perl/mg.c 2008-05-28 13:51:40.000000000 -0700
@@ -3039,7 +3039,7 @@
}
/*
-=for apidoc magic_sethint
+=for apidoc magic_clearhint
Triggered by a delete from %^H, records the key to
C<PL_compiling.cop_hints_hash>.
==== //depot/maint-5.10/perl/pod/perlintern.pod#4 (text+w) ====
Index: perl/pod/perlintern.pod
--- perl/pod/perlintern.pod#3~33135~ 2008-01-30 11:50:56.000000000 -0800
+++ perl/pod/perlintern.pod 2008-05-28 13:51:40.000000000 -0700
@@ -433,12 +433,25 @@
=over 8
-=item magic_sethint
-X<magic_sethint>
+=item magic_clearhint
+X<magic_clearhint>
Triggered by a delete from %^H, records the key to
C<PL_compiling.cop_hints_hash>.
+ int magic_clearhint(SV* sv, MAGIC* mg)
+
+=for hackers
+Found in file mg.c
+
+=item magic_sethint
+X<magic_sethint>
+
+Triggered by a store to %^H, records the key/value pair to
+C<PL_compiling.cop_hints_hash>. It is assumed that hints aren't storing
+anything that would need a deep copy. Maybe we should warn if we find a
+reference.
+
int magic_sethint(SV* sv, MAGIC* mg)
=for hackers
==== //depot/maint-5.10/perl/pp.c#7 (text) ====
Index: perl/pp.c
--- perl/pp.c#6~33921~ 2008-05-24 09:32:36.000000000 -0700
+++ perl/pp.c 2008-05-28 13:51:40.000000000 -0700
@@ -3172,6 +3172,8 @@
repl = SvPV_const(repl_sv_copy, repl_len);
repl_is_utf8 = DO_UTF8(repl_sv_copy) && SvCUR(sv);
}
+ if (!SvOK(sv))
+ sv_setpvs(sv, "");
sv_insert(sv, pos, rem, repl, repl_len);
if (repl_is_utf8)
SvUTF8_on(sv);
@@ -3191,7 +3193,7 @@
else if (SvOK(sv)) /* is it defined ? */
(void)SvPOK_only_UTF8(sv);
else
- sv_setpvn(sv,"",0); /* avoid lexical reincarnation */
+ sv_setpvs(sv, ""); /* avoid lexical reincarnation */
}
if (SvTYPE(TARG) < SVt_PVLV) {
==== //depot/maint-5.10/perl/regen.pl#4 (text) ====
Index: perl/regen.pl
--- perl/regen.pl#3~33944~ 2008-05-28 11:24:46.000000000 -0700
+++ perl/regen.pl 2008-05-28 13:51:40.000000000 -0700
@@ -11,7 +11,6 @@
use strict;
my $perl = $^X;
-require 'regen_lib.pl';
# keep warnings.pl in sync with the CPAN distribution by not requiring core
# changes. Um, what ?
# safer_unlink ("warnings.h", "lib/warnings.pm");
@@ -45,10 +44,11 @@
}
foreach my $pl (keys %gen) {
- print "$^X $pl\n";
+ my @command = ($^X, $pl, @ARGV);
+ print "@command\n";
my %cksum0;
%cksum0 = do_cksum($pl) unless $pl eq 'warnings.pl'; # the files were removed
- system "$^X $pl";
+ system @command;
next if $pl eq 'warnings.pl'; # the files were removed
my %cksum1 = do_cksum($pl);
my @chg;
==== //depot/maint-5.10/perl/regen_lib.pl#3 (text) ====
Index: perl/regen_lib.pl
--- perl/regen_lib.pl#2~33944~ 2008-05-28 11:24:46.000000000 -0700
+++ perl/regen_lib.pl 2008-05-28 13:51:40.000000000 -0700
@@ -1,6 +1,6 @@
#!/usr/bin/perl -w
use strict;
-use vars qw($Is_W32 $Is_OS2 $Is_Cygwin $Is_NetWare $Needs_Write);
+use vars qw($Is_W32 $Is_OS2 $Is_Cygwin $Is_NetWare $Needs_Write $Verbose);
use Config; # Remember, this is running using an existing perl
use File::Compare;
use Symbol;
@@ -17,6 +17,8 @@
$Needs_Write = $Is_OS2 || $Is_W32 || $Is_Cygwin || $Is_NetWare;
+@ARGV = grep { not($_ eq '-v' and $Verbose = 1) } @ARGV;
+
sub safer_unlink {
my @names = @_;
my $cnt = 0;
@@ -44,7 +46,7 @@
my ($from, $to) = @_;
if (compare($from, $to) == 0) {
- warn "no changes between '$from' & '$to'\n";
+ warn "no changes between '$from' & '$to'\n" if $Verbose;
safer_unlink($from);
return;
}
==== //depot/maint-5.10/perl/t/lib/warnings/9uninit#6 (text) ====
Index: perl/t/lib/warnings/9uninit
--- perl/t/lib/warnings/9uninit#5~33944~ 2008-05-28 11:24:46.000000000 -0700
+++ perl/t/lib/warnings/9uninit 2008-05-28 13:51:40.000000000 -0700
@@ -874,7 +874,6 @@
Use of uninitialized value $m2 in substr at - line 7.
Use of uninitialized value $g1 in substr at - line 7.
Use of uninitialized value $m1 in substr at - line 7.
-Use of uninitialized value $m1 in substr at - line 7.
Use of uninitialized value $g1 in substr at - line 8.
Use of uninitialized value $m1 in substr at - line 8.
Use of uninitialized value in scalar assignment at - line 8.
==== //depot/maint-5.10/perl/t/op/reg_namedcapture.t#2 (text) ====
Index: perl/t/op/reg_namedcapture.t
--- perl/t/op/reg_namedcapture.t#1~32694~ 2007-12-22 01:23:09.000000000 -0800
+++ perl/t/op/reg_namedcapture.t 2008-05-28 13:51:40.000000000 -0700
@@ -3,9 +3,13 @@
BEGIN {
chdir 't' if -d 't';
@INC = '../lib';
+ unless ( -r "$INC[0]/Errno.pm") {
+ print "1..0 # Skip: Errno.pm not yet available\n";
+ exit 0;
+ }
}
-# WARNING: Do not use anymodules as part of this test code.
+# WARNING: Do not directly use any modules as part of this test code.
# We could get action at a distance that would invalidate the tests.
print "1..2\n";
@@ -15,6 +19,8 @@
'X'=~/(?<X>X)/;
print eval '*X{HASH}{X} || 1' ? "" :"not ","ok ",++$test,"\n";
-# And since its a similar case we check %! as well
+# And since it's a similar case we check %! as well. Note that
+# this can't be done until ../lib/Errno.pm is in place, as the
+# glob hits $!, which needs that module.
*Y = *!;
print 0<keys(%Y) ? "" :"not ","ok ",++$test,"\n";
==== //depot/maint-5.10/perl/t/op/switch.t#3 (text) ====
Index: perl/t/op/switch.t
--- perl/t/op/switch.t#2~33943~ 2008-05-28 08:54:22.000000000 -0700
+++ perl/t/op/switch.t 2008-05-28 13:51:40.000000000 -0700
@@ -802,98 +802,101 @@
is($ok2, 1, "Calling sub indirectly (false)");
}
-# Test overloading
-{ package OverloadTest;
-
- use overload '""' => sub{"string value of obj"};
-
- use overload "~~" => sub {
- my ($self, $other, $reversed) = @_;
- if ($reversed) {
- $self->{left} = $other;
- $self->{right} = $self;
- $self->{reversed} = 1;
- } else {
- $self->{left} = $self;
- $self->{right} = $other;
- $self->{reversed} = 0;
- }
- $self->{called} = 1;
- return $self->{retval};
- };
+SKIP: {
+ skip "Scalar/Util.pm not yet available", 20
+ unless -r "$INC[0]/Scalar/Util.pm";
+ # Test overloading
+ { package OverloadTest;
+
+ use overload '""' => sub{"string value of obj"};
+
+ use overload "~~" => sub {
+ my ($self, $other, $reversed) = @_;
+ if ($reversed) {
+ $self->{left} = $other;
+ $self->{right} = $self;
+ $self->{reversed} = 1;
+ } else {
+ $self->{left} = $self;
+ $self->{right} = $other;
+ $self->{reversed} = 0;
+ }
+ $self->{called} = 1;
+ return $self->{retval};
+ };
- sub new {
- my ($pkg, $retval) = @_;
- bless {
- called => 0,
- retval => $retval,
- }, $pkg;
+ sub new {
+ my ($pkg, $retval) = @_;
+ bless {
+ called => 0,
+ retval => $retval,
+ }, $pkg;
+ }
+ }
+
+ {
+ my $test = "Overloaded obj in given (true)";
+ my $obj = OverloadTest->new(1);
+ my $matched;
+ given($obj) {
+ when ("other arg") {$matched = 1}
+ default {$matched = 0}
+ }
+
+ is($obj->{called}, 1, "$test: called");
+ ok($matched, "$test: matched");
+ is($obj->{left}, "string value of obj", "$test: left");
+ is($obj->{right}, "other arg", "$test: right");
+ ok(!$obj->{reversed}, "$test: not reversed");
}
-}
-{
- my $test = "Overloaded obj in given (true)";
- my $obj = OverloadTest->new(1);
- my $matched;
- given($obj) {
- when ("other arg") {$matched = 1}
- default {$matched = 0}
- }
+ {
+ my $test = "Overloaded obj in given (false)";
+ my $obj = OverloadTest->new(0);
+ my $matched;
+ given($obj) {
+ when ("other arg") {$matched = 1}
+ }
- is($obj->{called}, 1, "$test: called");
- ok($matched, "$test: matched");
- is($obj->{left}, "string value of obj", "$test: left");
- is($obj->{right}, "other arg", "$test: right");
- ok(!$obj->{reversed}, "$test: not reversed");
-}
-
-{
- my $test = "Overloaded obj in given (false)";
- my $obj = OverloadTest->new(0);
- my $matched;
- given($obj) {
- when ("other arg") {$matched = 1}
+ is($obj->{called}, 1, "$test: called");
+ ok(!$matched, "$test: not matched");
+ is($obj->{left}, "string value of obj", "$test: left");
+ is($obj->{right}, "other arg", "$test: right");
+ ok(!$obj->{reversed}, "$test: not reversed");
}
-
- is($obj->{called}, 1, "$test: called");
- ok(!$matched, "$test: not matched");
- is($obj->{left}, "string value of obj", "$test: left");
- is($obj->{right}, "other arg", "$test: right");
- ok(!$obj->{reversed}, "$test: not reversed");
-}
-{
- my $test = "Overloaded obj in when (true)";
- my $obj = OverloadTest->new(1);
- my $matched;
- given("topic") {
- when ($obj) {$matched = 1}
- default {$matched = 0}
- }
+ {
+ my $test = "Overloaded obj in when (true)";
+ my $obj = OverloadTest->new(1);
+ my $matched;
+ given("topic") {
+ when ($obj) {$matched = 1}
+ default {$matched = 0}
+ }
- is($obj->{called}, 1, "$test: called");
- ok($matched, "$test: matched");
- is($obj->{left}, "topic", "$test: left");
- is($obj->{right}, "string value of obj", "$test: right");
- ok($obj->{reversed}, "$test: reversed");
-}
-
-{
- my $test = "Overloaded obj in when (false)";
- my $obj = OverloadTest->new(0);
- my $matched;
- given("topic") {
- when ($obj) {$matched = 1}
- default {$matched = 0}
+ is($obj->{called}, 1, "$test: called");
+ ok($matched, "$test: matched");
+ is($obj->{left}, "topic", "$test: left");
+ is($obj->{right}, "string value of obj", "$test: right");
+ ok($obj->{reversed}, "$test: reversed");
}
+
+ {
+ my $test = "Overloaded obj in when (false)";
+ my $obj = OverloadTest->new(0);
+ my $matched;
+ given("topic") {
+ when ($obj) {$matched = 1}
+ default {$matched = 0}
+ }
- is($obj->{called}, 1, "$test: called");
- ok(!$matched, "$test: not matched");
- is($obj->{left}, "topic", "$test: left");
- is($obj->{right}, "string value of obj", "$test: right");
- ok($obj->{reversed}, "$test: reversed");
+ is($obj->{called}, 1, "$test: called");
+ ok(!$matched, "$test: not matched");
+ is($obj->{left}, "topic", "$test: left");
+ is($obj->{right}, "string value of obj", "$test: right");
+ ok($obj->{reversed}, "$test: reversed");
+ }
}
-
# Okay, that'll do for now. The intricacies of the smartmatch
# semantics are tested in t/op/smartmatch.t
__END__
End of Patch.
-
Change 33945: Integrate:
by Dave Mitchell