[PATCH 1 of 9 V2] mdiff: remove use of __slots__

[Gregory Szorc]

On Sat, Jun 25, 2016 at 7:21 PM, Pierre-Yves David <
pierre-yves.david at ens-lyon.org> wrote:

> On 06/25/2016 11:11 PM, Gregory Szorc wrote:
> > # HG changeset patch
> > # User Gregory Szorc <gregory.szorc at gmail.com>
> > # Date 1466887966 25200
> > #      Sat Jun 25 13:52:46 2016 -0700
> > # Node ID 8f5c54d6463b184fc337d524e3ad96cc6eb569d6
> > # Parent  591a8069b60e16f2331a53e9c608c37e14be6947
> > mdiff: remove use of __slots__
>
> Pushed patch 1 only as patch two requires some deeper review.
> I feel like patch 3-9 could come before patch 2. That would probably
> reduce the error churn while still producing valide code. Am I right?
>

Yes, many of the subsequent patches could land first. I'm hesitant to
reorder because many of them deal with fallout from the module loader. It
feels like cart before the horse to have them land before. But I may
reorder a few of them so the in-flight series is a bit smaller. Please
don't wait on that to look at patch 2...


> I don't think our important should push the magic too far so not
> detecting encode and decode seems the sensible path to take here.
>
> --
> Pierre-Yves David
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.mercurial-scm.org/pipermail/mercurial-devel/attachments/20160626/ac8c827a/attachment.html>