use Carp 'confess';
use Scalar::Util 'blessed', 'weaken';
+use Try::Tiny;
-our $VERSION = '0.77';
+our $VERSION = '1.12';
$VERSION = eval $VERSION;
our $AUTHORITY = 'cpan:STEVAN';
# needed
weaken($self->{'attribute'});
- $self->initialize_body;
+ $self->_initialize_body;
return $self;
}
sub _new {
my $class = shift;
- my $options = @_ == 1 ? $_[0] : {@_};
- $options->{is_inline} ||= 0;
+ return Class::MOP::Class->initialize($class)->new_object(@_)
+ if $class ne __PACKAGE__;
- return bless $options, $class;
+ my $params = @_ == 1 ? $_[0] : {@_};
+
+ return bless {
+ # inherited from Class::MOP::Method
+ body => $params->{body},
+ associated_metaclass => $params->{associated_metaclass},
+ package_name => $params->{package_name},
+ name => $params->{name},
+ original_method => $params->{original_method},
+
+ # inherit from Class::MOP::Generated
+ is_inline => $params->{is_inline} || 0,
+ definition_context => $params->{definition_context},
+
+ # defined in this class
+ attribute => $params->{attribute},
+ accessor_type => $params->{accessor_type},
+ } => $class;
}
## accessors
## factory
-sub initialize_body {
+sub _initialize_body {
my $self = shift;
my $method_name = join "_" => (
- 'generate',
+ '_generate',
$self->accessor_type,
'method',
($self->is_inline ? 'inline' : ())
## generators
-sub generate_accessor_method {
- my $attr = (shift)->associated_attribute;
- return sub {
- $attr->set_value($_[0], $_[1]) if scalar(@_) == 2;
- $attr->get_value($_[0]);
- };
-}
+sub _generate_accessor_method {
+ my $self = shift;
+ my $attr = $self->associated_attribute;
-sub generate_reader_method {
- my $attr = (shift)->associated_attribute;
return sub {
- confess "Cannot assign a value to a read-only accessor" if @_ > 1;
+ if (@_ >= 2) {
+ $attr->set_value($_[0], $_[1]);
+ }
$attr->get_value($_[0]);
};
}
-sub generate_writer_method {
- my $attr = (shift)->associated_attribute;
- return sub {
- $attr->set_value($_[0], $_[1]);
+sub _generate_accessor_method_inline {
+ my $self = shift;
+ my $attr = $self->associated_attribute;
+
+ return try {
+ $self->_compile_code([
+ 'sub {',
+ 'if (@_ > 1) {',
+ $attr->_inline_set_value('$_[0]', '$_[1]'),
+ '}',
+ $attr->_inline_get_value('$_[0]'),
+ '}',
+ ]);
+ }
+ catch {
+ confess "Could not generate inline accessor because : $_";
};
}
-sub generate_predicate_method {
- my $attr = (shift)->associated_attribute;
+sub _generate_reader_method {
+ my $self = shift;
+ my $attr = $self->associated_attribute;
+
return sub {
- $attr->has_value($_[0])
+ confess "Cannot assign a value to a read-only accessor"
+ if @_ > 1;
+ $attr->get_value($_[0]);
};
}
-sub generate_clearer_method {
- my $attr = (shift)->associated_attribute;
- return sub {
- $attr->clear_value($_[0])
+sub _generate_reader_method_inline {
+ my $self = shift;
+ my $attr = $self->associated_attribute;
+
+ return try {
+ $self->_compile_code([
+ 'sub {',
+ 'if (@_ > 1) {',
+ # XXX: this is a hack, but our error stuff is terrible
+ $self->_inline_throw_error(
+ '"Cannot assign a value to a read-only accessor"',
+ 'data => \@_'
+ ) . ';',
+ '}',
+ $attr->_inline_get_value('$_[0]'),
+ '}',
+ ]);
+ }
+ catch {
+ confess "Could not generate inline reader because : $_";
};
}
-## Inline methods
-
+sub _inline_throw_error {
+ my $self = shift;
+ return 'confess ' . $_[0];
+}
-sub generate_accessor_method_inline {
- my $self = shift;
- my $attr = $self->associated_attribute;
- my $attr_name = $attr->name;
- my $meta_instance = $attr->associated_class->instance_metaclass;
+sub _generate_writer_method {
+ my $self = shift;
+ my $attr = $self->associated_attribute;
- return $self->_eval_closure(
- {},
- 'sub {'
- . $meta_instance->inline_set_slot_value('$_[0]', $attr_name, '$_[1]')
- . ' if scalar(@_) == 2; '
- . $meta_instance->inline_get_slot_value('$_[0]', $attr_name)
- . '}'
- );
+ return sub {
+ $attr->set_value($_[0], $_[1]);
+ };
}
-sub generate_reader_method_inline {
- my $self = shift;
- my $attr = $self->associated_attribute;
- my $attr_name = $attr->name;
- my $meta_instance = $attr->associated_class->instance_metaclass;
-
- return $self->_eval_closure(
- {},
- 'sub {'
- . 'confess "Cannot assign a value to a read-only accessor" if @_ > 1;'
- . $meta_instance->inline_get_slot_value('$_[0]', "'$attr_name'")
- . '}'
- );
+sub _generate_writer_method_inline {
+ my $self = shift;
+ my $attr = $self->associated_attribute;
+
+ return try {
+ $self->_compile_code([
+ 'sub {',
+ $attr->_inline_set_value('$_[0]', '$_[1]'),
+ '}',
+ ]);
+ }
+ catch {
+ confess "Could not generate inline writer because : $_";
+ };
}
-sub generate_writer_method_inline {
- my $self = shift;
- my $attr = $self->associated_attribute;
- my $attr_name = $attr->name;
- my $meta_instance = $attr->associated_class->instance_metaclass;
-
- return $self->_eval_closure(
- {},
- 'sub {'
- . $meta_instance->inline_set_slot_value('$_[0]', "'$attr_name'", '$_[1]')
- . '}'
- );
+sub _generate_predicate_method {
+ my $self = shift;
+ my $attr = $self->associated_attribute;
+
+ return sub {
+ $attr->has_value($_[0])
+ };
}
+sub _generate_predicate_method_inline {
+ my $self = shift;
+ my $attr = $self->associated_attribute;
+
+ return try {
+ $self->_compile_code([
+ 'sub {',
+ $attr->_inline_has_value('$_[0]'),
+ '}',
+ ]);
+ }
+ catch {
+ confess "Could not generate inline predicate because : $_";
+ };
+}
-sub generate_predicate_method_inline {
- my $self = shift;
- my $attr = $self->associated_attribute;
- my $attr_name = $attr->name;
- my $meta_instance = $attr->associated_class->instance_metaclass;
+sub _generate_clearer_method {
+ my $self = shift;
+ my $attr = $self->associated_attribute;
- return $self->_eval_closure(
- {},
- 'sub {'
- . $meta_instance->inline_is_slot_initialized('$_[0]', "'$attr_name'")
- . '}'
- );
+ return sub {
+ $attr->clear_value($_[0])
+ };
}
-sub generate_clearer_method_inline {
- my $self = shift;
- my $attr = $self->associated_attribute;
- my $attr_name = $attr->name;
- my $meta_instance = $attr->associated_class->instance_metaclass;
-
- return $self->_eval_closure(
- {},
- 'sub {'
- . $meta_instance->inline_deinitialize_slot('$_[0]', "'$attr_name'")
- . '}'
- );
+sub _generate_clearer_method_inline {
+ my $self = shift;
+ my $attr = $self->associated_attribute;
+
+ return try {
+ $self->_compile_code([
+ 'sub {',
+ $attr->_inline_clear_value('$_[0]'),
+ '}',
+ ]);
+ }
+ catch {
+ confess "Could not generate inline clearer because : $_";
+ };
}
1;
=head1 DESCRIPTION
-This is a C<Class::MOP::Method> subclass which is used interally
-by C<Class::MOP::Attribute> to generate accessor code. It can
-handle generation of readers, writers, predicate and clearer
-methods, both as closures and as more optimized inline methods.
+This is a subclass of C<Class::MOP::Method> which is used by
+C<Class::MOP::Attribute> to generate accessor code. It handles
+generation of readers, writers, predicates and clearers. For each type
+of method, it can either create a subroutine reference, or actually
+inline code by generating a string and C<eval>'ing it.
=head1 METHODS
=over 4
-=item B<new (%options)>
+=item B<< Class::MOP::Method::Accessor->new(%options) >>
-This creates the method based on the criteria in C<%options>,
-these options are:
+This returns a new C<Class::MOP::Method::Accessor> based on the
+C<%options> provided.
=over 4
-=item I<attribute>
-
-This must be an instance of C<Class::MOP::Attribute> which this
-accessor is being generated for. This paramter is B<required>.
-
-=item I<accessor_type>
-
-This is a string from the following set; reader, writer, accessor,
-predicate or clearer. This is used to determine which type of
-method is to be generated.
-
-=item I<is_inline>
-
-This is a boolean to indicate if the method should be generated
-as a closure, or as a more optimized inline version.
+=item * attribute
-=back
+This is the C<Class::MOP::Attribute> for which accessors are being
+generated. This option is required.
-=item B<accessor_type>
+=item * accessor_type
-This returns the accessor type which was passed into C<new>.
+This is a string which should be one of "reader", "writer",
+"accessor", "predicate", or "clearer". This is the type of method
+being generated. This option is required.
-=item B<is_inline>
+=item * is_inline
-This returns the boolean which was passed into C<new>.
+This indicates whether or not the accessor should be inlined. This
+defaults to false.
-=item B<associated_attribute>
+=item * name
-This returns the attribute instance which was passed into C<new>.
+The method name (without a package name). This is required.
-=item B<initialize_body>
+=item * package_name
-This will actually generate the method based on the specified
-criteria passed to the constructor.
+The package name for the method. This is required.
=back
-=head2 Method Generators
-
-These methods will generate appropriate code references for
-the various types of accessors which are supported by
-C<Class::MOP::Attribute>. The names pretty much explain it all.
-
-=over 4
-
-=item B<generate_accessor_method>
-
-=item B<generate_accessor_method_inline>
-
-=item B<generate_clearer_method>
+=item B<< $metamethod->accessor_type >>
-=item B<generate_clearer_method_inline>
+Returns the accessor type which was passed to C<new>.
-=item B<generate_predicate_method>
+=item B<< $metamethod->is_inline >>
-=item B<generate_predicate_method_inline>
+Returns a boolean indicating whether or not the accessor is inlined.
-=item B<generate_reader_method>
+=item B<< $metamethod->associated_attribute >>
-=item B<generate_reader_method_inline>
+This returns the L<Class::MOP::Attribute> object which was passed to
+C<new>.
-=item B<generate_writer_method>
+=item B<< $metamethod->body >>
-=item B<generate_writer_method_inline>
+The method itself is I<generated> when the accessor object is
+constructed.
=back
=head1 COPYRIGHT AND LICENSE
-Copyright 2006-2008 by Infinity Interactive, Inc.
+Copyright 2006-2010 by Infinity Interactive, Inc.
L<http://www.iinteractive.com>
projects / gitmo/Class-MOP.git / blobdiff