From f5fb0b4cf87b50c86531a7f0e98fa0637134c925 Mon Sep 17 00:00:00 2001 From: Rosen Penev Date: Sat, 30 May 2020 19:14:39 -0700 Subject: Remove pointless return in void functions [clang-tidy] Found with readability-redundant-control-flow Signed-off-by: Rosen Penev --- src/bplist.c | 2 -- src/plist.c | 10 ---------- src/xplist.c | 2 -- 3 files changed, 14 deletions(-) diff --git a/src/bplist.c b/src/bplist.c index 14db755..24326b7 100644 --- a/src/bplist.c +++ b/src/bplist.c @@ -941,8 +941,6 @@ static void serialize_plist(node_t* node, void* data) for (ch = node_first_child(node); ch; ch = node_next_sibling(ch)) { serialize_plist(ch, data); } - - return; } #define Log2(x) (x == 8 ? 3 : (x == 4 ? 2 : (x == 2 ? 1 : 0))) diff --git a/src/plist.c b/src/plist.c index 27f90b1..7ac146a 100644 --- a/src/plist.c +++ b/src/plist.c @@ -525,7 +525,6 @@ PLIST_API void plist_array_set_item(plist_t node, plist_t item, uint32_t n) } } } - return; } PLIST_API void plist_array_append_item(plist_t node, plist_t item) @@ -535,7 +534,6 @@ PLIST_API void plist_array_append_item(plist_t node, plist_t item) node_attach(node, item); _plist_array_post_insert(node, item, -1); } - return; } PLIST_API void plist_array_insert_item(plist_t node, plist_t item, uint32_t n) @@ -545,7 +543,6 @@ PLIST_API void plist_array_insert_item(plist_t node, plist_t item, uint32_t n) node_insert(node, n, item); _plist_array_post_insert(node, item, (long)n); } - return; } PLIST_API void plist_array_remove_item(plist_t node, uint32_t n) @@ -562,7 +559,6 @@ PLIST_API void plist_array_remove_item(plist_t node, uint32_t n) plist_free(old_item); } } - return; } PLIST_API void plist_array_item_remove(plist_t node) @@ -587,7 +583,6 @@ PLIST_API void plist_array_new_iter(plist_t node, plist_array_iter *iter) *iter = malloc(sizeof(node_t*)); *((node_t**)(*iter)) = node_first_child(node); } - return; } PLIST_API void plist_array_next_item(plist_t node, plist_array_iter iter, plist_t *item) @@ -607,7 +602,6 @@ PLIST_API void plist_array_next_item(plist_t node, plist_array_iter iter, plist_ } *iter_node = node_next_sibling(*iter_node); } - return; } PLIST_API uint32_t plist_dict_get_size(plist_t node) @@ -627,7 +621,6 @@ PLIST_API void plist_dict_new_iter(plist_t node, plist_dict_iter *iter) *iter = malloc(sizeof(node_t*)); *((node_t**)(*iter)) = node_first_child(node); } - return; } PLIST_API void plist_dict_next_item(plist_t node, plist_dict_iter iter, char **key, plist_t *val) @@ -656,7 +649,6 @@ PLIST_API void plist_dict_next_item(plist_t node, plist_dict_iter iter, char **k } *iter_node = node_next_sibling(*iter_node); } - return; } PLIST_API void plist_dict_get_item_key(plist_t node, char **key) @@ -752,7 +744,6 @@ PLIST_API void plist_dict_set_item(plist_t node, const char* key, plist_t item) } } } - return; } PLIST_API void plist_dict_insert_item(plist_t node, const char* key, plist_t item) @@ -776,7 +767,6 @@ PLIST_API void plist_dict_remove_item(plist_t node, const char* key) plist_free(old_item); } } - return; } PLIST_API void plist_dict_merge(plist_t *target, plist_t source) diff --git a/src/xplist.c b/src/xplist.c index a7d52e5..537d315 100644 --- a/src/xplist.c +++ b/src/xplist.c @@ -370,8 +370,6 @@ static void node_to_xml(node_t* node, bytearray_t **outbuf, uint32_t depth) str_buf_append(*outbuf, ">", 1); } str_buf_append(*outbuf, "\n", 1); - - return; } static void parse_date(const char *strval, struct TM *btime) -- cgit v1.1-32-gdbae