unofficial mirror of notmuch@notmuchmail.org
 help / color / mirror / code / Atom feed
From: Mark Walters <markwalters1009@gmail.com>
To: notmuch@notmuchmail.org
Subject: [PATCH v4 0/4] Allow JSON to use non-entire thread, and use for elide
Date: Tue, 24 Apr 2012 10:11:11 +0100	[thread overview]
Message-ID: <1335258675-29439-1-git-send-email-markwalters1009@gmail.com> (raw)

This is version 4 of this patch set (previous version at [1]) which
removes the --entire-thread constraint from json output in
notmuch-show, and then uses to implement the emacs elide
functionality.

Changes from the previous version: 

	split the cli patch 

	update devel/schemata 

	output "null" rather than "{}" for a null message and do so
	using a format string remove this feature

	from devel/TODO

The first three patches implement the client side changes: one to
allow null messages to be output in Json (without upsetting the
parsing/structure), one to do the command line parsing for
--entire-thread=false and one to update the schemata.

The final emacs patch is independent (and the first three make sense
without it) and uses this new functionality to implement the elide
option.

I think this addresses all review comments

Best wishes

Mark

[1] id:"1334999750-21811-1-git-send-email-markwalters1009@gmail.com"

Mark Walters (4):
  cli: Let json output "null" messages for non --entire-thread
  cli: make --entire-thread=false work for format=json.
  Update devel/schemata for --entire-thread=false
  emacs: make elide messages use notmuch-show for omitting messages.

 devel/TODO            |    2 --
 devel/schemata        |    2 +-
 emacs/notmuch-show.el |   18 ++++++++++--------
 notmuch-client.h      |    1 +
 notmuch-show.c        |   36 ++++++++++++++++++++++++++++++------
 5 files changed, 42 insertions(+), 17 deletions(-)

-- 
1.7.9.1

             reply	other threads:[~2012-04-24  9:11 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-04-24  9:11 Mark Walters [this message]
2012-04-24  9:11 ` [PATCH v4 1/4] cli: Let json output "null" messages for non --entire-thread Mark Walters
2012-04-24  9:11 ` [PATCH v4 2/4] cli: make --entire-thread=false work for format=json Mark Walters
2012-05-26  2:15   ` Peter Wang
2012-05-26  8:28     ` Mark Walters
2012-05-26 13:31       ` David Bremner
2012-04-24  9:11 ` [PATCH v4 3/4] Update devel/schemata for --entire-thread=false Mark Walters
2012-04-24  9:11 ` [PATCH v4 4/4] emacs: make elide messages use notmuch-show for omitting messages Mark Walters
2012-05-27 22:30 ` [PATCH v4 0/4] Allow JSON to use non-entire thread, and use for elide Daniel Schoepe

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=1335258675-29439-1-git-send-email-markwalters1009@gmail.com \
    --to=markwalters1009@gmail.com \
    --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).