[notmuch] [PATCH] Make reply/show/tag all require at least one search term

[Carl Worth]

On Wed, 18 Nov 2009 13:45:58 -0800, Keith Packard  wrote:
> In particular, notmuch tag -inbox "" tended to take a long time to
> run, happened if you hit 'a' on a blank line in the search view and
> probably didn't have the desired effect.

Another nice fix. Thanks again. This is pushed.

-Carl

[notmuch] [PATCH] Make reply/show/tag all require at least one search term

[Keith Packard]

In particular, notmuch tag -inbox "" tended to take a long time to
run, happened if you hit 'a' on a blank line in the search view and
probably didn't have the desired effect.

Signed-off-by: Keith Packard 
---
 notmuch-reply.c |   13 +
 notmuch-show.c  |   13 +
 notmuch-tag.c   |8 
 3 files changed, 22 insertions(+), 12 deletions(-)

diff --git a/notmuch-reply.c b/notmuch-reply.c
index 4a4a782..31e2c87 100644
--- a/notmuch-reply.c
+++ b/notmuch-reply.c
@@ -196,16 +196,21 @@ notmuch_reply_command (void *ctx, int argc, char *argv[])
 if (config == NULL)
return 1;

-notmuch = notmuch_database_open (notmuch_config_get_database_path 
(config));
-if (notmuch == NULL)
-   return 1;
-
 query_string = query_string_from_args (ctx, argc, argv);
 if (query_string == NULL) {
fprintf (stderr, "Out of memory\n");
return 1;
 }

+if (*query_string == '\0') {
+   fprintf (stderr, "Error: notmuch reply requires at least one search 
term.\n");
+   return 1;
+}
+
+notmuch = notmuch_database_open (notmuch_config_get_database_path 
(config));
+if (notmuch == NULL)
+   return 1;
+
 query = notmuch_query_create (notmuch, query_string);
 if (query == NULL) {
fprintf (stderr, "Out of memory\n");
diff --git a/notmuch-show.c b/notmuch-show.c
index 59a58c2..c0c3ca3 100644
--- a/notmuch-show.c
+++ b/notmuch-show.c
@@ -196,16 +196,21 @@ notmuch_show_command (void *ctx, unused (int argc), 
unused (char *argv[]))
 if (config == NULL)
return 1;

-notmuch = notmuch_database_open (notmuch_config_get_database_path 
(config));
-if (notmuch == NULL)
-   return 1;
-
 query_string = query_string_from_args (ctx, argc, argv);
 if (query_string == NULL) {
fprintf (stderr, "Out of memory\n");
return 1;
 }

+if (*query_string == '\0') {
+   fprintf (stderr, "Error: notmuch show requires at least one search 
term.\n");
+   return 1;
+}
+
+notmuch = notmuch_database_open (notmuch_config_get_database_path 
(config));
+if (notmuch == NULL)
+   return 1;
+
 query = notmuch_query_create (notmuch, query_string);
 if (query == NULL) {
fprintf (stderr, "Out of memory\n");
diff --git a/notmuch-tag.c b/notmuch-tag.c
index 7d92ec4..12ab86c 100644
--- a/notmuch-tag.c
+++ b/notmuch-tag.c
@@ -83,8 +83,10 @@ notmuch_tag_command (void *ctx, unused (int argc), unused 
(char *argv[]))
return 1;
 }

-if (i == argc) {
-   fprintf (stderr, "Error: 'notmuch tag' requires at least one search 
term.\n");
+query_string = query_string_from_args (ctx, argc - i, [i]);
+
+if (*query_string == '\0') {
+   fprintf (stderr, "Error: notmuch tag requires at least one search 
term.\n");
return 1;
 }

@@ -96,8 +98,6 @@ notmuch_tag_command (void *ctx, unused (int argc), unused 
(char *argv[]))
 if (notmuch == NULL)
return 1;

-query_string = query_string_from_args (ctx, argc - i, [i]);
-
 query = notmuch_query_create (notmuch, query_string);
 if (query == NULL) {
fprintf (stderr, "Out of memory.\n");
-- 
1.6.5.2