X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FMooseX%2FClassAttribute%2FTrait%2FAttribute.pm;h=14323315c6ab3fa081cb7182965e8f726bbc3447;hb=a5ed69bcccc24de628b7bd416423c9f2fcace667;hp=c1d0b38a69e9ce02b10758a056e602fbabea214e;hpb=0cee5df4e6c398079ade79a540e683f5ecaf47c6;p=gitmo%2FMooseX-ClassAttribute.git diff --git a/lib/MooseX/ClassAttribute/Trait/Attribute.pm b/lib/MooseX/ClassAttribute/Trait/Attribute.pm index c1d0b38..1432331 100644 --- a/lib/MooseX/ClassAttribute/Trait/Attribute.pm +++ b/lib/MooseX/ClassAttribute/Trait/Attribute.pm @@ -3,10 +3,6 @@ package MooseX::ClassAttribute::Trait::Attribute; use strict; use warnings; -our $VERSION = '0.11'; - -use MooseX::ClassAttribute::Meta::Method::Accessor; - use namespace::autoclean; use Moose::Role; @@ -14,11 +10,7 @@ use Moose::Role; # because it overrides a lot of behavior. However, as a subclass it # won't cooperate with _other_ subclasses. -around 'accessor_metaclass' => sub { - return 'MooseX::ClassAttribute::Meta::Method::Accessor'; -}; - -around '_process_options' => sub { +around _process_options => sub { my $orig = shift; my $class = shift; my $name = shift; @@ -30,25 +22,19 @@ around '_process_options' => sub { return $class->$orig( $name, $options ); }; -around attach_to_class => sub { - my $orig = shift; +after attach_to_class => sub { my $self = shift; my $meta = shift; - $self->$orig($meta); - $self->_initialize($meta) unless $self->is_lazy(); }; -around 'detach_from_class' => sub { - my $orig = shift; +before detach_from_class => sub { my $self = shift; my $meta = shift; $self->clear_value($meta); - - $self->$orig($meta); }; sub _initialize { @@ -63,7 +49,7 @@ sub _initialize { } } -around 'default' => sub { +around default => sub { my $orig = shift; my $self = shift; @@ -76,7 +62,7 @@ around 'default' => sub { return $default; }; -around '_call_builder' => sub { +around _call_builder => sub { shift; my $self = shift; my $class = shift; @@ -93,7 +79,7 @@ around '_call_builder' => sub { . "'" ); }; -around 'set_value' => sub { +around set_value => sub { shift; my $self = shift; shift; # ignoring instance or class name @@ -103,7 +89,7 @@ around 'set_value' => sub { ->set_class_attribute_value( $self->name() => $value ); }; -around 'get_value' => sub { +around get_value => sub { shift; my $self = shift; @@ -111,7 +97,7 @@ around 'get_value' => sub { ->get_class_attribute_value( $self->name() ); }; -around 'has_value' => sub { +around has_value => sub { shift; my $self = shift; @@ -119,7 +105,7 @@ around 'has_value' => sub { ->has_class_attribute_value( $self->name() ); }; -around 'clear_value' => sub { +around clear_value => sub { shift; my $self = shift; @@ -127,16 +113,110 @@ around 'clear_value' => sub { ->clear_class_attribute_value( $self->name() ); }; +if ( $Moose::VERSION < 1.99 ) { + around inline_get => sub { + shift; + my $self = shift; + + return $self->associated_class() + ->_inline_get_class_slot_value( $self->slots() ); + }; + + around inline_set => sub { + shift; + my $self = shift; + shift; + my $value = shift; + + my $meta = $self->associated_class(); + + my $code + = $meta->_inline_set_class_slot_value( $self->slots(), $value ) + . ";"; + $code + .= $meta->_inline_weaken_class_slot_value( $self->slots(), $value ) + . " if ref $value;" + if $self->is_weak_ref(); + + return $code; + }; + + around inline_has => sub { + shift; + my $self = shift; + + return $self->associated_class() + ->_inline_is_class_slot_initialized( $self->slots() ); + }; + + around inline_clear => sub { + shift; + my $self = shift; + + return $self->associated_class() + ->_inline_deinitialize_class_slot( $self->slots() ); + }; +} +else { + around _inline_instance_get => sub { + shift; + my $self = shift; + + return $self->associated_class() + ->_inline_get_class_slot_value( $self->slots() ); + }; + + around _inline_instance_set => sub { + shift; + my $self = shift; + shift; + my $value = shift; + + return $self->associated_class() + ->_inline_set_class_slot_value( $self->slots(), $value ); + }; + + around _inline_instance_has => sub { + shift; + my $self = shift; + + return $self->associated_class() + ->_inline_is_class_slot_initialized( $self->slots() ); + }; + + around _inline_instance_clear => sub { + shift; + my $self = shift; + + return $self->associated_class() + ->_inline_deinitialize_class_slot( $self->slots() ); + }; + + around _inline_weaken_value => sub { + shift; + my $self = shift; + shift; + my $value = shift; + + return unless $self->is_weak_ref(); + + return ( + $self->associated_class->_inline_weaken_class_slot_value( + $self->slots(), $value + ), + 'if ref ' . $value . ';', + ); + }; +} + 1; +# ABSTRACT: A trait for class attributes + __END__ =pod -=head1 NAME - -MooseX::ClassAttribute::Trait::Attribute - A trait for class attributes - =head1 DESCRIPTION This role modifies the behavior of class attributes in various @@ -147,19 +227,8 @@ metaclasses, like C. There are no new public methods implemented by this role. All it does is change the behavior of a number of existing methods. -=head1 AUTHOR - -Dave Rolsky, C<< >> - =head1 BUGS See L for details. -=head1 COPYRIGHT & LICENSE - -Copyright 2007-2010 Dave Rolsky, All Rights Reserved. - -This program is free software; you can redistribute it and/or modify -it under the same terms as Perl itself. - =cut