I wrote:
> Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)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