[Moose-commits] r7592 - Moose/trunk/lib/Moose/Meta/Method

[doy at code2.0beta.co.uk]

Author: doy
Date: 2009-02-09 16:26:31 -0800 (Mon, 09 Feb 2009)
New Revision: 7592

Modified:
   Moose/trunk/lib/Moose/Meta/Method/Delegation.pm
Log:
add a note about not inlining delegation methods



Modified: Moose/trunk/lib/Moose/Meta/Method/Delegation.pm
===================================================================
--- Moose/trunk/lib/Moose/Meta/Method/Delegation.pm	2009-02-09 22:18:13 UTC (rev 7591)
+++ Moose/trunk/lib/Moose/Meta/Method/Delegation.pm	2009-02-10 00:26:31 UTC (rev 7592)
@@ -72,6 +72,10 @@
     # out. However, the more I thought about it, the less I liked it
     # doing the goto, and I prefered the act of delegation being
     # actually represented in the stack trace.  - SL
+    # not inlining this, since it won't really speed things up at
+    # all... the only thing that would end up different would be
+    # interpolating in $method_to_call, and a bunch of things in the
+    # error handling that mostly never gets called - doy
     $self->{body} = sub {
         my $instance = shift;
         my $proxy    = $instance->$accessor();