From: Tom Lane [mailto:tgl@sss.pgh.pa.us]
> I temporarily changed the Assert to be "==" rather than "<=", and
> it still passed check-world, so evidently we are not testing any
> cases where the descriptors are of different lengths. This explains
> the lack of symptoms. It's still a bug though, so pushed.
Thank you for committing.
> > BTW, I thought that in PostgreSQL coding convention, local variables
> should be defined at the top of blocks, but this function writes "for (int
> natts;".
>
> Yeah, we've agreed to join the 21st century to the extent of allowing
> local for-loop variables.
That's good news. It'll help a bit to code comfortably.
Regards
Takayuki Tsunakawa
