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
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>
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
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
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.
> ---
>
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
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
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 --
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
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
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>
ze: 835 bytes
Desc: not available
URL:
<http://notmuchmail.org/pipermail/notmuch/attachments/20120110/f3fc081f/attachment.pgp>
Name: not available
Type: application/pgp-signature
Size: 835 bytes
Desc: not available
URL:
<http://notmuchmail.org/pipermail/notmuch/attachments/20120110/20df6bf6/attachment.pgp>
-- 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>
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>
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
> >
e: application/pgp-signature
Size: 197 bytes
Desc: not available
URL:
<http://notmuchmail.org/pipermail/notmuch/attachments/20120110/c0543dc2/attachment.pgp>
ilable
URL:
<http://notmuchmail.org/pipermail/notmuch/attachments/20120110/2a6e139c/attachment-0001.pgp>
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
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
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 |
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
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
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
/pgp-signature
Size: 197 bytes
Desc: not available
URL:
<http://notmuchmail.org/pipermail/notmuch/attachments/20120110/e6bb92c9/attachment.pgp>
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>
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>
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>
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
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
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
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
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
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
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.
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
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
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
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
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
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
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 |
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
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
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
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
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,
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
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
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
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(+),
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
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
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
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
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
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
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
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 |
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 |
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
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
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.
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.
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
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
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.
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
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
---
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
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
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
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
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
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
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
76 matches
Mail list logo