Re: [PATCH] clarify palloc comment on quote_literal_cstr

On Sun, Apr 06, 2025 at 12:37:24PM -0500, Steve Chavez wrote:
> I found the numbers in `quote_literal_cstr` palloc quite magical. So I've
> added a comment clarifying what they mean. The change is small:
>
> /* We make a worst-case result area; wasting a little space is OK */
> - result = palloc(len * 2 + 3 + 1);
> + result = palloc(
> + (len * 2) /* worst-case doubling for every character if
> each one is a quote */
> + + 3 /* two outer quotes + possibly 'E' if needed */
> + + 1 /* null terminator */
> + );

Agreed that this is a good idea to have a better documentation here if
someone decides to tweak this area in the future, rather than have
them guess the numbers.

Looking at what quote_literal_internal() does, it seems to me that you
are right based on ESCAPE_STRING_SYNTAX, SQL_STR_DOUBLE(), and the
extra backslashes added to the destination buffer.
--
Michael