[PATCH 2/4] emacs: repurpose notmuch-show-archive-thread-internal function for general thread tagging

2012-01-10 Thread Jameson Graef Rollins
oint, Aaron. I'll rework it to make it a little clean and more flexible when I resend. Thanks again. jamie. -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 835 bytes Desc: not available URL: <http://not

[PATCH 0/3] Automatic tag-based exclusion

2012-01-10 Thread Jameson Graef Rollins
bbed... Name: not available Type: application/pgp-signature Size: 835 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120110/79554dd1/attachment.pgp>

[PATCH] emacs: Improve `notmuch-hello' display on ttys.

2012-01-10 Thread Jani Nikula
On Tue, 10 Jan 2012 22:41:49 +0200, Jani Nikula wrote: > On Tue, 10 Jan 2012 16:18:21 +, David Edmondson wrote: > > I have a vague recollection that someone reported a bug where > > `(window-width)' was not the right value to use if `line-number-mode' is > > enabled, but I can't find it

[PATCH v2] emacs: Cycle through notmuch buffers rather than jumping to the last.

2012-01-10 Thread Jani Nikula
On Tue, 10 Jan 2012 16:55:07 +, David Edmondson wrote: > On Wed, 28 Dec 2011 08:29:58 +, David Edmondson wrote: > > As suggested by j4ni in #notmuch, rename > > `notmuch-jump-to-recent-buffer' as `notmuch-cycle-notmuch-buffers' and > > have it behave accordingly. > > > > Consider

[PATCH] emacs: Improve `notmuch-hello' display on ttys.

2012-01-10 Thread Jani Nikula
On Tue, 10 Jan 2012 10:15:28 +, David Edmondson wrote: > Inserting spaces to pad out columns is good, except when the padding > makes the line wider than the window. This looks particularly bad on a > tty where there is no fringe. > > Hence, avoid padding the last column on each row. > --- >

[PATCH v3 4/4] test: add tests for "notmuch reply" --reply-to=sender

2012-01-10 Thread Jani Nikula
From: Mark Walters --- test/notmuch-test|1 + test/reply-to-sender | 209 ++ 2 files changed, 210 insertions(+), 0 deletions(-) create mode 100755 test/reply-to-sender diff --git a/test/notmuch-test

[PATCH v3 3/4] emacs: bind 'r' to reply-to-sender and 'R' to reply-to-all

2012-01-10 Thread Jani Nikula
Change the default reply key bindings, making 'r' reply-to-sender and 'R' reply-to-all. Signed-off-by: Jani Nikula --- There were mixed feelings about this. This as a separate patch so it's easy to drop if needed. --- emacs/notmuch-show.el |4 ++-- emacs/notmuch.el |4 ++-- 2

[PATCH v3 2/4] emacs: add support for replying just to the sender

2012-01-10 Thread Jani Nikula
Provide reply to sender counterparts to the search and show reply functions. Add key binding 'R' to reply to sender, while keeping 'r' as reply to all, both in search and show views. Signed-off-by: Jani Nikula --- emacs/notmuch-mua.el |9 ++--- emacs/notmuch-show.el | 10 --

[PATCH v3 1/4] cli: add support for replying just to the sender in "notmuch reply"

2012-01-10 Thread Jani Nikula
Add new option --reply-to=(all|sender) to "notmuch reply" to select whether to reply to all (sender and all recipients), or just sender. Reply to all remains the default. Credits to Mark Walters for his similar earlier work where I picked up the basic idea of handling reply-to-sender in

[PATCH v3 0/4] reply to sender

2012-01-10 Thread Jani Nikula
Hi all, v3 of the reply-to-sender series. Changes since v2: - Patches 1 and 2 of the original series were pushed already. - Patch 1: Don't force recipient type, based on comment by Mark (id:"877h11eq3g.fsf at qmul.ac.uk"). Fix man page accordinly, and clean up the wording a bit otherwise

another attempt to add delete functionality in emacs

2012-01-10 Thread Jameson Graef Rollins
deleted") (notmuch-show-next-open-message))) -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 835 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120110/53d26c97/attachment.pgp>

[PATCH 2/4] emacs: repurpose notmuch-show-archive-thread-internal function for general thread tagging

2012-01-10 Thread Jameson Graef Rollins
ze: 835 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120110/f3fc081f/attachment.pgp>

another attempt to add delete functionality in emacs

2012-01-10 Thread Jameson Graef Rollins
Name: not available Type: application/pgp-signature Size: 835 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120110/20df6bf6/attachment.pgp>

[PATCH v2] emacs: Cycle through notmuch buffers rather than jumping to the last.

2012-01-10 Thread David Edmondson
-- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 197 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120110/4db2bc11/attachment.pgp>

[PATCH] emacs: Improve `notmuch-hello' display on ttys.

