[PATCH 04 of 19] commit: refactor mergestate management into workingctx

Pierre-Yves David pierre-yves.david at logilab.fr
Thu Feb 14 04:27:23 CST 2013


On Sun, Feb 10, 2013 at 03:29:54PM -0800, David Schleimer wrote:

> +    def unresolved(self):
> +        assert self._unresolved is not None # must call status first
> +        return self._unresolved

I would be happy If we could get ride of those assert statement by either
- having the status automatically called if cache is empty
- having a real exception raise

It looks like there were alredy here before you started your refactoring, so it
is not really related to your series


-- 
Pierre-Yves David

http://www.logilab.fr/

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://selenic.com/pipermail/mercurial-devel/attachments/20130214/b4dc9351/attachment.pgp>


More information about the Mercurial-devel mailing list