D3561: dispatch: drop redundant setting of "ret" to -1

martinvonz (Martin von Zweigbergk) phabricator at mercurial-scm.org
Tue May 15 17:34:04 UTC 2018


martinvonz created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.

REVISION SUMMARY
  It has already been set to -1 since https://phab.mercurial-scm.org/rHG6f9ac3cb098750529b832ab902ad93c72dc5b8c8 (dispatch: unify
  handling of None returned by a command function, 2018-05-12).

REPOSITORY
  rHG Mercurial

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

AFFECTED FILES
  mercurial/dispatch.py

CHANGE DETAILS

diff --git a/mercurial/dispatch.py b/mercurial/dispatch.py
--- a/mercurial/dispatch.py
+++ b/mercurial/dispatch.py
@@ -230,7 +230,6 @@
         except IOError as inst:
             if inst.errno != errno.EPIPE:
                 raise
-        ret = -1
     finally:
         duration = util.timer() - starttime
         req.ui.flush()



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


More information about the Mercurial-devel mailing list