[glib/signal-performance: 7/10] Add g_signal_set_va_marshaller

[Alexander Larsson <alexl src gnome org>]

commit 7964cda8e6c9dc779e99b65150e158e29d373ea4
Author: Alexander Larsson <alexl redhat com>
Date:   Wed Feb 22 19:34:05 2012 +0100

    Add g_signal_set_va_marshaller
    
    This lets you set a va_marshaller on your signal which will be
    propagated to all closures for the signal. Also, automatically
    uses the generica va_marshaller if you specify a NULL c_marshaller.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=661140

 gobject/gsignal.c |   58 ++++++++++++++++++++++++++++++++++++++++++++++++----
 gobject/gsignal.h |   11 ++++++++++
 2 files changed, 64 insertions(+), 5 deletions(-)
---
diff --git a/gobject/gsignal.c b/gobject/gsignal.c
index 1346929..20f11c6 100644
--- a/gobject/gsignal.c
+++ b/gobject/gsignal.c
@@ -213,6 +213,7 @@ struct _SignalNode
   GBSearchArray     *class_closure_bsa;
   SignalAccumulator *accumulator;
   GSignalCMarshaller c_marshaller;
+  GSignalCVaMarshaller va_marshaller;
   GHookList         *emission_hooks;
 };
 #define	MAX_TEST_CLASS_OFFSET	(4096)	/* 2^12, 12 bits for test_class_offset */
@@ -1492,7 +1493,11 @@ signal_add_class_closure (SignalNode *node,
 						    &key);
   g_closure_sink (closure);
   if (node->c_marshaller && closure && G_CLOSURE_NEEDS_MARSHAL (closure))
-    g_closure_set_marshal (closure, node->c_marshaller);
+    {
+      g_closure_set_marshal (closure, node->c_marshaller);
+      if (node->va_marshaller)
+	_g_closure_set_va_marshal (closure, node->va_marshaller);
+    }
 }
 
 /**
@@ -1645,8 +1650,12 @@ g_signal_newv (const gchar       *signal_name,
     }
   else
     node->accumulator = NULL;
+  node->va_marshaller = NULL;
   if (c_marshaller == NULL)
+    {
       c_marshaller = g_cclosure_marshal_generic;
+      node->va_marshaller = g_cclosure_marshal_generic_va;
+    }
   node->c_marshaller = c_marshaller;
   node->emission_hooks = NULL;
   if (class_closure)
@@ -1665,6 +1674,32 @@ g_signal_newv (const gchar       *signal_name,
   return signal_id;
 }
 
+void
+g_signal_set_va_marshaller (guint              signal_id,
+			    GType              instance_type,
+			    GSignalCVaMarshaller va_marshaller)
+{
+  SignalNode *node;
+  
+  g_return_if_fail (signal_id > 0);
+  g_return_if_fail (va_marshaller != NULL);
+  
+  SIGNAL_LOCK ();
+  node = LOOKUP_SIGNAL_NODE (signal_id);
+  if (node)
+    {
+      node->va_marshaller = va_marshaller;
+      if (node->class_closure_bsa)
+	{
+	  ClassClosure *cc = g_bsearch_array_get_nth (node->class_closure_bsa, &g_class_closure_bconfig, 0);
+	  if (cc->closure->marshal == node->c_marshaller)
+	    _g_closure_set_va_marshal (cc->closure, va_marshaller);
+	}
+    }
+  SIGNAL_UNLOCK ();
+}
+
+
 /**
  * g_signal_new_valist:
  * @signal_name: the name for the signal
@@ -1741,6 +1776,7 @@ signal_destroy_R (SignalNode *signal_node)
   signal_node->class_closure_bsa = NULL;
   signal_node->accumulator = NULL;
   signal_node->c_marshaller = NULL;
+  signal_node->va_marshaller = NULL;
   signal_node->emission_hooks = NULL;
   
 #ifdef	G_ENABLE_DEBUG
@@ -2166,7 +2202,11 @@ g_signal_connect_closure_by_id (gpointer  instance,
 	  g_closure_sink (closure);
 	  handler_insert (signal_id, instance, handler);
 	  if (node->c_marshaller && G_CLOSURE_NEEDS_MARSHAL (closure))
-	    g_closure_set_marshal (closure, node->c_marshaller);
+	    {
+	      g_closure_set_marshal (closure, node->c_marshaller);
+	      if (node->va_marshaller)
+		_g_closure_set_va_marshal (closure, node->va_marshaller);
+	    }
 	}
     }
   else
@@ -2224,7 +2264,11 @@ g_signal_connect_closure (gpointer     instance,
 	  g_closure_sink (closure);
 	  handler_insert (signal_id, instance, handler);
 	  if (node->c_marshaller && G_CLOSURE_NEEDS_MARSHAL (handler->closure))
-	    g_closure_set_marshal (handler->closure, node->c_marshaller);
+	    {
+	      g_closure_set_marshal (handler->closure, node->c_marshaller);
+	      if (node->va_marshaller)
+		_g_closure_set_va_marshal (handler->closure, node->va_marshaller);
+	    }
 	}
     }
   else
@@ -2318,8 +2362,12 @@ g_signal_connect_data (gpointer       instance,
 	  g_closure_sink (handler->closure);
 	  handler_insert (signal_id, instance, handler);
 	  if (node->c_marshaller && G_CLOSURE_NEEDS_MARSHAL (handler->closure))
-	    g_closure_set_marshal (handler->closure, node->c_marshaller);
-	}
+	    {
+	      g_closure_set_marshal (handler->closure, node->c_marshaller);
+	      if (node->va_marshaller)
+		_g_closure_set_va_marshal (handler->closure, node->va_marshaller);
+	    }
+        }
     }
   else
     g_warning ("%s: signal `%s' is invalid for instance `%p'", G_STRLOC, detailed_signal, instance);
diff --git a/gobject/gsignal.h b/gobject/gsignal.h
index 73dcf47..966b6c7 100644
--- a/gobject/gsignal.h
+++ b/gobject/gsignal.h
@@ -44,6 +44,14 @@ typedef struct _GSignalInvocationHint	 GSignalInvocationHint;
  */
 typedef GClosureMarshal			 GSignalCMarshaller;
 /**
+ * GSignalCVaMarshaller:
+ * 
+ * This is the signature of va_list marshaller functions, an optional
+ * marshaller that can be used in some situations to avoid
+ * marshalling the signal argument into GValues.
+ */
+typedef GVaClosureMarshal		 GSignalCVaMarshaller;
+/**
  * GSignalEmissionHook:
  * @ihint: Signal invocation hint, see #GSignalInvocationHint.
  * @n_param_values: the number of parameters to the function, including
@@ -294,6 +302,9 @@ guint            g_signal_new_class_handler (const gchar        *signal_name,
                                              GType               return_type,
                                              guint               n_params,
                                              ...);
+void             g_signal_set_va_marshaller (guint              signal_id,
+					     GType              instance_type,
+					     GSignalCVaMarshaller va_marshaller);
 
 void                  g_signal_emitv        (const GValue       *instance_and_params,
 					     guint               signal_id,