On Fri, Nov 12, 2010 at 9:11 AM, Matt Sergeant <matt@sergeant.org> wrote:
Sherm Pendley wrote:
On Thu, Nov 11, 2010 at 8:36 PM, Matt Sergeant <matt@sergeant.org> wrote:
Another idea I had

IMHO we've already put *far* more effort into this than is warranted for old code that's I hope to make obsolete soon anyway. So, to keep things moving forward I'm making an Executive Decision. :-)

Let's keep the code that's currently committed, with one minor change - where it currently caches all methods that get caught by the autoload, it will instead check $isSuperMethod, and only cache methods that were *not* called with SUPER::foo(). Given typical usage patterns, that means your cache code will be able to improve performance for the huge majority of method calls - probably 95% or better. That's a huge win.
OK I'm cool with that. Sorry this just got under my skin :-D

Don't be sorry - as I said, it's a huge win. A small change that makes a big difference, in 752 out of 755 method calls in the example code is nothing to sneeze at.

I hope you're just as enthusiastic about 2.0. :-)

sherm--

--
Cocoa programming in Perl:
http://camelbones.sourceforge.net