[pacman-dev] [PATCH] Add remove counterparts to alpm_option_add_* functions

[Nagy Gabor]

> Nagy Gabor wrote:
> >> Updated patch attached without the memory leaks and adding return
> >> of removal success.
> >>
> >> Cheers,
> >> Allan
> >>     
> >
> > Is newcachedir freed?;-)
> > Bye
> >
> >   
> No... but it wasn't in alpm_option_add_cachedir() either and I was 
> trying for coding style consistency. :)  Both are fixed in attached
> patch.
No. You mustn't free it in add_cachedir, because alpm_list_add just adds
a POINTER to the (handle->cachedirs) list. (So you can always think
alpm_list as a list of pointers.)

Bye