Patchwork [2,of,5] histedit: remove useless 'else' clause

login
register
mail settings
Submitter Pierre-Yves David
Date Aug. 3, 2015, 9:25 p.m.
Message ID <4e57b7b30543c50f009e.1438637124@marginatus.alto.octopoid.net>
Download mbox | patch
Permalink /patch/10095/
State Accepted
Headers show

Comments

Pierre-Yves David - Aug. 3, 2015, 9:25 p.m.
# HG changeset patch
# User Pierre-Yves David <pierre-yves.david@fb.com>
# Date 1438606665 25200
#      Mon Aug 03 05:57:45 2015 -0700
# Node ID 4e57b7b30543c50f009e65c8cf0bf05589bc510b
# Parent  7eb4f2a653e9412c51c4163ad0b5a6e0bea6e43a
histedit: remove useless 'else' clause

This 'else: pass' clause have no effect. We drop it for clarity.

Patch

diff --git a/hgext/histedit.py b/hgext/histedit.py
--- a/hgext/histedit.py
+++ b/hgext/histedit.py
@@ -794,12 +794,10 @@  def _histedit(ui, repo, state, *freeargs
         parentnodes = [c.node() for c in repo[None].parents()]
         for n in leafs | set([state.parentctxnode]):
             if n in parentnodes:
                 hg.clean(repo, state.topmost)
                 break
-        else:
-            pass
         cleanupnode(ui, repo, 'created', tmpnodes)
         cleanupnode(ui, repo, 'temp', leafs)
         state.clear()
         return
     else: