From 10afd9000f07a648d41b65267a17b7c32823730b Mon Sep 17 00:00:00 2001 From: Avery Pennarun Date: Thu, 9 Dec 2010 06:44:47 -0800 Subject: [PATCH] Add some conditionals around some high-bandwidth debug statements. When you have lots of unmodified dependencies, building these printout strings (which aren't even printed unless you're using -d) ends up taking something like 5% of the runtime. --- redo-ifchange.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/redo-ifchange.py b/redo-ifchange.py index 2a433cc..cd706b0 100755 --- a/redo-ifchange.py +++ b/redo-ifchange.py @@ -5,7 +5,7 @@ from helpers import debug, debug2, err, unlink def dirty_deps(f, depth, max_changed): - debug('%s?%s\n' % (depth, f.name)) + if vars.DEBUG >= 1: debug('%s?%s\n' % (depth, f.name)) if f.changed_runid == None: debug('%s-- DIRTY (never built)\n' % depth) @@ -14,7 +14,7 @@ def dirty_deps(f, depth, max_changed): debug('%s-- DIRTY (built)\n' % depth) return True # has been built more recently than parent if f.is_checked(): - debug('%s-- CLEAN (checked)\n' % depth) + if vars.DEBUG >= 1: debug('%s-- CLEAN (checked)\n' % depth) return False # has already been checked during this session if not f.stamp: