Hello Doug,
This patch refactors all of the connection state PQfinish() and NULL’ing into a single function. Excludes PQfinish() in doConnect().
My 0.02€: The argument could be "PGconn **" instead of a "CState *"? If so, it may be used in a few more places. What is your opinion?I'm fine with this kind of factorization which takes out a three-line pattern, but I'm wondering whether it would please committers.
-- Fabien.
