Clean up code a bit
[gitmo/Class-MOP.git] / lib / Class / MOP / Attribute.pm
index 8a51a5c..3c5e7a2 100644 (file)
@@ -4,395 +4,481 @@ package Class::MOP::Attribute;
 use strict;
 use warnings;
 
+use Class::MOP::Method::Accessor;
+
 use Carp         'confess';
-use Scalar::Util 'blessed', 'reftype', 'weaken';
+use Scalar::Util 'blessed', 'weaken';
 
-our $VERSION   = '0.12';
+our $VERSION   = '0.78';
+$VERSION = eval $VERSION;
 our $AUTHORITY = 'cpan:STEVAN';
 
 use base 'Class::MOP::Object';
 
-sub meta { 
-    require Class::MOP::Class;
-    Class::MOP::Class->initialize(blessed($_[0]) || $_[0]);
-}
-
 # NOTE: (meta-circularity)
-# This method will be replaced in the 
-# boostrap section of Class::MOP, by 
-# a new version which uses the 
+# This method will be replaced in the
+# boostrap section of Class::MOP, by
+# a new version which uses the
 # &Class::MOP::Class::construct_instance
 # method to build an attribute meta-object
 # which itself is described with attribute
-# meta-objects. 
+# meta-objects.
 #     - Ain't meta-circularity grand? :)
 sub new {
-    my $class   = shift;
-    my $name    = shift;
-    my %options = @_;    
-        
+    my ( $class, @args ) = @_;
+
+    unshift @args, "name" if @args % 2 == 1;
+    my %options = @args;
+
+    my $name = $options{name};
+
     (defined $name && $name)
         || confess "You must provide a name for the attribute";
-    $options{init_arg} = $name 
+
+    $options{init_arg} = $name
         if not exists $options{init_arg};
-            
-    (is_default_a_coderef(\%options))
-        || confess("References are not allowed as default values, you must ". 
-                   "wrap then in a CODE reference (ex: sub { [] } and not [])")
-            if exists $options{default} && ref $options{default};      
-            
+    if(exists $options{builder}){
+        confess("builder must be a defined scalar value which is a method name")
+            if ref $options{builder} || !(defined $options{builder});
+        confess("Setting both default and builder is not allowed.")
+            if exists $options{default};
+    } else {
+        (is_default_a_coderef(\%options))
+            || confess("References are not allowed as default values, you must ".
+                       "wrap the default of '$name' in a CODE reference (ex: sub { [] } and not [])")
+                if exists $options{default} && ref $options{default};
+    }
+    if( $options{required} and not( defined($options{builder}) || defined($options{init_arg}) || exists $options{default} ) ) {
+        confess("A required attribute must have either 'init_arg', 'builder', or 'default'");
+    }
+
+    $class->_new(\%options);
+}
+
+sub _new {
+    my $class = shift;
+    my $options = @_ == 1 ? $_[0] : {@_};
+
     bless {
-        name      => $name,
-        accessor  => $options{accessor},
-        reader    => $options{reader},
-        writer    => $options{writer},
-        predicate => $options{predicate},
-        clearer   => $options{clearer},
-        init_arg  => $options{init_arg},
-        default   => $options{default},
-        # keep a weakened link to the 
+        'name'               => $options->{name},
+        'accessor'           => $options->{accessor},
+        'reader'             => $options->{reader},
+        'writer'             => $options->{writer},
+        'predicate'          => $options->{predicate},
+        'clearer'            => $options->{clearer},
+        'builder'            => $options->{builder},
+        'init_arg'           => $options->{init_arg},
+        'default'            => $options->{default},
+        'initializer'        => $options->{initializer},
+        'definition_context' => $options->{definition_context},
+        # keep a weakened link to the
         # class we are associated with
-        associated_class => undef,
-    } => $class;
+        'associated_class' => undef,
+        # and a list of the methods
+        # associated with this attr
+        'associated_methods' => [],
+    }, $class;
 }
 
 # NOTE:
-# this is a primative (and kludgy) clone operation 
+# this is a primative (and kludgy) clone operation
 # for now, it will be replaced in the Class::MOP
