Patch applied.
Tom Lane wrote:
> Neil Conway <neilc@samurai.com> writes:
>>BTW, this idiom occurs a few times:
>
>
>> if (BufferIsValid(buf))
>> {
>> ReleaseBuffer(buf);
>> buf = InvalidBuffer;
>> }
>
>
> I'd leave it as-is; ISTM to be more easily understandable than the
> alternatives you suggest.
Yeah, fair enough.
-Neil