These functions cannot fail, so return void.
authorgraham.gower <graham.gower@e8e0d7a0-c8d9-11dd-a880-a1081c7ac358>
Thu, 5 Nov 2009 04:52:10 +0000 (04:52 +0000)
committergraham.gower <graham.gower@e8e0d7a0-c8d9-11dd-a880-a1081c7ac358>
Thu, 5 Nov 2009 04:52:10 +0000 (04:52 +0000)
git-svn-id: http://opkg.googlecode.com/svn/trunk@260 e8e0d7a0-c8d9-11dd-a880-a1081c7ac358

12 files changed:
libopkg/conffile_list.c
libopkg/conffile_list.h
libopkg/nv_pair_list.c
libopkg/nv_pair_list.h
libopkg/pkg_dest_list.c
libopkg/pkg_dest_list.h
libopkg/pkg_src_list.c
libopkg/pkg_src_list.h
libopkg/str_list.c
libopkg/str_list.h
libopkg/void_list.c
libopkg/void_list.h

index 401c6b408c14229fc36d30fb013788d7cb42bb07..61ea55dd2f5909f7a750836502cf5c05062ae9bc 100644 (file)
@@ -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)
index b044d63b6ddc416bc268a7919e7bb16e887b3584..942f68ebd16c520af99b7f4dde35dc97436dc937 100644 (file)
@@ -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
index 5b6d4ff5b1c412de9455c6448850f2438757398c..b6c8f8e84cc6517d97d70e561f313d22588ffe39 100644 (file)
@@ -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)
index fc9d9e250443dde60f4d02de05082369834a4904..e37f612844e000ce6aa4533ce14b6aa5ee169f2c 100644 (file)
@@ -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);
 
index f8a7c52e61dc6fbc40a376cbd57f4d6d03dd59ae..b117bbba2949ca127f6a18bd295febf366a0af51 100644 (file)
@@ -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)
index 3e976bc3bf36d5a6ce07d9ccf112e5ef5d93aeb7..33aef19e4568585b8ae601d2cf2335571b99415c 100644 (file)
@@ -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
index cd7fce4d20a88d31e33ad0a7aeb37c27ee6c0c01..13bff5fe25d5f79c2902300e7cbc865660b18bd7 100644 (file)
@@ -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)
index ce95eb4c89649ab1fdf3cbb6dc07702b7607b04e..529f013fcc944493122a0f2f4079e934142e464c 100644 (file)
@@ -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
index a8e179d2d2893697af48c09cc97050cddac2c956..2323c8ecd32df291702c154cf091130cf265df81 100644 (file)
@@ -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)
index 4f62fe7a33375dad17e4ba65c7839666e7db5a32..3690820bd09479acc207f95b4b45f2c3f081a755 100644 (file)
@@ -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);
index e36b4517688248af5a3f4000364dcecb58b76588..cc4d25142ac2a77ce65c57bcf69908877e44e931 100644 (file)
 #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)
index ca380b82f61f5370ab76efa5b4067e32668a34c0..b63a68d533baf3f0f8e82765cba31032e548ae0c 100644 (file)
@@ -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);