use Carp 'confess';
use Scalar::Util 'blessed', 'weaken';
+use Try::Tiny;
-our $VERSION = '0.78';
+our $VERSION = '1.11';
$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' : ())
);
- eval { $self->{'body'} = $self->$method_name() };
- die $@ if $@;
+ $self->{'body'} = $self->$method_name();
}
## 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 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 _inline_throw_error {
+ my $self = shift;
+ return 'confess ' . $_[0];
+}
- my $code = $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)
- . '}'
- );
- confess "Could not generate inline accessor because : $@" if $@;
+sub _generate_writer_method {
+ my $self = shift;
+ my $attr = $self->associated_attribute;
- return $code;
+ 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;
-
- my $code = $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)
- . '}'
- );
- confess "Could not generate inline reader because : $@" if $@;
-
- return $code;
+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;
-
- my $code = $self->_eval_closure(
- {},
- 'sub {'
- . $meta_instance->inline_set_slot_value('$_[0]', $attr_name, '$_[1]')
- . '}'
- );
- confess "Could not generate inline writer because : $@" if $@;
+sub _generate_predicate_method {
+ my $self = shift;
+ my $attr = $self->associated_attribute;
- return $code;
+ 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;
-
- my $code = $self->_eval_closure(
- {},
- 'sub {'
- . $meta_instance->inline_is_slot_initialized('$_[0]', $attr_name)
- . '}'
- );
- confess "Could not generate inline predicate because : $@" if $@;
+sub _generate_clearer_method {
+ my $self = shift;
+ my $attr = $self->associated_attribute;
- return $code;
+ 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;
-
- my $code = $self->_eval_closure(
- {},
- 'sub {'
- . $meta_instance->inline_deinitialize_slot('$_[0]', $attr_name)
- . '}'
- );
- confess "Could not generate inline clearer because : $@" if $@;
-
- return $code;
+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 subclass of <Class::MOP::Method> which is used by
+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
=over 4
-=item attribute
+=item * attribute
This is the C<Class::MOP::Attribute> for which accessors are being
generated. This option is required.
-=item accessor_type
+=item * accessor_type
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 is_inline
+=item * is_inline
This indicates whether or not the accessor should be inlined. This
-default to false.
+defaults to false.
+
+=item * name
+
+The method name (without a package name). This is required.
+
+=item * package_name
+
+The package name for the method. This is required.
=back
=head1 COPYRIGHT AND LICENSE
-Copyright 2006-2009 by Infinity Interactive, Inc.
+Copyright 2006-2010 by Infinity Interactive, Inc.
L<http://www.iinteractive.com>