Joe Conway <mail@joeconway.com> writes:
> [ improved patch ]
Still a couple quibbles:
> + ereport(level,
> + (errcode(sqlstate),
> + errmsg(message_primary),
This *must* be errmsg("%s", message_primary), else you have big problems
with % in the text. Also, I think it's at least theoretically possible
for message_primary to be null, in which case you'd better substitute
"unknown error" or some such.
You could avoid the ugly cast-away-const by making
dblink_context_conname be const char *, no?
Since dblink_res_error isn't going to return if fail = true, seems
like you could skip the "if (!fail)" tests occurring after calls to it.
regards, tom lane
