On Thu, Jul 21, 2011 at 12:19 PM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Robert Haas <robertmhaas@gmail.com> writes:
>> So I finally got around to taking a look at this patch, and I guess my
>> basic feeling is that I like it. The existing code is pretty weird
>> and inconsistent: the logic in PQsetvalue() basically does the same
>> thing as the logic in pqAddTuple(), but incompatibly and less
>> efficiently. Unifying them seems sensible, and the fix looks simple
>> enough to back-patch.
>
> Yeah, I've been looking at it too. For some reason I had had the
> idea that the proposed patch complicated the code, but actually it's
> simplifying it by removing almost-duplicate code. So that's good.
>
> The patch as proposed adds back a bug in return for the one it fixes
> (you can not free() the result of pqResultAlloc()), but that's easily
> fixed.
>
> Will fix and commit.
Cool. I believe that's the last patch for CommitFest 2011-06.
*bangs gavel*
I believe that makes it time for 9.2alph1.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
