From af66c658642635a3c951bb0ee130e40e7f084fd9 Mon Sep 17 00:00:00 2001
From: "graham.gower" <graham.gower@e8e0d7a0-c8d9-11dd-a880-a1081c7ac358>
Date: Thu, 5 Nov 2009 04:52:10 +0000
Subject: [PATCH] These functions cannot fail, so return void.

git-svn-id: http://opkg.googlecode.com/svn/trunk@260 e8e0d7a0-c8d9-11dd-a880-a1081c7ac358
---
 libopkg/conffile_list.c |  8 ++++----
 libopkg/conffile_list.h |  4 ++--
 libopkg/nv_pair_list.c  | 16 +++++-----------
 libopkg/nv_pair_list.h  |  4 ++--
 libopkg/pkg_dest_list.c | 19 +++++++------------
 libopkg/pkg_dest_list.h |  6 +++---
 libopkg/pkg_src_list.c  | 15 +++++----------
 libopkg/pkg_src_list.h  |  6 +++---
 libopkg/str_list.c      | 16 ++++++++--------
 libopkg/str_list.h      |  8 ++++----
 libopkg/void_list.c     | 17 ++++-------------
 libopkg/void_list.h     |  8 ++++----
 12 files changed, 51 insertions(+), 76 deletions(-)

diff --git a/libopkg/conffile_list.c b/libopkg/conffile_list.c
index 401c6b4..61ea55d 100644
--- a/libopkg/conffile_list.c
+++ b/libopkg/conffile_list.c
@@ -19,9 +19,9 @@
 
 #include "conffile_list.h"
 
-int conffile_list_init(conffile_list_t *list)
+void conffile_list_init(conffile_list_t *list)
 {
-    return nv_pair_list_init(list);
+    nv_pair_list_init(list);
 }
 
 void conffile_list_deinit(conffile_list_t *list)
@@ -35,9 +35,9 @@ conffile_t *conffile_list_append(conffile_list_t *list, const char *file_name,
     return nv_pair_list_append(list, file_name, md5sum);
 }
 
-int conffile_list_push(conffile_list_t *list, conffile_t *data)
+void conffile_list_push(conffile_list_t *list, conffile_t *data)
 {
-    return nv_pair_list_push(list, data);
+    nv_pair_list_push(list, data);
 }
 
 conffile_list_elt_t *conffile_list_pop(conffile_list_t *list)
diff --git a/libopkg/conffile_list.h b/libopkg/conffile_list.h
index b044d63..942f68e 100644
--- a/libopkg/conffile_list.h
+++ b/libopkg/conffile_list.h
@@ -25,12 +25,12 @@ typedef nv_pair_list_t conffile_list_t;
 
 #include "conffile.h"
 
-int conffile_list_init(conffile_list_t *list);
+void conffile_list_init(conffile_list_t *list);
 void conffile_list_deinit(conffile_list_t *list);
 
 conffile_t *conffile_list_append(conffile_list_t *list, const char *name,
 			       const char *root_dir);
-int conffile_list_push(conffile_list_t *list, conffile_t *data);
+void conffile_list_push(conffile_list_t *list, conffile_t *data);
 conffile_list_elt_t *conffile_list_pop(conffile_list_t *list);
 
 #endif
diff --git a/libopkg/nv_pair_list.c b/libopkg/nv_pair_list.c
index 5b6d4ff..b6c8f8e 100644
--- a/libopkg/nv_pair_list.c
+++ b/libopkg/nv_pair_list.c
@@ -22,9 +22,9 @@
 #include "nv_pair_list.h"
 #include "libbb/libbb.h"
 
-int nv_pair_list_init(nv_pair_list_t *list)
+void nv_pair_list_init(nv_pair_list_t *list)
 {
-    return void_list_init((void_list_t *) list);
+    void_list_init((void_list_t *) list);
 }
 
 void nv_pair_list_deinit(nv_pair_list_t *list)
@@ -48,23 +48,17 @@ void nv_pair_list_deinit(nv_pair_list_t *list)
 
 nv_pair_t *nv_pair_list_append(nv_pair_list_t *list, const char *name, const char *value)
 {
-    int err;
-
     /* freed in nv_pair_list_deinit */
     nv_pair_t *nv_pair = xcalloc(1, sizeof(nv_pair_t));
     nv_pair_init(nv_pair, name, value);
-
-    err = void_list_append((void_list_t *) list, nv_pair);
-    if (err) {
-	return NULL;
-    }
+    void_list_append((void_list_t *) list, nv_pair);
 
     return nv_pair;
 }
 
