> Added a safety guard before calling PQescapeLiteral() to ensure pset.db is
> non-NULL and in CONNECTION_OK state, falling back to ALL otherwise.
Hi, VASUKI M
I just looked at this patch and thought about it briefly, but I'm a little
confused about the purpose of this conditional check.
If the condition is met, then returning anything at this point makes no sense,
right?
```
if (!pset.db || PQstatus(pset.db) != CONNECTION_OK)
{
COMPLETE_WITH("ALL");
}
```
Furthermore, I believe this should be guaranteed by the function `exec_query`.
```
COMPLETE_WITH_QUERY_PLUS -> COMPLETE_WITH_QUERY_LIST -> complete_from_query ->
exec_query
```
However, I haven't had time to debug it yet, so I might need your help to
double-check it.
--
Regards,
Man Zeng
www.openhalo.org
