Use appendStringInfoSpaces in more places
authorDavid Rowley <drowley@postgresql.org>
Fri, 20 Jan 2023 00:07:24 +0000 (13:07 +1300)
committerDavid Rowley <drowley@postgresql.org>
Fri, 20 Jan 2023 00:07:24 +0000 (13:07 +1300)
commit9f1ca6ce6596f1f77adc323769f2f8303ddad14f
tree422728e355dbe1bc246e72a3545b8d09184a5c3f
parent1ca604c2018456f2da357d944e0f03c3efaeb00d
Use appendStringInfoSpaces in more places

This adjusts a few places which were appending a string constant
containing spaces onto a StringInfo.  We have appendStringInfoSpaces for
that job, so let's use that instead.

For the change to jsonb.c's add_indent() function, appendStringInfoString
was being called inside a loop to append 4 spaces on each loop.  This
meant that enlargeStringInfo would get called once per loop.  Here it
should be much more efficient to get rid of the loop and just calculate
the number of spaces with "level * 4" and just append all the spaces in
one go.

Here we additionally adjust the appendStringInfoSpaces function so it
makes use of memset rather than a while loop to apply the required spaces
to the StringInfo.  One of the problems with the while loop was that it
was incrementing one variable and decrementing another variable once per
loop.  That's more work than what's required to get the job done.  We may
as well use memset for this rather than trying to optimize the existing
loop.  Some testing has shown memset is faster even for very small sizes.

Discussion: https://postgr.es/m/CAApHDvp_rKkvwudBKgBHniNRg67bzXVjyvVKfX0G2zS967K43A@mail.gmail.com
src/backend/commands/explain.c
src/backend/utils/adt/jsonb.c
src/common/stringinfo.c