From: stevenknight <stevenknight@fdb21ef1-2011-0410-befe-b5e4ea1792b1>
Date: Mon, 4 Apr 2005 03:45:45 +0000 (+0000)
Subject: Performance optimization in the last patch:  eliminate implicit_factory() entirely.
X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=c6124ecb6d61ff555b63bdd6ccf5dad6177de5bb;p=scons.git

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 bc10e217..8754c7de 100644
--- 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):