From: Christopher H. Laco <claco@cpan.org>
Date: Wed, 8 Nov 2006 02:36:41 +0000 (+0000)
Subject: Big speedup for get_inherited under heavy usage
X-Git-Tag: v0.04000~1
X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=commitdiff_plain;h=a49c32d971911283265695ba8549a4052aba6026;p=p5sagit%2FClass-Accessor-Grouped.git

Big speedup for get_inherited under heavy usage
Check current class variable before calculating super paths
Cache super paths locally to speed up multiple calls on the same class
---

diff --git a/Changes b/Changes
index d5e491a..9663152 100644
--- a/Changes
+++ b/Changes
@@ -1,5 +1,10 @@
 Revision history for Class::Accessor::Grouped.
 
+0.03  2006-11-07 21:33::35
+    - big speedup for get_inherited
+    - get_inherited now checks the current class first before calculating super_path
+    - get_inherited now caches super_path results
+
 0.02  2006-06-26 19:23:13
     - Added return statement to end of get_inherited
     - Fixed pod NAME
diff --git a/lib/Class/Accessor/Grouped.pm b/lib/Class/Accessor/Grouped.pm
index 9ac75d1..94a1281 100644
--- a/lib/Class/Accessor/Grouped.pm
+++ b/lib/Class/Accessor/Grouped.pm
@@ -6,7 +6,7 @@ use Class::ISA;
 use Scalar::Util qw/blessed reftype/;
 use vars qw($VERSION);
 
-$VERSION = '0.02';
+$VERSION = '0.03';
 
 =head1 NAME
 
@@ -290,20 +290,30 @@ base class, inherited and changed in subclasses, and inherited and changed for o
 
 sub get_inherited {
     my ($self, $get) = @_;
+    my $class;
 
     if (blessed $self) {
-        if (reftype($self) eq 'HASH' && exists $self->{$get}) {
+        my $reftype = reftype $self;
+        $class = ref $self;
+
+        if ($reftype eq 'HASH' && exists $self->{$get}) {
             return $self->{$get};
-        } elsif (reftype($self) ne 'HASH') {
+        } elsif ($reftype ne 'HASH') {
             croak('Cannot get inherited value on an object instance that is not hash-based');
         };
+    } else {
+        $class = $self;
     };
 
     no strict 'refs';
+    return ${$class.'::__cag_'.$get} if defined(${$class.'::__cag_'.$get});
 
-    my @supers = Class::ISA::self_and_super_path(ref $self || $self);
-    foreach (@supers) {
-        return ${$_.'::_'.$get} if defined(${$_.'::_'.$get});
+    if (!@{$class.'::__cag_supers'}) {
+        @{$class.'::__cag_supers'} = $self->get_super_paths;
+    };
+
+    foreach (@{$class.'::__cag_supers'}) {
+        return ${$_.'::__cag_'.$get} if defined(${$_.'::__cag_'.$get});
     };
 
     return;
@@ -339,10 +349,22 @@ sub set_inherited {
     } else {
         no strict 'refs';
 
-        return ${$self.'::_'.$set} = $val;
+        return ${$self.'::__cag_'.$set} = $val;
     };
 }
 
+=head2 get_super_paths
+
+Returns a list of 'parent' or 'super' class names that the current class inherited from.
+
+=cut
+
+sub get_super_paths {
+    my $class = blessed $_[0] || $_[0];
+
+    return Class::ISA::super_path($class);
+};
+
 1;
 
 =head1 AUTHORS