[PATCH 1/2] emacs: Use text properties rather than overlays in `notmuch-show-mode'.

2012-01-24 Thread David Edmondson
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

[PATCH 1/2] emacs: Use text properties rather than overlays in `notmuch-show-mode'.

2012-01-24 Thread David Edmondson
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

[PATCH 1/2] emacs: Use text properties rather than overlays in `notmuch-show-mode'.

2012-01-24 Thread Austin Clements
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

Re: [PATCH 1/2] emacs: Use text properties rather than overlays in `notmuch-show-mode'.

2012-01-24 Thread David Edmondson
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

Re: [PATCH 1/2] emacs: Use text properties rather than overlays in `notmuch-show-mode'.

2012-01-24 Thread Austin Clements
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

[PATCH 1/2] emacs: Use text properties rather than overlays in `notmuch-show-mode'.

2012-01-24 Thread David Edmondson
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