Re: proposal: psql autocomplete for casting

Hello, I considered on this,

At Thu, 3 Mar 2016 10:05:06 +0100, Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> wrote in <CAFj8pRDZ456OKbpV9jDJ_VCgTWprqxYu1kQp6z_Eu_WgNvs57Q(at)mail(dot)gmail(dot)com>
> We have not autocomplete for casting introduced by symbol "::". A
> implementation should not be hard.

Completion needs a sequence of break characters between two
succeeding words and the break characters cannot be a part of a
word. If cast operators are allowed to be preceded by white
space, the following diff gives maybe-desired behavior.

===
--- a/src/bin/psql/tab-complete.c
+++ b/src/bin/psql/tab-complete.c
@@ -1322,6 +1322,8 @@ psql_completion(const char *text, int start, int end)
else
matches = complete_from_variables(text, ":", "", true);
}
+ else if (text[0] == ':' && text[1] == ':')
+ COMPLETE_WITH_LIST3("::integer", "::text", "::date");

/* If no previous word, suggest one of the basic sql commands */
else if (previous_words_count == 0)
===

> =# select 123 ::<tab>
> ::DATE ::INTEGER ::TEXT
> =# select 123 ::T<tab>
> =# select 123 ::TEXT

It might not be instinctive but would be doable by making such a
candidate list.

====
diff --git a/src/bin/psql/tab-complete.c b/src/bin/psql/tab-complete.c
index 5f27120..179c9f0 100644
--- a/src/bin/psql/tab-complete.c
+++ b/src/bin/psql/tab-complete.c
@@ -1322,6 +1322,8 @@ psql_completion(const char *text, int start, int end)
else
matches = complete_from_variables(text, ":", "", true);
}
+ else if (text[0] == ':' && text[1] == ':')
+ COMPLETE_WITH_QUERY("SELECT distinct('::' || typname) FROM pg_type where typname !~'^(pg)?_.*'");

/* If no previous word, suggest one of the basic sql commands */
else if (previous_words_count == 0)
====

> =# select 2314 ::t<tab>
> ::t ::tinterval
> ::table_constraints ::transforms
> ::table_privileges ::trigger
> ::tables ::triggered_update_columns
> ::text ::triggers
> ::tid ::tsm_handler
> ::time ::tsquery
> ::timestamp ::tsrange
> ::time_stamp ::tstzrange
> ::timestamptz ::tsvector
> ::timetz ::txid_snapshot

Does this make sense?

regards,

--
Kyotaro Horiguchi
NTT Open Source Software Center