Remove alpm_list_first
The only thing this accessor did was remove the const qualifier given our entire list implementation requires passing around the head anyway. Signed-off-by: Allan McRae <allan@archlinux.org> Signed-off-by: Dan McGee <dan@archlinux.org>
This commit is contained in:
parent
c748eadc80
commit
97103f860d
|
@ -506,22 +506,6 @@ alpm_list_t SYMEXPORT *alpm_list_reverse(alpm_list_t *list)
|
|||
|
||||
/* Accessors */
|
||||
|
||||
/**
|
||||
* @brief Get the first element of a list.
|
||||
*
|
||||
* @param list the list
|
||||
*
|
||||
* @return the first element in the list
|
||||
*/
|
||||
inline alpm_list_t SYMEXPORT *alpm_list_first(const alpm_list_t *list)
|
||||
{
|
||||
if(list) {
|
||||
return (alpm_list_t *)list;
|
||||
} else {
|
||||
return NULL;
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
* @brief Return nth element from list (starting from 0).
|
||||
*
|
||||
|
|
|
@ -67,7 +67,6 @@ alpm_list_t *alpm_list_copy_data(const alpm_list_t *list, size_t size);
|
|||
alpm_list_t *alpm_list_reverse(alpm_list_t *list);
|
||||
|
||||
/* item accessors */
|
||||
alpm_list_t *alpm_list_first(const alpm_list_t *list);
|
||||
alpm_list_t *alpm_list_nth(const alpm_list_t *list, size_t n);
|
||||
alpm_list_t *alpm_list_next(const alpm_list_t *list);
|
||||
alpm_list_t *alpm_list_previous(const alpm_list_t *list, const alpm_list_t *node);
|
||||
|
|
|
@ -841,7 +841,7 @@ out:
|
|||
/* cleanup */
|
||||
if(config->verbosepkglists) {
|
||||
/* targets is a list of lists of strings, free inner lists here */
|
||||
for(j = alpm_list_first(targets); j; j = alpm_list_next(j)) {
|
||||
for(j = targets; j; j = alpm_list_next(j)) {
|
||||
lp = alpm_list_getdata(j);
|
||||
FREELIST(lp);
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue