On 11/1/21, 9:58 AM, "Tom Lane" <t...@sss.pgh.pa.us> wrote: > "Bossart, Nathan" <bossa...@amazon.com> writes: >> Should there be a list_free(trgmNFA->queue) at the end of >> transformGraph()? > > There could be, but that's visibly invoked only once per > createTrgmNFAInternal call, so I didn't think it was worthwhile > to do so (unlike the case for processState). If we were concerned > about leakage in that function, the hash table would be a far > bigger issue.
Ah, I see it now. The patch looks good to me, then. Nathan