# HG changeset patch # User Mads Kiilerich # Date 2014-08-15 02:37:46 # Node ID f8dc6599da5d2220aeee736f6e59f95046421157 # Parent 35c2ea4ca26fdf50802bad41c73fd93fbcf905aa cleanup: name unused variables using convention of leading _ This helps checker tools ... and readability for those who knows and follows the convention. diff --git a/mercurial/transaction.py b/mercurial/transaction.py --- a/mercurial/transaction.py +++ b/mercurial/transaction.py @@ -24,7 +24,7 @@ def active(func): return _active def _playback(journal, report, opener, entries, backupentries, unlink=True): - for f, o, ignore in entries: + for f, o, _ignore in entries: if o or not unlink: try: fp = opener(f, 'a') @@ -41,7 +41,7 @@ def _playback(journal, report, opener, e raise backupfiles = [] - for f, b, ignore in backupentries: + for f, b, _ignore in backupentries: filepath = opener.join(f) backuppath = opener.join(b) try: @@ -265,7 +265,7 @@ class transaction(object): self.opener.unlink(self.journal) if self.opener.isfile(self.backupjournal): self.opener.unlink(self.backupjournal) - for f, b, _ignore in self.backupentries: + for _f, b, _ignore in self.backupentries: self.opener.unlink(b) self.backupentries = [] self.journal = None