From 24324ff0e12a3e19cada669a4c5c391e486790a3 Mon Sep 17 00:00:00 2001 From: Allan McRae Date: Mon, 4 Jul 2011 11:17:12 +1000 Subject: Simplify alpm_list_previous We can readily detect the first node in a list by checking if node->prev->next is NULL. So there is no need to pass the head of the list to this function and its prototype now looks like all the other item accessors. Signed-off-by: Allan McRae Signed-off-by: Dan McGee --- lib/libalpm/remove.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'lib/libalpm/remove.c') diff --git a/lib/libalpm/remove.c b/lib/libalpm/remove.c index 59374dcb..9b8517cb 100644 --- a/lib/libalpm/remove.c +++ b/lib/libalpm/remove.c @@ -329,7 +329,7 @@ int _alpm_upgraderemove_package(alpm_handle_t *handle, _alpm_log(handle, ALPM_LOG_DEBUG, "removing %ld files\n", (unsigned long)filenum); /* iterate through the list backwards, unlinking files */ - for(lp = alpm_list_last(files); lp; lp = alpm_list_previous(files, lp)) { + for(lp = alpm_list_last(files); lp; lp = alpm_list_previous(lp)) { unlink_file(handle, oldpkg, lp->data, skip_remove, 0); } FREELIST(skip_remove); @@ -406,7 +406,7 @@ int _alpm_remove_packages(alpm_handle_t *handle) pkg_count, (pkg_count - targcount + 1)); /* iterate through the list backwards, unlinking files */ - for(lp = alpm_list_last(files); lp; lp = alpm_list_previous(files, lp)) { + for(lp = alpm_list_last(files); lp; lp = alpm_list_previous(lp)) { int percent; unlink_file(handle, info, lp->data, NULL, trans->flags & ALPM_TRANS_FLAG_NOSAVE); -- cgit v1.2.3