[PATCH v2 4/4] schemata: update for --body=true|false option

2012-07-23 Thread Mark Walters
Previously body: was a compulsory field in a message. The new --body=false option causes notmuch show to omit this field so update schemata to reflect this. --- devel/schemata |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/devel/schemata b/devel/schemata index

[PATCH v2 3/4] man: update man page for the new --body=true|false option

2012-07-23 Thread Mark Walters
--- man/man1/notmuch-show.1 | 16 1 files changed, 16 insertions(+), 0 deletions(-) diff --git a/man/man1/notmuch-show.1 b/man/man1/notmuch-show.1 index 5fa590e..91c19d8 100644 --- a/man/man1/notmuch-show.1 +++ b/man/man1/notmuch-show.1 @@ -154,6 +154,22 @@ The default is

[PATCH v2 2/4] test: add tests for the new --body=true|false option

2012-07-23 Thread Mark Walters
--- test/json |9 + 1 files changed, 9 insertions(+), 0 deletions(-) diff --git a/test/json b/test/json index 6439788..709b34f 100755 --- a/test/json +++ b/test/json @@ -7,6 +7,15 @@ add_message "[subject]=\"json-show-subject\"" "[date]=\"Sat, 01 Jan 2000 12:00:0 output=$(notmuch

[PATCH v2 1/4] cli: add --body=true|false option to notmuch-show.c

