From 6c323f5b0d8a82aa88c10f62cd8a2b824a08db69 Mon Sep 17 00:00:00 2001 From: Tim-Philipp Müller Date: Sun, 2 Aug 2009 11:44:03 +0100 Subject: multiudpsink: don't do things with side-effects inside g_return_val_if_fail() Someone might compile this code with -DG_DISABLE_ASSERT some day. --- gst/udp/gstmultiudpsink.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) (limited to 'gst/udp') diff --git a/gst/udp/gstmultiudpsink.c b/gst/udp/gstmultiudpsink.c index 68eea75c..a6afc9ae 100644 --- a/gst/udp/gstmultiudpsink.c +++ b/gst/udp/gstmultiudpsink.c @@ -117,7 +117,7 @@ static void gst_multiudpsink_finalize (GObject * object); static GstFlowReturn gst_multiudpsink_render (GstBaseSink * sink, GstBuffer * buffer); -#ifndef G_OS_WIN32 /*sendmsg() is not available on Windows*/ +#ifndef G_OS_WIN32 /* sendmsg() is not available on Windows */ static GstFlowReturn gst_multiudpsink_render_list (GstBaseSink * bsink, GstBufferList * list); #endif @@ -451,8 +451,9 @@ gst_multiudpsink_render_list (GstBaseSink * bsink, GstBufferList * list) sink = GST_MULTIUDPSINK (bsink); g_return_val_if_fail (list != NULL, GST_FLOW_ERROR); - g_return_val_if_fail ((it = gst_buffer_list_iterate (list)) != NULL, - GST_FLOW_ERROR); + + it = gst_buffer_list_iterate (list); + g_return_val_if_fail (it != NULL, GST_FLOW_ERROR); while (gst_buffer_list_iterator_next_group (it)) { msg.msg_iovlen = 0; -- cgit