Comments
Patch
@@ -995,6 +995,7 @@
return 0
parents = self.dirstate.parents()
+ self.destroying()
transaction.rollback(self.sopener, self.sjoin('undo'), ui.warn)
if os.path.exists(self.join('undo.bookmarks')):
util.rename(self.join('undo.bookmarks'),
@@ -1475,6 +1476,19 @@
tr.release()
lock.release()
+ def destroying(self):
+ '''Inform the repository that nodes are about to be destroyed.
+ Intended for use by strip and rollback, so there's a common
+ place for anything that has to be done before destroying history.
+
+ This is mostly useful for saving state that is in memory and waiting
+ to be flushed when the current lock is released. Because a call to
+ destroyed is imminent, the repo will be invalidated causing those
+ changes to stay in memory (waiting for the next unlock), or vanish
+ completely.
+ '''
+ pass
+
def destroyed(self, newheadnodes=None):
'''Inform the repository that nodes have been destroyed.
Intended for use by strip and rollback, so there's a common
@@ -60,6 +60,7 @@
# have to consider the effect of the stripped revisions and not revisions
# missing because the cache is out-of-date.
repo.updatebranchcache()
+ repo.destroying()
cl = repo.changelog
# TODO handle undo of merge sets