unofficial mirror of notmuch@notmuchmail.org
 help / color / mirror / code / Atom feed
From: Mark Walters <markwalters1009@gmail.com>
To: Peter Feigl <craven@gmx.net>, notmuch@notmuchmail.org
Subject: Re: [PATCH v5 0/5] New output format sexp (Lisp S-Expressions)
Date: Thu, 06 Dec 2012 22:13:57 +0000	[thread overview]
Message-ID: <87obi67rru.fsf@qmul.ac.uk> (raw)
In-Reply-To: <1354828335-18151-1-git-send-email-craven@gmx.net>


On Thu, 06 Dec 2012, Peter Feigl <craven@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, Emacs 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.
>
> This version adresses comments by Tomi Ollila and Austin Clements.

This looks good to me except there are a couple of trailing whitespace
errors

Best wishes

Mark


>
> Old versions:
> v4: id:cover.1354794428.git.craven@gmx.net
> v3: id:1354779189-12231-1-git-send-email-craven@gmx.net
> v2: id:1354632382-15609-1-git-send-email-craven@gmx.net
> v1: 1354264143-30173-1-git-send-email-craven@gmx.net
>
> 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.
>   Adding tests for --format=sexp.
>   Updating man pages for new S-Expression output format.
>
>  Makefile.local            |   1 +
>  devel/schemata            |  24 +++--
>  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           |  48 ++++++----
>  notmuch-search.c          |   6 +-
>  notmuch-show.c            |  76 ++++++++-------
>  sprinter-sexp.c           | 235 ++++++++++++++++++++++++++++++++++++++++++++++
>  sprinter.h                |   4 +
>  test/notmuch-test         |   1 +
>  test/sexp                 |  48 ++++++++++
>  13 files changed, 434 insertions(+), 82 deletions(-)
>  create mode 100644 sprinter-sexp.c
>  create mode 100755 test/sexp
>
> -- 
> 1.8.0
>
> _______________________________________________
> notmuch mailing list
> notmuch@notmuchmail.org
> http://notmuchmail.org/mailman/listinfo/notmuch

  parent reply	other threads:[~2012-12-06 22:14 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-12-06 21:12 [PATCH v5 0/5] New output format sexp (Lisp S-Expressions) Peter Feigl
2012-12-06 21:12 ` [PATCH v5 1/5] Adding an S-expression structured output printer Peter Feigl
2012-12-06 21:12 ` [PATCH v5 2/5] Rename the -json printer functions in notmuch-reply and notmuch-show to generic -sprinter functions Peter Feigl
2012-12-06 21:12 ` [PATCH v5 3/5] Use the S-Expression structured printer in notmuch-show, notmuch-reply and notmuch-search Peter Feigl
2012-12-06 21:12 ` [PATCH v5 4/5] Adding tests for --format=sexp Peter Feigl
2012-12-06 21:12 ` [PATCH v5 5/5] Updating man pages for new S-Expression output format Peter Feigl
2012-12-06 22:13 ` Mark Walters [this message]
2012-12-07 13:11   ` [PATCH v5 0/5] New output format sexp (Lisp S-Expressions) David Bremner
2012-12-07 16:03 ` Austin Clements
2012-12-08 13:34 ` David Bremner

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://notmuchmail.org/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87obi67rru.fsf@qmul.ac.uk \
    --to=markwalters1009@gmail.com \
    --cc=craven@gmx.net \
    --cc=notmuch@notmuchmail.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://yhetil.org/notmuch.git/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).