[PATCH v4 5/5] emacs: show: set default show-all-multipart/alternatives to nil

2012-12-17 Thread Mark Walters
Now that the invisibility display of parts is present we no longer need to force the display of all multipart/alternatives: users can toggle them for themselves when needed. --- emacs/notmuch-show.el |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/emacs/notmuch-show.el

[PATCH v4 4/5] emacs: wash: fix fake-diff part to include msg parameter

2012-12-17 Thread Mark Walters
The fake-diff part from notmuch wash did not pass the msg parameter to notmuch-show-insert-bodypart. Previously this did not matter, but is now needed to get the overlays right for invisibility. --- emacs/notmuch-wash.el |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git

[PATCH v4 3/5] emacs: show: add invisibility button action

2012-12-17 Thread Mark Walters
This adds a button action to show hidden parts. In this version "t" toggles the visibility of a part. In addition "RET" on a non-shown part shows it. The button is used to hide parts when appropriate (eg text/html in multipart/alternative). --- emacs/notmuch-show.el | 35

[PATCH v4 2/5] emacs: show: add overlays for each part

2012-12-17 Thread Mark Walters
This makes notmuch-show-insert-bodypart add an overlay for any non-trivial part with a button header (currently the first text/plain part does not have a button). At this point the overlay is available to the button but there is no action using it yet. In addition the argument HIDE is passed down

[PATCH v4 1/5] emacs: show: modify insert-part-header to save the button text

2012-12-17 Thread Mark Walters
This just make notmuch-show-insert-part-header save the basic button text for parts as an attribute. This makes it simpler for the button action (added in a later patch) to reword the label as appropriate (eg append "(not shown)" or not as appropriate). --- emacs/notmuch-show.el | 18

[PATCH v4 0/5] Use invisibility to toggle display of all parts including multipart

2012-12-17 Thread Mark Walters
This is version 4 of this series (previous version at id:139338-14313-1-git-send-email-markwalters1009 at gmail.com). The only change should be a bugfix which, for reasons I don't understand, only causes a problem on emacs 24. The problem is that the part invisibility code looks for a part

[PATCH] contrib: pick: Do not indent messages in the message pane

2012-12-17 Thread Tomi Ollila
On Sun, Dec 09 2012, Mark Walters wrote: > Currently pick just uses notmuch-show to display messages in the > message pane: this means that they get indented just as show > would. However, since pick is only displaying one message at a time > there is no need to indent so override the

[PATCH 2/3] tag-utils: use the tag_opt_list_t as talloc context, if possible.

2012-12-17 Thread Tomi Ollila
On Mon, Dec 17 2012, david at tethera.net wrote: > From: David Bremner > > This code is no less correct than the previous version, since it does > not make sense for the array to live longer than the wrapping struct. > > By not relying on the context passed into tag_parse_line, we can allow >

[PATCH] contrib: pick: Do not indent messages in the message pane

2012-12-17 Thread Jameson Graef Rollins
indentation. Tested and LGTM. 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/20121217/c8561e74/attachment.pgp>

[PATCH v5 0/4] indicate length of omitted body content

2012-12-17 Thread David Bremner
Peter Wang writes: > This obsoletes 1355057796-19260-1-git-send-email-markwalters1009 at gmail.com > > Peter Wang (4): > test: normalize only message filenames in show json > show: indicate charset for all omitted parts > show: indicate length, encoding of omitted body content > test:

[PATCH v3 3/5] emacs: show: add invisibility button action

2012-12-17 Thread David Bremner
Mark Walters writes: > This adds a button action to show hidden parts. In this version "t" > toggles the visibility of a part. In addition "RET" on a non-shown > part shows it. After applying this patch (and the two before it), I get emacs-show: Testing emacs notmuch-show view FAIL Hiding

gmail importer script

2012-12-17 Thread Patrick Totzke
ture.asc Type: application/pgp-signature Size: 198 bytes Desc: signature URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20121217/9349b9c0/attachment.pgp>

[PATCH v4 0/5] add --format=text0 to notmuch search