-int nv_pair_list_push(nv_pair_list_t *list, nv_pair_t *data)
+void nv_pair_list_push(nv_pair_list_t *list, nv_pair_t *data)
 {
-    return void_list_push((void_list_t *) list, data);
+    void_list_push((void_list_t *) list, data);
 }
 
 nv_pair_list_elt_t *nv_pair_list_pop(nv_pair_list_t *list)
diff --git a/libopkg/nv_pair_list.h b/libopkg/nv_pair_list.h
index fc9d9e2..e37f612 100644
--- a/libopkg/nv_pair_list.h
+++ b/libopkg/nv_pair_list.h
@@ -30,12 +30,12 @@ static inline int nv_pair_list_empty(nv_pair_list_t *list)
     return void_list_empty ((void_list_t *)list);
 }
 
-int nv_pair_list_init(nv_pair_list_t *list);
+void nv_pair_list_init(nv_pair_list_t *list);
 void nv_pair_list_deinit(nv_pair_list_t *list);
 
 nv_pair_t *nv_pair_list_append(nv_pair_list_t *list,
 			       const char *name, const char *value);
-int nv_pair_list_push(nv_pair_list_t *list, nv_pair_t *data);
+void nv_pair_list_push(nv_pair_list_t *list, nv_pair_t *data);
 nv_pair_list_elt_t *nv_pair_list_pop(nv_pair_list_t *list);
 char *nv_pair_list_find(nv_pair_list_t *list, char *name);
 
diff --git a/libopkg/pkg_dest_list.c b/libopkg/pkg_dest_list.c
index f8a7c52..b117bbb 100644
--- a/libopkg/pkg_dest_list.c
+++ b/libopkg/pkg_dest_list.c
@@ -22,9 +22,9 @@
 #include "pkg_dest_list.h"
 #include "libbb/libbb.h"
 
-int pkg_dest_list_elt_init(pkg_dest_list_elt_t *elt, pkg_dest_t *data)
+void pkg_dest_list_elt_init(pkg_dest_list_elt_t *elt, pkg_dest_t *data)
 {
-    return void_list_elt_init((void_list_elt_t *) elt, data);
+    void_list_elt_init((void_list_elt_t *) elt, data);
 }
 
 void pkg_dest_list_elt_deinit(pkg_dest_list_elt_t *elt)
@@ -32,9 +32,9 @@ void pkg_dest_list_elt_deinit(pkg_dest_list_elt_t *elt)
     void_list_elt_deinit((void_list_elt_t *) elt);
 }
 
-int pkg_dest_list_init(pkg_dest_list_t *list)
+void pkg_dest_list_init(pkg_dest_list_t *list)
 {
-    return void_list_init((void_list_t *) list);
+    void_list_init((void_list_t *) list);
 }
 
 void pkg_dest_list_deinit(pkg_dest_list_t *list)
@@ -56,24 +56,19 @@ void pkg_dest_list_deinit(pkg_dest_list_t *list)
 pkg_dest_t *pkg_dest_list_append(pkg_dest_list_t *list, const char *name,
 				 const char *root_dir,const char *lists_dir)
 {
-    int err;
     pkg_dest_t *pkg_dest;
 
     /* freed in pkg_dest_list_deinit */
     pkg_dest = xcalloc(1, sizeof(pkg_dest_t));
-
     pkg_dest_init(pkg_dest, name, root_dir,lists_dir);
-    err = void_list_append((void_list_t *) list, pkg_dest);
-    if (err) {
-	return NULL;
-    }
+    void_list_append((void_list_t *) list, pkg_dest);
 
     return pkg_dest;
 }
 
-int pkg_dest_list_push(pkg_dest_list_t *list, pkg_dest_t *data)
+void pkg_dest_list_push(pkg_dest_list_t *list, pkg_dest_t *data)
 {
-    return void_list_push((void_list_t *) list, data);
+    void_list_push((void_list_t *) list, data);
 }
 
 pkg_dest_list_elt_t *pkg_dest_list_pop(pkg_dest_list_t *list)
diff --git a/libopkg/pkg_dest_list.h b/libopkg/pkg_dest_list.h
index 3e976bc..33aef19 100644
--- a/libopkg/pkg_dest_list.h
+++ b/libopkg/pkg_dest_list.h
@@ -24,15 +24,15 @@ typedef struct void_list_elt pkg_dest_list_elt_t;
 
 typedef struct void_list pkg_dest_list_t;
 
-int pkg_dest_list_elt_init(pkg_dest_list_elt_t *elt, pkg_dest_t *data);
+void pkg_dest_list_elt_init(pkg_dest_list_elt_t *elt, pkg_dest_t *data);
 void pkg_dest_list_elt_deinit(pkg_dest_list_elt_t *elt);
 