-# bootstrap with a proper one, however we know 
+# bootstrap with a proper one, however we know
 # that this one will work fine for now.
 sub clone {
     my $self    = shift;
     my %options = @_;
     (blessed($self))
         || confess "Can only clone an instance";
-    return bless { %{$self}, %options } => blessed($self);
+    return bless { %{$self}, %options } => ref($self);
 }
 
 sub initialize_instance_slot {
     my ($self, $meta_instance, $instance, $params) = @_;
-    my $init_arg = $self->{init_arg};
+    my $init_arg = $self->{'init_arg'};
+
     # try to fetch the init arg from the %params ...
-    my $val;        
-    $val = $params->{$init_arg} if exists $params->{$init_arg};
-    # if nothing was in the %params, we can use the 
+
+    # if nothing was in the %params, we can use the
     # attribute's default value (if it has one)
-    if (!defined $val && defined $self->{default}) {
-        $val = $self->default($instance);
+    if(defined $init_arg and exists $params->{$init_arg}){
+        $self->_set_initial_slot_value(
+            $meta_instance, 
+            $instance,
+            $params->{$init_arg},
+        );
+    } 
+    elsif (defined $self->{'default'}) {
+        $self->_set_initial_slot_value(
+            $meta_instance, 
+            $instance,
+            $self->default($instance),
+        );
+    } 
+    elsif (defined( my $builder = $self->{'builder'})) {
+        if ($builder = $instance->can($builder)) {
+            $self->_set_initial_slot_value(
+                $meta_instance, 
+                $instance,
+                $instance->$builder,
+            );
+        } 
+        else {
+            confess(ref($instance)." does not support builder method '". $self->{'builder'} ."' for attribute '" . $self->name . "'");
+        }
     }
-    $meta_instance->set_slot_value($instance, $self->name, $val);
 }
 
-# NOTE:
-# the next bunch of methods will get bootstrapped 
-# away in the Class::MOP bootstrapping section
+sub _set_initial_slot_value {
+    my ($self, $meta_instance, $instance, $value) = @_;
+
+    my $slot_name = $self->name;
+
+    return $meta_instance->set_slot_value($instance, $slot_name, $value)
+        unless $self->has_initializer;
 
-sub name { $_[0]->{name} }
+    my $callback = sub {
+        $meta_instance->set_slot_value($instance, $slot_name, $_[0]);
+    };
+    
+    my $initializer = $self->initializer;
 
-sub associated_class { $_[0]->{associated_class} }
+    # most things will just want to set a value, so make it first arg
+    $instance->$initializer($value, $callback, $self);
+}
 
-sub has_accessor  { defined($_[0]->{accessor})  ? 1 : 0 }
-sub has_reader    { defined($_[0]->{reader})    ? 1 : 0 }
-sub has_writer    { defined($_[0]->{writer})    ? 1 : 0 }
-sub has_predicate { defined($_[0]->{predicate}) ? 1 : 0 }
-sub has_clearer   { defined($_[0]->{clearer})   ? 1 : 0 }
-sub has_init_arg  { defined($_[0]->{init_arg})  ? 1 : 0 }
-sub has_default   { defined($_[0]->{default})   ? 1 : 0 }
+# NOTE:
+# the next bunch of methods will get bootstrapped
+# away in the Class::MOP bootstrapping section
 
-sub accessor  { $_[0]->{accessor}  } 
-sub reader    { $_[0]->{reader}    }
-sub writer    { $_[0]->{writer}    }
-sub predicate { $_[0]->{predicate} }
-sub clearer   { $_[0]->{clearer}   }
-sub init_arg  { $_[0]->{init_arg}  }
+sub associated_class   { $_[0]->{'associated_class'}   }
+sub associated_methods { $_[0]->{'associated_methods'} }
+
+sub has_accessor    { defined($_[0]->{'accessor'}) }
+sub has_reader      { defined($_[0]->{'reader'}) }
+sub has_writer      { defined($_[0]->{'writer'}) }
+sub has_predicate   { defined($_[0]->{'predicate'}) }
+sub has_clearer     { defined($_[0]->{'clearer'}) }
+sub has_builder     { defined($_[0]->{'builder'}) }
+sub has_init_arg    { defined($_[0]->{'init_arg'}) }
+sub has_default     { defined($_[0]->{'default'}) }
+sub has_initializer { defined($_[0]->{'initializer'}) }
+
+sub accessor           { $_[0]->{'accessor'}    }
+sub reader             { $_[0]->{'reader'}      }
+sub writer             { $_[0]->{'writer'}      }
+sub predicate          { $_[0]->{'predicate'}   }
+sub clearer            { $_[0]->{'clearer'}     }
+sub builder            { $_[0]->{'builder'}     }
+sub init_arg           { $_[0]->{'init_arg'}    }
+sub initializer        { $_[0]->{'initializer'} }
+sub definition_context { $_[0]->{'definition_context'} }
 
 # end bootstrapped away method section.
 # (all methods below here are kept intact)
 
-sub is_default_a_coderef { 
-    ('CODE' eq (reftype($_[0]->{default}) || ''))    
+sub has_read_method  { $_[0]->has_reader || $_[0]->has_accessor }
+sub has_write_method { $_[0]->has_writer || $_[0]->has_accessor }
+
+sub get_read_method  { 
+    my $self   = shift;    
+    my $reader = $self->reader || $self->accessor;
+    # normal case ...
+    return $reader unless ref $reader;
+    # the HASH ref case
+    my ($name) = %$reader;
+    return $name;
+}
+
+sub get_write_method { 
+    my $self   = shift;
+    my $writer = $self->writer || $self->accessor; 
+    # normal case ...
+    return $writer unless ref $writer;
+    # the HASH ref case
+    my ($name) = %$writer;
+    return $name;    
+}
+
+sub get_read_method_ref {
+    my $self = shift;
+    if ((my $reader = $self->get_read_method) && $self->associated_class) {   
+        return $self->associated_class->get_method($reader);
+    }
+    else {
+        my $code = sub { $self->get_value(@_) };
+        if (my $class = $self->associated_class) {
+            return $class->method_metaclass->wrap(
+                $code,
+                package_name => $class->name,
+                name         => '__ANON__'
+            );
+        }
+        else {
+            return $code;
+        }
+    }
 }
 
-sub default { 
+sub get_write_method_ref {
+    my $self = shift;    
+    if ((my $writer = $self->get_write_method) && $self->associated_class) {         
+        return $self->associated_class->get_method($writer);
+    }
+    else {
+        my $code = sub { $self->set_value(@_) };
+        if (my $class = $self->associated_class) {
+            return $class->method_metaclass->wrap(
+                $code,
+                package_name => $class->name,
+                name         => '__ANON__'
+            );
+        }
+        else {
+            return $code;
+        }
+    }
+}
+
+sub is_default_a_coderef {
+    ('CODE' eq ref($_[0]->{'default'}))
+}
+
+sub default {
     my ($self, $instance) = @_;
-    if ($instance && $self->is_default_a_coderef) {
-        # if the default is a CODE ref, then 
+    if (defined $instance && $self->is_default_a_coderef) {
+        # if the default is a CODE ref, then
         # we pass in the instance and default
-        # can return a value based on that 
+        # can return a value based on that
         # instance. Somewhat crude, but works.
-        return $self->{default}->($instance);
-    }           
-    $self->{default};
+        return $self->{'default'}->($instance);
+    }
+    $self->{'default'};
 }
 
 # slots
 
 sub slots { (shift)->name }
 
-# class association 
+# class association
 
 sub attach_to_class {
     my ($self, $class) = @_;
     (blessed($class) && $class->isa('Class::MOP::Class'))
         || confess "You must pass a Class::MOP::Class instance (or a subclass)";
-    weaken($self->{associated_class} = $class);    
+    weaken($self->{'associated_class'} = $class);
 }
 
 sub detach_from_class {
     my $self = shift;
-    $self->{associated_class} = undef;        
+    $self->{'associated_class'} = undef;
+}
+
+# method association
+
+sub associate_method {
+    my ($self, $method) = @_;
+    push @{$self->{'associated_methods'}} => $method;
 }
 
 ## Slot management
 
+sub set_initial_value {
+    my ($self, $instance, $value) = @_;
+    $self->_set_initial_slot_value(
+        Class::MOP::Class->initialize(ref($instance))->get_meta_instance,
+        $instance,
+        $value
+    );
+}
+
 sub set_value {
     my ($self, $instance, $value) = @_;
 
-    Class::MOP::Class->initialize(Scalar::Util::blessed($instance))
+    Class::MOP::Class->initialize(ref($instance))
                      ->get_meta_instance
-                     ->set_slot_value( $instance, $self->name, $value );
+                     ->set_slot_value($instance, $self->name, $value);
 }
 
 sub get_value {
     my ($self, $instance) = @_;
 
-    Class::MOP::Class->initialize(Scalar::Util::blessed($instance))
+    Class::MOP::Class->initialize(ref($instance))
                      ->get_meta_instance
                      ->get_slot_value($instance, $self->name);
 }
 
-## Method generation helpers
-
-sub generate_accessor_method {
-    my $attr = shift; 
-    return sub {
-        $attr->set_value($_[0], $_[1]) if scalar(@_) == 2;
-        $attr->get_value($_[0]);
-    };
-}
-
-sub generate_accessor_method_inline {
-    my $self          = shift; 
-    my $attr_name     = $self->name;
-    my $meta_instance = $self->associated_class->instance_metaclass;
-
-    my $code = eval '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 $@;
-
-    return $code;
-}
-
-sub generate_reader_method {
-    my $attr = shift;
-    return sub { 
-        confess "Cannot assign a value to a read-only accessor" if @_ > 1;
-        $attr->get_value($_[0]);
-    };   
-}
-
-sub generate_reader_method_inline {
-    my $self          = shift; 
-    my $attr_name     = $self->name;
-    my $meta_instance = $self->associated_class->instance_metaclass;
-
-    my $code = eval '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 accessor because : $@" if $@;
-
-    return $code;
-}
-
-sub generate_writer_method {
-    my $attr = shift;
-    return sub {
-        $attr->set_value($_[0], $_[1]);
-    };
-}
-
-sub generate_writer_method_inline {
-    my $self          = shift; 
-    my $attr_name     = $self->name;
-    my $meta_instance = $self->associated_class->instance_metaclass;
-
-    my $code = eval 'sub {'
-        . $meta_instance->inline_set_slot_value('$_[0]', "'$attr_name'", '$_[1]')
-    . '}';
-    confess "Could not generate inline accessor because : $@" if $@;
+sub has_value {
+    my ($self, $instance) = @_;
 
-    return $code;
+    Class::MOP::Class->initialize(ref($instance))
+                     ->get_meta_instance
+                     ->is_slot_initialized($instance, $self->name);
 }
 
-sub generate_predicate_method {
-    my $self = shift;
-    my $attr_name  = $self->name;
-    return sub { 
-        defined Class::MOP::Class->initialize(Scalar::Util::blessed($_[0]))
-                                 ->get_meta_instance
-                                 ->get_slot_value($_[0], $attr_name) ? 1 : 0;
-    };
-}
+sub clear_value {
+    my ($self, $instance) = @_;
 
-sub generate_clearer_method {
-    my $self = shift;
-    my $attr_name  = $self->name;
-    return sub { 
-        Class::MOP::Class->initialize(Scalar::Util::blessed($_[0]))
-                         ->get_meta_instance
-                         ->deinitialize_slot($_[0], $attr_name);
-    };
+    Class::MOP::Class->initialize(ref($instance))
+                     ->get_meta_instance
+                     ->deinitialize_slot($instance, $self->name);
 }
 
-sub generate_predicate_method_inline {
-    my $self          = shift; 
-    my $attr_name     = $self->name;
-    my $meta_instance = $self->associated_class->instance_metaclass;
+## load em up ...
 
-    my $code = eval 'sub {'
-        . 'defined ' . $meta_instance->inline_get_slot_value('$_[0]', "'$attr_name'") . ' ? 1 : 0'
-    . '}';
-    confess "Could not generate inline predicate because : $@" if $@;
+sub accessor_metaclass { 'Class::MOP::Method::Accessor' }
 
-    return $code;
+sub process_accessors {
+    warn "The process_accessors method has been made private and this public alias will be removed in a future release.";
+    goto &_process_accessors;
 }
 
-sub generate_clearer_method_inline {
-    my $self          = shift; 
-    my $attr_name     = $self->name;
-    my $meta_instance = $self->associated_class->instance_metaclass;
+sub _process_accessors {
+    my ($self, $type, $accessor, $generate_as_inline_methods) = @_;
 
-    my $code = eval 'sub {'
-        . $meta_instance->inline_deinitialize_slot('$_[0]', "'$attr_name'")
-    . '}';
-    confess "Could not generate inline clearer because : $@" if $@;
+    my $method_ctx;
 
-    return $code;
-}
+    if ( my $ctx = $self->definition_context ) {
+        $method_ctx = { %$ctx };
+    }
 
-sub process_accessors {
-    my ($self, $type, $accessor, $generate_as_inline_methods) = @_;
-    if (reftype($accessor)) {
-        (reftype($accessor) eq 'HASH')
+    if (ref($accessor)) {
+        (ref($accessor) eq 'HASH')
             || confess "bad accessor/reader/writer/predicate/clearer format, must be a HASH ref";
         my ($name, $method) = %{$accessor};
-        return ($name, Class::MOP::Attribute::Accessor->wrap($method));        
+        $method = $self->accessor_metaclass->wrap(
+            $method,
+            package_name => $self->associated_class->name,
+            name         => $name,
+            definition_context => $method_ctx,
+        );
+        $self->associate_method($method);
+        return ($name, $method);
     }
     else {
-        my $inline_me = ($generate_as_inline_methods && $self->associated_class->instance_metaclass->is_inlinable); 
-        my $generator = $self->can('generate_' . $type . '_method' . ($inline_me ? '_inline' : ''));
-        ($generator)
-            || confess "There is no method generator for the type='$type'";
-        if (my $method = $self->$generator($self->name)) {
-            return ($accessor => Class::MOP::Attribute::Accessor->wrap($method));            
-        }
-        confess "Could not create the '$type' method for " . $self->name . " because : $@";
-    }    
+        my $inline_me = ($generate_as_inline_methods && $self->associated_class->instance_metaclass->is_inlinable);
+        my $method;
+        eval {
+            if ( $method_ctx ) {
+                my $desc = "accessor $accessor";
+                if ( $accessor ne $self->name ) {
+                    $desc .= " of attribute " . $self->name;
+                }
+
+                $method_ctx->{description} = $desc;
+            }
+
+            $method = $self->accessor_metaclass->new(
+                attribute     => $self,
+                is_inline     => $inline_me,
+                accessor_type => $type,
+                package_name  => $self->associated_class->name,
+                name          => $accessor,
+                definition_context => $method_ctx,
+            );
+        };
+        confess "Could not create the '$type' method for " . $self->name . " because : $@" if $@;
+        $self->associate_method($method);
+        return ($accessor, $method);
+    }
 }
 
 sub install_accessors {
     my $self   = shift;
     my $inline = shift;
     my $class  = $self->associated_class;
-    
+
     $class->add_method(
-        $self->process_accessors('accessor' => $self->accessor(), $inline)
+        $self->_process_accessors('accessor' => $self->accessor(), $inline)
     ) if $self->has_accessor();
 
-    $class->add_method(            
-        $self->process_accessors('reader' => $self->reader(), $inline)
+    $class->add_method(
+        $self->_process_accessors('reader' => $self->reader(), $inline)
     ) if $self->has_reader();
 
     $class->add_method(
-        $self->process_accessors('writer' => $self->writer(), $inline)
+        $self->_process_accessors('writer' => $self->writer(), $inline)
     ) if $self->has_writer();
 
     $class->add_method(
-        $self->process_accessors('predicate' => $self->predicate(), $inline)
+        $self->_process_accessors('predicate' => $self->predicate(), $inline)
     ) if $self->has_predicate();
-    
+
     $class->add_method(
-        $self->process_accessors('clearer' => $self->clearer(), $inline)
+        $self->_process_accessors('clearer' => $self->clearer(), $inline)
     ) if $self->has_clearer();
-    
+
     return;
 }
 
 {
     my $_remove_accessor = sub {
         my ($accessor, $class) = @_;
-        if (reftype($accessor) && reftype($accessor) eq 'HASH') {
+        if (ref($accessor) && ref($accessor) eq 'HASH') {
             ($accessor) = keys %{$accessor};
-        }        
-        my $method = $class->get_method($accessor);   
-        $class->remove_method($accessor) 
-            if (blessed($method) && $method->isa('Class::MOP::Attribute::Accessor'));
+        }
+        my $method = $class->get_method($accessor);
+        $class->remove_method($accessor)
+            if (ref($method) && $method->isa('Class::MOP::Method::Accessor'));
     };
-    
+
     sub remove_accessors {
         my $self = shift;
+        # TODO:
+        # we really need to make sure to remove from the
+        # associates methods here as well. But this is
+        # such a slimly used method, I am not worried
+        # about it right now.
         $_remove_accessor->($self->accessor(),  $self->associated_class()) if $self->has_accessor();
         $_remove_accessor->($self->reader(),    $self->associated_class()) if $self->has_reader();
         $_remove_accessor->($self->writer(),    $self->associated_class()) if $self->has_writer();
         $_remove_accessor->($self->predicate(), $self->associated_class()) if $self->has_predicate();
         $_remove_accessor->($self->clearer(),   $self->associated_class()) if $self->has_clearer();
-        return;                        
+        return;
     }
 
 }
 
-package Class::MOP::Attribute::Accessor;
-
-use strict;
-use warnings;
-
-use Class::MOP::Method;
-
-our $VERSION   = '0.02';
-our $AUTHORITY = 'cpan:STEVAN';
-
-use base 'Class::MOP::Method';
-
 1;
 
 __END__
 
 =pod
 
-=head1 NAME 
+=head1 NAME
 
 Class::MOP::Attribute - Attribute Meta Object
 
 =head1 SYNOPSIS
-  
-  Class::MOP::Attribute->new('$foo' => (
-      accessor  => 'foo',        # dual purpose get/set accessor
-      predicate => 'has_foo'     # predicate check for defined-ness      
-      init_arg  => '-foo',       # class->new will look for a -foo key
-      default   => 'BAR IS BAZ!' # if no -foo key is provided, use this
-  ));
-  
-  Class::MOP::Attribute->new('$.bar' => (
-      reader    => 'bar',        # getter
-      writer    => 'set_bar',    # setter     
-      predicate => 'has_bar'     # predicate check for defined-ness      
-      init_arg  => ':bar',       # class->new will look for a :bar key
-      # no default value means it is undef
-  ));
+
+  Class::MOP::Attribute->new(
+      foo => (
+          accessor  => 'foo',           # dual purpose get/set accessor
+          predicate => 'has_foo',       # predicate check for defined-ness
+          init_arg  => '-foo',          # class->new will look for a -foo key
+          default   => 'BAR IS BAZ!'    # if no -foo key is provided, use this
+      )
+  );
+
+  Class::MOP::Attribute->new(
+      bar => (
+          reader    => 'bar',           # getter
+          writer    => 'set_bar',       # setter
+          predicate => 'has_bar',       # predicate check for defined-ness
+          init_arg  => ':bar',          # class->new will look for a :bar key
+                                        # no default value means it is undef
+      )
+  );
 
 =head1 DESCRIPTION
 
-The Attribute Protocol is almost entirely an invention of this module,
-and is completely optional to this MOP. This is because Perl 5 does not 
-have consistent notion of what is an attribute of a class. There are 
-so many ways in which this is done, and very few (if any) are 
-easily discoverable by this module.
+The Attribute Protocol is almost entirely an invention of
+C<Class::MOP>. Perl 5 does not have a consistent notion of
+attributes. There are so many ways in which this is done, and very few
+(if any) are easily discoverable by this module.
 
-So, all that said, this module attempts to inject some order into this 
-chaos, by introducing a consistent API which can be used to create 
+With that said, this module attempts to inject some order into this
+chaos, by introducing a consistent API which can be used to create
 object attributes.
 
 =head1 METHODS
@@ -401,181 +487,268 @@ object attributes.
 
 =over 4
 
-=item B<new ($name, ?%options)>
+=item B<< Class::MOP::Attribute->new($name, ?%options) >>
 
-An attribute must (at the very least), have a C<$name>. All other 
-C<%options> are contained added as key-value pairs. Acceptable keys
-are as follows:
+An attribute must (at the very least), have a C<$name>. All other
+C<%options> are added as key-value pairs.
 
-=over 4
+=over 8
 
 =item I<init_arg>
 
-This should be a string value representing the expected key in 
-an initialization hash. For instance, if we have an I<init_arg> 
-value of C<-foo>, then the following code will Just Work.
+This is a string value representing the expected key in an
+initialization hash. For instance, if we have an C<init_arg> value of
+C<-foo>, then the following code will Just Work.
+
+  MyClass->meta->construct_instance( -foo => 'Hello There' );
+
+If an init_arg is not assigned, it will automatically use the
+attribute's name. If C<init_arg> is explicitly set to C<undef>, the
+attribute cannot be specified during initialization.
 
-  MyClass->meta->construct_instance(-foo => "Hello There");
+=item I<builder>
 
-In an init_arg is not assigned, it will automatically use the 
-value of C<$name>.
+This provides the name of a method that will be called to initialize
+the attribute. This method will be called on the object after it is
+constructed. It is expected to return a valid value for the attribute.
 
 =item I<default>
 
-The value of this key is the default value which 
-C<Class::MOP::Class::construct_instance> will initialize the 
-attribute to. 
-
-B<NOTE:>
-If the value is a simple scalar (string or number), then it can 
-be just passed as is. However, if you wish to initialize it with 
-a HASH or ARRAY ref, then you need to wrap that inside a CODE 
-reference, like so:
-
-  Class::MOP::Attribute->new('@foo' => (
-      default => sub { [] },
-  ));
-  
-  # or ...  
-  
-  Class::MOP::Attribute->new('%foo' => (
-      default => sub { {} },
-  ));  
-
-If you wish to initialize an attribute with a CODE reference 
-itself, then you need to wrap that in a subroutine as well, like
-so:
-  
-  Class::MOP::Attribute->new('&foo' => (
-      default => sub { sub { print "Hello World" } },
-  ));
-
-And lastly, if the value of your attribute is dependent upon 
-some other aspect of the instance structure, then you can take 
-advantage of the fact that when the I<default> value is a CODE 
-reference, it is passed the raw (unblessed) instance structure 
-as it's only argument. So you can do things like this:
-
-  Class::MOP::Attribute->new('$object_identity' => (
-      default => sub { Scalar::Util::refaddr($_[0]) },
-  ));
-
-This last feature is fairly limited as there is no gurantee of 
-the order of attribute initializations, so you cannot perform 
-any kind of dependent initializations. However, if this is 
-something you need, you could subclass B<Class::MOP::Class> and 
-this class to acheive it. However, this is currently left as 
-an exercise to the reader :).
+This can be used to provide an explicit default for initializing the
+attribute. If the default you provide is a subroutine reference, then
+this reference will be called I<as a method> on the object.
+
+If the value is a simple scalar (string or number), then it can be
+just passed as is. However, if you wish to initialize it with a HASH
+or ARRAY ref, then you need to wrap that inside a subroutine
+reference:
+
+  Class::MOP::Attribute->new(
+      'foo' => (
+          default => sub { [] },
+      )
+  );
+
+  # or ...
+
+  Class::MOP::Attribute->new(
+      'foo' => (
+          default => sub { {} },
+      )
+  );
+
+If you wish to initialize an attribute with a subroutine reference
+itself, then you need to wrap that in a subroutine as well:
+
+  Class::MOP::Attribute->new(
+      'foo' => (
+          default => sub {
+              sub { print "Hello World" }
+          },
+      )
+  );
+
+And lastly, if the value of your attribute is dependent upon some
+other aspect of the instance structure, then you can take advantage of
+the fact that when the C<default> value is called as a method:
+
+  Class::MOP::Attribute->new(
+      'object_identity' => (
+          default => sub { Scalar::Util::refaddr( $_[0] ) },
+      )
+  );
+
+Note that there is no guarantee that attributes are initialized in any
+particular order, so you cannot rely on the value of some other
+attribute when generating the default.
+
+=item I<initializer>
+
+This option can be either a method name or a subroutine
+reference. This method will be called when setting the attribute's
+value in the constructor. Unlike C<default> and C<builder>, the
+initializer is only called when a value is provided to the
+constructor. The initializer allows you to munge this value during
+object construction.
+
+The initializer is called as a method with three arguments. The first
+is the value that was passed to the constructor. The second is a
+subroutine reference that can be called to actually set the
+attribute's value, and the last is the associated
+C<Class::MOP::Attribute> object.
+
+This contrived example shows an initializer that sets the attribute to
+twice the given value.
+
+  Class::MOP::Attribute->new(
+      'doubled' => (
+          initializer => sub {
+              my ( $instance, $value, $set ) = @_;
+              $set->( $value * 2 );
+          },
+      )
+  );
+
+Since an initializer can be a method name, you can easily make
+attribute initialization use the writer:
+
+  Class::MOP::Attribute->new(
+      'some_attr' => (
+          writer      => 'some_attr',
+          initializer => 'some_attr',
+      )
+  );
+
+Your writer will need to examine C<@_> and determine under which
+context it is being called.
 
 =back
 
-The I<accessor>, I<reader>, I<writer>, I<predicate> and I<clearer> keys can
-contain either; the name of the method and an appropriate default one will be
-generated for you, B<or> a HASH ref containing exactly one key (which will be
-used as the name of the method) and one value, which should contain a CODE
-reference which will be installed as the method itself.
+The C<accessor>, C<reader>, C<writer>, C<predicate> and C<clearer>
+options all accept the same parameters. You can provide the name of
+the method, in which case an appropriate default method will be
+generated for you. Or instead you can also provide hash reference
+containing exactly one key (the method name) and one value. The value
+should be a subroutine reference, which will be installed as the
+method itself.
 
 =over 4
 
 =item I<accessor>
 
-The I<accessor> is a standard perl-style read/write accessor. It will 
-return the value of the attribute, and if a value is passed as an argument, 
-it will assign that value to the attribute.
+An C<accessor> is a standard Perl-style read/write accessor. It will
+return the value of the attribute, and if a value is passed as an
+argument, it will assign that value to the attribute.
 
-B<NOTE:>
-This method will properly handle the following code, by assigning an 
-C<undef> value to the attribute.
+Note that C<undef> is a legitimate value, so this will work:
 
   $object->set_something(undef);
 
 =item I<reader>
 
-This is a basic read-only accessor, it will just return the value of 
-the attribute.
+This is a basic read-only accessor. It returns the value of the
+attribute.
 
 =item I<writer>
 
-This is a basic write accessor, it accepts a single argument, and 
-assigns that value to the attribute. This method does not intentially 
-return a value, however perl will return the result of the last 
-expression in the subroutine, which returns in this returning the 
-same value that it was passed. 
+This is a basic write accessor, it accepts a single argument, and
+assigns that value to the attribute.
 
-B<NOTE:>
-This method will properly handle the following code, by assigning an 
-C<undef> value to the attribute.
+Note that C<undef> is a legitimate value, so this will work:
 
-  $object->set_something();
+  $object->set_something(undef);
 
 =item I<predicate>
 
-This is a basic test to see if the value of the attribute is not 
-C<undef>. It will return true (C<1>) if the attribute's value is 
-defined, and false (C<0>) otherwise.
+The predicate method returns a boolean indicating whether or not the
+attribute has been explicitly set.
+
+Note that the predicate returns true even if the attribute was set to
+a false value (C<0> or C<undef>).
 
 =item I<clearer>
 
-This is the a method that will uninitialize the attr, reverting lazy values
-back to their "unfulfilled" state.
+This method will uninitialize the attribute. After an attribute is
+cleared, its C<predicate> will return false.
+
+=item I<definition_context>
+
+Mostly, this exists as a hook for the benefit of Moose.
+
+This option should be a hash reference containing several keys which
+will be used when inlining the attribute's accessors. The keys should
+include C<line>, the line number where the attribute was created, and
+either C<file> or C<description>.
+
+This information will ultimately be used when eval'ing inlined
+accessor code so that error messages report a useful line and file
+name.
 
 =back
 
-=item B<clone (%options)>
+=item B<< $attr->clone(%options) >>
 
-=item B<initialize_instance_slot ($instance, $params)>
+This clones the attribute. Any options you provide will override the
+settings of the original attribute. You can change the name of the new
+attribute by passing a C<name> key in C<%options>.
 
-=back 
+=back
 
-=head2 Value management
+=head2 Informational
+
+These are all basic read-only accessors for the values passed into
+the constructor.
 
 =over 4
 
-=item set_value $instance, $value
+=item B<< $attr->name >>
 
-Set the value without going through the accessor. Note that this may be done to
-even attributes with just read only accessors.
+=item B<< $attr->accessor >>
 
-=item get_value $instance
+=item B<< $attr->reader >>
 
-Return the value without going through the accessor. Note that this may be done
-even to attributes with just write only accessors.
+=item B<< $attr->writer >>
 
-=back
+=item B<< $attr->predicate >>
 
-=head2 Informational
+=item B<< $attr->clearer >>
 
-These are all basic read-only value accessors for the values 
-passed into C<new>. I think they are pretty much self-explanitory.
+The C<accessor>, C<reader>, C<writer>, C<predicate>, and C<clearer>
+methods all return exactly what was passed to the constructor, so it
+can be either a string containing a method name, or a hash refrence.
 
-=over 4
+=item B<< $attr->initializer >>
+
+Returns the intializer as passed to the constructor, so this may be
+either a method name or a subroutine reference.
+
+=item B<< $attr->init_arg >>
+
+=item B<< $attr->is_default_a_coderef >>
+
+=item B<< $attr->default($instance) >>
+
+The C<$instance> argument is optional. If you don't pass it, the
+return value for this method is exactly what was passed to the
+constructor, either a simple scalar or a subroutine reference.
+
+If you I<do> pass an C<$instance> and the default is a subroutine
+reference, then the reference is called as a method on the
+C<$instance> and the generated value is returned.
+
+=item B<< $attr->slots >>
 
-=item B<name>
+Return a list of slots required by the attribute. This is usually just
+one, the name of the attribute.
 
-=item B<accessor>
+A slot is the name of the hash key used to store the attribute in an
+object instance.
 
-=item B<reader>
+=item B<< $attr->get_read_method >>
 
-=item B<writer>
+=item B<< $attr->get_write_method >>
 
-=item B<predicate>
+Returns the name of a method suitable for reading or writing the value
+of the attribute in the associated class.
 
-=item B<clearer>
+If an attribute is read- or write-only, then these methods can return
+C<undef> as appropriate.
 
-=item B<init_arg>
+=item B<< $attr->has_read_method >>
 
-=item B<is_default_a_coderef>
+=item B<< $attr->has_write_method >>
 
-=item B<default (?$instance)>
+This returns a boolean indicating whether the attribute has a I<named>
+read or write method.
 
-As noted in the documentation for C<new> above, if the I<default> 
-value is a CODE reference, this accessor will pass a single additional
-argument C<$instance> into it and return the value.
+=item B<< $attr->get_read_method_ref >>
 
-=item B<slots>
+=item B<< $attr->get_write_method_ref >>
 
-Returns a list of slots required by the attribute. This is usually 
-just one, which is the name of the attribute.
+Returns the subroutine reference of a method suitable for reading or
+writing the attribute's value in the associated class. These methods
+always return a subroutine reference, regardless of whether or not the
+attribute is read- or write-only.
 
 =back
 
@@ -585,94 +758,153 @@ These are all basic predicate methods for the values passed into C<new>.
 
 =over 4
 
-=item B<has_accessor>
+=item B<< $attr->has_accessor >>
 
-=item B<has_reader>
+=item B<< $attr->has_reader >>
 
-=item B<has_writer>
+=item B<< $attr->has_writer >>
 
-=item B<has_predicate>
+=item B<< $attr->has_predicate >>
 
-=item B<has_clearer>
+=item B<< $attr->has_clearer >>
 
-=item B<has_init_arg>
+=item B<< $attr->has_initializer >>
 
-=item B<has_default>
+=item B<< $attr->has_init_arg >>
+
+This will be I<false> if the C<init_arg> was set to C<undef>.
+
+=item B<< $attr->has_default >>
+
+This will be I<false> if the C<default> was set to C<undef>, since
+C<undef> is the default C<default> anyway.
+
+=item B<< $attr->has_builder >>
 
 =back
 
-=head2 Class association
+=head2 Value management
+
+These methods are basically "backdoors" to the instance, and can be
+used to bypass the regular accessors, but still stay within the MOP.
+
+These methods are not for general use, and should only be used if you
+really know what you are doing.
 
 =over 4
 
-=item B<associated_class>
+=item B<< $attr->initialize_instance_slot($meta_instance, $instance, $params) >>
 
-=item B<attach_to_class ($class)>
+This method is used internally to initialize the attribute's slot in
+the object C<$instance>.
 
-=item B<detach_from_class>
+The C<$params> is a hash reference of the values passed to the object
+constructor.
 
-=item B<slot_name>
+It's unlikely that you'll need to call this method yourself.
 
-=item B<allocate_slots>
+=item B<< $attr->set_value($instance, $value) >>
 
-=item B<deallocate_slots>
+Sets the value without going through the accessor. Note that this
+works even with read-only attributes.
 
-=back
+=item B<< $attr->set_initial_value($instance, $value) >>
 
-=head2 Attribute Accessor generation
+Sets the value without going through the accessor. This method is only
+called when the instance is first being initialized.
 
-=over 4
+=item B<< $attr->get_value($instance) >>
+
+Returns the value without going through the accessor. Note that this
+works even with write-only accessors.
+
+=item B<< $attr->has_value($instance) >>
+
+Return a boolean indicating whether the attribute has been set in
+C<$instance>. This how the default C<predicate> method works.
+
+=item B<< $attr->clear_value($instance) >>
 
-=item B<install_accessors>
+This will clear the attribute's value in C<$instance>. This is what
+the default C<clearer> calls.
 
-This allows the attribute to generate and install code for it's own 
-I<accessor/reader/writer/predicate> methods. This is called by 
-C<Class::MOP::Class::add_attribute>.
+Note that this works even if the attribute does not have any
+associated read, write or clear methods.
 
-This method will call C<process_accessors> for each of the possible 
-method types (accessor, reader, writer & predicate).
+=back
 
-=item B<process_accessors ($type, $value)>
+=head2 Class association
 
-This takes a C<$type> (accessor, reader, writer or predicate), and 
-a C<$value> (the value passed into the constructor for each of the
-different types). It will then either generate the method itself 
-(using the C<generate_*_method> methods listed below) or it will 
-use the custom method passed through the constructor. 
+These methods allow you to manage the attributes association with
+the class that contains it. These methods should not be used
+lightly, nor are they very magical, they are mostly used internally
+and by metaclass instances.
 
 =over 4
 
-=item B<generate_accessor_method>
+=item B<< $attr->associated_class >>
+
+This returns the C<Class::MOP::Class> with which this attribute is
+associated, if any.
+
+=item B<< $attr->attach_to_class($metaclass) >>
+
+This method stores a weakened reference to the C<$metaclass> object
+internally.
 
-=item B<generate_predicate_method>
+This method does not remove the attribute from its old class,
+nor does it create any accessors in the new class.
 
-=item B<generate_clearer_method>
+It is probably best to use the L<Class::MOP::Class> C<add_attribute>
+method instead.
 
-=item B<generate_reader_method>
+=item B<< $attr->detach_from_class >>
 
-=item B<generate_writer_method>
+This method removes the associate metaclass object from the attribute
+it has one.
+
+This method does not remove the attribute itself from the class, or
+remove its accessors.
+
+It is probably best to use the L<Class::MOP::Class>
+C<remove_attribute> method instead.
 
 =back
 
+=head2 Attribute Accessor generation
+
 =over 4
 
-=item B<generate_accessor_method_inline>
+=item B<< $attr->accessor_metaclass >>
 
-=item B<generate_predicate_method_inline>
+Accessor methods are generated using an accessor metaclass. By
+default, this is L<Class::MOP::Method::Accessor>. This method returns
+the name of the accessor metaclass that this attribute uses.
 
-=item B<generate_clearer_method_inline>
+=item B<< $attr->associate_method($method) >>
 
-=item B<generate_reader_method_inline>
+This associates a L<Class::MOP::Method> object with the
+attribute. Typically, this is called internally when an attribute
+generates its accessors.
 
-=item B<generate_writer_method_inline>
+=item B<< $attr->associated_methods >>
 
-=back
+This returns the list of methods which have been associated with the
+attribute.
 
-=item B<remove_accessors>
+=item B<< $attr->install_accessors >>
 
-This allows the attribute to remove the method for it's own 
-I<accessor/reader/writer/predicate/clearer>. This is called by 
-C<Class::MOP::Class::remove_attribute>.
+This method generates and installs code the attributes various
+accessors. It is typically called from the L<Class::MOP::Class>
+C<add_attribute> method.
+
+=item B<< $attr->remove_accessors >>
+
+This method removes all of the accessors associated with the
+attribute.
+
+This does not currently remove methods from the list returned by
+C<associated_methods>.
 
 =back
 
@@ -680,15 +912,13 @@ C<Class::MOP::Class::remove_attribute>.
 
 =over 4
 
-=item B<meta>
+=item B<< $attr->meta >>
 
-This will return a B<Class::MOP::Class> instance which is related 
-to this class.
+This will return a L<Class::MOP::Class> instance for this class.
 
-It should also be noted that B<Class::MOP> will actually bootstrap 
-this module by installing a number of attribute meta-objects into 
-it's metaclass. This will allow this class to reap all the benifits 
-of the MOP when subclassing it. 
+It should also be noted that L<Class::MOP> will actually bootstrap
+this module by installing a number of attribute meta-objects into its
+metaclass.
 
 =back
 
@@ -696,16 +926,14 @@ of the MOP when subclassing it.
 
 Stevan Little E<lt>stevan@iinteractive.comE<gt>
 
-Yuval Kogman E<lt>nothingmuch@woobling.comE<gt>
-
 =head1 COPYRIGHT AND LICENSE
 
-Copyright 2006 by Infinity Interactive, Inc.
+Copyright 2006-2009 by Infinity Interactive, Inc.
 
 L<http://www.iinteractive.com>
 
 This library is free software; you can redistribute it and/or modify
-it under the same terms as Perl itself. 
+it under the same terms as Perl itself.
 
 =cut