2012-12-17 Thread Tomi Ollila
On Mon, Dec 17 2012, Tomi Ollila wrote: > On Mon, Dec 17 2012, Jani Nikula wrote: > >> Hi all, a quick rebase of id:cover.1355691124.git.jani at nikula.org to fix >> a tiny conflict in patch context of 3/5. >> >> BR, >> Jani. > > As Mark & Austin (also) gave +1 to v3 added notmuch::patch

[PATCH v4 0/5] add --format=text0 to notmuch search

2012-12-17 Thread Tomi Ollila
On Mon, Dec 17 2012, Jani Nikula wrote: > Hi all, a quick rebase of id:cover.1355691124.git.jani at nikula.org to fix > a tiny conflict in patch context of 3/5. > > BR, > Jani. As Mark & Austin (also) gave +1 to v3 added notmuch::patch (only). Tomi > > Jani Nikula (5): > sprinter: clarify

[PATCH 3/3] notmuch-restore: allocate a temporary talloc context for each line parsed.

2012-12-17 Thread Jani Nikula
On Sun, 16 Dec 2012, david at tethera.net wrote: > From: David Bremner > > This lets the high level code in notmuch restore be ignorant about > what the lower level code is doing as far as allocating memory. > --- > notmuch-restore.c | 12 ++-- > 1 file changed, 10 insertions(+), 2

[PATCH v4 5/5] man: document notmuch search --format=text0

2012-12-17 Thread Jani Nikula
--- man/man1/notmuch-search.1 | 26 -- 1 file changed, 16 insertions(+), 10 deletions(-) diff --git a/man/man1/notmuch-search.1 b/man/man1/notmuch-search.1 index 5c771fa..12f6719 100644 --- a/man/man1/notmuch-search.1 +++ b/man/man1/notmuch-search.1 @@ -25,9 +25,11 @@

[PATCH v4 4/5] test: notmuch search --format=text0

