gobject/: fully remove gobjectalias hacks
[platform/upstream/glib.git] / gobject / gsignal.c
index 3eaa7c6..df5a37b 100644 (file)
  * this code is based on the original GtkSignal implementation
  * for the Gtk+ library by Peter Mattis <petm@xcf.berkeley.edu>
  */
+
+/*
+ * MT safe
+ */
+
+#include "config.h"
+
+#include <string.h>
+#include <signal.h>
+
+#include "gsignal.h"
+#include "gbsearcharray.h"
+#include "gvaluecollector.h"
+#include "gvaluetypes.h"
+#include "gboxed.h"
+#include "gobject.h"
+#include "genums.h"
+#include "gobject_trace.h"
+
+
 /**
  * SECTION:signals
- * @Short_description: A means for customization of object behaviour and a general purpose notification mechanism
- * @Title: Signals
- * 
- * The basic concept of the signal system is that of the <emphasis>emission</emphasis>
- * of a signal.
- * Signals are introduced per-type and are identified through strings.
- * Signals introduced for a parent type are available in derived types as well,
- * so basically they are a per-type facility that is inherited.
- * A signal emission mainly involves invocation of a certain set of callbacks in
- * precisely defined manner. There are two main categories of such callbacks,
- * per-object
- *     i'm referring to those types as "object types" in the following, simply
- *     because that is the context most users will encounter signals in.
- * 
+ * @short_description: A means for customization of object behaviour
+ *     and a general purpose notification mechanism
+ * @title: Signals
+ *
+ * The basic concept of the signal system is that of the
+ * <emphasis>emission</emphasis> of a signal. Signals are introduced
+ * per-type and are identified through strings.  Signals introduced
+ * for a parent type are available in derived types as well, so
+ * basically they are a per-type facility that is inherited.  A signal
+ * emission mainly involves invocation of a certain set of callbacks
+ * in precisely defined manner. There are two main categories of such
+ * callbacks, per-object 
+ * <footnote><para>Although signals can deal with any kind of instantiatable 
+ * type, i'm referring to those types as "object types" in the following, 
+ * simply because that is the context most users will encounter signals in.
+ * </para></footnote>
  * ones and user provided ones.
  * The per-object callbacks are most often referred to as "object method
  * handler" or "default (signal) handler", while user provided callbacks are
@@ -43,7 +65,7 @@
  * frequently happens at the end of an object class' creation), while user
  * provided handlers are frequently connected and disconnected to/from a certain
  * signal on certain object instances.
- * 
+ *
  * A signal emission consists of five stages, unless prematurely stopped:
  * <variablelist>
  * <varlistentry><term></term><listitem><para>
  * detail part of the signal specification upon connection) serves as a
  * wildcard and matches any detail argument passed in to emission.
  */
-/*
- * MT safe
- */
-
-#include <config.h>
-
-#include        "gsignal.h"
-
-#include        "gbsearcharray.h"
-#include        "gvaluecollector.h"
-#include       "gvaluetypes.h"
-#include       "gboxed.h"
-#include       "gobject.h"
-#include       "genums.h"
 
-#include       "gobjectalias.h"
-
-#include       <string.h> 
-#include       <signal.h>
-
-
-/* pre allocation configurations
- */
-#define        MAX_STACK_VALUES        (16)
 
 #define REPORT_BUG      "please report occurrence circumstances to gtk-devel-list@gnome.org"
 #ifdef G_ENABLE_DEBUG
@@ -579,7 +578,7 @@ handler_ref (Handler *handler)
 {
   g_return_if_fail (handler->ref_count > 0);
   
-  g_atomic_int_inc (&handler->ref_count);
+  g_atomic_int_inc ((int *)&handler->ref_count);
 }
 
 static inline void