2012-01-10 Thread David Edmondson
achment was scrubbed... Name: not available Type: application/pgp-signature Size: 197 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120110/057f090a/attachment.pgp>

another attempt to add delete functionality in emacs

2012-01-10 Thread David Bremner
On Tue, 10 Jan 2012 07:47:23 +, David Edmondson wrote: > On Sat, 7 Jan 2012 14:28:10 -0800, Jameson Graef Rollins finestructure.net> wrote: > > I try to address the concerns that have come up in previous attempts. > > In particular, I include a patch that creates a new customization > >

[PATCH] emacs: Improve `notmuch-hello' display on ttys.

2012-01-10 Thread David Edmondson
e: application/pgp-signature Size: 197 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120110/c0543dc2/attachment.pgp>

[PATCH 2/3 v2] test: Add tests for `notmuch-show-clean-address'.

2012-01-10 Thread David Edmondson
ilable URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120110/2a6e139c/attachment-0001.pgp>

[PATCH] emacs: Improve `notmuch-hello' display on ttys.

2012-01-10 Thread Austin Clements
Quoth David Edmondson on Jan 10 at 3:47 pm: > On Tue, 10 Jan 2012 10:36:50 -0500, Austin Clements > wrote: > > LGTM, though would it be easier to put this in the else clause of the > > if after the setq count? > > Agreed. I got confused thinking about it due to the empty elements in > the

[PATCH] emacs: Improve `notmuch-hello' display on ttys.

