I wrote:
> Peter Eisentraut <peter.eisentraut@2ndquadrant.com> writes:
>> The other adds a location field to the DefElem node.
> +1 for sure, lots of places where that would be a good thing
> (the duplicate check itself, for starters).
Forgot to mention: seems like you should have added a location
argument to makeDefElem.
regards, tom lane