Fix another small oversight in command_no_begin patch.

Need a "return false" to prevent tests from continuing after we've moved
the "query" pointer.  As it stood, it'd accept "DROP DISCARD ALL" as a
match.

diff --git a/src/bin/psql/common.c b/src/bin/psql/common.c
index 81a633a3d01f21ab3777985d3defb7fe6c0aca9e..76cba901e0c9d1d89b4d54e9b90ccd099169394c 100644 (file)
--- a/src/bin/psql/common.c
+++ b/src/bin/psql/common.c
@@ -1367,6 +1367,7 @@ command_no_begin(const char *query)
            return true;
        if (wordlen == 10 && pg_strncasecmp(query, "tablespace", 10) == 0)
            return true;
+       return false;
    }
 
    /* DISCARD ALL isn't allowed in xacts, but other variants are allowed. */
@@ -1382,6 +1383,7 @@ command_no_begin(const char *query)
 
        if (wordlen == 3 && pg_strncasecmp(query, "all", 3) == 0)
            return true;
+       return false;
    }
 
    return false;