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

Before I resubmit an updated patch with all the functions updated, is this function better? Specifically, do I free vdata correctly? int SYMEXPORT alpm_option_remove_noupgrade(const char *pkg) { void *vdata = NULL; handle->noupgrade = alpm_list_remove(handle->noupgrade, pkg, _alpm_str_cmp, &vdata); if(vdata != NULL) { FREELIST(vdata); return 1; } return 0; } Cheers, Allan