On 01/19/2016 07:55 AM, Michael Paquier wrote:
+ /* If we have COPY BINARY, compelete with list of tables */ s/compelete/complete
Fixed.
+ else if (TailMatches2("COPY|\\copy", "(")) + COMPLETE_WITH_LIST7("SELECT", "TABLE", "VALUES", "INSERT", "UPDATE", "DELETE", "WITH"); This one should be Matches, no?
Yep, fixed. Andreas
diff --git a/src/bin/psql/tab-complete.c b/src/bin/psql/tab-complete.c index 5a11c61..72e0255 100644 --- a/src/bin/psql/tab-complete.c +++ b/src/bin/psql/tab-complete.c @@ -1937,11 +1937,18 @@ psql_completion(const char *text, int start, int end) /* COPY */ /* - * If we have COPY [BINARY] (which you'd have to type yourself), offer - * list of tables (Also cover the analogous backslash command) + * If we have COPY, offer list of tables or "(" (Also cover the analogous + * backslash command). */ - else if (Matches1("COPY|\\copy") || Matches2("COPY", "BINARY")) + else if (Matches1("COPY|\\copy")) + COMPLETE_WITH_SCHEMA_QUERY(Query_for_list_of_tables, + " UNION ALL SELECT '('"); + /* If we have COPY BINARY, complete with list of tables */ + else if (Matches2("COPY", "BINARY")) COMPLETE_WITH_SCHEMA_QUERY(Query_for_list_of_tables, NULL); + /* If we have COPY (, complete it with legal commands */ + else if (Matches2("COPY|\\copy", "(")) + COMPLETE_WITH_LIST7("SELECT", "TABLE", "VALUES", "INSERT", "UPDATE", "DELETE", "WITH"); /* If we have COPY [BINARY] <sth>, complete it with "TO" or "FROM" */ else if (Matches2("COPY|\\copy", MatchAny) || Matches3("COPY", "BINARY", MatchAny))
-- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers