Don't call looping Module#constants from loop in Type.include_modules_from.

jemc · jemc · commit bd8361b8649e · 2015-05-02T18:47:18.000-07:00
This looping over the same series of direct_superclasses is exponentially
redundant and wasteful (not to mention the extra overhead in Module#constants).
diff --git a/kernel/common/type.rb b/kernel/common/type.rb
@@ -322,7 +322,7 @@ def self.include_modules_from(included_module, klass)
             changed = true
           end
 
-          constants_changed ||= mod.constants.any?
+          constants_changed ||= mod.constant_table.size > 0
         end
 
         mod = mod.direct_superclass
