Add debug function and some assert to lttv_hook_remove_list

Also add some const correctness to the remove_list function

Signed-off-by: Yannick Brosseau <yannick.brosseau@gmail.com>

diff --git a/lttv/lttv/hook.c b/lttv/lttv/hook.c
index 62918cd5e0a321f67f638e995ec09497a5a9415e..3d82f1114fbca808bd7f29cd1363140dd2af998b 100644 (file)
--- a/lttv/lttv/hook.c
+++ b/lttv/lttv/hook.c
@@ -193,19 +193,23 @@ void lttv_hooks_remove_data(LttvHooks *h, LttvHook f, void *hook_data)
 }
 
 
-void lttv_hooks_remove_list(LttvHooks *h, LttvHooks *list)
+void lttv_hooks_remove_list(LttvHooks *h, const LttvHooks *list)
 {
        guint i, j;
 
        LttvHookClosure *c, *c_list;
 
        if(list == NULL) return;
+       g_assert(h != list);
+       /* This iteration assume that both list are in the same order */
        for(i = 0, j = 0 ; i < h->len && j < list->len ;) {
                c = &g_array_index(h, LttvHookClosure, i);
                c_list = &g_array_index(list, LttvHookClosure, j);
                if(c->hook == c_list->hook && c->hook_data == c_list->hook_data) {
                        if(c->ref_count == 1) {
+                               int count = h->len;
                                lttv_hooks_remove_by_position(h, i);
+                               g_assert((count-1) == h->len);
                        } else {
                                g_assert(c->ref_count != 0);
                                c->ref_count--;
@@ -394,3 +398,22 @@ gboolean lttv_hooks_call_check_merge(LttvHooks *h1, void *call_data1,
 
        return FALSE;
 }
+
+
+void lttv_hooks_print(const LttvHooks *h)
+{
+       gboolean ret, sum_ret = FALSE;
+
+       LttvHookClosure *c;
+
+       guint i;
+
+       if(likely(h != NULL)) {
+               for(i = 0 ; i < h->len ; i++) {
+                       c = &g_array_index(h, LttvHookClosure, i);
+                       printf("%p:%i:%i,", c->hook, c->ref_count, c->prio);
+               }
+               printf("\n");
+       }
+
+}

diff --git a/lttv/lttv/hook.h b/lttv/lttv/hook.h
index 81383b5b4285c44ce14ccfb17f23632633e7b28a..65903b284b100c10c8eafd53cffb08402f179e3e 100644 (file)
--- a/lttv/lttv/hook.h
+++ b/lttv/lttv/hook.h
@@ -77,7 +77,7 @@ void lttv_hooks_remove_data(LttvHooks *h, LttvHook f, void *hook_data);
 
 /* Remove a list of hooks from the hooks list in h. */
 
-void lttv_hooks_remove_list(LttvHooks *h, LttvHooks *list);
+void lttv_hooks_remove_list(LttvHooks *h, const LttvHooks *list);
 
 
 /* Return the number of hooks in the list */
@@ -188,4 +188,8 @@ LttvHooks *lttv_hooks_by_id_channel_find(LttvHooksByIdChannelArray *h,
 
 #endif /* BABEL_CLEANUP */
 
+/* Print information about each hook in the list*/
+
+void lttv_hooks_print(const LttvHooks *h);
+
 #endif // HOOK_H