[PATCH 03 of 17 V4] dirstate: create class for status lists

Martin von Zweigbergk martinvonz at gmail.com
Sun Oct 12 16:08:12 CDT 2014


On Sun, Oct 12, 2014 at 4:54 AM, Mads Kiilerich <mads at kiilerich.com> wrote:
> This series LGTM.

Thanks for reviewing.

> The biggest question remains whether we want the status class to be this
> kind of named tuple and whether the use of __slots__ is an unnecessary micro
> optimization. The implementation can however easily be changed later if
> there should be a need for that.

Agreed, we can change it later if necessary.


More information about the Mercurial-devel mailing list