RE: Use appendStringInfoString and appendPQExpBufferStr where possible

Hou, Zhijie <houzj.fnst@cn.fujitsu.com>

View thread View on postgr.es

From: "Hou, Zhijie" <houzj.fnst@cn.fujitsu.com>

To: Justin Pryzby <pryzby@telsasoft.com>, David Rowley <dgrowleyml@gmail.com>

Cc: PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>

Date: 2020-09-28T00:47:12Z

Lists: pgsql-hackers

Attachments

0001-Latest-appendStringInfoString-instead-of-appendStringInfo.patch (application/octet-stream) patch 0001
0001-appendStringInfoChar-instead-of-appendStringInfoString.patch (application/octet-stream) patch 0001
0001-Latest-appendPQExpBufferStr-instead-of-appendPQExpBuffer.patch (application/octet-stream) patch 0001

> Good point.  There's another one:

> $ git grep -E 'appendStringInfoString.*".{,1}");'
> src/backend/utils/adt/ruleutils.c:      appendStringInfoString(buf, "(");

> I noticed you added a similar thread here.
> https://commitfest.postgresql.org/30/

> I think this one could be combined as a single patchset with the existing CF entry for the other thread.

Thanks for your response, combined them as a single patchset now.

Best regards,

Commits

Fixup some appendStringInfo and appendPQExpBuffer calls
- 110d81728a0a 14.0 landed