2012-12-17 Thread Jani Nikula
--- test/text | 33 + 1 file changed, 33 insertions(+) diff --git a/test/text b/test/text index 428c89b..b5ccefc 100755 --- a/test/text +++ b/test/text @@ -52,4 +52,37 @@ output=$(notmuch search --format=text "t?xt-search-m?ssage" | notmuch_search_s

[PATCH v4 3/5] cli: add --format=text0 to notmuch search

2012-12-17 Thread Jani Nikula
Add new format text0, which is otherwise the same as text, but use the null character as separator instead of the newline character. This is similar to find(1) -print0 option, and works together with the xargs(1) -0 option. --- notmuch-search.c | 16 ++-- 1 file changed, 14

[PATCH v4 2/5] sprinter: add text0 formatter for null character separated text

2012-12-17 Thread Jani Nikula
Same as the text formatter, but with each field separated by a null character rather than a newline character. --- sprinter-text.c | 22 ++ sprinter.h |6 ++ 2 files changed, 28 insertions(+) diff --git a/sprinter-text.c b/sprinter-text.c index 10343be..7779488

[PATCH v4 1/5] sprinter: clarify separator documentation

2012-12-17 Thread Jani Nikula
For text printers, the separator is a syntactic element. --- sprinter.h |9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/sprinter.h b/sprinter.h index 59776a9..f43a844 100644 --- a/sprinter.h +++ b/sprinter.h @@ -42,10 +42,11 @@ typedef struct sprinter { */

[PATCH v4 0/5] add --format=text0 to notmuch search

2012-12-17 Thread Jani Nikula
Hi all, a quick rebase of id:cover.1355691124.git.jani at nikula.org to fix a tiny conflict in patch context of 3/5. BR, Jani. Jani Nikula (5): sprinter: clarify separator documentation sprinter: add text0 formatter for null character separated text cli: add --format=text0 to notmuch

Re: gmail importer script

2012-12-17 Thread Patrick Totzke
BTW: I removed the whole maildir contents and restarted the downlowd over night. here's the result: --- Traceback (most recent call last):2| | 26% ETA:

Re: [PATCH v3 3/5] emacs: show: add invisibility button action

2012-12-17 Thread David Bremner
Mark Walters markwalters1...@gmail.com writes: This adds a button action to show hidden parts. In this version t toggles the visibility of a part. In addition RET on a non-shown part shows it. After applying this patch (and the two before it), I get emacs-show: Testing emacs notmuch-show

Re: [PATCH v5 0/4] indicate length of omitted body content

2012-12-17 Thread David Bremner
Peter Wang noval...@gmail.com writes: This obsoletes 1355057796-19260-1-git-send-email-markwalters1...@gmail.com Peter Wang (4): test: normalize only message filenames in show json show: indicate charset for all omitted parts show: indicate length, encoding of omitted body content

Re: [PATCH 2/3] tag-utils: use the tag_opt_list_t as talloc context, if possible.

2012-12-17 Thread Tomi Ollila
On Mon, Dec 17 2012, da...@tethera.net wrote: From: David Bremner brem...@debian.org This code is no less correct than the previous version, since it does not make sense for the array to live longer than the wrapping struct. By not relying on the context passed into tag_parse_line, we can

Re: [PATCH] contrib: pick: Do not indent messages in the message pane

2012-12-17 Thread Tomi Ollila
On Sun, Dec 09 2012, Mark Walters wrote: Currently pick just uses notmuch-show to display messages in the message pane: this means that they get indented just as show would. However, since pick is only displaying one message at a time there is no need to indent so override the indentation.

Re: [PATCH] contrib: pick: Do not indent messages in the message pane

2012-12-17 Thread Jameson Graef Rollins
On Sun, Dec 09 2012, Mark Walters markwalters1...@gmail.com wrote: Currently pick just uses notmuch-show to display messages in the message pane: this means that they get indented just as show would. However, since pick is only displaying one message at a time there is no need to indent so

[PATCH v4 0/5] Use invisibility to toggle display of all parts including multipart

2012-12-17 Thread Mark Walters
This is version 4 of this series (previous version at id:139338-14313-1-git-send-email-markwalters1...@gmail.com). The only change should be a bugfix which, for reasons I don't understand, only causes a problem on emacs 24. The problem is that the part invisibility code looks for a part

[PATCH v4 1/5] emacs: show: modify insert-part-header to save the button text

2012-12-17 Thread Mark Walters
This just make notmuch-show-insert-part-header save the basic button text for parts as an attribute. This makes it simpler for the button action (added in a later patch) to reword the label as appropriate (eg append (not shown) or not as appropriate). --- emacs/notmuch-show.el | 18

[PATCH v4 2/5] emacs: show: add overlays for each part

2012-12-17 Thread Mark Walters
This makes notmuch-show-insert-bodypart add an overlay for any non-trivial part with a button header (currently the first text/plain part does not have a button). At this point the overlay is available to the button but there is no action using it yet. In addition the argument HIDE is passed down

[PATCH v4 3/5] emacs: show: add invisibility button action

2012-12-17 Thread Mark Walters
This adds a button action to show hidden parts. In this version t toggles the visibility of a part. In addition RET on a non-shown part shows it. The button is used to hide parts when appropriate (eg text/html in multipart/alternative). --- emacs/notmuch-show.el | 35

[PATCH v4 4/5] emacs: wash: fix fake-diff part to include msg parameter

2012-12-17 Thread Mark Walters
The fake-diff part from notmuch wash did not pass the msg parameter to notmuch-show-insert-bodypart. Previously this did not matter, but is now needed to get the overlays right for invisibility. --- emacs/notmuch-wash.el |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git

[PATCH v4 5/5] emacs: show: set default show-all-multipart/alternatives to nil

2012-12-17 Thread Mark Walters
Now that the invisibility display of parts is present we no longer need to force the display of all multipart/alternatives: users can toggle them for themselves when needed. --- emacs/notmuch-show.el |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/emacs/notmuch-show.el

[DRAFT PATCH] emacs: Eliminate buffer invisibility specs from show and wash

2012-12-17 Thread Austin Clements
Previously, all visibility in show buffers for headers, message bodies, and washed text was specified by generating one or more symbols for each region and creating overlays with their 'invisible property set to carefully crafted combinations of these symbols. Visibility was controlled not by