use Sub::Name when installing attr methods into a class, so that they appear to be...

[gitmo/Moose.git] / lib / Moose / Object.pm

diff --git a/lib/Moose/Object.pm b/lib/Moose/Object.pm
index 56fea9f..6b723dd 100644 (file)
--- a/lib/Moose/Object.pm
+++ b/lib/Moose/Object.pm
@@ -9,38 +9,47 @@ use metaclass 'Moose::Meta::Class';
 
 use Carp 'confess';
 
-our $VERSION = '0.07';
+our $VERSION   = '0.09';
+our $AUTHORITY = 'cpan:STEVAN';
 
 sub new {
     my $class = shift;
     my %params;
     if (scalar @_ == 1) {
-        (ref($_[0]) eq 'HASH')
-            || confess "Single parameters to new() must be a HASH ref";
-        %params = %{$_[0]};
+        if (defined $_[0]) {
+            (ref($_[0]) eq 'HASH')
+                || confess "Single parameters to new() must be a HASH ref";
+            %params = %{$_[0]};
+        }
     }
     else {
         %params = @_;
     }
-       my $self = $class->meta->new_object(%params);
-       $self->BUILDALL(\%params);
-       return $self;
+    my $self = $class->meta->new_object(%params);
+    $self->BUILDALL(\%params);
+    return $self;
 }
 
 sub BUILDALL {
-       return unless $_[0]->can('BUILD');    
-       my ($self, $params) = @_;
-       foreach my $method (reverse $self->meta->find_all_methods_by_name('BUILD')) {
-               $method->{code}->($self, $params);
-       }
+    # NOTE: we ask Perl if we even 
+    # need to do this first, to avoid
+    # extra meta level calls
+    return unless $_[0]->can('BUILD');    
+    my ($self, $params) = @_;
+    foreach my $method (reverse $self->meta->find_all_methods_by_name('BUILD')) {
+        $method->{code}->($self, $params);
+    }
 }
 
 sub DEMOLISHALL {
-       return unless $_[0]->can('DEMOLISH');    
-       my $self = shift;       
-       foreach my $method ($self->meta->find_all_methods_by_name('DEMOLISH')) {
-               $method->{code}->($self);
-       }       
+    # NOTE: we ask Perl if we even 
+    # need to do this first, to avoid
+    # extra meta level calls    
+    return unless $_[0]->can('DEMOLISH');    
+    my $self = shift;    
+    foreach my $method ($self->meta->find_all_methods_by_name('DEMOLISH')) {
+        $method->{code}->($self);
+    }    
 }
 
 sub DESTROY { goto &DEMOLISHALL }
@@ -145,7 +154,7 @@ Stevan Little E<lt>stevan@iinteractive.comE<gt>
 
 =head1 COPYRIGHT AND LICENSE
 
-Copyright 2006 by Infinity Interactive, Inc.
+Copyright 2006-2008 by Infinity Interactive, Inc.
 
 L<http://www.iinteractive.com>