Patchwork [03,of,17] branchmap: remove superfluous pass statements

login
register
mail settings
Submitter Augie Fackler
Date Sept. 30, 2017, 12:02 p.m.
Message ID <fa24e8e3eb1bbd5f5dfa.1506772929@augie-macbookpro2.roam.corp.google.com>
Download mbox | patch
Permalink /patch/24236/
State Accepted
Headers show

Comments

Augie Fackler - Sept. 30, 2017, 12:02 p.m.
# HG changeset patch
# User Augie Fackler <augie@google.com>
# Date 1506771779 14400
#      Sat Sep 30 07:42:59 2017 -0400
# Node ID fa24e8e3eb1bbd5f5dfac4af9c54bf3fb196afe4
# Parent  f80c5a4dac76cb923f83b8a51f953e17bdc34a86
branchmap: remove superfluous pass statements

Patch

diff --git a/mercurial/branchmap.py b/mercurial/branchmap.py
--- a/mercurial/branchmap.py
+++ b/mercurial/branchmap.py
@@ -251,9 +251,8 @@  class branchcache(dict):
                         'wrote %s branch cache with %d labels and %d nodes\n',
                         repo.filtername, len(self), nodecount)
         except (IOError, OSError, error.Abort) as inst:
+            # Abort may be raised by read only opener, so log and continue
             repo.ui.debug("couldn't write branch cache: %s\n" % inst)
-            # Abort may be raise by read only opener
-            pass
 
     def update(self, repo, revgen):
         """Given a branchhead cache, self, that may have extra nodes or be