2012-07-23 Thread Mark Walters
This option allows the caller to suppress the output of the bodies of the messages. Currently this is only implemented for format=json. This is used by notmuch-pick.el (although not needed) because it gives a speed-up of at least a factor of a two (and in some cases a speed up of more than a

[PATCH v2 0/4] Add a --body option to notmuch show

2012-07-23 Thread Mark Walters
Version 2 of this patch set (version 1 at id:"1343060241-18283-1-git-send-email-markwalters1009 at gmail.com"). This corrects the errors Austin spotted in v1, and makes it explicit that --body=false and --part>0 are incompatible. Mark Walters (4): cli: add --body=true|false option to

[PATCH 4/4] schemata: update for --body=true|false option

2012-07-23 Thread Mark Walters
Previously body: was a compulsory field in a message. The new --body=false option causes notmuch show to omit this field so update schemata to reflect this. --- devel/schemata |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/devel/schemata b/devel/schemata index

[PATCH 3/4] man: update man page for the new --body=true|false option

2012-07-23 Thread Mark Walters
--- man/man1/notmuch-show.1 | 16 1 files changed, 16 insertions(+), 0 deletions(-) diff --git a/man/man1/notmuch-show.1 b/man/man1/notmuch-show.1 index 5fa590e..d781c47 100644 --- a/man/man1/notmuch-show.1 +++ b/man/man1/notmuch-show.1 @@ -154,6 +154,22 @@ The default is

[PATCH 2/4] test: add tests for the new --body=true|false option

2012-07-23 Thread Mark Walters
--- test/json |9 + 1 files changed, 9 insertions(+), 0 deletions(-) diff --git a/test/json b/test/json index 6439788..709b34f 100755 --- a/test/json +++ b/test/json @@ -7,6 +7,15 @@ add_message "[subject]=\"json-show-subject\"" "[date]=\"Sat, 01 Jan 2000 12:00:0 output=$(notmuch

[PATCH 1/4] cli: add --body=true|false option to notmuch-show.c

2012-07-23 Thread Mark Walters
This option allows the caller to suppress the output of the bodies of the messages. Currently this is only implemented for format=json. This is used by notmuch-pick.el (although not needed) because it gives a speed-up of at least a factor of a two (and in some cases a speed up of more than a

[PATCH 0/4] Add a --body option to notmuch show

2012-07-23 Thread Mark Walters
After discussion on irc today here is another version of the omit body patches. The conclusion on irc was that the option should just make the body: field in the JSON output optional. The --body=false option omits this field. The default remains --body=true including the body: field in the

[PATCH v8 0/3] notmuch-search: Structured Output Formatters

2012-07-23 Thread Tomi Ollila
On Mon, Jul 23 2012, craven at gmx.net wrote: > From: > > Currently there is no easy way to add support for different structured > formatters (like JSON). For example, adding support for S-Expressions > would result in code duplication. > > This patch series amends the situation by introducing

[PATCH v2 0/4] Add a --body option to notmuch show

2012-07-23 Thread Austin Clements
LGTM. Quoth Mark Walters on Jul 23 at 6:57 pm: > Version 2 of this patch set (version 1 at > id:"1343060241-18283-1-git-send-email-markwalters1009 at gmail.com"). This > corrects the errors Austin spotted in v1, and makes it explicit that > --body=false and --part>0 are incompatible. > > >

[PATCH 3/4] man: update man page for the new --body=true|false option

2012-07-23 Thread Austin Clements
Quoth Mark Walters on Jul 23 at 5:17 pm: > --- > man/man1/notmuch-show.1 | 16 > 1 files changed, 16 insertions(+), 0 deletions(-) > > diff --git a/man/man1/notmuch-show.1 b/man/man1/notmuch-show.1 > index 5fa590e..d781c47 100644 > --- a/man/man1/notmuch-show.1 > +++

[PATCH 1/4] cli: add --body=true|false option to notmuch-show.c

2012-07-23 Thread Austin Clements
Quoth Mark Walters on Jul 23 at 5:17 pm: > This option allows the caller to suppress the output of the bodies of > the messages. Currently this is only implemented for format=json. > > This is used by notmuch-pick.el (although not needed) because it gives > a speed-up of at least a factor of a

[PATCH v8 3/3] Use the structured formatters in notmuch-search.c.

2012-07-23 Thread cra...@gmx.net
From: This patch switches from the current ad-hoc printer to the structured formatters in sprinter.h, sprinter-text.c and sprinter-json.c. The JSON tests are changed slightly in order to make them PASS for the new structured output formatter. The text tests pass without

[PATCH v8 2/3] Add structured output formatter for JSON and plain text (but don't use them yet).

2012-07-23 Thread cra...@gmx.net
From: Using the new structured printer support in sprinter.h, implement sprinter_json_create, which returns a new JSON structured output formatter. The formatter prints output similar to the existing JSON, but with differences in whitespace (mostly newlines, --output=summary

[PATCH v8 1/3] Add support for structured output formatters.

2012-07-23 Thread cra...@gmx.net
From: This patch adds a new struct type sprinter_t, which is used for structured formatting, e.g. JSON or S-Expressions. The structure printer is heavily based on code from Austin Clements (id:87d34hsdx8.fsf at awakening.csail.mit.edu). It includes the following functions:

[PATCH v8 0/3] notmuch-search: Structured Output Formatters

2012-07-23 Thread cra...@gmx.net
From: Currently there is no easy way to add support for different structured formatters (like JSON). For example, adding support for S-Expressions would result in code duplication. This patch series amends the situation by introducing structured formatters, which allow different

[PATCH v2 0/4] Add a --body option to notmuch show

2012-07-23 Thread Jameson Graef Rollins
changed, 51 insertions(+), 11 deletions(-) Untested, but 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/20120723/2355377f/attachment.pgp>

[PATCH v8 0/3] notmuch-search: Structured Output Formatters

2012-07-23 Thread Austin Clements
LGTM Quoth craven at gmx.net on Jul 23 at 12:39 pm: > From: > > Currently there is no easy way to add support for different structured > formatters (like JSON). For example, adding support for S-Expressions > would result in code duplication. > > This patch series amends the situation by

[PATCH v8 2/3] Add structured output formatter for JSON and plain text (but don't use them yet).

2012-07-23 Thread craven
From: cra...@gmx.net Using the new structured printer support in sprinter.h, implement sprinter_json_create, which returns a new JSON structured output formatter. The formatter prints output similar to the existing JSON, but with differences in whitespace (mostly newlines, --output=summary prints

[PATCH v8 0/3] notmuch-search: Structured Output Formatters

2012-07-23 Thread craven
From: cra...@gmx.net Currently there is no easy way to add support for different structured formatters (like JSON). For example, adding support for S-Expressions would result in code duplication. This patch series amends the situation by introducing structured formatters, which allow different

[PATCH v8 3/3] Use the structured formatters in notmuch-search.c.

2012-07-23 Thread craven
From: cra...@gmx.net This patch switches from the current ad-hoc printer to the structured formatters in sprinter.h, sprinter-text.c and sprinter-json.c. The JSON tests are changed slightly in order to make them PASS for the new structured output formatter. The text tests pass without

Re: notmuch-emacs and bbdb

2012-07-23 Thread Daniel Bergey
Glad to hear I have another user! My function bbdb/notmuch-snarf-to doesn't work yet on more than one recipient. Once I get that working, I'd like to do what you suggest, and have some hook call bbdb/notmuch-snarf-to every time I send an email. Probably not before late August, though. bergey

Re: [PATCH v8 0/3] notmuch-search: Structured Output Formatters

2012-07-23 Thread Austin Clements
LGTM Quoth cra...@gmx.net on Jul 23 at 12:39 pm: From: cra...@gmx.net Currently there is no easy way to add support for different structured formatters (like JSON). For example, adding support for S-Expressions would result in code duplication. This patch series amends the situation by

Re: [PATCH v8 0/3] notmuch-search: Structured Output Formatters

2012-07-23 Thread Tomi Ollila
On Mon, Jul 23 2012, cra...@gmx.net wrote: From: cra...@gmx.net Currently there is no easy way to add support for different structured formatters (like JSON). For example, adding support for S-Expressions would result in code duplication. This patch series amends the situation by

[PATCH 0/4] Add a --body option to notmuch show

2012-07-23 Thread Mark Walters
After discussion on irc today here is another version of the omit body patches. The conclusion on irc was that the option should just make the body: field in the JSON output optional. The --body=false option omits this field. The default remains --body=true including the body: field in the

[PATCH 1/4] cli: add --body=true|false option to notmuch-show.c

2012-07-23 Thread Mark Walters
This option allows the caller to suppress the output of the bodies of the messages. Currently this is only implemented for format=json. This is used by notmuch-pick.el (although not needed) because it gives a speed-up of at least a factor of a two (and in some cases a speed up of more than a

[PATCH 2/4] test: add tests for the new --body=true|false option

2012-07-23 Thread Mark Walters
--- test/json |9 + 1 files changed, 9 insertions(+), 0 deletions(-) diff --git a/test/json b/test/json index 6439788..709b34f 100755 --- a/test/json +++ b/test/json @@ -7,6 +7,15 @@ add_message [subject]=\json-show-subject\ [date]=\Sat, 01 Jan 2000 12:00:0 output=$(notmuch show

[PATCH 3/4] man: update man page for the new --body=true|false option

2012-07-23 Thread Mark Walters
--- man/man1/notmuch-show.1 | 16 1 files changed, 16 insertions(+), 0 deletions(-) diff --git a/man/man1/notmuch-show.1 b/man/man1/notmuch-show.1 index 5fa590e..d781c47 100644 --- a/man/man1/notmuch-show.1 +++ b/man/man1/notmuch-show.1 @@ -154,6 +154,22 @@ The default is

[PATCH 4/4] schemata: update for --body=true|false option

2012-07-23 Thread Mark Walters
Previously body: was a compulsory field in a message. The new --body=false option causes notmuch show to omit this field so update schemata to reflect this. --- devel/schemata |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/devel/schemata b/devel/schemata index

Re: [PATCH 1/4] cli: add --body=true|false option to notmuch-show.c

2012-07-23 Thread Austin Clements
Quoth Mark Walters on Jul 23 at 5:17 pm: This option allows the caller to suppress the output of the bodies of the messages. Currently this is only implemented for format=json. This is used by notmuch-pick.el (although not needed) because it gives a speed-up of at least a factor of a two

Re: [PATCH 3/4] man: update man page for the new --body=true|false option

2012-07-23 Thread Austin Clements
Quoth Mark Walters on Jul 23 at 5:17 pm: --- man/man1/notmuch-show.1 | 16 1 files changed, 16 insertions(+), 0 deletions(-) diff --git a/man/man1/notmuch-show.1 b/man/man1/notmuch-show.1 index 5fa590e..d781c47 100644 --- a/man/man1/notmuch-show.1 +++

[PATCH v2 0/4] Add a --body option to notmuch show

2012-07-23 Thread Mark Walters
Version 2 of this patch set (version 1 at id:1343060241-18283-1-git-send-email-markwalters1...@gmail.com). This corrects the errors Austin spotted in v1, and makes it explicit that --body=false and --part0 are incompatible. Mark Walters (4): cli: add --body=true|false option to notmuch-show.c

[PATCH v2 1/4] cli: add --body=true|false option to notmuch-show.c

2012-07-23 Thread Mark Walters
This option allows the caller to suppress the output of the bodies of the messages. Currently this is only implemented for format=json. This is used by notmuch-pick.el (although not needed) because it gives a speed-up of at least a factor of a two (and in some cases a speed up of more than a

[PATCH v2 2/4] test: add tests for the new --body=true|false option

2012-07-23 Thread Mark Walters
--- test/json |9 + 1 files changed, 9 insertions(+), 0 deletions(-) diff --git a/test/json b/test/json index 6439788..709b34f 100755 --- a/test/json +++ b/test/json @@ -7,6 +7,15 @@ add_message [subject]=\json-show-subject\ [date]=\Sat, 01 Jan 2000 12:00:0 output=$(notmuch show

[PATCH v2 3/4] man: update man page for the new --body=true|false option

2012-07-23 Thread Mark Walters
--- man/man1/notmuch-show.1 | 16 1 files changed, 16 insertions(+), 0 deletions(-) diff --git a/man/man1/notmuch-show.1 b/man/man1/notmuch-show.1 index 5fa590e..91c19d8 100644 --- a/man/man1/notmuch-show.1 +++ b/man/man1/notmuch-show.1 @@ -154,6 +154,22 @@ The default is

[PATCH v2 4/4] schemata: update for --body=true|false option

2012-07-23 Thread Mark Walters
Previously body: was a compulsory field in a message. The new --body=false option causes notmuch show to omit this field so update schemata to reflect this. --- devel/schemata |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/devel/schemata b/devel/schemata index

Re: [PATCH v2 0/4] Add a --body option to notmuch show

2012-07-23 Thread Austin Clements
LGTM. Quoth Mark Walters on Jul 23 at 6:57 pm: Version 2 of this patch set (version 1 at id:1343060241-18283-1-git-send-email-markwalters1...@gmail.com). This corrects the errors Austin spotted in v1, and makes it explicit that --body=false and --part0 are incompatible. Mark Walters

Re: [PATCH v2 0/4] Add a --body option to notmuch show

2012-07-23 Thread Jameson Graef Rollins
On Mon, Jul 23 2012, Mark Walters markwalters1...@gmail.com wrote: Version 2 of this patch set (version 1 at id:1343060241-18283-1-git-send-email-markwalters1...@gmail.com). This corrects the errors Austin spotted in v1, and makes it explicit that --body=false and --part0 are incompatible.