drodil/libcore — comment on commit f805e521382db33b468fc89eb68de1ada3b3222b
File: darray.c line 254
By ChrisWinter:

I'm not sure if this is the best way to handle freeing of user data stored in the DArray, it kind of makes the DArray API a little confusing since darray_erase sounds like it does the same thing as darray_remove. :-)

I was thinking that we could fold darray_erase into darray_remove, and add a parameter to the function definition that the user can then set if they want their data to be freed on removal. Otherwise, darray_remove could just return the void * to the user's data and then we leave it up to the user as to what happens.

Do you have any thoughts on this?