-int pkg_dest_list_init(pkg_dest_list_t *list);
+void pkg_dest_list_init(pkg_dest_list_t *list);
 void pkg_dest_list_deinit(pkg_dest_list_t *list);
 
 pkg_dest_t *pkg_dest_list_append(pkg_dest_list_t *list, const char *name,
 				 const char *root_dir,const char* lists_dir);
-int pkg_dest_list_push(pkg_dest_list_t *list, pkg_dest_t *data);
+void pkg_dest_list_push(pkg_dest_list_t *list, pkg_dest_t *data);
 pkg_dest_list_elt_t *pkg_dest_list_pop(pkg_dest_list_t *list);
 
 #endif
diff --git a/libopkg/pkg_src_list.c b/libopkg/pkg_src_list.c
index cd7fce4..13bff5f 100644
--- a/libopkg/pkg_src_list.c
+++ b/libopkg/pkg_src_list.c
@@ -21,9 +21,9 @@
 #include "void_list.h"
 #include "libbb/libbb.h"
 
-int pkg_src_list_init(pkg_src_list_t *list)
+void pkg_src_list_init(pkg_src_list_t *list)
 {
-    return void_list_init((void_list_t *) list);
+    void_list_init((void_list_t *) list);
 }
 
 void pkg_src_list_deinit(pkg_src_list_t *list)
@@ -46,23 +46,18 @@ pkg_src_t *pkg_src_list_append(pkg_src_list_t *list,
 			       const char *name, const char *base_url, const char *extra_data,
 			       int gzip)
 {
-    int err;
-
     /* freed in pkg_src_list_deinit */
     pkg_src_t *pkg_src = xcalloc(1, sizeof(pkg_src_t));
     pkg_src_init(pkg_src, name, base_url, extra_data, gzip);
 
-    err = void_list_append((void_list_t *) list, pkg_src);
-    if (err) {
-	return NULL;
-    }
+    void_list_append((void_list_t *) list, pkg_src);
 
     return pkg_src;
 }
 
-int pkg_src_list_push(pkg_src_list_t *list, pkg_src_t *data)
+void pkg_src_list_push(pkg_src_list_t *list, pkg_src_t *data)
 {
-    return void_list_push((void_list_t *) list, data);
+    void_list_push((void_list_t *) list, data);
 }
 
 pkg_src_list_elt_t *pkg_src_list_pop(pkg_src_list_t *list)
diff --git a/libopkg/pkg_src_list.h b/libopkg/pkg_src_list.h
index ce95eb4..529f013 100644
--- a/libopkg/pkg_src_list.h
+++ b/libopkg/pkg_src_list.h
@@ -30,14 +30,14 @@ static inline int pkg_src_list_empty(pkg_src_list_t *list)
     return void_list_empty((void_list_t *)list);
 }
 
-int pkg_src_list_elt_init(pkg_src_list_elt_t *elt, nv_pair_t *data);
+void pkg_src_list_elt_init(pkg_src_list_elt_t *elt, nv_pair_t *data);
 void pkg_src_list_elt_deinit(pkg_src_list_elt_t *elt);
 
-int pkg_src_list_init(pkg_src_list_t *list);
+void pkg_src_list_init(pkg_src_list_t *list);
 void pkg_src_list_deinit(pkg_src_list_t *list);
 
 pkg_src_t *pkg_src_list_append(pkg_src_list_t *list, const char *name, const char *root_dir, const char *extra_data, int gzip);
-int pkg_src_list_push(pkg_src_list_t *list, pkg_src_t *data);
+void pkg_src_list_push(pkg_src_list_t *list, pkg_src_t *data);
 pkg_src_list_elt_t *pkg_src_list_pop(pkg_src_list_t *list);
 
 #endif
diff --git a/libopkg/str_list.c b/libopkg/str_list.c
index a8e179d..2323c8e 100644
--- a/libopkg/str_list.c
+++ b/libopkg/str_list.c
@@ -19,9 +19,9 @@
 #include "str_list.h"
 #include "libbb/libbb.h"
 
-int str_list_elt_init(str_list_elt_t *elt, char *data)
+void str_list_elt_init(str_list_elt_t *elt, char *data)
 {
-    return void_list_elt_init((void_list_elt_t *) elt, data);
+    void_list_elt_init((void_list_elt_t *) elt, data);
 }
 
 void str_list_elt_deinit(str_list_elt_t *elt)
@@ -38,9 +38,9 @@ str_list_t *str_list_alloc()
      return list;
 }
 
-int str_list_init(str_list_t *list)
+void str_list_init(str_list_t *list)
 {
-    return void_list_init((void_list_t *) list);
+    void_list_init((void_list_t *) list);
 }
 
 void str_list_deinit(str_list_t *list)
