Thomas Munro <thomas.mu...@enterprisedb.com> writes:
> ... and a patch.  Thoughts?

It's not real obvious why you're adding inclusion guards over just a
subset of the header.  A short comment about that would be worthwhile.

                        regards, tom lane

Reply via email to