Le 26/01/2010 00:04, Joe Conway a écrit :
> I'm reviewing the patch posted here:
> for this commitfest item:
First, thanks for reviewing my patch.
> Patch attached - a few minor changes:
> 1) Updated to apply cleanly against cvs tip
Sorry about this. I already updated it twice. I didn't think a new
update was needed.
> 2) Improved comments
> 3) Moved much of what was in PQconnectStartParams() to a new
> conninfo_array_parse() to be more consistent with existing code
You're right. It also makes the code more readable and understandable. I
should have done that.
> a) Do we want an analog to PQconninfoParse(), e.g.
> PQconninfoParseParams()? If not, it isn't worth keeping use_defaults
> as an argument to conninfo_array_parse().
No, I don't think so. I can't find a use case for it.
> b) I refrained from further consolidation even though there is room.
> For example, I considered leaving only the real parsing code in
> conninfo_parse(), and having it return keywords and values arrays.
> If we did that, the rest of the code could be modified to accept
> keywords and values instead of conninfo, and therefore shared. I was
> concerned about the probably small performance hit to the existing
> code path. Thoughts?
> c) Obviously I liked the "two-arrays approach" better -- any objections
> to that?
No objection. I prefer the other one, but it's just not that important.
I didn't put any documentation before knowing which one will be choosen.
So we still need to work on the manual.
In response to
pgsql-hackers by date
|Next:||From: KaiGai Kohei||Date: 2010-01-25 23:52:00|
|Subject: Re: RADIUS authentication|
|Previous:||From: Joe Conway||Date: 2010-01-25 23:04:43|
|Subject: Re: Patch: libpq new connect function (PQconnectParams)|