Thomas Lockhart <lockhart(at)fourpalms(dot)org> writes:
> If this should be done differently I'm happy for suggestions...
I think DEFAULT should probably be represented by a NULL, not by
a Value node containing a null string pointer.
I'm willing to do the work if no one else feels strongly about it ;-)
regards, tom lane