Re: Proposal: add new API to stringinfo

On 2024-12-28 Sa 12:45 AM, Tatsuo Ishii wrote:
> Michael said:
>> New APIs are materials for HEAD, so recompilation needs to happen
>> anyway.  Using a macro makes things slightly simpler and it would not
>> break unnecessarily the compilation of existing extensions.
> Ok.
>
> David said:
>> I didn't review the patch in detail, but I think "initsize" would be a
>> better parameter name than "size".
> Ok, will change to "initsize".
>
> With opinions from Michael and David , what about following additional APIs?
>
> #define STRINGINFO_DEFAULT_SIZE    1024    /* default initial allocation size */
> #define STRINGINFO_SMALL_SIZE    64    /* small initial allocation size */
>
> #define makeStringInfo        makeStringInfoExtended(STRINGINFO_DEFAULT_SIZE)
> #define    initStringInfo(str)    initStringInfoExtended(str, STRINGINFO_DEFAULT_SIZE)
>
> extern StringInfo makeStringInfoExtended(int initsize);
> extern void initStringInfoExtended(StringInfo str, int initsize);
>


Seems like a good idea.


cheers


andrew

--
Andrew Dunstan
EDB: https://www.enterprisedb.com