removes unnecessary assignment and local variable in AR::Base#subclas…

…ses, and revises its rdoc
rails · Apr 13, 2010 · 0e27463 · 0e27463
1 parent c88b933
commit 0e27463
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/activerecord/lib/active_record/base.rb b/activerecord/lib/active_record/base.rb
@@ -1275,11 +1275,10 @@ def with_exclusive_scope(method_scoping = {}, &block)
           with_scope(method_scoping, :overwrite, &block)
         end
 
-        # ActiveRecord::Base utilizes the inherited hook to know about new subclasses.
-        # You can access the list of currently loaded ActiveRecord::Base subclasses using this accessor.
+        # Returns a list of all subclasses of this class, meaning all descendants.
         def subclasses
           @@subclasses[self] ||= []
-          @@subclasses[self] + extra = @@subclasses[self].inject([]) {|list, subclass| list + subclass.subclasses }
+          @@subclasses[self] + @@subclasses[self].inject([]) {|list, subclass| list + subclass.subclasses }
         end
 
         public :subclasses