@@ -57,14 +57,14 @@ void str_list_deinit(str_list_t *list)
     }
 }
 
-int str_list_append(str_list_t *list, char *data)
+void str_list_append(str_list_t *list, char *data)
 {
-    return void_list_append((void_list_t *) list, xstrdup(data));
+    void_list_append((void_list_t *) list, xstrdup(data));
 }
 
-int str_list_push(str_list_t *list, char *data)
+void str_list_push(str_list_t *list, char *data)
 {
-    return void_list_push((void_list_t *) list, xstrdup(data));
+    void_list_push((void_list_t *) list, xstrdup(data));
 }
 
 str_list_elt_t *str_list_pop(str_list_t *list)
diff --git a/libopkg/str_list.h b/libopkg/str_list.h
index 4f62fe7..3690820 100644
--- a/libopkg/str_list.h
+++ b/libopkg/str_list.h
@@ -24,15 +24,15 @@ typedef struct void_list_elt str_list_elt_t;
 
 typedef struct void_list str_list_t;
 
-int str_list_elt_init(str_list_elt_t *elt, char *data);
+void str_list_elt_init(str_list_elt_t *elt, char *data);
 void str_list_elt_deinit(str_list_elt_t *elt);
 
 str_list_t *str_list_alloc(void);
-int str_list_init(str_list_t *list);
+void str_list_init(str_list_t *list);
 void str_list_deinit(str_list_t *list);
 
-int str_list_append(str_list_t *list, char *data);
-int str_list_push(str_list_t *list, char *data);
+void str_list_append(str_list_t *list, char *data);
+void str_list_push(str_list_t *list, char *data);
 str_list_elt_t *str_list_pop(str_list_t *list);
 void str_list_remove(str_list_t *list, str_list_elt_t **iter);
 void str_list_remove_elt(str_list_t *list, const char *target_str);
diff --git a/libopkg/void_list.c b/libopkg/void_list.c
index e36b451..cc4d251 100644
--- a/libopkg/void_list.c
+++ b/libopkg/void_list.c
@@ -21,12 +21,10 @@
 #include "void_list.h"
 #include "libbb/libbb.h"
 
-int void_list_elt_init(void_list_elt_t *elt, void *data)
+void void_list_elt_init(void_list_elt_t *elt, void *data)
 {
     INIT_LIST_HEAD(&elt->node);
     elt->data = data;
-
-    return 0;
 }
 
 void_list_elt_t * void_list_elt_new (void *data) {
@@ -44,10 +42,9 @@ void void_list_elt_deinit(void_list_elt_t *elt)
     free(elt);
 }
 
-int void_list_init(void_list_t *list)
+void void_list_init(void_list_t *list)
 {
     INIT_LIST_HEAD(&list->head);
-    return 0;
 }
 
 void void_list_deinit(void_list_t *list)
@@ -61,22 +58,16 @@ void void_list_deinit(void_list_t *list)
     INIT_LIST_HEAD(&list->head);
 }
 
-int void_list_append(void_list_t *list, void *data)
+void void_list_append(void_list_t *list, void *data)
 {
     void_list_elt_t *elt = void_list_elt_new(data);
-     
     list_add_tail(&elt->node, &list->head);
-
-    return 0;
 }
 
-int void_list_push(void_list_t *list, void *data)
+void void_list_push(void_list_t *list, void *data)
 {
     void_list_elt_t *elt = void_list_elt_new(data);
-
     list_add(&elt->node, &list->head);
-
-    return 0;
 }
 
 void_list_elt_t *void_list_pop(void_list_t *list)
diff --git a/libopkg/void_list.h b/libopkg/void_list.h
index ca380b8..b63a68d 100644
--- a/libopkg/void_list.h
+++ b/libopkg/void_list.h
@@ -38,14 +38,14 @@ static inline int void_list_empty(void_list_t *list)
     return list_empty(&list->head);
 }
 
-int void_list_elt_init(void_list_elt_t *elt, void *data);
+void void_list_elt_init(void_list_elt_t *elt, void *data);
 void void_list_elt_deinit(void_list_elt_t *elt);
 
-int void_list_init(void_list_t *list);
+void void_list_init(void_list_t *list);
 void void_list_deinit(void_list_t *list);
 
-int void_list_append(void_list_t *list, void *data);
-int void_list_push(void_list_t *list, void *data);
+void void_list_append(void_list_t *list, void *data);
+void void_list_push(void_list_t *list, void *data);
 void_list_elt_t *void_list_pop(void_list_t *list);
 
 void *void_list_remove(void_list_t *list, void_list_elt_t **iter);
-- 
2.30.2