[PATCH v5 0/4] Use invisibility to toggle display of all parts including multipart
Austin Clements
amdragon at MIT.EDU
Tue Dec 18 21:11:13 PST 2012
Quoth Mark Walters on Dec 18 at 7:27 pm:
> This is an alternative version of
> id:1355781287-6010-1-git-send-email-markwalters1009 at gmail.com based on
> top of Austin's patch at
> id:1355812810-32747-1-git-send-email-amdragon at mit.edu
>
> Austin's patch significantly simplifies the invisibility handling
> taking this series down from 90/27 to 68/26 in diffstat terms.
>
> In general terms Austin's approach has to be the right thing to do:
> what we want to do just before the freeze is less clear. My view is
> that we should go with Austin's approach now so that at least any bugs
> we get from it and (more likely) from this series apply to master as
> well.
>
> I am posting this series to make it easier for people to judge the two
> approaches when finished (ie with part invisibility too).
>
> I attach a trimmed diff from v4 below the diffstat (note this was a
> diff with -U10 which I have trimmed so it is purely for information)
>
> Note we no longer need patch 4/5 because in this approach the overlays
> do not need to know about other overlays.
>
> Best wishes
>
> Mark
LGTM.
More information about the notmuch
mailing list