mirror of
https://git.notmuchmail.org/git/notmuch
synced 2024-12-27 03:41:41 +01:00
f6ec7ca78f
Now that we have maildir synchronization turned on by default, it's advantageous to make all of the tests exercise it as much as possible.
33 lines
1.2 KiB
Text
33 lines
1.2 KiB
Text
From: "Carl Worth" <cworth@cworth.org>
|
|
To: notmuch@notmuchmail.org
|
|
Date: Wed, 18 Nov 2009 03:31:23 -0800
|
|
Subject: [notmuch] [PATCH] Error out if no query is supplied to search
|
|
instead of going into an infinite loop
|
|
In-Reply-To: <cf0c4d610911171623q3e27a0adx802e47039b57604b@mail.gmail.com>
|
|
References: <cf0c4d610911171623q3e27a0adx802e47039b57604b@mail.gmail.com>
|
|
Message-ID: <877htoqdbo.fsf@yoom.home.cworth.org>
|
|
|
|
On Tue, 17 Nov 2009 16:23:53 -0800, Alex Botero-Lowry <alex.boterolowry at gmail.com> wrote:
|
|
> In this case error out when no query is supplied. There seems to be an
|
|
> infinite-loop casued by i think notmuch_query_search_threads having
|
|
> an exception:
|
|
> A Xapian exception occurred: Syntax: <expression> AND <expression>
|
|
> A Xapian exception occurred: Syntax: <expression> AND <expression>
|
|
> A Xapian exception occurred: Syntax: <expression> AND <expression>
|
|
>
|
|
> I'll look into that bug specifically a bit later.
|
|
>
|
|
> It might be better to do a usage instead of just throwing an error here?
|
|
|
|
Definitely.
|
|
|
|
Priit Laes reported the same thing in IRC and I've just committed a
|
|
patch to give a nice error message:
|
|
|
|
$ ./notmuch search
|
|
Error: notmuch search requires at least one search term.
|
|
|
|
Thanks for the report!
|
|
|
|
-Carl
|
|
|