summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMatthias Clasen <mclasen@redhat.com>2013-12-02 22:26:48 -0500
committerMatthias Clasen <mclasen@redhat.com>2013-12-03 06:00:47 -0500
commit2baa50ee4fde91f89692e00b1fc4fa39713be799 (patch)
tree59da81bf6a87e293db5842e12b776324109e8508
parent28c2706da73d7dddb12d336b26a08218e761817c (diff)
downloadglib-2baa50ee4fde91f89692e00b1fc4fa39713be799.tar.gz
Remove g_trap_instance_signals as well
This is another crude conditional breakpoint mechanism and can be done better with actual conditional breakpoints or with systemtap tracepoints. https://bugzilla.gnome.org/show_bug.cgi?id=719687
-rw-r--r--gobject/gsignal.c34
1 files changed, 2 insertions, 32 deletions
diff --git a/gobject/gsignal.c b/gobject/gsignal.c
index ebdf8d901..831ded3cc 100644
--- a/gobject/gsignal.c
+++ b/gobject/gsignal.c
@@ -105,14 +105,6 @@
#define REPORT_BUG "please report occurrence circumstances to gtk-devel-list@gnome.org"
-#ifdef G_ENABLE_DEBUG
-#define COND_DEBUG(debug_type, cond) ((_g_type_debug_flags & G_TYPE_DEBUG_ ## debug_type) || (cond))
-#define IF_DEBUG(debug_type, cond) if (COND_DEBUG(debug_type, cond))
-
-static volatile gpointer g_trace_instance_signals = NULL;
-static volatile gpointer g_trap_instance_signals = NULL;
-#endif /* G_ENABLE_DEBUG */
-
/* --- typedefs --- */
typedef struct _SignalNode SignalNode;
@@ -3042,12 +3034,7 @@ g_signal_emitv (const GValue *instance_and_params,
if (node->single_va_closure != NULL &&
(node->single_va_closure == SINGLE_VA_CLOSURE_EMPTY_MAGIC ||
- _g_closure_is_void (node->single_va_closure, instance))
-#ifdef G_ENABLE_DEBUG
- && !COND_DEBUG (SIGNALS, g_trace_instance_signals != instance &&
- g_trap_instance_signals == instance)
-#endif /* G_ENABLE_DEBUG */
- )
+ _g_closure_is_void (node->single_va_closure, instance)))
{
HandlerList* hlist = handler_list_lookup (node->signal_id, instance);
if (hlist == NULL || hlist->handlers == NULL)
@@ -3129,12 +3116,7 @@ g_signal_emit_valist (gpointer instance,
if (!node->single_va_closure_is_valid)
node_update_single_va_closure (node);
- if (node->single_va_closure != NULL
-#ifdef G_ENABLE_DEBUG
- && !COND_DEBUG (SIGNALS, g_trace_instance_signals != instance &&
- g_trap_instance_signals == instance)
-#endif /* G_ENABLE_DEBUG */
- )
+ if (node->single_va_closure != NULL)
{
HandlerList* hlist = handler_list_lookup (node->signal_id, instance);
Handler *fastpath_handler = NULL;
@@ -3448,18 +3430,6 @@ signal_emit_unlocked_R (SignalNode *node,
gulong max_sequential_handler_number;
gboolean return_value_altered = FALSE;
-#ifdef G_ENABLE_DEBUG
- IF_DEBUG (SIGNALS, g_trace_instance_signals == instance || g_trap_instance_signals == instance)
- {
- g_message ("%s::%s(%u) emitted (instance=%p, signal-node=%p)",
- g_type_name (G_TYPE_FROM_INSTANCE (instance)),
- node->name, detail,
- instance, node);
- if (g_trap_instance_signals == instance)
- G_BREAKPOINT ();
- }
-#endif /* G_ENABLE_DEBUG */
-
TRACE(GOBJECT_SIGNAL_EMIT(node->signal_id, detail, instance, G_TYPE_FROM_INSTANCE (instance)));
SIGNAL_LOCK ();