2012-01-10 Thread Austin Clements
LGTM, though would it be easier to put this in the else clause of the if after the setq count? Is it possible for a tag in the last column to be just long enough to make the line still wrap? Somehow my current tag set doesn't trigger this bug, so I can't test this case (and I admit I can't

[PATCH] emacs: Improve `notmuch-hello' display on ttys.

2012-01-10 Thread David Edmondson
Inserting spaces to pad out columns is good, except when the padding makes the line wider than the window. This looks particularly bad on a tty where there is no fringe. Hence, avoid padding the last column on each row. --- Thanks to j4ni in #notmuch for spotting this. emacs/notmuch-hello.el |

[PATCH 2/2] notmuch-reply.c: uncrustify

2012-01-10 Thread David Bremner
From: David Bremner This patch shows the raw result of running uncrustify on notmuch-reply.c. The re-indenting of "format_reply" would probably not be desirable. --- notmuch-reply.c | 160 +- 1 files changed, 74

[PATCH 1/2] uncrustify.cfg: initial support for notmuch coding style

2012-01-10 Thread David Bremner
From: David Bremner Uncrustify is a free (as in GPL2+) tool that indents and beautifies C/C++ code. It is similar to GNU indent in functionality although probably more configurable (in fairness, indent has better documentation). Uncrustify does not have the indent

No subject

2012-01-10 Thread David Bremner
After a little fine-tuning, and some consensus about brace style on the list, I think this is getting pretty useful. I attach a demo uncrustification, where IMHO the output is mostly good. Note that it catches some style mistakes in quite recent changes. One thing I wondered about where to put

change to default archive/delete key bindings

2012-01-10 Thread David Edmondson
/pgp-signature Size: 197 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120110/e6bb92c9/attachment.pgp>

another attempt to add delete functionality in emacs

2012-01-10 Thread David Edmondson
as a non-English speaker (for example) might want to use something else. -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 197 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120110/3ab755cf/attachment-0001.pgp>

[PATCH 1/4 v2] emacs: add show-mode functions to archive/delete only current message

2012-01-10 Thread David Edmondson
ttachment was scrubbed... Name: not available Type: application/pgp-signature Size: 197 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120110/038cedd1/attachment-0001.pgp>

[PATCH 4/4] emacs: Use the new JSON reply format.

2012-01-10 Thread David Edmondson
p in `notmuch-lib.el'). -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 197 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120110/1da5c38a/attachment-0001.pgp>

[PATCH] Properly handle short writes in sigint handlers

2012-01-10 Thread David Bremner
On Fri, 23 Dec 2011 23:10:35 +0400, Dmitry Kurochkin wrote: > Hi Austin. > > I think we should put the write loop into a separate function and reuse > it. I could go either way on this, unless there is somewhere else the code is actually needed at the moment. > > Also, does it make sense to

[PATCH 2/3 v2] test: Add tests for `notmuch-show-clean-address'.

2012-01-10 Thread David Bremner
On Fri, 30 Dec 2011 09:39:37 +, David Edmondson wrote: > --- > > Added backslash test. UTF8 round-trip still isn't right. > Hi David. Do you mind explaining a bit better in the commit message what the issue is here? d

[PATCH] emacs: Mark the quoted region during reply.

2012-01-10 Thread David Bremner
On Fri, 6 Jan 2012 10:03:40 +, David Edmondson wrote: > Mark the quoted region of text during a reply, making it easy for the > user to delete it quickly. Pushed to master. d

[PATCH] man: add missing SEE ALSO header to notmuch reply man page

2012-01-10 Thread David Bremner
On Sun, 8 Jan 2012 22:57:22 +0200, Jani Nikula wrote: > Signed-off-by: Jani Nikula > --- > man/man1/notmuch-reply.1 |3 +++ > 1 files changed, 3 insertions(+), 0 deletions(-) Pushed to master d

[RFC PATCH 3/9] lib: fix messages.c build warn

2012-01-10 Thread David Bremner
On Sun, 8 Jan 2012 01:26:17 +0200, Jani Nikula wrote: > lib/messages.c: In function ?notmuch_messages_move_to_next?: > lib/messages.c:131:2: warning: ISO C forbids ?return? with expression, in > function returning void [-pedantic] Pushed to master. d

Distributed Notmuch

2012-01-10 Thread Jan Pobrislo
Quoting Ethan Glasser-Camp (2012-01-08 11:23:59) >Hi guys, > > ... > >In brainstorming about the One True Mail Setup, my friend suggested to >me that Maildir/IMAP are not really the best choices for mail storage. In my opinion Maildirs are very good mail storage format, the issue is just that

Re: Python bindings for adoption

2012-01-10 Thread Jameson Graef Rollins
On Sun, 08 Jan 2012 16:16:06 -, Justus Winter 4win...@informatik.uni-hamburg.de wrote: I've decided to step up as a new maintainer for the libnotmuch python bindings. I assume that I'll have to mail an ssh public key to someone for repository access, right? That's great to hear, Justus.

Re: [PATCH v2 0/6] reply to sender

2012-01-10 Thread Dmitry Kurochkin
Hi Jani. I prefer to leave the Emacs UI default reply behavior as is. Changing it in CLI would not affect me, but I think the default should be the same as in the Emacs UI. Regards, Dmitry ___ notmuch mailing list notmuch@notmuchmail.org

Re: [PATCH 4/4] emacs: Use the new JSON reply format.

2012-01-10 Thread Adam Wolfe Gordon
On Sun, Jan 8, 2012 at 18:27, Aaron Ecay aarone...@gmail.com wrote: +(defun w3m-region (start end)) ;; From `w3m.el'. What is the purpose of the above line?  If it is to make the compiler aware of the function, you should use ‘declare-function’ instead.  Defun will erase the original

Re: [PATCH] NEWS: add news entry for notmuch reply uninitialized variable bugfix

2012-01-10 Thread David Bremner
On Mon, 9 Jan 2012 11:49:56 +, Jani Nikula j...@nikula.org wrote: --- This is against release branch. --- Pushed to release. d ___ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch

Re: Distributed Notmuch

2012-01-10 Thread Jan Pobrislo
Quoting Ethan Glasser-Camp (2012-01-08 11:23:59) Hi guys, ... In brainstorming about the One True Mail Setup, my friend suggested to me that Maildir/IMAP are not really the best choices for mail storage. In my opinion Maildirs are very good mail storage format, the issue is just that IMAP

Re: [PATCH 4/4] emacs: Use the new JSON reply format.

2012-01-10 Thread David Edmondson
On Mon, 9 Jan 2012 19:10:48 -0700, Adam Wolfe Gordon a...@xvx.ca wrote: Using w3m means that you should `require' it. What happens when a user doesn't have it? (Either the elisp or the command.) This was my initial thought, but when I looked at notmuch-show.el, which uses w3m features, I

Re: another attempt to add delete functionality in emacs

2012-01-10 Thread David Edmondson
On Sat, 7 Jan 2012 14:28:10 -0800, Jameson Graef Rollins jroll...@finestructure.net wrote: I try to address the concerns that have come up in previous attempts. In particular, I include a patch that creates a new customization variable, notmuch-search-exclude-deleted, that will exclude any

[PATCH] emacs: Improve `notmuch-hello' display on ttys.

2012-01-10 Thread David Edmondson
Inserting spaces to pad out columns is good, except when the padding makes the line wider than the window. This looks particularly bad on a tty where there is no fringe. Hence, avoid padding the last column on each row. --- Thanks to j4ni in #notmuch for spotting this. emacs/notmuch-hello.el |

Re: [RFC PATCH 3/9] lib: fix messages.c build warn

2012-01-10 Thread David Bremner
On Sun, 8 Jan 2012 01:26:17 +0200, Jani Nikula j...@nikula.org wrote: lib/messages.c: In function ‘notmuch_messages_move_to_next’: lib/messages.c:131:2: warning: ISO C forbids ‘return’ with expression, in function returning void [-pedantic] Pushed to master. d

Re: [PATCH] man: add missing SEE ALSO header to notmuch reply man page

2012-01-10 Thread David Bremner
On Sun, 8 Jan 2012 22:57:22 +0200, Jani Nikula j...@nikula.org wrote: Signed-off-by: Jani Nikula j...@nikula.org --- man/man1/notmuch-reply.1 |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) Pushed to master d ___ notmuch mailing list

