Discussed on IRC for a bit, this makes the following changes for clarity:

* alpm_list_is_in    --> alpm_list_find
* alpm_list_is_strin --> alpm_list_find_str
* Flip parameters of both functions to be inline with rest of alpm_list.

First commit, woohoo.

1 files changed, 2 insertions, 2 deletions

diff --git a/src/pacman/sync.c b/src/pacman/sync.c
index 6b1cb432..9054d5ee 100644
--- a/src/pacman/sync.c
+++ b/src/pacman/sync.c
@@ -159,7 +159,7 @@ static int sync_cleancache(int level)
 				/* TODO Do not remove the currently installed version EITHER */
 				if(!strcmp(name, n)) {
 					char *ptr = (alpm_pkg_vercmp(version, v) < 0) ? str : s;
-					if(!alpm_list_is_strin(ptr, clean)) {
+					if(!alpm_list_find_str(clean, ptr)) {
 						clean = alpm_list_add(clean, strdup(ptr));
 					}
 				}
@@ -645,7 +645,7 @@ int pacman_sync(alpm_list_t *targets)
 				for(j = data; j; j = alpm_list_next(j)) {
 					pmpkg_t *p = alpm_list_getdata(j);
 					const char *pkgname = alpm_pkg_get_name(p);
-					if(!alpm_list_is_strin(pkgname, list_remove)) {
+					if(!alpm_list_find_str(list_remove, pkgname)) {
 						list_remove = alpm_list_add(list_remove, strdup(pkgname));
 					}
 				}