From 5171831813716fa09e64d37a9d6f6ba4fc418c5b Mon Sep 17 00:00:00 2001
From: Carl Worth <cworth@cworth.org>
Date: Tue, 23 Feb 2010 12:41:59 -0800
Subject: [PATCH] notmuch search: Use "thread" rather than "id" when formatting
 with JSON

The text output uses thread:<foo>, (which is a syntax directly supported
by "notmuch show"), so make the json output be "thread", "<foo>" rather
than "id", "<foo>". This should help avoid confusion of thread IDs with
message IDs, (which do use the "id" prefix in searches).
---
 notmuch-search.c | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/notmuch-search.c b/notmuch-search.c
index 482c6e8e..25dd6eba 100644
--- a/notmuch-search.c
+++ b/notmuch-search.c
@@ -24,7 +24,7 @@ typedef struct search_format {
     const char *results_start;
     const char *thread_start;
     void (*thread) (const void *ctx,
-		    const char *id,
+		    const char *thread_id,
 		    const time_t date,
 		    const int matched,
 		    const int total,
@@ -41,7 +41,7 @@ typedef struct search_format {
 
 static void
 format_thread_text (const void *ctx,
-		    const char *id,
+		    const char *thread_id,
 		    const time_t date,
 		    const int matched,
 		    const int total,
@@ -60,7 +60,7 @@ static const search_format_t format_text = {
 
 static void
 format_thread_json (const void *ctx,
-		    const char *id,
+		    const char *thread_id,
 		    const time_t date,
 		    const int matched,
 		    const int total,
@@ -79,7 +79,7 @@ static const search_format_t format_json = {
 
 static void
 format_thread_text (const void *ctx,
-		    const char *id,
+		    const char *thread_id,
 		    const time_t date,
 		    const int matched,
 		    const int total,
@@ -87,7 +87,7 @@ format_thread_text (const void *ctx,
 		    const char *subject)
 {
     printf ("thread:%s %12s [%d/%d] %s; %s",
-	    id,
+	    thread_id,
 	    notmuch_time_relative_date (ctx, date),
 	    matched,
 	    total,
@@ -97,7 +97,7 @@ format_thread_text (const void *ctx,
 
 static void
 format_thread_json (const void *ctx,
-		    const char *id,
+		    const char *thread_id,
 		    const time_t date,
 		    const int matched,
 		    const int total,
@@ -110,18 +110,18 @@ format_thread_json (const void *ctx,
 
     tm = gmtime (&date);
     if (tm == NULL)
-	INTERNAL_ERROR ("gmtime failed on thread %s.", id);
+	INTERNAL_ERROR ("gmtime failed on thread %s.", thread_id);
 
     if (strftime (timestamp, sizeof (timestamp), "%s", tm) == 0)
-	INTERNAL_ERROR ("strftime failed on thread %s.", id);
+	INTERNAL_ERROR ("strftime failed on thread %s.", thread_id);
 
-    printf ("\"id\": %s,\n"
+    printf ("\"thread\": %s,\n"
 	    "\"timestamp\": %s,\n"
 	    "\"matched\": %d,\n"
 	    "\"total\": %d,\n"
 	    "\"authors\": %s,\n"
 	    "\"subject\": %s,\n",
-	    json_quote_str (ctx_quote, id),
+	    json_quote_str (ctx_quote, thread_id),
 	    timestamp,
 	    matched,
 	    total,