Add source printing code to Method::Generated that is contralled by $ENV{MOP_PRINT_SO...
[gitmo/Class-MOP.git] / lib / Class / MOP / Method / Generated.pm
index 06cf9c5..63e2f76 100644 (file)
@@ -6,84 +6,108 @@ use warnings;
 
 use Carp 'confess';
 
-our $VERSION   = '0.02';
+our $VERSION   = '0.91';
+$VERSION = eval $VERSION;
 our $AUTHORITY = 'cpan:STEVAN';
 
 use base 'Class::MOP::Method';
 
+use constant _PRINT_SOURCE => $ENV{MOP_PRINT_SOURCE} ? 1 : 0;
+
+## accessors
+
 sub new {
-    my $class   = shift;
-    my %options = @_;  
-        
-    ($options{package_name} && $options{name})
-        || confess "You must supply the package_name and name parameters";     
-        
-    my $self = bless {
-        # from our superclass
-        '&!body'          => undef,
-        '$!package_name'  => $options{package_name},
-        '$!name'          => $options{name},        
-        # specific to this subclass
-        '$!is_inline'     => ($options{is_inline} || 0),
-    } => $class;
-    
-    $self->initialize_body;
-    
-    return $self;
+    confess __PACKAGE__ . " is an abstract base class, you must provide a constructor.";
 }
 
-## accessors
+sub is_inline { $_[0]{is_inline} }
 
-sub is_inline { (shift)->{'$!is_inline'} }
+sub definition_context { $_[0]{definition_context} }
 
-sub initialize_body {
+sub _initialize_body {
     confess "No body to initialize, " . __PACKAGE__ . " is an abstract base class";
 }
 
+sub _eval_closure {
+    # my ($self, $captures, $sub_body) = @_;
+    my $__captures = $_[1];
+
+    my $code;
+
+    my $e = do {
+        local $@;
+        local $SIG{__DIE__};
+        my $source = join
+            "\n", (
+            map {
+                /^([\@\%\$])/
+                    or die "capture key should start with \@, \% or \$: $_";
+                q[my ] 
+                    . $_ . q[ = ] 
+                    . $1
+                    . q[{$__captures->{']
+                    . $_ . q['}};];
+                } keys %$__captures
+            ),
+            $_[2];
+        print STDERR $_[0]->name, ' ', $source, "\n" if _PRINT_SOURCE;
+        $code = eval $source;
+        $@;
+    };
+
+    return ( $code, $e );
+}
 
+sub _add_line_directive {
+    my ( $self, %args ) = @_;
 
-1;
-
-__END__
+    my ( $line, $file );
 
-=pod
+    if ( my $ctx = ( $args{context} || $self->definition_context ) ) {
+        $line = $ctx->{line};
+        if ( my $desc = $ctx->{description} ) {
+            $file = "$desc defined at $ctx->{file}";
+        } else {
+            $file = $ctx->{file};
+        }
+    } else {
+        ( $line, $file ) = ( 0, "generated method (unknown origin)" );
+    }
 
-=head1 NAME 
+    my $code = $args{code};
 
-Class::MOP::Method::Generated - Abstract base class for generated methods
+    # if it's an array of lines, join it up
+    # don't use newlines so that the definition context is more meaningful
+    $code = join(@$code, ' ') if ref $code;
 
-=head1 DESCRIPTION
-
-This is a C<Class::MOP::Method> subclass which is used interally 
-by C<Class::MOP::Method::Accessor> and C<Class::MOP::Method::Constructor>.
-
-=head1 METHODS
-
-=over 4
+    return qq{#line $line "$file"\n} . $code;
+}
 
-=item B<new (%options)>
+sub _compile_code {
+    my ( $self, %args ) = @_;
 
-This creates the method based on the criteria in C<%options>, 
-these options are:
+    my $code = $self->_add_line_directive(%args);
 
-=over 4
+    return $self->_eval_closure($args{environment}, $code);
+}
 
-=item I<is_inline>
+1;
 
-This is a boolean to indicate if the method should be generated
-as a closure, or as a more optimized inline version.
+__END__
 
-=back
+=pod
 
-=item B<is_inline>
+=head1 NAME 
 
-This returns the boolean which was passed into C<new>.
+Class::MOP::Method::Generated - Abstract base class for generated methods
 
-=item B<initialize_body>
+=head1 DESCRIPTION
 
-This is an abstract method and will throw an exception if called.
+This is a C<Class::MOP::Method> subclass which is subclassed by
+C<Class::MOP::Method::Accessor> and
+C<Class::MOP::Method::Constructor>.
 
-=back
+It is not intended to be used directly.
 
 =head1 AUTHORS
 
@@ -91,7 +115,7 @@ Stevan Little E<lt>stevan@iinteractive.comE<gt>
 
 =head1 COPYRIGHT AND LICENSE
 
-Copyright 2006-2008 by Infinity Interactive, Inc.
+Copyright 2006-2009 by Infinity Interactive, Inc.
 
 L<http://www.iinteractive.com>