From 91e8b0bb9122d35c74a343c64d59ab33bd006476 Mon Sep 17 00:00:00 2001
From: Thomas Vander Stichele <thomas@apestaart.org>
Date: Thu, 29 Sep 2005 18:35:38 +0000
Subject: [PATCH] gst/gstobject.c: for 2.6 refcounting, make debug log more
 correct by printing the actual refcounts at the time of swa...

Original commit message from CVS:

* gst/gstobject.c:
for 2.6 refcounting, make debug log more correct by printing
the actual refcounts at the time of swap (Wim)
---
 ChangeLog       |  6 ++++++
 common          |  2 +-
 gst/gstobject.c | 50 ++++++++++++++++++++++++--------------------------
 3 files changed, 31 insertions(+), 27 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index be54db3..4222476 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2005-09-29  Thomas Vander Stichele  <thomas at apestaart dot org>
+
+	* gst/gstobject.c:
+	  for 2.6 refcounting, make debug log more correct by printing
+	  the actual refcounts at the time of swap (Wim)
+
 2005-09-29  Andy Wingo  <wingo@pobox.com>
 
 	* gst/gstbus.c (gst_bus_remove_signal_watch): New function,
diff --git a/common b/common
index 5cd4217..e6246e8 160000
--- a/common
+++ b/common
@@ -1 +1 @@
-Subproject commit 5cd4217979c25fc1fcbb82282e4ad6cbdea9e6cf
+Subproject commit e6246e87e86c75b2c9c00d0748f0cd2332295eaa
diff --git a/gst/gstobject.c b/gst/gstobject.c
index 1157755..4e3e47c 100644
--- a/gst/gstobject.c
+++ b/gst/gstobject.c
@@ -357,25 +357,25 @@ gst_object_constructor (GType type, guint n_construct_properties,
 gpointer
 gst_object_ref (gpointer object)
 {
+#ifdef REFCOUNT_HACK
+  gint old;
+#endif
+
   g_return_val_if_fail (GST_IS_OBJECT (object), NULL);
 
-#ifdef DEBUG_REFCOUNT
 #ifdef REFCOUNT_HACK
+  old = g_atomic_int_exchange_and_add (&((GstObject *) object)->refcount, 1);
+#ifdef DEBUG_REFCOUNT
   GST_CAT_LOG_OBJECT (GST_CAT_REFCOUNTING, object, "%p ref %d->%d",
-      object,
-      GST_OBJECT_REFCOUNT_VALUE (object),
-      GST_OBJECT_REFCOUNT_VALUE (object) + 1);
+      object, old, old + 1);
+#endif
+  PATCH_REFCOUNT (object);
 #else
+#ifdef DEBUG_REFCOUNT
   GST_CAT_LOG_OBJECT (GST_CAT_REFCOUNTING, object, "%p ref %d->%d",
       object,
       ((GObject *) object)->ref_count, ((GObject *) object)->ref_count + 1);
 #endif
-#endif
-
-#ifdef REFCOUNT_HACK
-  g_atomic_int_inc (&((GstObject *) object)->refcount);
-  PATCH_REFCOUNT (object);
-#else
   g_object_ref (object);
 #endif
 
@@ -396,36 +396,34 @@ gst_object_ref (gpointer object)
 void
 gst_object_unref (gpointer object)
 {
+#ifdef REFCOUNT_HACK
+  gint old;
+#endif
   g_return_if_fail (GST_IS_OBJECT (object));
 
 #ifdef REFCOUNT_HACK
   g_return_if_fail (GST_OBJECT_REFCOUNT_VALUE (object) > 0);
-#else
-  g_return_if_fail (((GObject *) object)->ref_count > 0);
-#endif
+
+  old = g_atomic_int_exchange_and_add (&((GstObject *) object)->refcount, -1);
 
 #ifdef DEBUG_REFCOUNT
-#ifdef REFCOUNT_HACK
   GST_CAT_LOG_OBJECT (GST_CAT_REFCOUNTING, object, "%p unref %d->%d",
-      object,
-      GST_OBJECT_REFCOUNT_VALUE (object),
-      GST_OBJECT_REFCOUNT_VALUE (object) - 1);
-#else
-  GST_CAT_LOG_OBJECT (GST_CAT_REFCOUNTING, object, "%p unref %d->%d",
-      object,
-      ((GObject *) object)->ref_count, ((GObject *) object)->ref_count - 1);
-#endif
+      object, old, old - 1);
 #endif
-
-#ifdef REFCOUNT_HACK
-  if (G_UNLIKELY (g_atomic_int_dec_and_test (&((GstObject *) object)->
-              refcount))) {
+  if (G_UNLIKELY (old == 1)) {
     PATCH_REFCOUNT1 (object);
     g_object_unref (object);
   } else {
     PATCH_REFCOUNT (object);
   }
 #else
+  g_return_if_fail (((GObject *) object)->ref_count > 0);
+
+#ifdef DEBUG_REFCOUNT
+  GST_CAT_LOG_OBJECT (GST_CAT_REFCOUNTING, object, "%p unref %d->%d",
+      object,
+      ((GObject *) object)->ref_count, ((GObject *) object)->ref_count - 1);
+#endif
   g_object_unref (object);
 #endif
 }
-- 
2.7.4