X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FMoose%2FMeta%2FMethod%2FDestructor.pm;h=ee10bdc9d6b28b41147cf8ab7f93cbfa227d2452;hb=3eb89f709f04907580b508f821d6be2316fcb65f;hp=d0a4a7662af511ab7fb7fb1ae22cbe227983bd22;hpb=e3d94ac0990080399851812a76928c893c3cd4fa;p=gitmo%2FMoose.git diff --git a/lib/Moose/Meta/Method/Destructor.pm b/lib/Moose/Meta/Method/Destructor.pm index d0a4a76..ee10bdc 100644 --- a/lib/Moose/Meta/Method/Destructor.pm +++ b/lib/Moose/Meta/Method/Destructor.pm @@ -4,9 +4,11 @@ package Moose::Meta::Method::Destructor; use strict; use warnings; +use Devel::GlobalDestruction (); use Scalar::Util 'blessed', 'weaken'; +use Try::Tiny (); -our $VERSION = '0.82'; +our $VERSION = '1.01'; $VERSION = eval $VERSION; our $AUTHORITY = 'cpan:STEVAN'; @@ -79,21 +81,31 @@ sub _initialize_body { my @DEMOLISH_methods = $self->associated_metaclass->find_all_methods_by_name('DEMOLISH'); my $source; + $source = 'sub {' . "\n"; + $source .= 'my $self = shift;' . "\n"; + $source .= 'return $self->Moose::Object::DESTROY(@_)' . "\n"; + $source .= ' if Scalar::Util::blessed($self) ne '; + $source .= "'" . $self->associated_metaclass->name . "'"; + $source .= ';' . "\n"; + if ( @DEMOLISH_methods ) { - $source = 'sub {'; + $source .= 'local $?;' . "\n"; + + $source .= 'my $in_global_destruction = Devel::GlobalDestruction::in_global_destruction;' . "\n"; + + $source .= 'Try::Tiny::try {' . "\n"; - my @DEMOLISH_calls; - foreach my $method (@DEMOLISH_methods) { - push @DEMOLISH_calls => '$_[0]->' . $method->{class} . '::DEMOLISH()'; - } + $source .= '$self->' . $_->{class} . '::DEMOLISH($in_global_destruction);' . "\n" + for @DEMOLISH_methods; - $source .= join ";\n" => @DEMOLISH_calls; + $source .= '}'; + $source .= q[ Try::Tiny::catch { no warnings 'misc'; die $_ };] . "\n"; + $source .= 'return;' . "\n"; - $source .= ";\n" . '}'; - } else { - $source = 'sub { }'; } + $source .= '}'; + warn $source if $self->options->{debug}; my ( $code, $e ) = $self->_compile_code( @@ -168,13 +180,17 @@ of its parents defines a C method, it needs a destructor. =back +=head1 BUGS + +See L for details on reporting bugs. + =head1 AUTHORS Stevan Little Estevan@iinteractive.comE =head1 COPYRIGHT AND LICENSE -Copyright 2006-2009 by Infinity Interactive, Inc. +Copyright 2006-2010 by Infinity Interactive, Inc. L