[Differential] [Accepted] D65: histedit: create transaction outside of try

durham (Durham Goode) phabricator at mercurial-scm.org
Thu Jul 13 15:52:51 EDT 2017


durham accepted this revision.
durham added inline comments.

INLINE COMMENTS

> histedit.py:1120
> +        # external process it may choose to commit the transaction first.
> +        tr = repo.transaction('histedit')
>  

It might be more technically correct to do this after state.write() happens.  state.write() is intended to be called before the transaction, and it only technically works right here because it's ignoring the pending transaction (which could be fixed in the future).

REPOSITORY
  rHG Mercurial

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

EMAIL PREFERENCES
  https://phab.mercurial-scm.org/settings/panel/emailpreferences/

To: martinvonz, #hg-reviewers, dsp, durham
Cc: durham, mercurial-devel


More information about the Mercurial-devel mailing list