Remove StatIfNecessary call that is never necessary
authorBrad King <brad.king@kitware.com>
Tue, 17 Nov 2015 19:28:11 +0000 (14:28 -0500)
committerBrad King <brad.king@kitware.com>
Thu, 25 Feb 2016 21:01:21 +0000 (16:01 -0500)
commit1a8e5455939ed3f701141cc72262add9a7d75055
treeee95c94021b3fe965c211f41d15db732c2e2c0e0
parentd1763746b65cc7349d4ed9478befdb651aa24589
Remove StatIfNecessary call that is never necessary

The call to StatIfNecessary in DependencyScan::RecomputeOutputsDirty was
added by commit v1.4.0^2~7^2~1 (Share more code between CleanNode() and
RecomputeDirty(), 2013-09-02) while consolidating code paths.  However,
it was needed only when called from RecomputeDirty because prior to
refactoring the CleanNode code path did not call it.

Later commit v1.6.0^2~46^2 (Let DependencyScan::RecomputeDirty() work
correclty with cyclic graphs, 2014-12-07) added back to RecomputeDirty a
loop over outputs that calls StatIfNecessary.  Therefore
RecomputeOutputsDirty no longer needs to call StatIfNecessary for either
of its own callers.
src/graph.cc