On 2 July 2015 at 21:59, Heikki Linnakangas <hlinn...@iki.fi> wrote:

> Applied the straightforward parts.


Thanks for committing.


> I left out the changes like
>
>  -               appendStringInfoString(&collist, buf.data);
>> +               appendBinaryStringInfo(&collist, buf.data, buf.len);
>>
>
> because they're not an improvement in readablity, IMHO, and they were not
> in performance-critical paths.
>
>
Perhaps we can come up with appendStringInfoStringInfo at some later date.

--
 David Rowley                   http://www.2ndQuadrant.com/
<http://www.2ndquadrant.com/>
 PostgreSQL Development, 24x7 Support, Training & Services

Reply via email to