That's consistent with the documented behaviour of g_hash_table_remove_all().
Reviewed-by: Darren Kenny <darren.ke...@oracle.com> Thanks, Darren. On Tue, Oct 24, 2017 at 03:27:59AM +0800, zhiyong...@ucloud.cn wrote:
From: Zhi Yong Wu <zhiyong...@ucloud.cn> When hash table is created with g_hash_table_new_full(), the free function has been registered. So it isn't necessary since g_hash_table_remove_all() will call connection_destroy() for each connection struct automatically. Signed-off-by: Zhi Yong Wu <zhiyong...@ucloud.cn> --- net/colo.c | 6 ------ 1 files changed, 0 insertions(+), 6 deletions(-) diff --git a/net/colo.c b/net/colo.c index 28ce7c8..668abb7 100644 --- a/net/colo.c +++ b/net/colo.c @@ -197,12 +197,6 @@ Connection *connection_get(GHashTable *connection_track_table, trace_colo_proxy_main("colo proxy connection hashtable full," " clear it"); connection_hashtable_reset(connection_track_table); - /* - * clear the conn_list - */ - while (!g_queue_is_empty(conn_list)) { - connection_destroy(g_queue_pop_head(conn_list)); - } } g_hash_table_insert(connection_track_table, new_key, conn); -- 1.7.1