X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2Fbase.pm;h=d8baa95d0a1063f4ca5ea69d44da78030ce1c5fe;hb=99ffb1cb9ad68572b442d79446caa8bcbb2964fc;hp=832b6a4a9a1610549547be1b5a32cb0b1e68f548;hpb=446e776fb7618d56ecd9043c4060e08967abe22c;p=p5sagit%2Fp5-mst-13.2.git diff --git a/lib/base.pm b/lib/base.pm index 832b6a4..d8baa95 100644 --- a/lib/base.pm +++ b/lib/base.pm @@ -2,7 +2,7 @@ package base; use strict 'vars'; use vars qw($VERSION); -$VERSION = '2.06'; +$VERSION = '2.08'; # constant.pm is slow sub SUCCESS () { 1 } @@ -71,6 +71,10 @@ sub import { my $inheritor = caller(0); foreach my $base (@_) { + if ( $inheritor eq $base ) { + warn "Class '$inheritor' tried to inherit from itself\n"; + } + next if $inheritor->isa($base); if (has_version($base)) { @@ -78,7 +82,7 @@ sub import { unless defined ${$base.'::VERSION'}; } else { - local $SIG{__DIE__} = 'IGNORE'; + local $SIG{__DIE__}; eval "require $base"; # Only ignore "Can't locate" errors from our eval require. # Other fatal errors (syntax etc) must be reported. @@ -97,7 +101,7 @@ ERROR push @{"$inheritor\::ISA"}, $base; if ( has_fields($base) || has_attr($base) ) { - # No multiple fields inheritence *suck* + # No multiple fields inheritance *suck* if ($fields_base) { require Carp; Carp::croak("Can't multiply inherit %FIELDS"); @@ -192,7 +196,7 @@ it, will define $VERSION in the base package, setting it to the string C<-1, set by base.pm>. Will also initialize the fields if one of the base classes has it. -Multiple inheritence of fields is B supported, if two or more +Multiple inheritance of fields is B supported, if two or more base classes each have inheritable fields the 'base' pragma will croak. See L, L and L for a description of this feature. @@ -212,6 +216,12 @@ found in your path. This module was introduced with Perl 5.004_04. +Attempting to inherit from yourself generates a warning: + + use Foo; + use base 'Foo'; + + # Class 'Foo' tried to inherit from itself =head1 CAVEATS