use Carp 'confess';
use Scalar::Util 'blessed';
-our $VERSION = '0.05';
+our $VERSION = '0.71_01';
+$VERSION = eval $VERSION;
our $AUTHORITY = 'cpan:STEVAN';
use base 'Class::MOP::Object';
sub new {
- my ($class, $metaclass, $options) = @_;
+ my ($class, @args) = @_;
- my $self = bless {
- '$!metaclass' => $metaclass,
- '%!options' => $options,
- '$!immutable_metaclass' => undef,
- } => $class;
+ my ( $metaclass, $options );
- # NOTE:
- # we initialize the immutable
- # version of the metaclass here
- $self->create_immutable_metaclass;
+ if ( @args == 2 ) {
+ # compatibility args
+ ( $metaclass, $options ) = @args;
+ } else {
+ unshift @args, "metaclass" if @args % 2 == 1;
+
+ # default named args
+ my %options = @args;
+ $options = \%options;
+ $metaclass = $options{metaclass};
+ }
+
+ my $self = $class->_new(
+ 'metaclass' => $metaclass,
+ 'options' => $options,
+ 'immutable_metaclass' => undef,
+ );
return $self;
}
-sub immutable_metaclass { (shift)->{'$!immutable_metaclass'} }
-sub metaclass { (shift)->{'$!metaclass'} }
-sub options { (shift)->{'%!options'} }
+sub _new {
+ my $class = shift;
+ my $options = @_ == 1 ? $_[0] : {@_};
+
+ bless $options, $class;
+}
+
+sub immutable_metaclass {
+ my $self = shift;
+
+ $self->create_immutable_metaclass unless $self->{'immutable_metaclass'};
+
+ return $self->{'immutable_metaclass'};
+}
+
+sub metaclass { (shift)->{'metaclass'} }
+sub options { (shift)->{'options'} }
sub create_immutable_metaclass {
my $self = shift;
# metaclass is just a anon-class
# which shadows the methods
# appropriately
- $self->{'$!immutable_metaclass'} = Class::MOP::Class->create_anon_class(
+ $self->{'immutable_metaclass'} = Class::MOP::Class->create_anon_class(
superclasses => [ blessed($self->metaclass) ],
methods => $self->create_methods_for_immutable_metaclass,
);
return Class::MOP::Class->initialize($self) unless blessed($self);
# otherwise, they are asking for the metaclass
# which has been made immutable, which is itself
- return $self;
+ # except in the cases where it is a metaclass itself
+ # that has been made immutable and for that we need
+ # to dig a bit ...
+ if ($self->isa('Class::MOP::Class')) {
+ return $self->{'___original_class'}->meta;
+ }
+ else {
+ return $self;
+ }
},
is_mutable => sub { 0 },
is_immutable => sub { 1 },
sub make_metaclass_immutable {
my ($self, $metaclass, $options) = @_;
- foreach my $pair (
- [ inline_accessors => 1 ],
- [ inline_constructor => 1 ],
- [ inline_destructor => 0 ],
- [ constructor_name => 'new' ],
- [ debug => 0 ],
- ) {
- $options->{$pair->[0]} = $pair->[1] unless exists $options->{$pair->[0]};
- }
+ my %options = (
+ inline_accessors => 1,
+ inline_constructor => 1,
+ inline_destructor => 0,
+ constructor_name => 'new',
+ debug => 0,
+ %$options,
+ );
- my %options = %$options;
+ %$options = %options; # FIXME who the hell is relying on this?!? tests fail =(
if ($options{inline_accessors}) {
foreach my $attr_name ($metaclass->get_attribute_list) {
$metaclass->add_method(
$options{constructor_name},
$constructor_class->new(
- options => \%options,
- metaclass => $metaclass,
- is_inline => 1,
+ options => \%options,
+ metaclass => $metaclass,
+ is_inline => 1,
+ package_name => $metaclass->name,
+ name => $options{constructor_name}
)
- ) unless $metaclass->has_method($options{constructor_name});
+ ) if $options{replace_constructor} or !$metaclass->has_method($options{constructor_name});
}
if ($options{inline_destructor}) {
my $destructor_class = $options{destructor_class};
- my $destructor = $destructor_class->new(
- options => \%options,
- metaclass => $metaclass,
- );
-
- $metaclass->add_method('DESTROY' => $destructor)
- # NOTE:
- # we allow the destructor to determine
- # if it is needed or not, it can perform
- # all sorts of checks because it has the
- # metaclass instance
- if $destructor->is_needed;
+ # NOTE:
+ # we allow the destructor to determine
+ # if it is needed or not before we actually
+ # create the destructor too
+ # - SL
+ if ($destructor_class->is_needed($metaclass)) {
+ my $destructor = $destructor_class->new(
+ options => \%options,
+ metaclass => $metaclass,
+ package_name => $metaclass->name,
+ name => 'DESTROY'
+ );
+
+ $metaclass->add_method('DESTROY' => $destructor)
+ # NOTE:
+ # we allow the destructor to determine
+ # if it is needed or not, it can perform
+ # all sorts of checks because it has the
+ # metaclass instance
+ if $destructor->is_needed;
+ }
}
my $memoized_methods = $self->options->{memoize};
($metaclass->can($method_name))
|| confess "Could not find the method '$method_name' in " . $metaclass->name;
-
- if ($type eq 'ARRAY') {
- $metaclass->{'___' . $method_name} = [ $metaclass->$method_name ];
- }
- elsif ($type eq 'HASH') {
- $metaclass->{'___' . $method_name} = { $metaclass->$method_name };
- }
- elsif ($type eq 'SCALAR') {
- $metaclass->{'___' . $method_name} = $metaclass->$method_name;
- }
}
$metaclass->{'___original_class'} = blessed($metaclass);
if ($options{inline_destructor} && $immutable->has_method('DESTROY')) {
$immutable->remove_method('DESTROY')
- if $immutable->get_method('DESTROY')->blessed eq $options{destructor_class};
+ if blessed($immutable->get_method('DESTROY')) eq $options{destructor_class};
}
# NOTE:
# 14:26 <@stevan> the only user of ::Method::Constructor is immutable
# 14:27 <@stevan> if someone uses it outside of immutable,.. they are either: mst or groditi
# 14:27 <@stevan> so I am not worried
- if ($options{inline_constructor}) {
+ if ($options{inline_constructor} && $immutable->has_method($options{constructor_name})) {
my $constructor_class = $options{constructor_class} || 'Class::MOP::Method::Constructor';
$immutable->remove_method( $options{constructor_name} )
- if $immutable->get_method($options{constructor_name})->blessed eq $constructor_class;
+ if blessed($immutable->get_method($options{constructor_name})) eq $constructor_class;
}
}
my $self = shift;
my %methods = %DEFAULT_METHODS;
+ my $metaclass = $self->metaclass;
+ my $meta = $metaclass->meta;
foreach my $read_only_method (@{$self->options->{read_only}}) {
- my $method = $self->metaclass->meta->find_method_by_name($read_only_method);
+ my $method = $meta->find_method_by_name($read_only_method);
(defined $method)
- || confess "Could not find the method '$read_only_method' in " . $self->metaclass->name;
+ || confess "Could not find the method '$read_only_method' in " . $metaclass->name;
$methods{$read_only_method} = sub {
confess "This method is read-only" if scalar @_ > 1;
my $memoized_methods = $self->options->{memoize};
foreach my $method_name (keys %{$memoized_methods}) {
my $type = $memoized_methods->{$method_name};
+ my $key = '___' . $method_name;
+ my $method = $meta->find_method_by_name($method_name);
+
if ($type eq 'ARRAY') {
- $methods{$method_name} = sub { @{$_[0]->{'___' . $method_name}} };
+ $methods{$method_name} = sub {
+ @{$_[0]->{$key}} = $method->execute($_[0])
+ if !exists $_[0]->{$key};
+ return @{$_[0]->{$key}};
+ };
}
elsif ($type eq 'HASH') {
- $methods{$method_name} = sub { %{$_[0]->{'___' . $method_name}} };
+ $methods{$method_name} = sub {
+ %{$_[0]->{$key}} = $method->execute($_[0])
+ if !exists $_[0]->{$key};
+ return %{$_[0]->{$key}};
+ };
}
elsif ($type eq 'SCALAR') {
- $methods{$method_name} = sub { $_[0]->{'___' . $method_name} };
+ $methods{$method_name} = sub {
+ $_[0]->{$key} = $method->execute($_[0])
+ if !exists $_[0]->{$key};
+ return $_[0]->{$key};
+ };
}
}
+
+ my $wrapped_methods = $self->options->{wrapped};
+
+ foreach my $method_name (keys %{ $wrapped_methods }) {
+ my $method = $meta->find_method_by_name($method_name);
+
+ (defined $method)
+ || confess "Could not find the method '$method_name' in " . $metaclass->name;
+
+ my $wrapper = $wrapped_methods->{$method_name};
+
+ $methods{$method_name} = sub { $wrapper->($method, @_) };
+ }
$methods{get_mutable_metaclass_name} = sub { (shift)->{'___original_class'} };
projects / gitmo/Class-MOP.git / blobdiff