On Tue, 24 Jan 2012 11:15:11 -0500, Austin Clements wrote:
> LGTM other than one very minor nit, though I pretty reliably make
> fence post errors when dealing with text properties, so perhaps
> somebody else should check that.
>
> Quoth David Edmondson on Jan 24 at 11:36 am:
> > Except for where
Except for where invisibility is involved, replace the use of overlays
in `notmuch-show-mode' with text properties, which are more efficient
and can be merged together more effectively.
---
emacs/notmuch-show.el | 66 +++-
emacs/notmuch-wash.el |8
LGTM other than one very minor nit, though I pretty reliably make
fence post errors when dealing with text properties, so perhaps
somebody else should check that.
Quoth David Edmondson on Jan 24 at 11:36 am:
> Except for where invisibility is involved, replace the use of overlays
> in `notmuch-sho
On Tue, 24 Jan 2012 11:15:11 -0500, Austin Clements wrote:
> LGTM other than one very minor nit, though I pretty reliably make
> fence post errors when dealing with text properties, so perhaps
> somebody else should check that.
>
> Quoth David Edmondson on Jan 24 at 11:36 am:
> > Except for where
LGTM other than one very minor nit, though I pretty reliably make
fence post errors when dealing with text properties, so perhaps
somebody else should check that.
Quoth David Edmondson on Jan 24 at 11:36 am:
> Except for where invisibility is involved, replace the use of overlays
> in `notmuch-sho
Except for where invisibility is involved, replace the use of overlays
in `notmuch-show-mode' with text properties, which are more efficient
and can be merged together more effectively.
---
emacs/notmuch-show.el | 66 +++-
emacs/notmuch-wash.el |8