Performance optimization in the last patch: eliminate implicit_factory() entirely.

git-svn-id: http://scons.tigris.org/svn/scons/trunk@1275 fdb21ef1-2011-0410-befe-b5e4ea1792b1

diff --git a/src/engine/SCons/Node/__init__.py b/src/engine/SCons/Node/__init__.py
index bc10e21744a5667d9d83491a662bb2093371f1d2..8754c7de0f525d873d667b5b4fdbf82523c8ea57 100644 (file)
--- a/src/engine/SCons/Node/__init__.py
+++ b/src/engine/SCons/Node/__init__.py
@@ -417,16 +417,6 @@ class Node:
 
         return deps
 
-    def implicit_factory(self, path):
-        """
-        Turn a cache implicit dependency path into a node.
-        This is called so many times that doing caching
-        here is a significant performance boost.
-        __cacheable__
-        """
-        env = self.get_build_env()
-        return env.get_factory(self.builder.source_factory)(path)
-
     def get_scanner(self, env, kw={}):
         return env.get_scanner(self.scanner_key())
 
@@ -482,7 +472,8 @@ class Node:
         if implicit_cache and not implicit_deps_changed:
             implicit = self.get_stored_implicit()
             if implicit:
-                implicit = map(self.implicit_factory, implicit)
+                factory = build_env.get_factory(self.builder.source_factory)
+                implicit = map(factory, implicit)
                 self._add_child(self.implicit, self.implicit_dict, implicit)
                 calc = build_env.get_calculator()
                 if implicit_deps_unchanged or self.current(calc):