Re: [PATCH] emacs: Mark the quoted region during reply.

2012-01-10 Thread David Bremner
On Fri, 6 Jan 2012 10:03:40 +, David Edmondson d...@dme.org wrote: Mark the quoted region of text during a reply, making it easy for the user to delete it quickly. Pushed to master. d ___ notmuch mailing list notmuch@notmuchmail.org

Re: [PATCH 2/3 v2] test: Add tests for `notmuch-show-clean-address'.

2012-01-10 Thread David Bremner
On Fri, 30 Dec 2011 09:39:37 +, David Edmondson d...@dme.org wrote: --- Added backslash test. UTF8 round-trip still isn't right. Hi David. Do you mind explaining a bit better in the commit message what the issue is here? d ___ notmuch

Re: [PATCH] Properly handle short writes in sigint handlers

2012-01-10 Thread David Bremner
On Fri, 23 Dec 2011 23:10:35 +0400, Dmitry Kurochkin dmitry.kuroch...@gmail.com wrote: Hi Austin. I think we should put the write loop into a separate function and reuse it. I could go either way on this, unless there is somewhere else the code is actually needed at the moment. Also,

Re: [PATCH 2/3 v2] test: Add tests for `notmuch-show-clean-address'.

2012-01-10 Thread David Edmondson
On Tue, 10 Jan 2012 07:06:58 -0400, David Bremner da...@tethera.net wrote: On Fri, 30 Dec 2011 09:39:37 +, David Edmondson d...@dme.org wrote: --- Added backslash test. UTF8 round-trip still isn't right. Do you mind explaining a bit better in the commit message what the issue is

