Re: [PATCH] cex: fix leaks

Akim Demaille <[email protected]>
Newsgroups gmane.comp.parsers.bison.patches
Message-ID <[email protected]>
Hi Vincent,

> Le 18 mai 2020 à 23:09, Vincent Imbimbo <[email protected]> a écrit :
> 
> Hey Akim,
> 
> I realized this afternoon that it would probably be fine to put all of these edges in the state-items themselves:
> 
> struct state_item
> {
>     ...
>     struct state_item *transition;
>     bitset productions;
>     bitset revs;
> };

That sounds much clearer!

> I think the derivation leaks are going to take a little bit more thought. I
> probably need API in gl_list to free the list without calling its dispose
> function on elements. I guess I could also just set it to NULL and free
> everything manually instead.

Or maybe you can steal the elements from the list before?  And set NULL in the
list instead.  I don't know you use case, so it might not be applicable.

Cheers!
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.