I'd suggest linking to a page about the search_path [0].
Done
+extern const char *session_search_path(void);
This function appears to be unused.
+ /* + * Distinguish unknown (NULL) from an empty but valid search_path (""). + * If not connected or older server doesn't report it via ParameterStatus, + * show "?". + */ + if (!pset.db || PQparameterStatus(pset.db, "search_path") == NULL) + strlcpy(buf, "?", sizeof(buf)); + else + strlcpy(buf, PQparameterStatus(pset.db, "search_path"), sizeof(buf)); + break;
Shouldn't we move most of this logic to session_search_path() and use it here?