On Mon, 13 Apr 2026 at 01:34, David Rowley <[email protected]> wrote: > I think the author of copy_sequences() doesn't know what > resetStringInfo() does. I expect they think that it'll pfree all the > memory, but that's what destroyStringInfo() is for.
That was poor analysis. The resetStringInfo is there to reset the StringInfo before the next loop. So nothing is wrong with it. I've now pushed your patch. David