[no subject]

2012-01-10 Thread David Bremner
After a little fine-tuning, and some consensus about brace style on the list, I think this is getting pretty useful. I attach a demo uncrustification, where IMHO the output is mostly good. Note that it catches some style mistakes in quite recent changes. One thing I wondered about where to put

[PATCH 1/2] uncrustify.cfg: initial support for notmuch coding style

2012-01-10 Thread David Bremner
From: David Bremner brem...@debian.org Uncrustify is a free (as in GPL2+) tool that indents and beautifies C/C++ code. It is similar to GNU indent in functionality although probably more configurable (in fairness, indent has better documentation). Uncrustify does not have the indent mis-feature

[PATCH 2/2] notmuch-reply.c: uncrustify

2012-01-10 Thread David Bremner
From: David Bremner brem...@debian.org This patch shows the raw result of running uncrustify on notmuch-reply.c. The re-indenting of format_reply would probably not be desirable. --- notmuch-reply.c | 160 +- 1 files changed, 74 insertions(+),

Re: [PATCH] man: add missing SEE ALSO header to notmuch reply man page

2012-01-10 Thread Xavier Maillard
On Sun, 8 Jan 2012 22:57:22 +0200, Jani Nikula j...@nikula.org wrote: Signed-off-by: Jani Nikula j...@nikula.org --- man/man1/notmuch-reply.1 |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) +1 /Xavier ___ notmuch mailing list

Re: [ANNOUNCE] mutt with notmuch support

2012-01-10 Thread Jeremy Nickurak
FWIW, here's the patch I ended up using to play with this: diff --git a/mutt_notmuch.c b/mutt_notmuch.c index 2f21407..a07b1ba 100644 --- a/mutt_notmuch.c +++ b/mutt_notmuch.c @@ -636,11 +636,15 @@ char *nm_uri_from_query(CONTEXT *ctx, char *buf, size_t bufsz)  static notmuch_message_t

Re: change to default archive/delete key bindings

2012-01-10 Thread David Edmondson
On Sat, 7 Jan 2012 17:26:51 -0800, Jameson Graef Rollins jroll...@finestructure.net wrote: While working on the delete message handling patches, I was reminded how much I really dislike the default show-mode key bindings. Why can't I just archive/delete the current message, without archiving

Re: [PATCH] emacs: Improve `notmuch-hello' display on ttys.

2012-01-10 Thread Austin Clements
LGTM, though would it be easier to put this in the else clause of the if after the setq count? Is it possible for a tag in the last column to be just long enough to make the line still wrap? Somehow my current tag set doesn't trigger this bug, so I can't test this case (and I admit I can't

Re: [PATCH] emacs: Improve `notmuch-hello' display on ttys.

2012-01-10 Thread David Edmondson
On Tue, 10 Jan 2012 10:36:50 -0500, Austin Clements amdra...@mit.edu wrote: LGTM, though would it be easier to put this in the else clause of the if after the setq count? Agreed. I got confused thinking about it due to the empty elements in the matrix, but perhaps that doesn't matter. I'll

