On 1/30/18, 03:41, "Fabien COELHO" <coe...@cri.ensmp.fr> wrote:
Hello Doug,
Hi Fabien,
> 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 should have named finishCon() as finishCStateCon() since it was specific
to that use pattern.
I'll resubmit with that change if you think it helps.
I'm fine with this kind of factorization which takes out a three-line
pattern, but I'm wondering whether it would please committers.
Guess we'll find out ...
--
Fabien.
Thanks!
doug
