diff options
| author | Adam Spiers <git@adamspiers.org> | 2016-07-03 09:13:03 -0600 |
|---|---|---|
| committer | Adam Spiers <git@adamspiers.org> | 2018-05-15 13:42:16 +0100 |
| commit | 1297a749c7842f49ef45fbda14163b4ad62eb639 (patch) | |
| tree | 14aa287e2101871156edd8009bf91335cfc3e849 /git_deps | |
| parent | bab7283b2cedebdf7ac46a903165d072bcef3a9e (diff) | |
| download | git-deps-1297a749c7842f49ef45fbda14163b4ad62eb639.tar.gz | |
use dependent_sha1 temporary variable
Avoid calling hex() repeatedly
Diffstat (limited to 'git_deps')
| -rw-r--r-- | git_deps/detector.py | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/git_deps/detector.py b/git_deps/detector.py index e395330..88c784c 100644 --- a/git_deps/detector.py +++ b/git_deps/detector.py @@ -106,19 +106,20 @@ class DependencyDetector(object): else: self.logger.debug(" TODO list now: %s" % " ".join(sha1s)) dependent = self.todo.pop(0) - del self.todo_d[dependent.hex] + dependent_sha1 = dependent.hex + del self.todo_d[dependent_sha1] self.logger.debug(" Processing %s from TODO list" % - dependent.hex[:8]) + dependent_sha1[:8]) self.notify_listeners('new_commit', dependent) for parent in dependent.parents: self.find_dependencies_with_parent(dependent, parent) - self.done.append(dependent.hex) - self.done_d[dependent.hex] = True + self.done.append(dependent_sha1) + self.done_d[dependent_sha1] = True self.logger.debug(" Found all dependencies for %s" % - dependent.hex[:8]) + dependent_sha1[:8]) # A commit won't have any dependencies if it only added new files - dependencies = self.dependencies.get(dependent.hex, {}) + dependencies = self.dependencies.get(dependent_sha1, {}) self.notify_listeners('dependent_done', dependent, dependencies) self.logger.debug("Finished processing TODO list") |