Re: [PATCH] emacs: Improve `notmuch-hello' display on ttys.

2012-01-10 Thread David Edmondson
On Tue, 10 Jan 2012 11:05:02 -0500, Austin Clements amdra...@mit.edu wrote: Is it possible for a tag in the last column to be just long enough to make the line still wrap? Somehow my current tag set doesn't trigger this bug, so I can't test this case (and I admit I can't follow

[PATCH v3 1/4] cli: add support for replying just to the sender in notmuch reply

2012-01-10 Thread Jani Nikula
Add new option --reply-to=(all|sender) to notmuch reply to select whether to reply to all (sender and all recipients), or just sender. Reply to all remains the default. Credits to Mark Walters markwalters1...@gmail.com for his similar earlier work where I picked up the basic idea of handling

[PATCH v3 2/4] emacs: add support for replying just to the sender

2012-01-10 Thread Jani Nikula
Provide reply to sender counterparts to the search and show reply functions. Add key binding 'R' to reply to sender, while keeping 'r' as reply to all, both in search and show views. Signed-off-by: Jani Nikula j...@nikula.org --- emacs/notmuch-mua.el |9 ++--- emacs/notmuch-show.el |

[PATCH v3 3/4] emacs: bind 'r' to reply-to-sender and 'R' to reply-to-all

2012-01-10 Thread Jani Nikula
Change the default reply key bindings, making 'r' reply-to-sender and 'R' reply-to-all. Signed-off-by: Jani Nikula j...@nikula.org --- There were mixed feelings about this. This as a separate patch so it's easy to drop if needed. --- emacs/notmuch-show.el |4 ++-- emacs/notmuch.el |

[PATCH v3 4/4] test: add tests for notmuch reply --reply-to=sender

2012-01-10 Thread Jani Nikula
From: Mark Walters markwalters1...@gmail.com --- test/notmuch-test|1 + test/reply-to-sender | 209 ++ 2 files changed, 210 insertions(+), 0 deletions(-) create mode 100755 test/reply-to-sender diff --git a/test/notmuch-test

Re: another attempt to add delete functionality in emacs

2012-01-10 Thread David Bremner
On Tue, 10 Jan 2012 07:47:23 +, David Edmondson d...@dme.org wrote: On Sat, 7 Jan 2012 14:28:10 -0800, Jameson Graef Rollins jroll...@finestructure.net wrote: I try to address the concerns that have come up in previous attempts. In particular, I include a patch that creates a new

Re: [PATCH] emacs: Improve `notmuch-hello' display on ttys.

2012-01-10 Thread Jani Nikula
On Tue, 10 Jan 2012 10:15:28 +, David Edmondson d...@dme.org wrote: Inserting spaces to pad out columns is good, except when the padding makes the line wider than the window. This looks particularly bad on a tty where there is no fringe. Hence, avoid padding the last column on each row.

Re: [PATCH v2] emacs: Cycle through notmuch buffers rather than jumping to the last.

2012-01-10 Thread Jani Nikula
On Tue, 10 Jan 2012 16:55:07 +, David Edmondson d...@dme.org wrote: On Wed, 28 Dec 2011 08:29:58 +, David Edmondson d...@dme.org wrote: As suggested by j4ni in #notmuch, rename `notmuch-jump-to-recent-buffer' as `notmuch-cycle-notmuch-buffers' and have it behave accordingly.

Re: [PATCH] emacs: Improve `notmuch-hello' display on ttys.

2012-01-10 Thread Jani Nikula
On Tue, 10 Jan 2012 22:41:49 +0200, Jani Nikula j...@nikula.org wrote: On Tue, 10 Jan 2012 16:18:21 +, David Edmondson d...@dme.org wrote: I have a vague recollection that someone reported a bug where `(window-width)' was not the right value to use if `line-number-mode' is enabled, but

Re: another attempt to add delete functionality in emacs

2012-01-10 Thread Jameson Graef Rollins
On Tue, 10 Jan 2012 07:47:23 +, David Edmondson d...@dme.org wrote: I honestly don't understand the reason for this. If someone wants to not see messages that they have tagged as 'deleted', they add 'and not tag:deleted' to the end of the search expression. Adding and not tag:deleted in

Re: [PATCH 2/4] emacs: repurpose notmuch-show-archive-thread-internal function for general thread tagging

