X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FMoose%2FMeta%2FMethod%2FConstructor.pm;h=2aec2614d93624cf700c75e2bd086599659bf371;hb=a125746004b00e978de096e9de12d1cbdaf8eb35;hp=caefb14490c5329f8d481b14538a2f32e63a88d3;hpb=849811463e6a7531a72f545a9a78902724b4085e;p=gitmo%2FMoose.git diff --git a/lib/Moose/Meta/Method/Constructor.pm b/lib/Moose/Meta/Method/Constructor.pm index caefb14..2aec261 100644 --- a/lib/Moose/Meta/Method/Constructor.pm +++ b/lib/Moose/Meta/Method/Constructor.pm @@ -4,10 +4,9 @@ package Moose::Meta::Method::Constructor; use strict; use warnings; -use Carp 'confess'; use Scalar::Util 'blessed', 'weaken', 'looks_like_number'; -our $VERSION = '0.05'; +our $VERSION = '0.62_02'; our $AUTHORITY = 'cpan:STEVAN'; use base 'Moose::Meta::Method', @@ -17,41 +16,110 @@ sub new { my $class = shift; my %options = @_; - (exists $options{options} && ref $options{options} eq 'HASH') - || confess "You must pass a hash of options"; + my $meta = $options{metaclass}; + + (ref $options{options} eq 'HASH') + || $class->throw_error("You must pass a hash of options", data => $options{options}); + + ($options{package_name} && $options{name}) + || $class->throw_error("You must supply the package_name and name parameters $Class::MOP::Method::UPGRADE_ERROR_TEXT"); my $self = bless { # from our superclass - '&!body' => undef, + 'body' => undef, + 'package_name' => $options{package_name}, + 'name' => $options{name}, # specific to this subclass - '%!options' => $options{options}, - '$!meta_instance' => $options{metaclass}->get_meta_instance, - '@!attributes' => [ $options{metaclass}->compute_all_applicable_attributes ], + 'options' => $options{options}, + 'meta_instance' => $meta->get_meta_instance, + 'attributes' => [ $meta->compute_all_applicable_attributes ], # ... - '$!associated_metaclass' => $options{metaclass}, + 'associated_metaclass' => $meta, } => $class; # we don't want this creating # a cycle in the code, if not # needed - weaken($self->{'$!associated_metaclass'}); + weaken($self->{'associated_metaclass'}); - $self->intialize_body; + $self->initialize_body; return $self; } +sub can_be_inlined { + my $self = shift; + my $metaclass = $self->associated_metaclass; + + my $expected_class = $self->_expected_constructor_class; + + # If any of our parents have been made immutable, we are okay to + # inline our own new method. The assumption is that an inlined new + # method provided by a parent does not actually get used by + # children anyway. + for my $meta ( + grep { $_->is_immutable } + map { ( ref $metaclass )->initialize($_) } + grep { $_ ne $expected_class } + $metaclass->linearized_isa + ) { + my $transformer = $meta->get_immutable_transformer; + + # This is actually a false positive if we're in a subclass of + # this class, _and_ the expected class is not overridden (but + # should be), and the real expected class is actually + # immutable itself (see Fey::Object::Table for an example of + # how this can happen). I'm not sure how to actually handle + # that case, since it's effectively a bug in the subclass (for + # not overriding _expected_constructor_class). + return 1 if $transformer->inlined_constructor; + } + + if ( my $constructor = $metaclass->find_method_by_name( $self->name ) ) { + my $class = $self->associated_metaclass->name; + + if ( $constructor->body != $expected_class->can('new') ) { + my $warning + = "Not inlining a constructor for $class since it is not" + . " inheriting the default $expected_class constructor\n"; + + $warning .= " (constructor has method modifiers which would be lost if it were inlined)\n" + if $constructor->isa('Class::MOP::Method::Wrapped'); + + warn $warning; + + return 0; + } + else { + return 1; + } + } + + # This would be a rather weird case where we have no constructor + # in the inheritance chain. + return 1; +} + +# This is here so can_be_inlined can be inherited by MooseX modules. +sub _expected_constructor_class { + return 'Moose::Object'; +} + ## accessors -sub options { (shift)->{'%!options'} } -sub meta_instance { (shift)->{'$!meta_instance'} } -sub attributes { (shift)->{'@!attributes'} } +sub options { (shift)->{'options'} } +sub meta_instance { (shift)->{'meta_instance'} } +sub attributes { (shift)->{'attributes'} } -sub associated_metaclass { (shift)->{'$!associated_metaclass'} } +sub associated_metaclass { (shift)->{'associated_metaclass'} } ## method -sub intialize_body { +# this was changed in 0.41, but broke MooseX::Singleton, so try to catch +# any other code using the original broken spelling +sub intialize_body { Moose->throw_error("Please correct the spelling of 'intialize_body' to 'initialize_body'") } + +sub initialize_body { my $self = shift; # TODO: # the %options should also include a both @@ -66,14 +134,15 @@ sub intialize_body { $source .= "\n" . 'return $class->Moose::Object::new(@_)'; $source .= "\n" . ' if $class ne \'' . $self->associated_metaclass->name . '\';'; - $source .= "\n" . 'my %params = (scalar @_ == 1) ? %{$_[0]} : @_;'; + $source .= "\n" . 'my $params = ' . $self->_generate_BUILDARGS('$class', '@_'); - $source .= "\n" . 'my $instance = ' . $self->meta_instance->inline_create_instance('$class'); + $source .= ";\n" . 'my $instance = ' . $self->meta_instance->inline_create_instance('$class'); $source .= ";\n" . (join ";\n" => map { $self->_generate_slot_initializer($_) } 0 .. (@{$self->attributes} - 1)); + $source .= ";\n" . $self->_generate_triggers(); $source .= ";\n" . $self->_generate_BUILDALL(); $source .= ";\n" . 'return $instance'; @@ -82,6 +151,8 @@ sub intialize_body { my $code; { + my $meta = $self; # FIXME for _inline_throw_error... + # NOTE: # create the nessecary lexicals # to be picked up in the eval @@ -105,20 +176,64 @@ sub intialize_body { } @type_constraints; $code = eval $source; - confess "Could not eval the constructor :\n\n$source\n\nbecause :\n\n$@" if $@; + $self->throw_error("Could not eval the constructor :\n\n$source\n\nbecause :\n\n$@", error => $@, data => $source ) if $@; + } + $self->{'body'} = $code; +} + +sub _generate_BUILDARGS { + my ( $self, $class, $args ) = @_; + + my $buildargs = $self->associated_metaclass->find_method_by_name("BUILDARGS"); + + if ( $args eq '@_' and ( !$buildargs or $buildargs->body == \&Moose::Object::BUILDARGS ) ) { + return join("\n", + 'do {', + $self->_inline_throw_error('"Single parameters to new() must be a HASH ref"', 'data => $_[0]'), + ' if scalar @_ == 1 && defined $_[0] && ref($_[0]) ne q{HASH};', + '(scalar @_ == 1) ? {%{$_[0]}} : {@_};', + '}', + ); + } else { + return $class . "->BUILDARGS($args)"; } - $self->{'&!body'} = $code; } sub _generate_BUILDALL { my $self = shift; my @BUILD_calls; foreach my $method (reverse $self->associated_metaclass->find_all_methods_by_name('BUILD')) { - push @BUILD_calls => '$instance->' . $method->{class} . '::BUILD(\%params)'; + push @BUILD_calls => '$instance->' . $method->{class} . '::BUILD($params)'; } return join ";\n" => @BUILD_calls; } +sub _generate_triggers { + my $self = shift; + my @trigger_calls; + foreach my $i (0 .. $#{ $self->attributes }) { + my $attr = $self->attributes->[$i]; + if ($attr->can('has_trigger') && $attr->has_trigger) { + if (defined(my $init_arg = $attr->init_arg)) { + push @trigger_calls => ( + '(exists $params->{\'' . $init_arg . '\'}) && do {' . "\n " + . '$attrs->[' . $i . ']->trigger->(' + . '$instance, ' + . $self->meta_instance->inline_get_slot_value( + '$instance', + ("'" . $attr->name . "'") + ) + . ', ' + . '$attrs->[' . $i . ']' + . ');' + ."\n}" + ); + } + } + } + return join ";\n" => @trigger_calls; +} + sub _generate_slot_initializer { my $self = shift; my $index = shift; @@ -130,28 +245,20 @@ sub _generate_slot_initializer { my $is_moose = $attr->isa('Moose::Meta::Attribute'); # XXX FIXME if ($is_moose && defined($attr->init_arg) && $attr->is_required && !$attr->has_default && !$attr->has_builder) { - push @source => ('(exists $params{\'' . $attr->init_arg . '\'}) ' . - '|| confess "Attribute (' . $attr->name . ') is required";'); + push @source => ('(exists $params->{\'' . $attr->init_arg . '\'}) ' . + '|| ' . $self->_inline_throw_error('"Attribute (' . $attr->name . ') is required"') .';'); } if (($attr->has_default || $attr->has_builder) && !($is_moose && $attr->is_lazy)) { if ( defined( my $init_arg = $attr->init_arg ) ) { - push @source => 'if (exists $params{\'' . $init_arg . '\'}) {'; - - push @source => ('my $val = $params{\'' . $init_arg . '\'};'); - - if ($is_moose && $attr->has_type_constraint) { - if ($attr->should_coerce && $attr->type_constraint->has_coercion) { - push @source => $self->_generate_type_coercion($attr, '$type_constraints[' . $index . ']', '$val', '$val'); - } - push @source => $self->_generate_type_constraint_check($attr, '$type_constraint_bodies[' . $index . ']', '$val'); - } - push @source => $self->_generate_slot_assignment($attr, '$val'); - + push @source => 'if (exists $params->{\'' . $init_arg . '\'}) {'; + push @source => ('my $val = $params->{\'' . $init_arg . '\'};'); + push @source => $self->_generate_type_constraint_and_coercion($attr, $index) + if $is_moose; + push @source => $self->_generate_slot_assignment($attr, '$val', $index); push @source => "} else {"; } - my $default; if ( $attr->has_default ) { $default = $self->_generate_default_value($attr, $index); @@ -160,27 +267,37 @@ sub _generate_slot_initializer { my $builder = $attr->builder; $default = '$instance->' . $builder; } + + push @source => '{'; # wrap this to avoid my $val overwrite warnings push @source => ('my $val = ' . $default . ';'); - push @source => $self->_generate_type_constraint_check( - $attr, - ('$type_constraint_bodies[' . $index . ']'), - '$val' - ) if ($is_moose && $attr->has_type_constraint); - push @source => $self->_generate_slot_assignment($attr, $default); + push @source => $self->_generate_type_constraint_and_coercion($attr, $index) + if $is_moose; + push @source => $self->_generate_slot_assignment($attr, '$val', $index); + push @source => '}'; # close - wrap this to avoid my $val overrite warnings push @source => "}" if defined $attr->init_arg; } elsif ( defined( my $init_arg = $attr->init_arg ) ) { - push @source => '(exists $params{\'' . $init_arg . '\'}) && do {'; + push @source => '(exists $params->{\'' . $init_arg . '\'}) && do {'; - push @source => ('my $val = $params{\'' . $init_arg . '\'};'); + push @source => ('my $val = $params->{\'' . $init_arg . '\'};'); if ($is_moose && $attr->has_type_constraint) { if ($attr->should_coerce && $attr->type_constraint->has_coercion) { - push @source => $self->_generate_type_coercion($attr, '$type_constraints[' . $index . ']', '$val', '$val'); + push @source => $self->_generate_type_coercion( + $attr, + '$type_constraints[' . $index . ']', + '$val', + '$val' + ); } - push @source => $self->_generate_type_constraint_check($attr, '$type_constraint_bodies[' . $index . ']', '$val'); + push @source => $self->_generate_type_constraint_check( + $attr, + '$type_constraint_bodies[' . $index . ']', + '$type_constraints[' . $index . ']', + '$val' + ); } - push @source => $self->_generate_slot_assignment($attr, '$val'); + push @source => $self->_generate_slot_assignment($attr, '$val', $index); push @source => "}"; } @@ -189,16 +306,26 @@ sub _generate_slot_initializer { } sub _generate_slot_assignment { - my ($self, $attr, $value) = @_; - my $source = ( - $self->meta_instance->inline_set_slot_value( - '$instance', - ("'" . $attr->name . "'"), - $value - ) . ';' - ); - - my $is_moose = $attr->isa('Moose::Meta::Attribute'); # XXX FIXME + my ($self, $attr, $value, $index) = @_; + + my $source; + + if ($attr->has_initializer) { + $source = ( + '$attrs->[' . $index . ']->set_initial_value($instance, ' . $value . ');' + ); + } + else { + $source = ( + $self->meta_instance->inline_set_slot_value( + '$instance', + ("'" . $attr->name . "'"), + $value + ) . ';' + ); + } + + my $is_moose = $attr->isa('Moose::Meta::Attribute'); # XXX FIXME if ($is_moose && $attr->is_weak_ref) { $source .= ( @@ -214,18 +341,42 @@ sub _generate_slot_assignment { return $source; } +sub _generate_type_constraint_and_coercion { + my ($self, $attr, $index) = @_; + + return unless $attr->has_type_constraint; + + my @source; + if ($attr->should_coerce && $attr->type_constraint->has_coercion) { + push @source => $self->_generate_type_coercion( + $attr, + '$type_constraints[' . $index . ']', + '$val', + '$val' + ); + } + push @source => $self->_generate_type_constraint_check( + $attr, + ('$type_constraint_bodies[' . $index . ']'), + ('$type_constraints[' . $index . ']'), + '$val' + ); + return @source; +} + sub _generate_type_coercion { my ($self, $attr, $type_constraint_name, $value_name, $return_value_name) = @_; return ($return_value_name . ' = ' . $type_constraint_name . '->coerce(' . $value_name . ');'); } sub _generate_type_constraint_check { - my ($self, $attr, $type_constraint_cv, $value_name) = @_; + my ($self, $attr, $type_constraint_cv, $type_constraint_obj, $value_name) = @_; return ( - $type_constraint_cv . '->(' . $value_name . ')' - . "\n\t" . '|| confess "Attribute (' . $attr->name . ') does not pass the type constraint (' - . $attr->type_constraint->name - . ') with " . (defined(' . $value_name . ') ? overload::StrVal(' . $value_name . ') : "undef");' + $self->_inline_throw_error('"Attribute (' # FIXME add 'dad' + . $attr->name + . ') does not pass the type constraint because: " . ' + . $type_constraint_obj . '->get_message(' . $value_name . ')') + . "\n\t unless " . $type_constraint_cv . '->(' . $value_name . ');' ); } @@ -241,13 +392,7 @@ sub _generate_default_value { return '$attrs->[' . $index . ']->default($instance)'; } else { - my $default = $attr->default; - # make sure to quote strings ... - unless (looks_like_number($default)) { - $default = "'$default'"; - } - - return $default; + return q{"} . quotemeta( $attr->default ) . q{"}; } } @@ -274,16 +419,20 @@ not particularly useful. =item B +=item B + =item B =item B =item B -=item B +=item B =item B +=item B + =back =head1 AUTHORS