> > Ok. One way that I can think of is that since we have the typeoids
> handily
> > available, we can check the same and see if it is of stringlike type
> > (PGOID_TYPE_CHAR, PGOID_TYPE_NAME, PGOID_TYPE_TEXT, etc..). If yes, then
> we
> > retain the "", else we ignore it. Sounds reasonable?
>
> Yes.
>
>
PFA, version 2 of this patch. We now check if the type of the argument is
stringlike. If yes, we retain the "", else we default to nothing.
Regards,
Nikhils