2012-01-10 Thread Jameson Graef Rollins
On Mon, 09 Jan 2012 00:02:20 -0500, Aaron Ecay aarone...@gmail.com wrote: On Sun, 08 Jan 2012 18:49:56 -0800, Jameson Graef Rollins jroll...@finestructure.net wrote: On Sun, 08 Jan 2012 20:08:59 -0500, Aaron Ecay aarone...@gmail.com wrote: - It would be good to make show-next optional.

Re: another attempt to add delete functionality in emacs

2012-01-10 Thread Jameson Graef Rollins
On Tue, 10 Jan 2012 16:01:32 -0400, David Bremner da...@tethera.net wrote: Just thinking out loud here, but it does seem a bit unfortunate to me that it represents a pretty fundamental divergence between the CLI and the emacs interface. Mind you, I guess one could make the same argument about

[PATCH 0/3] Automatic tag-based exclusion

2012-01-10 Thread Austin Clements
This implements essentially the same idea as Jamie's patch, but does it in the library/CLI and operates on the parsed query rather than using an ad hoc regexp. Which tags are automatically excluded is set in the config file and defaults to deleted and spam. My comment for the config file is

[PATCH 1/3] count: Convert to new-style argument parsing

2012-01-10 Thread Austin Clements
--- notmuch-count.c | 53 + 1 files changed, 25 insertions(+), 28 deletions(-) diff --git a/notmuch-count.c b/notmuch-count.c index 20ce334..fb7401b 100644 --- a/notmuch-count.c +++ b/notmuch-count.c @@ -21,6 +21,11 @@ #include

[PATCH 2/3] lib: Add support for automatically excluding tags from queries

2012-01-10 Thread Austin Clements
This is useful for tags like deleted and spam that people generally want to exclude from query results. These exclusions will be overridden if a tag is explicitly mentioned in a query. --- lib/notmuch.h |6 ++ lib/query.cc | 33 + 2 files changed, 39

Re: another attempt to add delete functionality in emacs

2012-01-10 Thread Jani Nikula
On Tue, 10 Jan 2012 19:12:16 -0800, Jameson Graef Rollins jroll...@finestructure.net wrote: On Tue, 10 Jan 2012 16:01:32 -0400, David Bremner da...@tethera.net wrote: Just thinking out loud here, but it does seem a bit unfortunate to me that it represents a pretty fundamental divergence

Re: another attempt to add delete functionality in emacs

2012-01-10 Thread Austin Clements
Quoth Jani Nikula on Jan 11 at 7:16 am: On Tue, 10 Jan 2012 19:12:16 -0800, Jameson Graef Rollins jroll...@finestructure.net wrote: On Tue, 10 Jan 2012 16:01:32 -0400, David Bremner da...@tethera.net wrote: Just thinking out loud here, but it does seem a bit unfortunate to me that it

Re: [PATCH 2/4] emacs: repurpose notmuch-show-archive-thread-internal function for general thread tagging

2012-01-10 Thread Aaron Ecay
On Tue, 10 Jan 2012 18:56:29 -0800, Jameson Graef Rollins jroll...@finestructure.net wrote: Actually, the show-next argument was already part of the function. I did not introduce it. And it wasn't optional originally, so if we want to change that behavior we should probably do so in a

Re: [PATCH 0/3] Automatic tag-based exclusion

2012-01-10 Thread Jameson Graef Rollins
On Wed, 11 Jan 2012 00:02:50 -0500, Austin Clements amdra...@mit.edu wrote: This implements essentially the same idea as Jamie's patch, but does it in the library/CLI and operates on the parsed query rather than using an ad hoc regexp. Which tags are automatically excluded is set in the

Re: [PATCH 2/4] emacs: repurpose notmuch-show-archive-thread-internal function for general thread tagging

2012-01-10 Thread Jameson Graef Rollins
On Wed, 11 Jan 2012 00:53:35 -0500, Aaron Ecay aarone...@gmail.com wrote: On Tue, 10 Jan 2012 18:56:29 -0800, Jameson Graef Rollins jroll...@finestructure.net wrote: Actually, the show-next argument was already part of the function. I did not introduce it. And it wasn't optional