Patchwork D3877: tests: don't reimplement enumerate() in run-tests

login
register
mail settings
Submitter phabricator
Date July 3, 2018, 11:50 p.m.
Message ID <9a69dffb0c9eee0f8b531b4e00be906e@localhost.localdomain>
Download mbox | patch
Permalink /patch/32605/
State Not Applicable
Headers show

Comments

phabricator - July 3, 2018, 11:50 p.m.
martinvonz updated this revision to Diff 9424.

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST UPDATE
  https://phab.mercurial-scm.org/D3877?vs=9419&id=9424

REVISION DETAIL
  https://phab.mercurial-scm.org/D3877

AFFECTED FILES
  tests/run-tests.py

CHANGE DETAILS




To: martinvonz, #hg-reviewers
Cc: mercurial-devel

Patch

diff --git a/tests/run-tests.py b/tests/run-tests.py
--- a/tests/run-tests.py
+++ b/tests/run-tests.py
@@ -1480,10 +1480,8 @@ 
                 if expected.get(pos, None):
                     els = expected[pos]
 
-                i = 0
                 optional = []
-                while i < len(els):
-                    el = els[i]
+                for i, el in enumerate(els):
 
                     r = self.linematch(el, lout)
                     if isinstance(r, str):
@@ -1512,8 +1510,6 @@ 
                                 if not self._iftest(conditions):
                                     optional.append(i)
 
-                    i += 1
-
                 if r:
                     if r == "retry":
                         continue