@@ -591,7 +590,7 @@ handler_unref_R (guint    signal_id,
 
   g_return_if_fail (handler->ref_count > 0);
   
-  is_zero = g_atomic_int_dec_and_test (&handler->ref_count);
+  is_zero = g_atomic_int_dec_and_test ((int *)&handler->ref_count);
 
   if (G_UNLIKELY (is_zero))
     {
@@ -811,13 +810,13 @@ _g_signals_destroy (GType itype)
  * @instance: the object whose signal handlers you wish to stop.
  * @signal_id: the signal identifier, as returned by g_signal_lookup().
  * @detail: the detail which the signal was emitted with.
- * 
+ *
  * Stops a signal's current emission.
- * 
+ *
  * This will prevent the default method from running, if the signal was
- * %G_SIGNAL_RUN_LAST and you connected normally (i.e. without the "after" 
+ * %G_SIGNAL_RUN_LAST and you connected normally (i.e. without the "after"
  * flag).
- * 
+ *
  * Prints a warning if used on a signal which isn't being emitted.
  */
 void
@@ -882,11 +881,11 @@ signal_finalize_hook (GHookList *hook_list,
  * @hook_func: a #GSignalEmissionHook function.
  * @hook_data: user data for @hook_func.
  * @data_destroy: a #GDestroyNotify for @hook_data.
- * 
+ *
  * Adds an emission hook for a signal, which will get called for any emission
  * of that signal, independent of the instance. This is possible only
  * for signals which don't have #G_SIGNAL_NO_HOOKS flag set.
- * 
+ *
  * Returns: the hook id, for later use with g_signal_remove_emission_hook().
  */
 gulong
@@ -947,9 +946,9 @@ g_signal_add_emission_hook (guint               signal_id,
 /**
  * g_signal_remove_emission_hook:
  * @signal_id: the id of the signal
- * @hook_id: the id of the emission hook, as returned by 
+ * @hook_id: the id of the emission hook, as returned by
  *  g_signal_add_emission_hook()
- * 
+ *
  * Deletes an emission hook.
  */
 void
@@ -1021,10 +1020,10 @@ signal_parse_name (const gchar *name,
  * @signal_id_p: Location to store the signal id.
  * @detail_p: Location to store the detail quark.
  * @force_detail_quark: %TRUE forces creation of a #GQuark for the detail.
- * 
+ *
  * Internal function to parse a signal name into its @signal_id
  * and @detail quark.
- * 
+ *
  * Returns: Whether the signal name could successfully be parsed and @signal_id_p and @detail_p contain valid return values.
  */
 gboolean
@@ -1062,10 +1061,10 @@ g_signal_parse_name (const gchar *detailed_signal,
  * g_signal_stop_emission_by_name:
  * @instance: the object whose signal handlers you wish to stop.
  * @detailed_signal: a string of the form "signal-name::detail".
- * 
+ *
  * Stops a signal's current emission.
- * 
- * This is just like g_signal_stop_emission() except it will look up the 
+ *
+ * This is just like g_signal_stop_emission() except it will look up the
  * signal id for you.
  */
 void
@@ -1117,15 +1116,15 @@ g_signal_stop_emission_by_name (gpointer     instance,
  * g_signal_lookup:
  * @name: the signal's name.
  * @itype: the type that the signal operates on.
- * 
- * Given the name of the signal and the type of object it connects to, gets 
- * the signal's identifying integer. Emitting the signal by number is 
+ *
+ * Given the name of the signal and the type of object it connects to, gets
+ * the signal's identifying integer. Emitting the signal by number is
  * somewhat faster than using the name each time.
- * 
+ *
  * Also tries the ancestors of the given type.
- * 
+ *
  * See g_signal_new() for details on allowed signal names.
- * 
+ *
  * Returns: the signal's identifying number, or 0 if no signal was found.
  */
 guint
@@ -1160,11 +1159,11 @@ g_signal_lookup (const gchar *name,
  * g_signal_list_ids:
  * @itype: Instance or interface type.
  * @n_ids: Location to store the number of signal ids for @itype.
- * 
+ *
  * Lists the signals by id that a certain instance or interface type
  * created. Further information about the signals can be acquired through
  * g_signal_query().
- * 
+ *
  * Returns: Newly allocated array of signal IDs.
  */
 guint*
@@ -1217,11 +1216,11 @@ g_signal_list_ids (GType  itype,
 /**
  * g_signal_name:
  * @signal_id: the signal's identifying number.
- * 
+ *
  * Given the signal's identifier, finds its name.
- * 
+ *
  * Two different signals may have the same name, if they have differing types.
- * 
+ *
  * Returns: the signal name, or %NULL if the signal number was invalid.
  */
 G_CONST_RETURN gchar*
@@ -1243,7 +1242,7 @@ g_signal_name (guint signal_id)
  * @signal_id: The signal id of the signal to query information for.
  * @query: A user provided structure that is filled in with constant
  *  values upon success.
- * 
+ *
  * Queries the signal system for in-depth information about a
  * specific signal. This function will fill in a user-provided
  * structure to hold signal-specific information. If an invalid
@@ -1279,33 +1278,38 @@ g_signal_query (guint         signal_id,
 /**
  * g_signal_new:
  * @signal_name: the name for the signal
- * @itype: the type this signal pertains to. It will also pertain to 
+ * @itype: the type this signal pertains to. It will also pertain to
  *  types which are derived from this type.
- * @signal_flags: a combination of #GSignalFlags specifying detail of when 
- *  the default handler is to be invoked. You should at least specify 
+ * @signal_flags: a combination of #GSignalFlags specifying detail of when
+ *  the default handler is to be invoked. You should at least specify
  *  %G_SIGNAL_RUN_FIRST or %G_SIGNAL_RUN_LAST.
- * @class_offset: The offset of the function pointer in the class structure 
+ * @class_offset: The offset of the function pointer in the class structure
  *  for this type. Used to invoke a class method generically. Pass 0 to
- *  not associate a class method with this signal.
+ *  not associate a class method slot with this signal.
  * @accumulator: the accumulator for this signal; may be %NULL.
  * @accu_data: user data for the @accumulator.
- * @c_marshaller: the function to translate arrays of parameter values to 
+ * @c_marshaller: the function to translate arrays of parameter values to
  *  signal emissions into C language callback invocations.
- * @return_type: the type of return value, or #G_TYPE_NONE for a signal 
+ * @return_type: the type of return value, or #G_TYPE_NONE for a signal
  *  without a return value.
  * @n_params: the number of parameter types to follow.
  * @...: a list of types, one for each parameter.
- * 
+ *
  * Creates a new signal. (This is usually done in the class initializer.)
- * 
+ *
  * A signal name consists of segments consisting of ASCII letters and
  * digits, separated by either the '-' or '_' character. The first
  * character of a signal name must be a letter. Names which violate these
- * rules lead to undefined behaviour of the GSignal system. 
- * 
+ * rules lead to undefined behaviour of the GSignal system.
+ *
  * When registering a signal and looking up a signal, either separator can
- * be used, but they cannot be mixed. 
- * 
+ * be used, but they cannot be mixed.
+ *
+ * If 0 is used for @class_offset subclasses cannot override the class handler
+ * in their <code>class_init</code> method by doing
+ * <code>super_class->signal_handler = my_signal_handler</code>. Instead they
+ * will have to use g_signal_override_class_handler().
+ *
  * Returns: the signal id
  */
 guint
@@ -1349,6 +1353,73 @@ g_signal_new (const gchar         *signal_name,
   return signal_id;
 }
 
+/**
+ * g_signal_new_class_handler:
+ * @signal_name: the name for the signal
+ * @itype: the type this signal pertains to. It will also pertain to
+ *  types which are derived from this type.
+ * @signal_flags: a combination of #GSignalFlags specifying detail of when
+ *  the default handler is to be invoked. You should at least specify
+ *  %G_SIGNAL_RUN_FIRST or %G_SIGNAL_RUN_LAST.
+ * @class_handler: a #GCallback which acts as class implementation of
+ *  this signal. Used to invoke a class method generically. Pass %NULL to
+ *  not associate a class method with this signal.
+ * @accumulator: the accumulator for this signal; may be %NULL.
+ * @accu_data: user data for the @accumulator.
+ * @c_marshaller: the function to translate arrays of parameter values to
+ *  signal emissions into C language callback invocations.
+ * @return_type: the type of return value, or #G_TYPE_NONE for a signal
+ *  without a return value.
+ * @n_params: the number of parameter types to follow.
+ * @...: a list of types, one for each parameter.
+ *
+ * Creates a new signal. (This is usually done in the class initializer.)
+ *
+ * This is a variant of g_signal_new() that takes a C callback instead
+ * off a class offset for the signal's class handler. This function
+ * doesn't need a function pointer exposed in the class structure of
+ * an object definition, instead the function pointer is passed
+ * directly and can be overriden by derived classes with
+ * g_signal_override_class_closure() or
+ * g_signal_override_class_handler()and chained to with
+ * g_signal_chain_from_overridden() or
+ * g_signal_chain_from_overridden_handler().
+ *
+ * See g_signal_new() for information about signal names.
+ *
+ * Returns: the signal id
+ *
+ * Since: 2.18
+ */
+guint
+g_signal_new_class_handler (const gchar        *signal_name,
+                            GType               itype,
+                            GSignalFlags        signal_flags,
+                            GCallback           class_handler,
+                            GSignalAccumulator  accumulator,
+                            gpointer            accu_data,
+                            GSignalCMarshaller  c_marshaller,
+                            GType               return_type,
+                            guint               n_params,
+                            ...)
+{
+  va_list args;
+  guint signal_id;
+
+  g_return_val_if_fail (signal_name != NULL, 0);
+
+  va_start (args, n_params);
+
+  signal_id = g_signal_new_valist (signal_name, itype, signal_flags,
+                                   class_handler ? g_cclosure_new (class_handler, NULL, NULL) : NULL,
+                                   accumulator, accu_data, c_marshaller,
+                                   return_type, n_params, args);
+
+  va_end (args);
+
+  return signal_id;
+}
+
 static inline ClassClosure*
 signal_find_class_closure (SignalNode *node,
                           GType       itype)
@@ -1382,9 +1453,12 @@ signal_lookup_closure (SignalNode    *node,
   ClassClosure *cc;
 
   if (node->class_closure_bsa && g_bsearch_array_get_n_nodes (node->class_closure_bsa) == 1)
-    cc = g_bsearch_array_get_nth (node->class_closure_bsa, &g_class_closure_bconfig, 0);
-  else
-    cc = signal_find_class_closure (node, G_TYPE_FROM_INSTANCE (instance));
+    {
+      cc = g_bsearch_array_get_nth (node->class_closure_bsa, &g_class_closure_bconfig, 0);
+      if (cc && cc->instance_type == 0) /* check for default closure */
+        return cc->closure;
+    }
+  cc = signal_find_class_closure (node, G_TYPE_FROM_INSTANCE (instance));
   return cc ? cc->closure : NULL;
 }
 
@@ -1413,25 +1487,25 @@ signal_add_class_closure (SignalNode *node,
 /**
  * g_signal_newv:
  * @signal_name: the name for the signal
- * @itype: the type this signal pertains to. It will also pertain to 
- *  types which are derived from this type.
- * @signal_flags: a combination of #GSignalFlags specifying detail of when 
- *  the default handler is to be invoked. You should at least specify 
- *  %G_SIGNAL_RUN_FIRST or %G_SIGNAL_RUN_LAST.
- * @class_closure: The closure to invoke on signal emission; may be %NULL.
- * @accumulator: the accumulator for this signal; may be %NULL.
- * @accu_data: user data for the @accumulator.
- * @c_marshaller: the function to translate arrays of parameter values to 
- *  signal emissions into C language callback invocations.
- * @return_type: the type of return value, or #G_TYPE_NONE for a signal 
- *  without a return value.
- * @n_params: the length of @param_types.
- * @param_types: an array types, one for each parameter.
- * 
+ * @itype: the type this signal pertains to. It will also pertain to
+ *     types which are derived from this type
+ * @signal_flags: a combination of #GSignalFlags specifying detail of when
+ *     the default handler is to be invoked. You should at least specify
+ *     %G_SIGNAL_RUN_FIRST or %G_SIGNAL_RUN_LAST
+ * @class_closure: The closure to invoke on signal emission; may be %NULL
+ * @accumulator: the accumulator for this signal; may be %NULL
+ * @accu_data: user data for the @accumulator
+ * @c_marshaller: the function to translate arrays of parameter values to
+ *     signal emissions into C language callback invocations
+ * @return_type: the type of return value, or #G_TYPE_NONE for a signal
+ *     without a return value
+ * @n_params: the length of @param_types
+ * @param_types: an array of types, one for each parameter
+ *
  * Creates a new signal. (This is usually done in the class initializer.)
- * 
+ *
  * See g_signal_new() for details on allowed signal names.
- * 
+ *
  * Returns: the signal id
  */
 guint
@@ -1534,6 +1608,8 @@ g_signal_newv (const gchar       *signal_name,
       node->name = g_intern_string (name);
       key.quark = g_quark_from_string (name);
       g_signal_key_bsa = g_bsearch_array_insert (g_signal_key_bsa, &g_signal_key_bconfig, &key);
+
+      TRACE(GOBJECT_SIGNAL_NEW(signal_id, name, itype));
     }
   node->destroyed = FALSE;
   node->test_class_offset = 0;
@@ -1571,25 +1647,25 @@ g_signal_newv (const gchar       *signal_name,
 /**
  * g_signal_new_valist:
  * @signal_name: the name for the signal
- * @itype: the type this signal pertains to. It will also pertain to 
+ * @itype: the type this signal pertains to. It will also pertain to
  *  types which are derived from this type.
- * @signal_flags: a combination of #GSignalFlags specifying detail of when 
- *  the default handler is to be invoked. You should at least specify 
+ * @signal_flags: a combination of #GSignalFlags specifying detail of when
+ *  the default handler is to be invoked. You should at least specify
  *  %G_SIGNAL_RUN_FIRST or %G_SIGNAL_RUN_LAST.
  * @class_closure: The closure to invoke on signal emission; may be %NULL.
  * @accumulator: the accumulator for this signal; may be %NULL.
  * @accu_data: user data for the @accumulator.
- * @c_marshaller: the function to translate arrays of parameter values to 
+ * @c_marshaller: the function to translate arrays of parameter values to
  *  signal emissions into C language callback invocations.
- * @return_type: the type of return value, or #G_TYPE_NONE for a signal 
+ * @return_type: the type of return value, or #G_TYPE_NONE for a signal
  *  without a return value.
  * @n_params: the number of parameter types in @args.
  * @args: va_list of #GType, one for each parameter.
- * 
+ *
  * Creates a new signal. (This is usually done in the class initializer.)
- * 
+ *
  * See g_signal_new() for details on allowed signal names.
- * 
+ *
  * Returns: the signal id
  */
 guint
@@ -1684,13 +1760,17 @@ signal_destroy_R (SignalNode *signal_node)
 /**
  * g_signal_override_class_closure:
  * @signal_id: the signal id
- * @instance_type: the instance type on which to override the class closure 
+ * @instance_type: the instance type on which to override the class closure
  *  for the signal.
  * @class_closure: the closure.
- * 
+ *
  * Overrides the class closure (i.e. the default handler) for the given signal
  * for emissions on instances of @instance_type. @instance_type must be derived
  * from the type to which the signal belongs.
+ *
+ * See g_signal_chain_from_overridden() and
+ * g_signal_chain_from_overridden_handler() for how to chain up to the
+ * parent class closure from inside the overridden one.
  */
 void
 g_signal_override_class_closure (guint     signal_id,
@@ -1719,15 +1799,56 @@ g_signal_override_class_closure (guint     signal_id,
 }
 
 /**
+ * g_signal_override_class_handler:
+ * @signal_name: the name for the signal
+ * @instance_type: the instance type on which to override the class handler
+ *  for the signal.
+ * @class_handler: the handler.
+ *
+ * Overrides the class closure (i.e. the default handler) for the
+ * given signal for emissions on instances of @instance_type with
+ * callabck @class_handler. @instance_type must be derived from the
+ * type to which the signal belongs.
+ *
+ * See g_signal_chain_from_overridden() and
+ * g_signal_chain_from_overridden_handler() for how to chain up to the
+ * parent class closure from inside the overridden one.
+ *
+ * Since: 2.18
+ */
+void
+g_signal_override_class_handler (const gchar *signal_name,
+                                GType        instance_type,
+                                GCallback    class_handler)
+{
+  guint signal_id;
+
+  g_return_if_fail (signal_name != NULL);
+  g_return_if_fail (instance_type != G_TYPE_NONE);
+  g_return_if_fail (class_handler != NULL);
+
+  signal_id = g_signal_lookup (signal_name, instance_type);
+
+  if (signal_id)
+    g_signal_override_class_closure (signal_id, instance_type,
+                                     g_cclosure_new (class_handler, NULL, NULL));
+  else
+    g_warning ("%s: signal name '%s' is invalid for type id '%"G_GSIZE_FORMAT"'",
+               G_STRLOC, signal_name, instance_type);
+
+}
+
+/**
  * g_signal_chain_from_overridden:
- * @instance_and_params: the argument list of the signal emission. The first 
- *  element in the array is a #GValue for the instance the signal is being 
+ * @instance_and_params: the argument list of the signal emission. The first
+ *  element in the array is a #GValue for the instance the signal is being
  *  emitted on. The rest are any arguments to be passed to the signal.
  * @return_value: Location for the return value.
- * 
+ *
  * Calls the original class closure of a signal. This function should only
- * be called from an overridden class closure; see 
- * g_signal_override_class_closure().
+ * be called from an overridden class closure; see
+ * g_signal_override_class_closure() and
+ * g_signal_override_class_handler().
  */
 void
 g_signal_chain_from_overridden (const GValue *instance_and_params,
@@ -1773,6 +1894,7 @@ g_signal_chain_from_overridden (const GValue *instance_and_params,
     }
   else
     g_warning ("%s: no signal is currently being emitted for instance `%p'", G_STRLOC, instance);
+
   if (closure)
     {
       emission->chain_type = chain_type;
@@ -1789,11 +1911,175 @@ g_signal_chain_from_overridden (const GValue *instance_and_params,
 }
 
 /**
+ * g_signal_chain_from_overridden_handler:
+ * @instance: the instance the signal is being emitted on.
+ * @...: parameters to be passed to the parent class closure, followed by a
+ *  location for the return value. If the return type of the signal
+ *  is #G_TYPE_NONE, the return value location can be omitted.
+ *
+ * Calls the original class closure of a signal. This function should
+ * only be called from an overridden class closure; see
+ * g_signal_override_class_closure() and
+ * g_signal_override_class_handler().
+ *
+ * Since: 2.18
+ */
+void
+g_signal_chain_from_overridden_handler (gpointer instance,
+                                        ...)
+{
+  GType chain_type = 0, restore_type = 0;
+  Emission *emission = NULL;
+  GClosure *closure = NULL;
+  SignalNode *node;
+  guint n_params = 0;
+
+  g_return_if_fail (G_TYPE_CHECK_INSTANCE (instance));
+
+  SIGNAL_LOCK ();
+  emission = emission_find_innermost (instance);
+  if (emission)
+    {
+      node = LOOKUP_SIGNAL_NODE (emission->ihint.signal_id);
+
+      g_assert (node != NULL); /* paranoid */
+
+      /* we should probably do the same parameter checks as g_signal_emit() here.
+       */
+      if (emission->chain_type != G_TYPE_NONE)
+       {
+         ClassClosure *cc = signal_find_class_closure (node, emission->chain_type);
+
+         g_assert (cc != NULL);        /* closure currently in call stack */
+
+         n_params = node->n_params;
+         restore_type = cc->instance_type;
+         cc = signal_find_class_closure (node, g_type_parent (cc->instance_type));
+         if (cc && cc->instance_type != restore_type)
+           {
+             closure = cc->closure;
+             chain_type = cc->instance_type;
+           }
+       }
+      else
+       g_warning ("%s: signal id `%u' cannot be chained from current emission stage for instance `%p'", G_STRLOC, node->signal_id, instance);
+    }
+  else
+    g_warning ("%s: no signal is currently being emitted for instance `%p'", G_STRLOC, instance);
+
+  if (closure)
+    {
+      GValue *instance_and_params;
+      GType signal_return_type;
+      GValue *param_values;
+      va_list var_args;
+      guint i;
+
+      va_start (var_args, instance);
+
+      signal_return_type = node->return_type;
+      instance_and_params = g_slice_alloc0 (sizeof (GValue) * (n_params + 1));
+      param_values = instance_and_params + 1;
+
+      for (i = 0; i < node->n_params; i++)
+        {
+          gchar *error;
+          GType ptype = node->param_types[i] & ~G_SIGNAL_TYPE_STATIC_SCOPE;
+          gboolean static_scope = node->param_types[i] & G_SIGNAL_TYPE_STATIC_SCOPE;
+
+          SIGNAL_UNLOCK ();
+          G_VALUE_COLLECT_INIT (param_values + i, ptype,
+                               var_args,
+                               static_scope ? G_VALUE_NOCOPY_CONTENTS : 0,
+                               &error);
+          if (error)
+            {
+              g_warning ("%s: %s", G_STRLOC, error);
+              g_free (error);
+
+              /* we purposely leak the value here, it might not be
+               * in a sane state if an error condition occoured
+               */
+              while (i--)
+                g_value_unset (param_values + i);
+
+              g_slice_free1 (sizeof (GValue) * (n_params + 1), instance_and_params);
+              va_end (var_args);
+              return;
+            }
+          SIGNAL_LOCK ();
+        }
+
+      SIGNAL_UNLOCK ();
+      instance_and_params->g_type = 0;
+      g_value_init (instance_and_params, G_TYPE_FROM_INSTANCE (instance));
+      g_value_set_instance (instance_and_params, instance);
+      SIGNAL_LOCK ();
+
+      emission->chain_type = chain_type;
+      SIGNAL_UNLOCK ();
+
+      if (signal_return_type == G_TYPE_NONE)
+        {
+          g_closure_invoke (closure,
+                            NULL,
+                            n_params + 1,
+                            instance_and_params,
+                            &emission->ihint);
+        }
+      else
+        {
+          GValue return_value = { 0, };
+          gchar *error = NULL;
+          GType rtype = signal_return_type & ~G_SIGNAL_TYPE_STATIC_SCOPE;
+          gboolean static_scope = signal_return_type & G_SIGNAL_TYPE_STATIC_SCOPE;
+
+          g_value_init (&return_value, rtype);
+
+          g_closure_invoke (closure,
+                            &return_value,
+                            n_params + 1,
+                            instance_and_params,
+                            &emission->ihint);
+
+          G_VALUE_LCOPY (&return_value,
+                         var_args,
+                         static_scope ? G_VALUE_NOCOPY_CONTENTS : 0,
+                         &error);
+          if (!error)
+            {
+              g_value_unset (&return_value);
+            }
+          else
+            {
+              g_warning ("%s: %s", G_STRLOC, error);
+              g_free (error);
+
+              /* we purposely leak the value here, it might not be
+               * in a sane state if an error condition occured
+               */
+            }
+        }
+
+      for (i = 0; i < n_params; i++)
+        g_value_unset (param_values + i);
+      g_value_unset (instance_and_params);
+      g_slice_free1 (sizeof (GValue) * (n_params + 1), instance_and_params);
+
+      va_end (var_args);
+
+      SIGNAL_LOCK ();
+      emission->chain_type = restore_type;
+    }
+  SIGNAL_UNLOCK ();
+}
+
+/**
  * g_signal_get_invocation_hint:
  * @instance: the instance to query
- * 
- * Returns the invocation hint of the innermost signal emission of instance. 
- * 
+ *
+ * Returns the invocation hint of the innermost signal emission of instance.
+ *
  * Returns: the invocation hint of the innermost signal emission.
  */
 GSignalInvocationHint*
@@ -1816,11 +2102,11 @@ g_signal_get_invocation_hint (gpointer instance)
  * @signal_id: the id of the signal.
  * @detail: the detail.
  * @closure: the closure to connect.
- * @after: whether the handler should be called before or after the 
+ * @after: whether the handler should be called before or after the
  *  default handler of the signal.
- * 
+ *
  * Connects a closure to a signal for a particular object.
- * 
+ *
  * Returns: the handler id
  */
 gulong
@@ -1870,11 +2156,11 @@ g_signal_connect_closure_by_id (gpointer  instance,
  * @instance: the instance to connect to.
  * @detailed_signal: a string of the form "signal-name::detail".
  * @closure: the closure to connect.
- * @after: whether the handler should be called before or after the 
+ * @after: whether the handler should be called before or after the
  *  default handler of the signal.
- * 
+ *
  * Connects a closure to a signal for a particular object.
- * 
+ *
  * Returns: the handler id
  */
 gulong
@@ -1931,13 +2217,13 @@ g_signal_connect_closure (gpointer     instance,
  * @data: data to pass to @c_handler calls.
  * @destroy_data: a #GClosureNotify for @data.
  * @connect_flags: a combination of #GConnectFlags.
- * 
+ *
  * Connects a #GCallback function to a signal for a particular object. Similar
  * to g_signal_connect(), but allows to provide a #GClosureNotify for the data
  * which will be called when the signal handler is disconnected and no longer
  * used. Specify @connect_flags if you need <literal>..._after()</literal> or
  * <literal>..._swapped()</literal> variants of this function.
- * 
+ *
  * Returns: the handler id
  */
 gulong
@@ -1996,14 +2282,14 @@ g_signal_connect_data (gpointer       instance,
  * g_signal_handler_block:
  * @instance: The instance to block the signal handler of.
  * @handler_id: Handler id of the handler to be blocked.
- * 
- * Blocks a handler of an instance so it will not be called during 
- * any signal emissions unless it is unblocked again. Thus "blocking" 
- * a signal handler means to temporarily deactive it, a signal handler 
- * has to be unblocked exactly the same amount of times it has been 
+ *
+ * Blocks a handler of an instance so it will not be called during any
+ * signal emissions unless it is unblocked again. Thus "blocking" a
+ * signal handler means to temporarily deactive it, a signal handler
+ * has to be unblocked exactly the same amount of times it has been
  * blocked before to become active again.
- * 
- * The @handler_id has to be a valid signal handler id, connected to a 
+ *
+ * The @handler_id has to be a valid signal handler id, connected to a
  * signal of @instance.
  */
 void
@@ -2034,19 +2320,19 @@ g_signal_handler_block (gpointer instance,
  * g_signal_handler_unblock:
  * @instance: The instance to unblock the signal handler of.
  * @handler_id: Handler id of the handler to be unblocked.
- * 
- * Undoes the effect of a previous g_signal_handler_block() call. 
- * A blocked handler is skipped during signal emissions and will not be 
- * invoked, unblocking it (for exactly the amount of times it has been 
- * blocked before) reverts its "blocked" state, so the handler will be 
- * recognized by the signal system and is called upon future or currently
- * ongoing signal emissions (since the order in which handlers are
- * called during signal emissions is deterministic, whether the
- * unblocked handler in question is called as part of a currently
- * ongoing emission depends on how far that emission has proceeded
- * yet).
- * 
- * The @handler_id has to be a valid id of a signal handler that is 
+ *
+ * Undoes the effect of a previous g_signal_handler_block() call.  A
+ * blocked handler is skipped during signal emissions and will not be
+ * invoked, unblocking it (for exactly the amount of times it has been
+ * blocked before) reverts its "blocked" state, so the handler will be
+ * recognized by the signal system and is called upon future or
+ * currently ongoing signal emissions (since the order in which
+ * handlers are called during signal emissions is deterministic,
+ * whether the unblocked handler in question is called as part of a
+ * currently ongoing emission depends on how far that emission has
+ * proceeded yet).
+ *
+ * The @handler_id has to be a valid id of a signal handler that is
  * connected to a signal of @instance and is currently blocked.
  */
 void
@@ -2076,12 +2362,12 @@ g_signal_handler_unblock (gpointer instance,
  * g_signal_handler_disconnect:
  * @instance: The instance to remove the signal handler from.
  * @handler_id: Handler id of the handler to be disconnected.
- * 
- * Disconnects a handler from an instance so it will not be called during 
- * any future or currently ongoing emissions of the signal it has been 
+ *
+ * Disconnects a handler from an instance so it will not be called during
+ * any future or currently ongoing emissions of the signal it has been
  * connected to. The @handler_id becomes invalid and may be reused.
- * 
- * The @handler_id has to be a valid signal handler id, connected to a 
+ *
+ * The @handler_id has to be a valid signal handler id, connected to a
  * signal of @instance.
  */
 void
@@ -2111,9 +2397,9 @@ g_signal_handler_disconnect (gpointer instance,
  * g_signal_handler_is_connected:
  * @instance: The instance where a signal handler is sought.
  * @handler_id: the handler id.
- * 
+ *
  * Returns whether @handler_id is the id of a handler connected to @instance.
- * 
+ *
  * Returns: whether @handler_id identifies a handler connected to @instance.
  */
 gboolean
@@ -2178,20 +2464,20 @@ g_signal_handlers_destroy (gpointer instance)
 /**
  * g_signal_handler_find:
  * @instance: The instance owning the signal handler to be found.
- * @mask: Mask indicating which of @signal_id, @detail, @closure, @func 
+ * @mask: Mask indicating which of @signal_id, @detail, @closure, @func
  *  and/or @data the handler has to match.
  * @signal_id: Signal the handler has to be connected to.
  * @detail: Signal detail the handler has to be connected to.
  * @closure: The closure the handler will invoke.
  * @func: The C closure callback of the handler (useless for non-C closures).
  * @data: The closure data of the handler's closure.
- * 
+ *
  * Finds the first signal handler that matches certain selection criteria.
  * The criteria mask is passed as an OR-ed combination of #GSignalMatchType
  * flags, and the criteria values are passed as arguments.
  * The match @mask has to be non-0 for successful matches.
  * If no handler was found, 0 is returned.
- * 
+ *
  * Returns: A valid non-0 signal handler id for a successful match.
  */
 gulong
@@ -2258,22 +2544,22 @@ signal_handlers_foreach_matched_R (gpointer         instance,
 /**
  * g_signal_handlers_block_matched:
  * @instance: The instance to block handlers from.
- * @mask: Mask indicating which of @signal_id, @detail, @closure, @func 
+ * @mask: Mask indicating which of @signal_id, @detail, @closure, @func
  *  and/or @data the handlers have to match.
  * @signal_id: Signal the handlers have to be connected to.
  * @detail: Signal detail the handlers have to be connected to.
  * @closure: The closure the handlers will invoke.
  * @func: The C closure callback of the handlers (useless for non-C closures).
  * @data: The closure data of the handlers' closures.
- * 
+ *
  * Blocks all handlers on an instance that match a certain selection criteria.
- * The criteria mask is passed as an OR-ed combination of #GSignalMatchType 
+ * The criteria mask is passed as an OR-ed combination of #GSignalMatchType
  * flags, and the criteria values are passed as arguments.
  * Passing at least one of the %G_SIGNAL_MATCH_CLOSURE, %G_SIGNAL_MATCH_FUNC
  * or %G_SIGNAL_MATCH_DATA match flags is required for successful matches.
  * If no handlers were found, 0 is returned, the number of blocked handlers
  * otherwise.
- * 
+ *
  * Returns: The number of handlers that matched.
  */
 guint
@@ -2305,14 +2591,14 @@ g_signal_handlers_block_matched (gpointer         instance,
 /**
  * g_signal_handlers_unblock_matched:
  * @instance: The instance to unblock handlers from.
- * @mask: Mask indicating which of @signal_id, @detail, @closure, @func 
+ * @mask: Mask indicating which of @signal_id, @detail, @closure, @func
  *  and/or @data the handlers have to match.
  * @signal_id: Signal the handlers have to be connected to.
  * @detail: Signal detail the handlers have to be connected to.
  * @closure: The closure the handlers will invoke.
  * @func: The C closure callback of the handlers (useless for non-C closures).
  * @data: The closure data of the handlers' closures.
- * 
+ *
  * Unblocks all handlers on an instance that match a certain selection
  * criteria. The criteria mask is passed as an OR-ed combination of
  * #GSignalMatchType flags, and the criteria values are passed as arguments.
@@ -2321,7 +2607,7 @@ g_signal_handlers_block_matched (gpointer         instance,
  * If no handlers were found, 0 is returned, the number of unblocked handlers
  * otherwise. The match criteria should not apply to any handlers that are
  * not currently blocked.
- * 
+ *
  * Returns: The number of handlers that matched.
  */
 guint
@@ -2353,22 +2639,23 @@ g_signal_handlers_unblock_matched (gpointer         instance,
 /**
  * g_signal_handlers_disconnect_matched:
  * @instance: The instance to remove handlers from.
- * @mask: Mask indicating which of @signal_id, @detail, @closure, @func 
+ * @mask: Mask indicating which of @signal_id, @detail, @closure, @func
  *  and/or @data the handlers have to match.
  * @signal_id: Signal the handlers have to be connected to.
  * @detail: Signal detail the handlers have to be connected to.
  * @closure: The closure the handlers will invoke.
  * @func: The C closure callback of the handlers (useless for non-C closures).
  * @data: The closure data of the handlers' closures.
- * 
- * Disconnects all handlers on an instance that match a certain selection 
- * criteria. The criteria mask is passed as an OR-ed combination of
- * #GSignalMatchType flags, and the criteria values are passed as arguments.
- * Passing at least one of the %G_SIGNAL_MATCH_CLOSURE, %G_SIGNAL_MATCH_FUNC
- * or %G_SIGNAL_MATCH_DATA match flags is required for successful matches.
- * If no handlers were found, 0 is returned, the number of disconnected 
- * handlers otherwise.
- * 
+ *
+ * Disconnects all handlers on an instance that match a certain
+ * selection criteria. The criteria mask is passed as an OR-ed
+ * combination of #GSignalMatchType flags, and the criteria values are
+ * passed as arguments.  Passing at least one of the
+ * %G_SIGNAL_MATCH_CLOSURE, %G_SIGNAL_MATCH_FUNC or
+ * %G_SIGNAL_MATCH_DATA match flags is required for successful
+ * matches.  If no handlers were found, 0 is returned, the number of
+ * disconnected handlers otherwise.
+ *
  * Returns: The number of handlers that matched.
  */
 guint
@@ -2403,17 +2690,17 @@ g_signal_handlers_disconnect_matched (gpointer         instance,
  * @signal_id: the signal id.
  * @detail: the detail.
  * @may_be_blocked: whether blocked handlers should count as match.
- * 
+ *
  * Returns whether there are any handlers connected to @instance for the
  * given signal id and detail.
- * 
- * One example of when you might use this is when the arguments to the 
- * signal are difficult to compute. A class implementor may opt to not emit 
- * the signal if no one is attached anyway, thus saving the cost of building
- * the arguments.
- * 
- * Returns: %TRUE if a handler is connected to the signal, 
- *  %FALSE otherwise.
+ *
+ * One example of when you might use this is when the arguments to the
+ * signal are difficult to compute. A class implementor may opt to not
+ * emit the signal if no one is attached anyway, thus saving the cost
+ * of building the arguments.
+ *
+ * Returns: %TRUE if a handler is connected to the signal, %FALSE
+ *          otherwise.
  */
 gboolean
 g_signal_has_handler_pending (gpointer instance,
@@ -2500,16 +2787,16 @@ signal_check_skip_emission (SignalNode *node,
 
 /**
  * g_signal_emitv:
- * @instance_and_params: argument list for the signal emission. The first 
- *  element in the array is a #GValue for the instance the signal is 
- *  being emitted on. The rest are any arguments to be passed to the 
+ * @instance_and_params: argument list for the signal emission. The first
+ *  element in the array is a #GValue for the instance the signal is
+ *  being emitted on. The rest are any arguments to be passed to the
  *  signal.
  * @signal_id: the signal id
  * @detail: the detail
  * @return_value: Location to store the return value of the signal emission.
- * 
- * Emits a signal. 
- * 
+ *
+ * Emits a signal.
+ *
  * Note that g_signal_emitv() doesn't change @return_value if no handlers are
  * connected, in contrast to g_signal_emit() and g_signal_emit_valist().
  */
@@ -2609,9 +2896,9 @@ g_signal_emitv (const GValue *instance_and_params,
  * @var_args: a list of parameters to be passed to the signal, followed by a
  *  location for the return value. If the return type of the signal
  *  is #G_TYPE_NONE, the return value location can be omitted.
- * 
- * Emits a signal. 
- * 
+ *
+ * Emits a signal.
+ *
  * Note that g_signal_emit_valist() resets the return value to the default
  * if no handlers are connected, in contrast to g_signal_emitv().
  */
@@ -2621,7 +2908,7 @@ g_signal_emit_valist (gpointer instance,
                      GQuark   detail,
                      va_list  var_args)
 {
-  GValue *instance_and_params, stack_values[MAX_STACK_VALUES], *free_me = NULL;
+  GValue *instance_and_params;
   GType signal_return_type;
   GValue *param_values;
   SignalNode *node;
@@ -2658,27 +2945,20 @@ g_signal_emit_valist (gpointer instance,
 
   n_params = node->n_params;
   signal_return_type = node->return_type;
-  if (node->n_params < MAX_STACK_VALUES)
-    instance_and_params = stack_values;
-  else
-    {
-      free_me = g_new (GValue, node->n_params + 1);
-      instance_and_params = free_me;
-    }
+  instance_and_params = g_slice_alloc0 (sizeof (GValue) * (n_params + 1));
   param_values = instance_and_params + 1;
+
   for (i = 0; i < node->n_params; i++)
     {
       gchar *error;
       GType ptype = node->param_types[i] & ~G_SIGNAL_TYPE_STATIC_SCOPE;
       gboolean static_scope = node->param_types[i] & G_SIGNAL_TYPE_STATIC_SCOPE;
-      
-      param_values[i].g_type = 0;
+
       SIGNAL_UNLOCK ();
-      g_value_init (param_values + i, ptype);
-      G_VALUE_COLLECT (param_values + i,
-                      var_args,
-                      static_scope ? G_VALUE_NOCOPY_CONTENTS : 0,
-                      &error);
+      G_VALUE_COLLECT_INIT (param_values + i, ptype,
+                           var_args,
+                           static_scope ? G_VALUE_NOCOPY_CONTENTS : 0,
+                           &error);
       if (error)
        {
          g_warning ("%s: %s", G_STRLOC, error);
@@ -2690,7 +2970,7 @@ g_signal_emit_valist (gpointer instance,
          while (i--)
            g_value_unset (param_values + i);
 
-         g_free (free_me);
+         g_slice_free1 (sizeof (GValue) * (n_params + 1), instance_and_params);
          return;
        }
       SIGNAL_LOCK ();
@@ -2731,8 +3011,7 @@ g_signal_emit_valist (gpointer instance,
   for (i = 0; i < n_params; i++)
     g_value_unset (param_values + i);
   g_value_unset (instance_and_params);
-  if (free_me)
-    g_free (free_me);
+  g_slice_free1 (sizeof (GValue) * (n_params + 1), instance_and_params);
 }
 
 /**
@@ -2743,9 +3022,9 @@ g_signal_emit_valist (gpointer instance,
  * @...: parameters to be passed to the signal, followed by a
  *  location for the return value. If the return type of the signal
  *  is #G_TYPE_NONE, the return value location can be omitted.
- * 
- * Emits a signal. 
- * 
+ *
+ * Emits a signal.
+ *
  * Note that g_signal_emit() resets the return value to the default
  * if no handlers are connected, in contrast to g_signal_emitv().
  */
@@ -2769,9 +3048,9 @@ g_signal_emit (gpointer instance,
  * @...: parameters to be passed to the signal, followed by a
  *  location for the return value. If the return type of the signal
  *  is #G_TYPE_NONE, the return value location can be omitted.
- * 
- * Emits a signal. 
- * 
+ *
+ * Emits a signal.
+ *
  * Note that g_signal_emit_by_name() resets the return value to the default
  * if no handlers are connected, in contrast to g_signal_emitv().
  */
@@ -2847,7 +3126,9 @@ signal_emit_unlocked_R (SignalNode   *node,
        G_BREAKPOINT ();
     }
 #endif /* G_ENABLE_DEBUG */
-  
+
+  TRACE(GOBJECT_SIGNAL_EMIT(node->signal_id, detail, instance, G_TYPE_FROM_INSTANCE (instance)));
+
   SIGNAL_LOCK ();
   signal_id = node->signal_id;
   if (node->flags & G_SIGNAL_NO_RECURSE)
@@ -3105,7 +3386,9 @@ signal_emit_unlocked_R (SignalNode   *node,
   SIGNAL_UNLOCK ();
   if (accumulator)
     g_value_unset (&accu);
-  
+
+  TRACE(GOBJECT_SIGNAL_EMIT_END(node->signal_id, detail, instance, G_TYPE_FROM_INSTANCE (instance)));
+
   return return_value_altered;
 }
 
@@ -3127,7 +3410,7 @@ type_debug_name (GType type)
  * @return_accu: standard #GSignalAccumulator parameter
  * @handler_return: standard #GSignalAccumulator parameter
  * @dummy: standard #GSignalAccumulator parameter
- * 
+ *
  * A predefined #GSignalAccumulator for signals that return a
  * boolean values. The behavior that this accumulator gives is
  * that a return of %TRUE stops the signal emission: no further
@@ -3135,8 +3418,9 @@ type_debug_name (GType type)
  * the emission to coninue. The idea here is that a %TRUE return
  * indicates that the callback <emphasis>handled</emphasis> the signal,
  * and no further handling is needed.
- * 
+ *
  * Since: 2.4
+ *
  * Returns: standard #GSignalAccumulator result
  */
 gboolean
@@ -3156,7 +3440,4 @@ g_signal_accumulator_true_handled (GSignalInvocationHint *ihint,
 }
 
 /* --- compile standard marshallers --- */
-#include        "gmarshal.c"
-
-#define __G_SIGNAL_C__
-#include "gobjectaliasdef.c"
+#include "gmarshal.c"