"Brendan Jurd" <[EMAIL PROTECTED]> writes: > On 9/22/07, Gregory Stark <[EMAIL PROTECTED]> wrote: >> The canonical way to do it is with >> >> DatumGetCString(DirectFunctionCall1(textout, t)) > > Ah, I see. Thanks. > > In that case, would it be helpful if I submitted a patch for the > various code fragments that do this locally, updating them to use > DatumGetCString?
I would be interested in seeing just a list of such places if you have it handy. I don't think we consider it wrong to violate the text data type abstraction barrier like you describe though. I'm interested because any such code is possibly either failing to take into account toasted data or is unnecessarily detoasting packed varlenas. -- Gregory Stark EnterpriseDB http://www.enterprisedb.com ---------------------------(end of broadcast)--------------------------- TIP 5: don't forget to increase your free space map settings
