> 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?
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