Andrew Dunstan <andrew(at)dunslane(dot)net> writes:
> On 12/06/2010 12:11 PM, Tom Lane wrote:
>> Also, why did you change the setup code to
>> not compute nfields in binary mode? That seems at best an unnecessary
>> change, and at worst a breakage of the binary path --- did you test it?
> AFAICT it's not used in binary mode at all. But I will double check.
Well, even if it is not used at the moment, it seems potentially of use
in that path. So I'd vote for continuing to set it correctly, rather
than making it deliberately incorrect as this patch is going out of its
way to do.
regards, tom lane