phillco accepted this revision as: phillco.
phillco added a comment.
This revision is now accepted and ready to land.
Actual LGTM.
Though, it's interesting, it looks like I have the ability to accept as `hg`.
I presume I'll need to keep unchecking that so I don't accidentally pretend to
martinvonz accepted this revision.
martinvonz added inline comments.
INLINE COMMENTS
> util.py:592
>
> +class nullcontextmanager(object):
> +"""A no-op context manager.
The body looks more like a "nulltransaction" (I think close(), release(),
abort() are part of transaction's interface),