This is version 7 of this patch set: the previous version is at id:"1338106946-7611-1-git-send-email-markwalters1009 at gmail.com".
I have fixed the whitespace errors, and simplified the logic in notmuch-show.c as suggested by Peter. I did not update the error output string as what is there is consistent with the other error cases. However, I have a followup patch which rejigs the error handling considerably, as there are some other oddities: for example currently notmuch show --part id:<a message> (with no number to "part") silently fails. It is a very simple but relatively large patch: I will post it as a reply to this thread but it is not logically part of the series (although it touches a lot of the same code). Best wishes Mark Mark Walters (6): cli: command line parsing: allow default for keyword options 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. cli: notmuch-show.c fix whitespace error command-line-arguments.c | 17 +++++++++++--- devel/TODO | 2 - devel/schemata | 2 +- emacs/notmuch-show.el | 18 ++++++++------ notmuch-client.h | 1 + notmuch-show.c | 55 +++++++++++++++++++++++++++++++++++++-------- 6 files changed, 70 insertions(+), 25 deletions(-) -- 1.7.9.1