exchange: drop now-unnecessary "local" from lock name variables
authorMartin von Zweigbergk <martinvonz@google.com>
Mon, 14 Aug 2017 16:14:14 -0700
changeset 33812 20d663a104fb
parent 33811 f2befc1f4cf4
child 33813 5904511fc9f8
exchange: drop now-unnecessary "local" from lock name variables Since fda0867cfe03 (exchange: drop support for lock-based unbundling (BC), 2017-08-06), there is no more remote locking. Differential Revision: https://phab.mercurial-scm.org/D390
mercurial/exchange.py
--- a/mercurial/exchange.py	Fri Jul 28 21:49:44 2017 -0700
+++ b/mercurial/exchange.py	Mon Aug 14 16:14:14 2017 -0700
@@ -438,17 +438,17 @@
         raise error.Abort(_('cannot push: destination does not support the '
                             'unbundle wire protocol command'))
 
-    # get local lock as we might write phase data
-    localwlock = locallock = None
-    locallocked = False
+    # get lock as we might write phase data
+    wlock = lock = None
+    locked = False
     try:
         # bundle2 push may receive a reply bundle touching bookmarks or other
         # things requiring the wlock. Take it now to ensure proper ordering.
         maypushback = pushop.ui.configbool('experimental', 'bundle2.pushback')
         if (not _forcebundle1(pushop)) and maypushback:
-            localwlock = pushop.repo.wlock()
-        locallock = pushop.repo.lock()
-        locallocked = True
+            wlock = pushop.repo.wlock()
+        lock = pushop.repo.lock()
+        locked = True
     except IOError as err:
         if err.errno != errno.EACCES:
             raise
@@ -458,7 +458,7 @@
         msg = 'cannot lock source repository: %s\n' % err
         pushop.ui.debug(msg)
     try:
-        if locallocked:
+        if locked:
             pushop.trmanager = transactionmanager(pushop.repo,
                                                   'push-response',
                                                   pushop.remote.url())
@@ -476,10 +476,10 @@
     finally:
         if pushop.trmanager:
             pushop.trmanager.release()
-        if locallock is not None:
-            locallock.release()
-        if localwlock is not None:
-            localwlock.release()
+        if lock is not None:
+            lock.release()
+        if wlock is not None:
+            wlock.release()
 
     return pushop