Patchwork D3878: tests: remove some redundant code in run-tests

login
register
mail settings
Submitter phabricator
Date July 3, 2018, 11:43 p.m.
Message ID <differential-rev-PHID-DREV-6fchrycnyz4vaqixnwus-req@phab.mercurial-scm.org>
Download mbox | patch
Permalink /patch/32601/
State Superseded
Headers show

Comments

phabricator - July 3, 2018, 11:43 p.m.
martinvonz created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.

REVISION SUMMARY
  These two lines also happen below if "r" is truthy, which it is in
  this case since it's equal to "retry".

REPOSITORY
  rHG Mercurial

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

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
@@ -1490,8 +1490,6 @@ 
                             r = '' # Warn only this line.
                         elif r == "retry":
                             postout.append(b'  ' + el)
-                            els.pop(i)
-                            break
                         else:
                             log('\ninfo, unknown linematch result: %r\n' % r)
                             r = False