diff options
author | Tim-Philipp Müller <tim@centricular.net> | 2006-01-25 22:05:28 +0000 |
---|---|---|
committer | Tim-Philipp Müller <tim@centricular.net> | 2006-01-25 22:05:28 +0000 |
commit | 916d083e9979df22327c4ae387444e24aefc052b (patch) | |
tree | 017f3ce4b9a972d35b349b7d4e4de12b74e3f3b7 /gst/id3demux | |
parent | 6021f2b54fcaec008fb5488bb521b5c5f089211a (diff) |
gst/id3demux/gstid3demux.c: Don't put function calls in g_return_if_fail() statements, or they'll be replaced with NO...
Original commit message from CVS:
* gst/id3demux/gstid3demux.c: (gst_id3demux_remove_srcpad):
Don't put function calls in g_return_if_fail() statements,
or they'll be replaced with NOOPs if someone compiles with
G_DISABLE_CHECKS defined.
Diffstat (limited to 'gst/id3demux')
-rw-r--r-- | gst/id3demux/gstid3demux.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/gst/id3demux/gstid3demux.c b/gst/id3demux/gstid3demux.c index 4f24b226..10dfcf29 100644 --- a/gst/id3demux/gstid3demux.c +++ b/gst/id3demux/gstid3demux.c @@ -291,15 +291,17 @@ gst_id3demux_add_srcpad (GstID3Demux * id3demux, GstCaps * new_caps) static gboolean gst_id3demux_remove_srcpad (GstID3Demux * id3demux) { + gboolean res = TRUE; + if (id3demux->srcpad != NULL) { GST_DEBUG_OBJECT (id3demux, "Removing src pad"); - g_return_val_if_fail (gst_element_remove_pad (GST_ELEMENT (id3demux), - id3demux->srcpad), FALSE); + res = gst_element_remove_pad (GST_ELEMENT (id3demux), id3demux->srcpad); + g_return_val_if_fail (res != FALSE, FALSE); gst_object_unref (id3demux->srcpad); id3demux->srcpad = NULL; } - return TRUE; + return res; }; static gboolean |