[PATCH 2/2] emacs: Combine string faces and combine under existing faces

2013-02-04 Thread Austin Clements
This improves notmuch-combine-face-text-property to support both applying faces to strings and to support combining the given face under existing faces, rather than over. --- emacs/notmuch-lib.el | 21 + 1 file changed, 13 insertions(+), 8 deletions(-) diff --git

[PATCH 1/2] emacs: Handle all face forms when combining faces

2013-02-04 Thread Austin Clements
Previously, notmuch-combine-face-text-property assumed that any existing face properties of the modified text were already in face list form. This was true as long as it was the only function manipulating faces (since it always produced a list form face), but if anything else has manipulated the

[PATCH 0/2] notmuch-combine-face-text-property improvements

2013-02-04 Thread Austin Clements
These two patches fix bugs in and improve on notmuch-combine-face-text-property. The bug fixed by the first patch was found by Damien and the features added in the second patch should help with notmuch-tagger. (Sorry for the delay with these. I've had them sitting in my tree for a while with

Reply all - issue

2013-02-04 Thread Suvayu Ali
On Mon, Feb 04, 2013 at 11:39:44AM +0100, Micha? Nazarewicz wrote: > 2 lut 2013 17:21, "Robert Mast" napisa?(a): > > Anyone interested in me patching Notmuch, or shall I keep the changes to > myself? > > I was actually wondering that instead of hard coding the logic into notmuch > itself, maybe

Reply all - issue

2013-02-04 Thread Michał Nazarewicz
search for all ancestors of a given message and possibly an option to sort results topologically which I dunno if notmuch has. -- next part -- An HTML attachment was scrubbed... URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20130204/e0759298/attachment.html>

RE: Reply all - issue

2013-02-04 Thread Michał Nazarewicz
2 lut 2013 17:21, Robert Mast beheer...@tekenbeetziekten.nl napisał(a): So Gmail-threading is still the best I suppose, I strongly disagree. Having said that, as long as it's configurable I obviously won't be blocking your efforts. Anyone interested in me patching Notmuch, or shall I keep the

Re: Reply all - issue

2013-02-04 Thread Suvayu Ali
On Mon, Feb 04, 2013 at 11:39:44AM +0100, Michał Nazarewicz wrote: 2 lut 2013 17:21, Robert Mast beheer...@tekenbeetziekten.nl napisał(a): Anyone interested in me patching Notmuch, or shall I keep the changes to myself? I was actually wondering that instead of hard coding the logic into

[PATCH 0/2] notmuch-combine-face-text-property improvements

2013-02-04 Thread Austin Clements
These two patches fix bugs in and improve on notmuch-combine-face-text-property. The bug fixed by the first patch was found by Damien and the features added in the second patch should help with notmuch-tagger. (Sorry for the delay with these. I've had them sitting in my tree for a while with

[PATCH 2/2] emacs: Combine string faces and combine under existing faces

2013-02-04 Thread Austin Clements
This improves notmuch-combine-face-text-property to support both applying faces to strings and to support combining the given face under existing faces, rather than over. --- emacs/notmuch-lib.el | 21 + 1 file changed, 13 insertions(+), 8 deletions(-) diff --git

[PATCH 1/2] emacs: Handle all face forms when combining faces

2013-02-04 Thread Austin Clements
Previously, notmuch-combine-face-text-property assumed that any existing face properties of the modified text were already in face list form. This was true as long as it was the only function manipulating faces (since it always produced a list form face), but if anything else has manipulated the