[PATCH v2 0/5] New output format sexp (Lisp S-Expressions)

[Jani Nikula]

Hi Peter -

On Tue, 04 Dec 2012, Peter Feigl <craven at gmx.net> wrote:
> This patch series adds a new output format "sexp" to notmuch-reply,
> notmuch-show and notmuch-search. These are useful for the Android mobile
> client and perhaps other Lisp programs as well.
> After the switch to a generic structured output printer, which was
> committed some months ago, these patches just add another one (like the
> json structured output printer).
> Basic tests and updates to the man pages are also included.
>
>
> Peter Feigl (5):
>   Adding an S-expression structured output printer.
>   Rename the -json printer functions in notmuch-reply and    
>     notmuch-show to generic -sprinter functions.
>   Use the S-Expression structured printer in notmuch-show,
>     notmuch-reply     and notmuch-search.

Patches 1-3 look good.

>   Adding tests for --format=sexp.

Did not review.

>   Updating man pages for new S-Expression output format.

Did not review, but noticed it adds lines with trailing whitespace.


BR,
Jani.


>
>  Makefile.local            |   1 +
>  man/man1/notmuch-reply.1  |  14 ++-
>  man/man1/notmuch-search.1 |  15 +--
>  man/man1/notmuch-show.1   |  36 +++++--
>  notmuch-client.h          |   8 +-
>  notmuch-reply.c           |  43 ++++----
>  notmuch-search.c          |   6 +-
>  notmuch-show.c            |  48 +++++----
>  sprinter-sexp.c           | 250 ++++++++++++++++++++++++++++++++++++++++++++++
>  sprinter.h                |   4 +
>  test/notmuch-test         |   1 +
>  test/sexp                 |  48 +++++++++
>  12 files changed, 414 insertions(+), 60 deletions(-)
>  create mode 100644 sprinter-sexp.c
>  create mode 100755 test/sexp
>
> -- 
> 1.8.0
>
> _______________________________________________
> notmuch mailing list
> notmuch at notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch