Treat empty '/' as "find next"

When the user requests a forward or backwards search, if
they provide no search string then we just abort the search.
However, many other programs (such as less and vi) treat
this as "repeat the last search", so users may have their
fingers trained to do this rather than use the explicit
"find next" feature.

This patch causes tig, if there was a previous search, to
find the next (or previous, if "?" was used) match in such a
case.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Jonas Fonseca <fonseca@diku.dk>

diff --git a/tig.c b/tig.c
index c8ae05395c875397fef7be41c9ebf4ac5473af62..53b2fdbf3dff95872a09c1e17dbc0ba0702ba1bd 100644 (file)
--- a/tig.c
+++ b/tig.c
@@ -6831,6 +6831,10 @@ main(int argc, const char *argv[])
 
                        if (search)
                                string_ncopy(opt_search, search, strlen(search));
+                       else if (*opt_search)
+                               request = request == REQ_SEARCH ?
+                                       REQ_FIND_NEXT :
+                                       REQ_FIND_PREV;
                        else
                                request = REQ_NONE;
                        break;