D7833: transaction: allow finalizer to add finalizer

martinvonz (Martin von Zweigbergk) phabricator at mercurial-scm.org
Thu Jan 16 11:51:09 EST 2020


This revision is now accepted and ready to land.
martinvonz added inline comments.
martinvonz accepted this revision.

INLINE COMMENTS

> martinvonz wrote in transaction.py:511
> Any reason to not set this to `None` as we do on line 516? Also, can we remove that line now?

Ah, never mind, I just found the previous patch in this stack and now it's clear.

REPOSITORY
  rHG Mercurial

CHANGES SINCE LAST ACTION
  https://phab.mercurial-scm.org/D7833/new/

REVISION DETAIL
  https://phab.mercurial-scm.org/D7833

To: marmoute, #hg-reviewers, martinvonz
Cc: martinvonz, mercurial-devel


More information about the Mercurial-devel mailing list