core: add pointer in some callbacks (closes #406)

This pointer is the first argument received by callbacks, and the
existing argument "data" is now automatically freed by WeeChat when the
object containing the callback is removed.

With this new pointer, the linked list of callbacks in scripts has been
removed. This will improve speed of scripts (using a lot of hooks),
reduce memory used by scripts and reduce time to unload scripts.

Following functions are affected in the C API:

* exec_on_files
* config_new
* config_new_section
* config_new_option
* hook_command
* hook_command_run
* hook_timer
* hook_fd
* hook_process
* hook_process_hashtable
* hook_connect
* hook_print
* hook_signal
* hook_hsignal
* hook_config
* hook_completion
* hook_modifier
* hook_info
* hook_info_hashtable
* hook_infolist
* hook_hdata
* hook_focus
* unhook_all_plugin
* buffer_new
* bar_item_new
* upgrade_new
* upgrade_read

1 files changed, 11 insertions, 6 deletions

diff --git a/src/plugins/trigger/trigger-buffer.c b/src/plugins/trigger/trigger-buffer.c
index 7fd28e9f4..aaf660204 100644
--- a/src/plugins/trigger/trigger-buffer.c
+++ b/src/plugins/trigger/trigger-buffer.c
@@ -114,10 +114,12 @@ trigger_buffer_set_title ()
  */
 
 int
-trigger_buffer_input_cb (void *data, struct t_gui_buffer *buffer,
-                        const char *input_data)
+trigger_buffer_input_cb (const void *pointer, void *data,
+                         struct t_gui_buffer *buffer,
+                         const char *input_data)
 {
     /* make C compiler happy */
+    (void) pointer;
     (void) data;
 
     /* close buffer */
@@ -144,9 +146,11 @@ trigger_buffer_input_cb (void *data, struct t_gui_buffer *buffer,
  */
 
 int
-trigger_buffer_close_cb (void *data, struct t_gui_buffer *buffer)
+trigger_buffer_close_cb (const void *pointer, void *data,
+                         struct t_gui_buffer *buffer)
 {
     /* make C compiler happy */
+    (void) pointer;
     (void) data;
     (void) buffer;
 
@@ -188,9 +192,10 @@ trigger_buffer_open (const char *filter, int switch_to_buffer)
 {
     if (!trigger_buffer)
     {
-        trigger_buffer = weechat_buffer_new (TRIGGER_BUFFER_NAME,
-                                             &trigger_buffer_input_cb, NULL,
-                                             &trigger_buffer_close_cb, NULL);
+        trigger_buffer = weechat_buffer_new (
+            TRIGGER_BUFFER_NAME,
+            &trigger_buffer_input_cb, NULL, NULL,
+            &trigger_buffer_close_cb, NULL, NULL);
 
         /* failed to create buffer ? then return */
         if (!trigger_buffer)