Re: [Patch] Fix Emacs interface bug dealing with invisible text

2011-10-30 Thread Ivy Foster
On Sun, 30 Oct 2011 10:58:55 +0100, Thomas Jost schno...@schnouki.net wrote: Thanks for this patch. I haven't tested it, but I will! Fair enough (-: However, I'm wondering something: wasn't this issue already addressed in Dmitry's patch series that I rebased and reposted almost a month ago,

Re: [PATCH 3/3] Add notmuch-hello-mode-hook

2011-10-30 Thread Ivy Foster
On Sun, 30 Oct 2011 11:06:38 +0100, Thomas Jost schno...@schnouki.net wrote: On Sat, 29 Oct 2011 12:35:04 -0400, Ivy Foster joyfulg...@archlinux.us wrote: The following patch adds a mode-hook to notmuch-hello-mode. Not much else to say, really. I agree it's useful, but

Re: [PATCH 3/3] Add notmuch-hello-mode-hook

2011-10-30 Thread Daniel Schoepe
On Sun, 30 Oct 2011 11:54:07 -0400, Ivy Foster joyfulg...@archlinux.us wrote: Oh, I meant populating the hello buffer with the list of tags, not around adding or removing a message's tags. I sent a patch[1] a while ago that changes notmuch-hello such that the buffer contents are generated by a

Re: [PATCH] xutil.c: remove duplicate copies, create new library libutil.a to contain xutil.

2011-10-30 Thread Austin Clements
LGTM. Code deduplication is always a good thing. Internal interfaces for notmuch. doesn't seem like the best summary for util/error_util.h On Sun, Oct 23, 2011 at 11:18 AM, David Bremner da...@tethera.net wrote: From: David Bremner brem...@debian.org We keep the lib/xutil.c version. As a

Re: [PATCH] xutil.c: remove duplicate copies, create new library libutil.a to contain xutil.

2011-10-30 Thread Jameson Graef Rollins
On Sun, 23 Oct 2011 12:18:53 -0300, David Bremner da...@tethera.net wrote: We keep the lib/xutil.c version. As a consequence, also factor out _internal_error and associated macros. It might be overkill to make a new file error_util.c for this, but _internal_error does not really belong in

Re: [PATCH] xutil.c: remove duplicate copies, create new library libutil.a to contain xutil.

2011-10-30 Thread David Bremner
On Sun, 30 Oct 2011 17:09:10 -0700, Jameson Graef Rollins jroll...@finestructure.net wrote: Once I got over the encoding issues, this patch applied cleanly and seems to be working fine. OK, thanks Jamie and Austin for having a look. I pushed this patch, and the followup (changing

[RFC PATCH 2/3] cli: add support for limiting the number of search results

2011-10-30 Thread Jani Nikula
On Sat, 29 Oct 2011 19:30:50 +0200, Daniel Schoepe wrote: > On Fri, 28 Oct 2011 23:59:30 +0300, Jani Nikula wrote: > > @@ -412,6 +413,14 @@ notmuch_search_command (void *ctx, int argc, char > > *argv[]) > > fprintf (stderr, "Invalid value for --sort: %s\n", opt); > >

[RFC PATCH 3/3] emacs: support limiting the number of messages shown in search results

2011-10-30 Thread Jani Nikula
On Sat, 29 Oct 2011 19:25:22 +0200, Daniel Schoepe wrote: > On Fri, 28 Oct 2011 23:59:31 +0300, Jani Nikula wrote: > > Add support for limiting the maximum number of messages initially displayed > > in search results. When enabled, the search results will contain push > > buttons to double the

Experimental Tag Sharing

2011-10-30 Thread David Bremner
On Sat, 29 Oct 2011 17:44:56 -0700, Jameson Graef Rollins wrote: > On Sat, 29 Oct 2011 13:45:06 -0300, David Bremner > wrote: > > These patches did not apply cleanly for me from the list. I was able to > get them from bremner's "nmbug" branch [0], though. > [0]

[Patch] Fix Emacs interface bug dealing with invisible text

2011-10-30 Thread Thomas Jost
schnouki.net". Regards, -- Thomas/Schnouki -- next part -- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 489 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20111030/a5e2e6d5/attachment.pgp>

[PATCH 3/3] Add notmuch-hello-mode-hook

2011-10-30 Thread Thomas Jost
t was scrubbed... Name: not available Type: application/pgp-signature Size: 489 bytes Desc: not available URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20111030/1caed0a9/attachment.pgp>

[Patch] Fix Emacs interface bug dealing with invisible text

2011-10-30 Thread Ivy Foster
On Sun, 30 Oct 2011 10:58:55 +0100, Thomas Jost wrote: > Thanks for this patch. I haven't tested it, but I will! Fair enough (-: > However, I'm wondering something: wasn't this issue > already addressed in Dmitry's patch series that I rebased > and reposted almost a month ago, specifically

[PATCH 3/3] Add notmuch-hello-mode-hook

2011-10-30 Thread Ivy Foster
r removing a message's tags. iff -- 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/20111030/85ed80d5/attachment-0001.pgp>

[PATCH] xutil.c: remove duplicate copies, create new library libutil.a to contain xutil.

2011-10-30 Thread Austin Clements
LGTM. Code deduplication is always a good thing. "Internal interfaces for notmuch." doesn't seem like the best summary for util/error_util.h On Sun, Oct 23, 2011 at 11:18 AM, David Bremner wrote: > From: David Bremner > > We keep the lib/xutil.c version. As a consequence, also factor out >

[PATCH] xutil.c: remove duplicate copies, create new library libutil.a to contain xutil.

2011-10-30 Thread Jameson Graef Rollins
ssues, this patch applied cleanly and seems to be working fine. jamie. -- 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/20111030/2ff80fbf/attachment.pgp>