From 7bc79aec0e5e1d3a4d4ed6482a9a4b06294c6e82 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Tim-Philipp=20M=C3=BCller?= Date: Fri, 3 Feb 2006 19:23:41 +0000 Subject: [PATCH] gst/playback/gstdecodebin.c: Don't put essential function call into g_return_*() macro, otherwise it'll all be replac... Original commit message from CVS: * gst/playback/gstdecodebin.c: (try_to_link_1): Don't put essential function call into g_return_*() macro, otherwise it'll all be replaced by NOOPs when compiling with G_DISABLE_CHECKS defined. --- ChangeLog | 8 ++++++++ gst/playback/gstdecodebin.c | 6 ++++-- 2 files changed, 12 insertions(+), 2 deletions(-) diff --git a/ChangeLog b/ChangeLog index 4cb5e03f9a..059fb0948e 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,11 @@ +2006-02-03 Tim-Philipp Müller + + * gst/playback/gstdecodebin.c: (try_to_link_1): + Don't put essential function call into + g_return_*() macro, otherwise it'll all be + replaced by NOOPs when compiling with + G_DISABLE_CHECKS defined. + 2006-02-03 Edgard Lima * ext/ogg/gstoggdemux.c: diff --git a/gst/playback/gstdecodebin.c b/gst/playback/gstdecodebin.c index 883d40180a..3631e56dc4 100644 --- a/gst/playback/gstdecodebin.c +++ b/gst/playback/gstdecodebin.c @@ -729,6 +729,8 @@ try_to_link_1 (GstDecodeBin * decode_bin, GstElement * srcelement, GstPad * pad, isdemux = is_demuxer_element (srcelement); if (isdemux && factories != NULL) { + GstPadLinkReturn dqlink; + /* Insert a queue between demuxer and decoder */ GST_DEBUG_OBJECT (decode_bin, "Element %s is a demuxer, inserting a queue", @@ -745,8 +747,8 @@ try_to_link_1 (GstDecodeBin * decode_bin, GstElement * srcelement, GstPad * pad, queuesinkpad = gst_element_get_pad (queue, "sink"); usedsrcpad = queuesrcpad = gst_element_get_pad (queue, "src"); - g_return_val_if_fail (gst_pad_link (pad, queuesinkpad) == GST_PAD_LINK_OK, - NULL); + dqlink = gst_pad_link (pad, queuesinkpad); + g_return_val_if_fail (dqlink == GST_PAD_LINK_OK, NULL); } /* loop over the factories */