transaction: simplify `undo.backupfiles` file creation with a variable
authorPierre-Yves David <pierre-yves.david@octobus.net>
Wed, 09 Jun 2021 00:59:44 +0200
changeset 47422 88439c6fbafc
parent 47421 d2e0226b511a
child 47423 be903d043099
transaction: simplify `undo.backupfiles` file creation with a variable This make the code shorter and clearer. (This is an opportunity improvement while looking at something next to that.) Differential Revision: https://phab.mercurial-scm.org/D10846
mercurial/transaction.py
--- a/mercurial/transaction.py	Wed Jun 09 00:59:04 2021 +0200
+++ b/mercurial/transaction.py	Wed Jun 09 00:59:44 2021 +0200
@@ -633,9 +633,9 @@
         """write transaction data for possible future undo call"""
         if self._undoname is None:
             return
-        undobackupfile = self._opener.open(
-            b"%s.backupfiles" % self._undoname, b'w'
-        )
+
+        undo_backup_path = b"%s.backupfiles" % self._undoname
+        undobackupfile = self._opener.open(undo_backup_path, b'w')
         undobackupfile.write(b'%d\n' % version)
         for l, f, b, c in self._backupentries:
             if not f:  # temporary file