decodebin2: Minor debug fix for decodepad
authorEdward Hervey <edward@centricular.com>
Thu, 17 Nov 2022 14:41:35 +0000 (15:41 +0100)
committerEdward Hervey <bilboed@bilboed.com>
Fri, 18 Nov 2022 07:22:23 +0000 (07:22 +0000)
decodedad might have their name changed when exposing, causing a race when
trying to get their name without taking a lock. Just use GST_PTR_POINTER in
debug statements instead

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/3428>

subprojects/gst-plugins-base/gst/playback/gstdecodebin2.c

index 532824f..7dcad54 100644 (file)
@@ -5015,7 +5015,7 @@ source_pad_event_probe (GstPad * pad, GstPadProbeInfo * info,
   GstDecodePad *dpad = user_data;
   gboolean res = TRUE;
 
-  GST_LOG_OBJECT (pad, "%s dpad:%p", GST_EVENT_TYPE_NAME (event), dpad);
+  GST_LOG_OBJECT (pad, "event %s", GST_EVENT_TYPE_NAME (event));
 
   if (GST_EVENT_TYPE (event) == GST_EVENT_EOS) {
     GST_DEBUG_OBJECT (pad, "we received EOS");
@@ -5162,8 +5162,8 @@ gst_decode_pad_query (GstPad * pad, GstObject * parent, GstQuery * query)
 
     ret = FALSE;
     GST_DEBUG_OBJECT (dpad->dbin,
-        "calling autoplug-query for %s (element %s): %" GST_PTR_FORMAT,
-        GST_PAD_NAME (dpad), GST_ELEMENT_NAME (delem->element), query);
+        "calling autoplug-query for %" GST_PTR_FORMAT " (element %s): %"
+        GST_PTR_FORMAT, dpad, GST_ELEMENT_NAME (delem->element), query);
     g_signal_emit (G_OBJECT (dpad->dbin),
         gst_decode_bin_signals[SIGNAL_AUTOPLUG_QUERY], 0, dpad, delem->element,
         query, &ret);