On 02.06.21 12:57, Alvaro Herrera wrote:
In the latest HEAD branch, I found some places were using
appendStringInfo/appendPQExpBuffer() when they could have been using
appendStringInfoString/ appendPQExpBufferStr() instead. I think we'd better
fix these places in case other developers will use these codes as a reference,
though, it seems will not bring noticeable performance gain.
hmm why didn't we get warnings about the PENDING DETACH one? Maybe we
need some decorator in PQExpBuffer.
I don't think there is anything wrong with the existing code there.
It's just like using printf() when you could use puts().
(I'm not against the proposed patch, just answering this question.)
