Robert Haas <robertmhaas(at)gmail(dot)com> writes:
> I was thinking more about a sprintf()-type function that only
> understands a handful of escapes, but adds the additional and novel
> escapes %I (quote as identifier) and %L (quote as literal). I think
> that would allow a great deal of code simplification, and it'd be more
> efficient, too.
Seems like a great idea. Are you offering to code it?
Note that this wouldn't entirely fix the fmtId problem, as not all the
uses of fmtId are directly in sprintf calls. Still, it might get rid of
most of the places where it'd be painful to avoid a memory leak with
a strdup'ing version of fmtId.
regards, tom lane
In response to
pgsql-hackers by date
|Next:||From: Robert Haas||Date: 2013-01-09 01:02:16|
|Subject: Re: [PATCH 2/5] Make relpathbackend return a statically
result instead of palloc()'ing it|
|Previous:||From: Robert Haas||Date: 2013-01-09 00:54:38|
|Subject: Re: proposal: Set effective_cache_size to greater of .conf