I wrote:
> Mark Dilger <hornschnorter@gmail.com> writes:
>> there are several places in the code where variables defined as
>> (char *) or as (const char *) are passed to the NameGetDatum()
>> macro. I believe it would be better form to use CStringGetDatum()
>> in these locations. I am aware that these two macros are internally
>> the same.
> I'm tempted to propose that we redefine NameGetDatum as
> #define NameGetDatum(X) CStringGetDatum(NameStr(*(X)))
Pushed with that change.
regards, tom lane