X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FClass%2FMOP%2FMethod%2FConstructor.pm;h=56f38c5c0db4557b43c3f55480a347e26a6fb870;hb=777b1f18dd34d5d656f15d14ce9836cfc9c2978f;hp=7f0fe514a7bb5f5a04f9e9311f0e97cbb29616fe;hpb=da1bebb6a46a770cee8fe206b15998ba8db84e51;p=gitmo%2FClass-MOP.git diff --git a/lib/Class/MOP/Method/Constructor.pm b/lib/Class/MOP/Method/Constructor.pm index 7f0fe51..56f38c5 100644 --- a/lib/Class/MOP/Method/Constructor.pm +++ b/lib/Class/MOP/Method/Constructor.pm @@ -5,13 +5,14 @@ use strict; use warnings; use Carp 'confess'; -use Scalar::Util 'blessed', 'weaken', 'looks_like_number'; +use Scalar::Util 'blessed', 'weaken'; +use Try::Tiny; -our $VERSION = '0.78_02'; +our $VERSION = '1.11'; $VERSION = eval $VERSION; our $AUTHORITY = 'cpan:STEVAN'; -use base 'Class::MOP::Method::Generated'; +use base 'Class::MOP::Method::Inlined'; sub new { my $class = shift; @@ -38,22 +39,33 @@ sub new { sub _new { my $class = shift; - my $options = @_ == 1 ? $_[0] : {@_}; - - bless { - # from our superclass - 'body' => undef, - 'package_name' => $options->{package_name}, - 'name' => $options->{name}, - # specific to this subclass - 'options' => $options->{options} || {}, - 'associated_metaclass' => $options->{metaclass}, - 'is_inline' => ($options->{is_inline} || 0), + + return Class::MOP::Class->initialize($class)->new_object(@_) + if $class ne __PACKAGE__; + + my $params = @_ == 1 ? $_[0] : {@_}; + + return bless { + # inherited from Class::MOP::Method + body => $params->{body}, + # associated_metaclass => $params->{associated_metaclass}, # overriden + package_name => $params->{package_name}, + name => $params->{name}, + original_method => $params->{original_method}, + + # inherited from Class::MOP::Generated + is_inline => $params->{is_inline} || 0, + definition_context => $params->{definition_context}, + + # inherited from Class::MOP::Inlined + _expected_method_class => $params->{_expected_method_class}, + + # defined in this subclass + options => $params->{options} || {}, + associated_metaclass => $params->{metaclass}, }, $class; } -sub can_be_inlined { 1 } - ## accessors sub options { (shift)->{'options'} } @@ -61,33 +73,13 @@ sub associated_metaclass { (shift)->{'associated_metaclass'} } ## cached values ... -sub meta_instance { - warn 'The meta_instance method has been made private.' - . " The public version is deprecated and will be removed in a future release.\n"; - shift->_meta_instance; -} - -sub _meta_instance { - my $self = shift; - $self->{'meta_instance'} ||= $self->associated_metaclass->get_meta_instance; -} - -sub attributes { - warn 'The attributes method is deprecated.' - . " Use ->associated_metaclass->compute_all_applicable_attributes instead.\n"; - +sub _attributes { my $self = shift; - $self->{'attributes'} ||= [ $self->associated_metaclass->compute_all_applicable_attributes ] + $self->{'attributes'} ||= [ $self->associated_metaclass->get_all_attributes ] } ## method -sub initialize_body { - warn 'The initialize_body method has been made private.' - . " The public version is deprecated and will be removed in a future release.\n"; - shift->_initialize_body; -} - sub _initialize_body { my $self = shift; my $method_name = '_generate_constructor_method'; @@ -97,58 +89,101 @@ sub _initialize_body { $self->{'body'} = $self->$method_name; } -sub generate_constructor_method { - warn 'The generate_constructor_method method has been made private.' - . " The public version is deprecated and will be removed in a future release.\n"; - shift->_generate_constructor_method; -} - sub _generate_constructor_method { return sub { Class::MOP::Class->initialize(shift)->new_object(@_) } } -sub generate_constructor_method_inline { - warn 'The generate_constructor_method_inline method has been made private.' - . " The public version is deprecated and will be removed in a future release.\n"; - shift->_generate_constructor_method_inline; -} - sub _generate_constructor_method_inline { my $self = shift; - my $close_over = {}; - - my $source = 'sub {'; - $source .= "\n" . 'my $class = shift;'; - - $source .= "\n" . 'return Class::MOP::Class->initialize($class)->new_object(@_)'; - $source .= "\n" . ' if $class ne \'' . $self->associated_metaclass->name . '\';'; - - $source .= "\n" . 'my $params = @_ == 1 ? $_[0] : {@_};'; + my $defaults = [map { $_->default } @{ $self->_attributes }]; + my $close_over = { + '$defaults' => \$defaults, + }; + + my $meta = $self->associated_metaclass; + + my $idx = 0; + my @source = ( + 'sub {', + 'my $class = shift;', + 'return Class::MOP::Class->initialize($class)->new_object(@_)', + 'if $class ne \'' . $meta->name . '\';', + 'my $params = @_ == 1 ? $_[0] : {@_};', + 'my $instance = ' . $meta->inline_create_instance('$class') . ';', + (map { $self->_generate_slot_initializer($_, $idx++) } + @{ $self->_attributes }), + $self->_preserve_weak_metaclasses, + 'return $instance', + '}', + ); - $source .= "\n" . 'my $instance = ' . $self->_meta_instance->inline_create_instance('$class'); - $source .= ";\n" . (join ";\n" => map { - $self->_generate_slot_initializer($_, $close_over) - } $self->associated_metaclass->compute_all_applicable_attributes); - $source .= ";\n" . 'return $instance'; - $source .= ";\n" . '}'; - warn $source if $self->options->{debug}; + warn join("\n", @source) if $self->options->{debug}; - my $code = $self->_eval_closure( - $close_over, - $source - ); - confess "Could not eval the constructor :\n\n$source\n\nbecause :\n\n$@" if $@; + my $code = try { + $self->_compile_code( + source => \@source, + environment => $close_over, + ); + } + catch { + my $source = join("\n", @source); + confess "Could not eval the constructor :\n\n$source\n\nbecause :\n\n$_"; + }; return $code; } sub _generate_slot_initializer { my $self = shift; - my $attr = shift; - my $close = shift; + my ($attr, $idx) = @_; + + my $default = $self->_generate_default_value($attr, $idx); + + if (defined(my $init_arg = $attr->init_arg)) { + my @source = ( + 'if (exists $params->{\'' . $init_arg . '\'}) {', + $attr->inline_set( + '$instance', '$params->{\'' . $init_arg . '\'}' + ) . ';', + '}', + ); + if (defined $default) { + push @source, ( + 'else {', + $attr->inline_set('$instance', $default) . ';', + '}', + ); + } + return @source; + } + elsif (defined $default) { + return ($attr->inline_set('$instance', $default) . ';'); + } + else { + return (); + } +} + +sub _preserve_weak_metaclasses { + my $self = shift; + my $meta = $self->associated_metaclass; + if (Class::MOP::metaclass_is_weak($meta->name)) { + return ( + $meta->_inline_set_mop_slot( + '$instance', 'Class::MOP::class_of($class)' + ) . ';' + ); + } + else { + return (); + } +} + +sub _generate_default_value { + my $self = shift; + my ($attr, $index) = @_; - my $default; if ($attr->has_default) { # NOTE: # default values can either be CODE refs @@ -157,43 +192,18 @@ sub _generate_slot_initializer { # in which case we can just deal with them # in the code we eval. if ($attr->is_default_a_coderef) { - my $idx = @{$close->{'@defaults'}||=[]}; - push(@{$close->{'@defaults'}}, $attr->default); - $default = '$defaults[' . $idx . ']->($instance)'; + return '$defaults->[' . $index . ']->($instance)'; } else { - $default = $attr->default; - # make sure to quote strings ... - unless (looks_like_number($default)) { - $default = "'$default'"; - } + return '$defaults->[' . $index . ']'; } - } elsif( $attr->has_builder ) { - $default = '$instance->'.$attr->builder; } - - if ( defined $attr->init_arg ) { - return ( - 'if(exists $params->{\'' . $attr->init_arg . '\'}){' . "\n" . - $self->_meta_instance->inline_set_slot_value( - '$instance', - $attr->name, - '$params->{\'' . $attr->init_arg . '\'}' ) . "\n" . - '} ' . (!defined $default ? '' : 'else {' . "\n" . - $self->_meta_instance->inline_set_slot_value( - '$instance', - $attr->name, - $default ) . "\n" . - '}') - ); - } elsif ( defined $default ) { - return ( - $self->_meta_instance->inline_set_slot_value( - '$instance', - $attr->name, - $default ) . "\n" - ); - } else { return '' } + elsif ($attr->has_builder) { + return '$instance->' . $attr->builder; + } + else { + return; + } } 1; @@ -264,12 +274,6 @@ inlined. This returns the L object for the method. -=item B<< $metamethod->can_be_inlined >> - -This method always returns true in this class. It exists so that -subclasses (as in Moose) can do some sort of checking to determine -whether or not inlining the constructor is safe. - =back =head1 AUTHORS @@ -278,7 +282,7 @@ Stevan Little Estevan@iinteractive.comE =head1 COPYRIGHT AND LICENSE -Copyright 2006-2009 by Infinity Interactive, Inc. +Copyright 2006-2010 by Infinity Interactive, Inc. L