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

Allan McRae mcrae_allan at hotmail.com
Sat Dec 22 11:49:50 EST 2007


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






More information about the pacman-dev mailing list