X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=lib%2FClass%2FMOP%2FMethod.pm;h=8b3c2b4df1363b62887f029a585e5e63beb79193;hb=fd662d66ae039fddacc0593f1bf732fb843c3a19;hp=912368aab02cf3dcfb7c54a28458482bc537e6d3;hpb=8768d570c91df985abc49095cf36f1a0825a0eab;p=gitmo%2FClass-MOP.git

diff --git a/lib/Class/MOP/Method.pm b/lib/Class/MOP/Method.pm
index 912368a..8b3c2b4 100644
--- a/lib/Class/MOP/Method.pm
+++ b/lib/Class/MOP/Method.pm
@@ -19,103 +19,14 @@ sub meta {
 
 # construction
 
-sub new { 
+sub wrap { 
     my $class = shift;
     my $code  = shift;
-    (reftype($code) && reftype($code) eq 'CODE')
+    ('CODE' eq (reftype($code) || ''))
         || confess "You must supply a CODE reference to bless";
     bless $code => blessed($class) || $class;
 }
 
-{
-	my %MODIFIERS;
-	
-	sub wrap {
-		my $code = shift;
-		(blessed($code))
-			|| confess "Can only ask the package name of a blessed CODE";
-		my $modifier_table = { 
-			orig   => $code,
-			before => [],
-			after  => [],		
-			around => {
-				cache   => $code,
-				methods => [],
-			},
-		};
-		my $method = $code->new(sub {
-			$_->(@_) for @{$modifier_table->{before}};
-			my (@rlist, $rval);
-			if (defined wantarray) {
-				if (wantarray) {
-					@rlist = $modifier_table->{around}->{cache}->(@_);
-				}
-				else {
-					$rval = $modifier_table->{around}->{cache}->(@_);
-				}
-			}
-			else {
-				$modifier_table->{around}->{cache}->(@_);
-			}
-			$_->(@_) for @{$modifier_table->{after}};			
-			return unless defined wantarray;
-			return wantarray ? @rlist : $rval;
-		});	
-		$MODIFIERS{$method} = $modifier_table;
-		$method;  
-	}
-	
-	sub add_before_modifier {
-		my $code     = shift;
-		my $modifier = shift;
-		(exists $MODIFIERS{$code})
-			|| confess "You must first wrap your method before adding a modifier";		
-		(blessed($code))
-			|| confess "Can only ask the package name of a blessed CODE";
-	    (reftype($modifier) && reftype($modifier) eq 'CODE')
-	        || confess "You must supply a CODE reference for a modifier";			
-		unshift @{$MODIFIERS{$code}->{before}} => $modifier;
-	}
-	
-	sub add_after_modifier {
-		my $code     = shift;
-		my $modifier = shift;
-		(exists $MODIFIERS{$code})
-			|| confess "You must first wrap your method before adding a modifier";		
-		(blessed($code))
-			|| confess "Can only ask the package name of a blessed CODE";
-	    (reftype($modifier) && reftype($modifier) eq 'CODE')
-	        || confess "You must supply a CODE reference for a modifier";			
-		push @{$MODIFIERS{$code}->{after}} => $modifier;
-	}
-	
-	{
-		my $compile_around_method = sub {{
-	    	my $f1 = pop;
-	    	return $f1 unless @_;
-	    	my $f2 = pop;
-	    	push @_, sub { $f2->( $f1, @_ ) };
-			redo;
-		}};
-	
-		sub add_around_modifier {
-			my $code     = shift;
-			my $modifier = shift;
-			(exists $MODIFIERS{$code})
-				|| confess "You must first wrap your method before adding a modifier";		
-			(blessed($code))
-				|| confess "Can only ask the package name of a blessed CODE";
-		    (reftype($modifier) && reftype($modifier) eq 'CODE')
-		        || confess "You must supply a CODE reference for a modifier";			
-			unshift @{$MODIFIERS{$code}->{around}->{methods}} => $modifier;		
-			$MODIFIERS{$code}->{around}->{cache} = $compile_around_method->(
-				@{$MODIFIERS{$code}->{around}->{methods}},
-				$MODIFIERS{$code}->{orig}
-			);
-		}	
-	}
-}
-
 # informational
 
 sub package_name { 
@@ -132,6 +43,163 @@ sub name {
 	svref_2object($code)->GV->NAME;
 }
 
+sub fully_qualified_name {
+	my $code = shift;
+	(blessed($code))
+		|| confess "Can only ask the package name of a blessed CODE";
+	$code->package_name . '::' . $code->name;		
+}
+
+package Class::MOP::Method::Wrapped;
+
+use strict;
+use warnings;
+
+use Carp         'confess';
+use Scalar::Util 'reftype', 'blessed';
+use Sub::Name    'subname';
+
+our $VERSION = '0.01';
+
+our @ISA = ('Class::MOP::Method');	
+
+# NOTE:
+# this ugly beast is the result of trying 
+# to micro optimize this as much as possible
+# while not completely loosing maintainability.
+# At this point it's "fast enough", after all
+# you can't get something for nothing :)
+my $_build_wrapped_method = sub {
+	my $modifier_table = shift;
+	my ($before, $after, $around) = (
+		$modifier_table->{before},
+		$modifier_table->{after},		
+		$modifier_table->{around},		
+	);
+	if (@$before && @$after) {
+		$modifier_table->{cache} = sub {
+			$_->(@_) for @{$before};
+			my @rval;
+			((defined wantarray) ?
+				((wantarray) ? 
+					(@rval = $around->{cache}->(@_)) 
+					: 
+					($rval[0] = $around->{cache}->(@_)))
+				:
+				$around->{cache}->(@_));
+			$_->(@_) for @{$after};			
+			return unless defined wantarray;
+			return wantarray ? @rval : $rval[0];
+		}		
+	}
+	elsif (@$before && !@$after) {
+		$modifier_table->{cache} = sub {
+			$_->(@_) for @{$before};
+			return $around->{cache}->(@_);
+		}		
+	}
+	elsif (@$after && !@$before) {
+		$modifier_table->{cache} = sub {
+			my @rval;
+			((defined wantarray) ?
+				((wantarray) ? 
+					(@rval = $around->{cache}->(@_)) 
+					: 
+					($rval[0] = $around->{cache}->(@_)))
+				:
+				$around->{cache}->(@_));
+			$_->(@_) for @{$after};			
+			return unless defined wantarray;
+			return wantarray ? @rval : $rval[0];
+		}		
+	}
+	else {
+		$modifier_table->{cache} = $around->{cache};
+	}
+};
+
+my %MODIFIERS;
+
+sub wrap {
+	my $class = shift;
+	my $code  = shift;
+	(blessed($code) && $code->isa('Class::MOP::Method'))
+		|| confess "Can only wrap blessed CODE";
+	my $modifier_table = { 
+		cache  => undef,
+		orig   => $code,
+		before => [],
+		after  => [],		
+		around => {
+			cache   => $code,
+			methods => [],		
+		},
+	};
+	$_build_wrapped_method->($modifier_table);
+	my $method = $class->SUPER::wrap(sub { $modifier_table->{cache}->(@_) });	
+	$MODIFIERS{$method} = $modifier_table;
+	$method;  
+}
+
+sub add_before_modifier {
+	my $code     = shift;
+	my $modifier = shift;
+	(exists $MODIFIERS{$code})
+		|| confess "You must first wrap your method before adding a modifier";		
+	(blessed($code))
+		|| confess "Can only ask the package name of a blessed CODE";
+	('CODE' eq (reftype($code) || ''))
+        || confess "You must supply a CODE reference for a modifier";			
+	unshift @{$MODIFIERS{$code}->{before}} => $modifier;
+	$_build_wrapped_method->($MODIFIERS{$code});
+}
+
+sub add_after_modifier {
+	my $code     = shift;
+	my $modifier = shift;
+	(exists $MODIFIERS{$code})
+		|| confess "You must first wrap your method before adding a modifier";		
+	(blessed($code))
+		|| confess "Can only ask the package name of a blessed CODE";
+    ('CODE' eq (reftype($code) || ''))
+        || confess "You must supply a CODE reference for a modifier";			
+	push @{$MODIFIERS{$code}->{after}} => $modifier;
+	$_build_wrapped_method->($MODIFIERS{$code});	
+}
+
+{
+	# NOTE:
+	# this is another possible canidate for 
+	# optimization as well. There is an overhead
+	# associated with the currying that, if 
+	# eliminated might make around modifiers
+	# more manageable.
+	my $compile_around_method = sub {{
+    	my $f1 = pop;
+    	return $f1 unless @_;
+    	my $f2 = pop;
+    	push @_, sub { $f2->( $f1, @_ ) };
+		redo;
+	}};
+
+	sub add_around_modifier {
+		my $code     = shift;
+		my $modifier = shift;
+		(exists $MODIFIERS{$code})
+			|| confess "You must first wrap your method before adding a modifier";		
+		(blessed($code))
+			|| confess "Can only ask the package name of a blessed CODE";
+	    ('CODE' eq (reftype($code) || ''))
+	        || confess "You must supply a CODE reference for a modifier";			
+		unshift @{$MODIFIERS{$code}->{around}->{methods}} => $modifier;		
+		$MODIFIERS{$code}->{around}->{cache} = $compile_around_method->(
+			@{$MODIFIERS{$code}->{around}->{methods}},
+			$MODIFIERS{$code}->{orig}
+		);
+		$_build_wrapped_method->($MODIFIERS{$code});		
+	}	
+}
+
 1;
 
 __END__
@@ -173,7 +241,7 @@ to this class.
 
 =over 4
 
-=item B<new (&code)>
+=item B<wrap (&code)>
 
 This simply blesses the C<&code> reference passed to it.
 
@@ -187,13 +255,33 @@ This simply blesses the C<&code> reference passed to it.
 
 =item B<package_name>
 
+=item B<fully_qualified_name>
+
 =back
 
-=head1 SEE ALSO
+=head1 Class::MOP::Method::Wrapped METHODS
+
+=head2 Construction
+
+=over 4
 
-http://dirtsimple.org/2005/01/clos-style-method-combination-for.html
+=item B<wrap (&code)>
+
+This simply blesses the C<&code> reference passed to it.
 
-http://www.gigamonkeys.com/book/object-reorientation-generic-functions.html
+=back
+
+=head2 Modifiers
+
+=over 4
+
+=item B<add_before_modifier ($code)>
+
+=item B<add_after_modifier ($code)>
+
+=item B<add_around_modifier ($code)>
+
+=back
 
 =head1 AUTHOR