[PATCH 15 of 23 v3] rebase: move local variable 'obsoletenotrebased' to the RRS class

Kostia Balytskyi ikostia at fb.com
Thu Jun 2 10:13:56 EDT 2016


# HG changeset patch
# User Kostia Balytskyi <ikostia at fb.com>
# Date 1464871663 -3600
#      Thu Jun 02 13:47:43 2016 +0100
# Node ID 6fd028f0df5cc4a769ada7df46bc5a4a51338fbc
# Parent  09786ee0bd37f2a0af30a84b583b77ca3f2bc282
rebase: move local variable 'obsoletenotrebased' to the RRS class

diff --git a/hgext/rebase.py b/hgext/rebase.py
--- a/hgext/rebase.py
+++ b/hgext/rebase.py
@@ -163,6 +163,7 @@
         # keepopen is not meant for use on the command line, but by
         # other extensions
         self.keepopen = opts.get('keepopen', False)
+        self.obsoletenotrebased = {}
 
     def validateoptions(self):
         if self.opts.get('interactive'):
@@ -402,15 +403,15 @@
                              rtstate.state,
                              activebookmark=rtstate.activebookmark)
 
-            obsoletenotrebased = {}
+            rtstate.obsoletenotrebased = {}
             if ui.configbool('experimental', 'rebaseskipobsolete',
                              default=True):
                 rebaseobsrevs = set([r for r, st in rtstate.state.items()
                                      if st == revprecursor])
                 rebasesetrevs = set(rtstate.state.keys())
-                obsoletenotrebased = _computeobsoletenotrebased(repo,
-                                        rebaseobsrevs, rtstate.target)
-                rebaseobsskipped = set(obsoletenotrebased)
+                rtstate.obsoletenotrebased = _computeobsoletenotrebased(repo,
+                                                rebaseobsrevs, rtstate.target)
+                rebaseobsskipped = set(rtstate.obsoletenotrebased)
                 _checkobsrebase(repo, ui, rebaseobsrevs, rebasesetrevs,
                                 rebaseobsskipped)
         else:
@@ -429,21 +430,21 @@
                       " unrebased descendants"),
                     hint=_('use --keep to keep original changesets'))
 
-            obsoletenotrebased = {}
+            rtstate.obsoletenotrebased = {}
             if ui.configbool('experimental', 'rebaseskipobsolete',
                              default=True):
                 rebasesetrevs = set(rebaseset)
                 rebaseobsrevs = _filterobsoleterevs(repo, rebasesetrevs)
-                obsoletenotrebased = _computeobsoletenotrebased(repo,
+                rtstate.obsoletenotrebased = _computeobsoletenotrebased(repo,
                                                                 rebaseobsrevs,
                                                                 dest)
-                rebaseobsskipped = set(obsoletenotrebased)
+                rebaseobsskipped = set(rtstate.obsoletenotrebased)
                 _checkobsrebase(repo, ui, rebaseobsrevs,
                                               rebasesetrevs,
                                               rebaseobsskipped)
 
             result = buildstate(repo, dest, rebaseset, rtstate.collapsef,
-                                obsoletenotrebased)
+                                rtstate.obsoletenotrebased)
 
             if not result:
                 # Empty state built, nothing to rebase
@@ -511,7 +512,7 @@
                 p1, p2, base = defineparents(repo, rev, rtstate.target,
                                              rtstate.state,
                                              rtstate.targetancestors,
-                                             obsoletenotrebased)
+                                             rtstate.obsoletenotrebased)
                 storestatus(repo, rtstate.originalwd, rtstate.target,
                             rtstate.state, rtstate.collapsef, rtstate.keepf,
                             rtstate.keepbranchesf, rtstate.external,
@@ -561,7 +562,7 @@
             elif rtstate.state[rev] == revignored:
                 ui.status(_('not rebasing ignored %s\n') % desc)
             elif rtstate.state[rev] == revprecursor:
-                targetctx = repo[obsoletenotrebased[rev]]
+                targetctx = repo[rtstate.obsoletenotrebased[rev]]
                 desctarget = '%d:%s "%s"' % (targetctx.rev(), targetctx,
                              targetctx.description().split('\n', 1)[0])
                 msg = _('note: not rebasing %s, already in destination as %s\n')
@@ -580,7 +581,7 @@
             p1, p2, _base = defineparents(repo, min(rtstate.state),
                                           rtstate.target, rtstate.state,
                                           rtstate.targetancestors,
-                                          obsoletenotrebased)
+                                          rtstate.obsoletenotrebased)
             editopt = opts.get('edit')
             editform = 'rebase.collapse'
             if rtstate.collapsemsg:


More information about the Mercurial-devel mailing list