D5622: localrepo: use context manager for transaction in commit()

lothiraldan (Boris Feld) phabricator at mercurial-scm.org
Fri Jan 18 01:59:21 EST 2019


lothiraldan accepted this revision.
lothiraldan added a comment.


  In https://phab.mercurial-scm.org/D5622#83013, @martinvonz wrote:
  
  > In https://phab.mercurial-scm.org/D5622#83011, @lothiraldan wrote:
  >
  > > I have the impression that the test changes are due to the previous patch in the stack.
  >
  >
  > I don't think so. See commit message for why I think they changed.
  
  
  Right, I was confused between the time the transaction was closed and the time the transaction was released.

REPOSITORY
  rHG Mercurial

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

To: martinvonz, durin42, #hg-reviewers, lothiraldan
Cc: lothiraldan, mercurial-devel


More information about the Mercurial-devel mailing list