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, Justu

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 http://notmu

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 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 definition of the w3m-

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 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 IM

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 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 notice

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

2012-01-10 Thread David Edmondson
On Sun, 8 Jan 2012 11:09:09 -0800, Jameson Graef Rollins wrote: > This adds two new function, notmuch-show-{archive,delete}-message, > that archive/delete the current message, and then move to the next > open one. Looks fine. pgpF5RhFNS8CD.pgp Description: PGP signature __

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 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 > messages with the "delet

[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 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 wrote: > Signed-off-by: Jani Nikula > --- > man/man1/notmuch-reply.1 |3 +++ > 1 files changed, 3 insertions(+), 0 deletions(-) Pushed to master d ___ notmuch mailing list notmuch@notmuchmail.org ht

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 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 http://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 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 mailing list no

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 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 a

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 wrote: > On Fri, 30 Dec 2011 09:39:37 +, David Edmondson 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 here? I'm not co

[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 pu

[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 mis-feature of needing to have

[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 insertions(+), 86 deletions(-)

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 wrote: > Signed-off-by: Jani Nikula > --- > man/man1/notmuch-reply.1 |3 +++ > 1 files changed, 3 insertions(+), 0 deletions(-) +1 /Xavier ___ notmuch mailing list notmuch@notmuchmail.org http://no

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 *get_nm_mes

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 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 the > entire thread? I

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 follow

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 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 continue to think,

Re: [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 matrix,

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 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 > > > notmuch-hel

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

2012-01-10 Thread David Edmondson
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 `message-mode' buffers to be of interest. Any reviewers? pgpW7uPCmz2l5.pgp D

[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@qmul.ac.uk"). Fix man page accordinly, and clean up the wording a bit otherwise too.

[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 add_recip

[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 -- e

[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 file

[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 b/test/notmuch-test index e40ef86..6a9

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 wrote: > On Sat, 7 Jan 2012 14:28:10 -0800, Jameson Graef Rollins > 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-se

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 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 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 `messag

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 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 again

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 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 all of you

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 wrote: > On Sun, 08 Jan 2012 18:49:56 -0800, Jameson Graef Rollins > wrote: > > On Sun, 08 Jan 2012 20:08:59 -0500, Aaron Ecay wrote: > > > > > > - It would be good to make show-next &optional. This will enable code > > > to call the fn with onl

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 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 the libs versu

[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 c

[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 "notmuch-cli

[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

[PATCH 3/3] search: Support automatic tag exclusions

2012-01-10 Thread Austin Clements
This adds a "search" section to the config file and an "auto_tag_exclusions" setting in that section. The search and count commands pass tag tags from the configuration to the library. --- notmuch-client.h |8 notmuch-config.c | 42 ++ notmuc

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 wrote: > On Tue, 10 Jan 2012 16:01:32 -0400, David Bremner 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

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 > wrote: > > On Tue, 10 Jan 2012 16:01:32 -0400, David Bremner wrote: > > > Just thinking out loud here, but it does seem a bit unfortunate to me > > > that it represents a pretty fundamental div

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 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 separate patch. Hrm. I didn’

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 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 config file and de

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 wrote: > On Tue, 10 Jan 2012 18:56:29 -0800, Jameson Graef Rollins > 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 beha

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 IM

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

2012-01-10 Thread David Edmondson
et integrated), which is itself just copied from w3m (and should end up 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 1/4 v2] emacs: add show-mode functions to archive/delete only current message

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

another attempt to add delete functionality in emacs

2012-01-10 Thread David Edmondson
'deleted' for the tag should be a configuration option, 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] 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 |

[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

[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

[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 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] 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 ad

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

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

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 pu

[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 mis-feature of needing to have

[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 insertions(+), 86 deletions(-)

change to default archive/delete key bindings

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

[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 follow

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

2012-01-10 Thread David Edmondson
scrubbed... Name: not available Type: application/pgp-signature Size: 197 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120110/c0543dc2/attachment.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 matri

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

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/057f090a/attachment.pgp>

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

2012-01-10 Thread David Edmondson
reviewers? -- 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 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 too

[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 add_recip

[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 -- e

[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 file

[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 b/test/notmuch-test index e40ef86..6a9

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 > > vari

[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 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 `messag

[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 again

another attempt to add delete functionality in emacs

2012-01-10 Thread Jameson Graef Rollins
tachment was scrubbed... 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 2/4] emacs: repurpose notmuch-show-archive-thread-internal function for general thread tagging

2012-01-10 Thread Jameson Graef Rollins
lable Type: application/pgp-signature Size: 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
tmuch-show-add-tag "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 0/3] Automatic tag-based exclusion

2012-01-10 Thread Jameson Graef Rollins
attachment was scrubbed... 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 2/4] emacs: repurpose notmuch-show-archive-thread-internal function for general thread tagging

2012-01-10 Thread Jameson Graef Rollins
le URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20120110/57bde9d1/attachment.pgp>