Re: [PATCH] pgbench - refactor some connection finish/null intocommon function

On 1/30/18, 03:41, "Fabien COELHO"  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