X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FMoose%2FObject.pm;h=387f248fceb8602b28265babbd03d78bea32c351;hb=55d05fb16429c2150b5337b1a0130ae334e129d0;hp=bef65068e417b136e225ba9270637a8f6679de0e;hpb=56d7c7453888ed2c684bcdb1d905841d8d03aeb1;p=gitmo%2FMoose.git diff --git a/lib/Moose/Object.pm b/lib/Moose/Object.pm index bef6506..387f248 100644 --- a/lib/Moose/Object.pm +++ b/lib/Moose/Object.pm @@ -4,26 +4,32 @@ package Moose::Object; use strict; use warnings; +use Devel::GlobalDestruction (); +use MRO::Compat (); +use Scalar::Util (); +use Try::Tiny (); + use if ( not our $__mx_is_compiled ), 'Moose::Meta::Class'; use if ( not our $__mx_is_compiled ), metaclass => 'Moose::Meta::Class'; -our $VERSION = '0.73'; +our $VERSION = '1.12'; $VERSION = eval $VERSION; our $AUTHORITY = 'cpan:STEVAN'; sub new { my $class = shift; - my $params = $class->BUILDARGS(@_); - my $self = $class->meta->new_object($params); - $self->BUILDALL($params); - return $self; + my $real_class = Scalar::Util::blessed($class) || $class; + + my $params = $real_class->BUILDARGS(@_); + + return Class::MOP::Class->initialize($real_class)->new_object($params); } sub BUILDARGS { my $class = shift; if ( scalar @_ == 1 ) { unless ( defined $_[0] && ref $_[0] eq 'HASH' ) { - $class->meta->throw_error( + Class::MOP::class_of($class)->throw_error( "Single parameters to new() must be a HASH ref", data => $_[0] ); } @@ -35,39 +41,61 @@ sub BUILDARGS { } sub BUILDALL { - # NOTE: we ask Perl if we even + # NOTE: we ask Perl if we even # need to do this first, to avoid # extra meta level calls - return unless $_[0]->can('BUILD'); + return unless $_[0]->can('BUILD'); my ($self, $params) = @_; - foreach my $method (reverse $self->meta->find_all_methods_by_name('BUILD')) { + foreach my $method (reverse Class::MOP::class_of($self)->find_all_methods_by_name('BUILD')) { $method->{code}->execute($self, $params); } } sub DEMOLISHALL { - my $self = shift; - # NOTE: we ask Perl if we even + my $self = shift; + my ($in_global_destruction) = @_; + + # NOTE: we ask Perl if we even # need to do this first, to avoid - # extra meta level calls + # extra meta level calls return unless $self->can('DEMOLISH'); - foreach my $method ($self->meta->find_all_methods_by_name('DEMOLISH')) { - $method->{code}->execute($self); + + my @isa; + if ( my $meta = Class::MOP::class_of($self ) ) { + @isa = $meta->linearized_isa; + } else { + # We cannot count on being able to retrieve a previously made + # metaclass, _or_ being able to make a new one during global + # destruction. However, we should still be able to use mro at + # that time (at least tests suggest so ;) + my $class_name = ref $self; + @isa = @{ mro::get_linear_isa($class_name) } + } + + foreach my $class (@isa) { + no strict 'refs'; + my $demolish = *{"${class}::DEMOLISH"}{CODE}; + $self->$demolish($in_global_destruction) + if defined $demolish; } } -sub DESTROY { - # if we have an exception here ... - if ($@) { - # localize the $@ ... - local $@; - # run DEMOLISHALL ourselves, ... - $_[0]->DEMOLISHALL; - # and return ... - return; +sub DESTROY { + my $self = shift; + + local $?; + + Try::Tiny::try { + $self->DEMOLISHALL(Devel::GlobalDestruction::in_global_destruction); } - # otherwise it is normal destruction - $_[0]->DEMOLISHALL; + Try::Tiny::catch { + # Without this, Perl will warn "\t(in cleanup)$@" because of some + # bizarre fucked-up logic deep in the internals. + no warnings 'misc'; + die $_; + }; + + return; } # support for UNIVERSAL::DOES ... @@ -80,22 +108,18 @@ BEGIN { }'; } -# new does() methods will be created +# new does() methods will be created # as appropiate see Moose::Meta::Role sub does { my ($self, $role_name) = @_; - my $meta = $self->meta; + my $meta = Class::MOP::class_of($self); (defined $role_name) - || $meta->throw_error("You much supply a role name to does()"); - foreach my $class ($meta->class_precedence_list) { - my $m = $meta->initialize($class); - return 1 - if $m->can('does_role') && $m->does_role($role_name); - } - return 0; + || $meta->throw_error("You must supply a role name to does()"); + return 1 if $meta->can('does_role') && $meta->does_role($role_name); + return 0; } -sub dump { +sub dump { my $self = shift; require Data::Dumper; local $Data::Dumper::Maxdepth = shift if @_; @@ -158,7 +182,8 @@ C. This will call every C method in the inheritance hierarchy, starting with the object's class and ending with the most distant -parent. +parent. C and C will receive a boolean +indicating whether or not we are currently in global destruction. =item B<< $object->does($role_name) >> @@ -184,9 +209,7 @@ the maximum depth is 1, to avoid making a mess. =head1 BUGS -All complex software has bugs lurking in it, and this module is no -exception. If you find a bug please either email me, or add the bug -to cpan-RT. +See L for details on reporting bugs. =head1 AUTHOR @@ -194,11 +217,11 @@ Stevan Little Estevan@iinteractive.comE =head1 COPYRIGHT AND LICENSE -Copyright 2006-2009 by Infinity Interactive, Inc. +Copyright 2006-2010 by Infinity Interactive, Inc. L This library is free software; you can redistribute it and/or modify -it under the same terms as Perl itself. +it under the same terms as Perl itself. =cut