At Mon, 1 Nov 2021 17:02:49 +0000, "Bossart, Nathan" <bossa...@amazon.com> wrote in > 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.
+1 -- Kyotaro Horiguchi NTT Open Source Software Center
