Daniel Gustafsson <dan...@yesql.se> writes:
> Being a two-space guy I would like the style to remain that, but I also agree
> that the churn is way too expensive and that it’s considered quite obsolete
> and
> old by many. Cutting that change from the patch makes the remainder more
> palatable.
This stuff seems reasonably non-controversial, so pushed.
I couldn't resist the temptation to mess about with dblink_res_error
a bit more, too.
BTW, really the point of what I'd mentioned before was to avoid having
dblink_res_error constructing a message out of fragments, which it's
still doing. I'd thought perhaps we would shove the responsibility for
mentioning the connection name out to the callers to get rid of that.
But handling the possibility of an unnamed connection seems like it'd
complicate the callers considerably. And as long as we don't actually
have translation support in that module, it'd just be make-work, so
I didn't do it.
regards, tom lane
