summaryrefslogtreecommitdiffstats
path: root/lib/libalpm/alpm_list.c
diff options
context:
space:
mode:
authorAndrew Gregory <andrew.gregory.8@gmail.com>2016-01-11 15:29:22 +0100
committerAllan McRae <allan@archlinux.org>2016-01-15 05:47:36 +0100
commitab50864a758a7c3d26d5423521743c5753a6cffe (patch)
treeceb784be7247152d58f77428ecae8fb901413803 /lib/libalpm/alpm_list.c
parent9813107c3368684784706cf2b77bd056bea58e3c (diff)
downloadpacman-ab50864a758a7c3d26d5423521743c5753a6cffe.tar.gz
pacman-ab50864a758a7c3d26d5423521743c5753a6cffe.tar.xz
add alpm_list_append
alpm_list_add always returns the provided list making it impossible for callers to check whether or not the operation actually succeeded without manually comparing the list length before and after. alpm_list_append instead returns a pointer to the newly created list item so that success can be checked. Signed-off-by: Andrew Gregory <andrew.gregory.8@gmail.com> Signed-off-by: Allan McRae <allan@archlinux.org>
Diffstat (limited to 'lib/libalpm/alpm_list.c')
-rw-r--r--lib/libalpm/alpm_list.c34
1 files changed, 24 insertions, 10 deletions
diff --git a/lib/libalpm/alpm_list.c b/lib/libalpm/alpm_list.c
index 62b28438..f3effc72 100644
--- a/lib/libalpm/alpm_list.c
+++ b/lib/libalpm/alpm_list.c
@@ -93,28 +93,42 @@ void SYMEXPORT alpm_list_free_inner(alpm_list_t *list, alpm_list_fn_free fn)
*/
alpm_list_t SYMEXPORT *alpm_list_add(alpm_list_t *list, void *data)
{
- alpm_list_t *ptr, *lp;
+ alpm_list_append(&list, data);
+ return list;
+}
+
+/**
+ * @brief Add a new item to the end of the list.
+ *
+ * @param list the list to add to
+ * @param data the new item to be added to the list
+ *
+ * @return the newly added item
+ */
+alpm_list_t SYMEXPORT *alpm_list_append(alpm_list_t **list, void *data)
+{
+ alpm_list_t *ptr;
ptr = malloc(sizeof(alpm_list_t));
if(ptr == NULL) {
- return list;
+ return NULL;
}
ptr->data = data;
ptr->next = NULL;
/* Special case: the input list is empty */
- if(list == NULL) {
+ if(*list == NULL) {
+ *list = ptr;
ptr->prev = ptr;
- return ptr;
+ } else {
+ alpm_list_t *lp = alpm_list_last(*list);
+ lp->next = ptr;
+ ptr->prev = lp;
+ (*list)->prev = ptr;
}
- lp = alpm_list_last(list);
- lp->next = ptr;
- ptr->prev = lp;
- list->prev = ptr;
-
- return list;
+ return ptr;
}
/**