rtsp-stream: Remove unused _locked() variant of a function
authorSebastian Dröge <sebastian@centricular.com>
Wed, 7 Sep 2016 15:44:34 +0000 (18:44 +0300)
committerSebastian Dröge <sebastian@centricular.com>
Wed, 7 Sep 2016 15:44:34 +0000 (18:44 +0300)
It was added during refactoring.

gst/rtsp-server/rtsp-stream.c

index 0ca31f9..2d3997c 100644 (file)
@@ -913,9 +913,21 @@ gst_rtsp_stream_get_multicast_iface (GstRTSPStream * stream)
   return result;
 }
 
-
-static GstRTSPAddress *
-gst_rtsp_stream_get_multicast_address_locked (GstRTSPStream * stream,
+/**
+ * gst_rtsp_stream_get_multicast_address:
+ * @stream: a #GstRTSPStream
+ * @family: the #GSocketFamily
+ *
+ * Get the multicast address of @stream for @family. The original
+ * #GstRTSPAddress is cached and copy is returned, so freeing the return value
+ * won't release the address from the pool.
+ *
+ * Returns: (transfer full) (nullable): the #GstRTSPAddress of @stream
+ * or %NULL when no address could be allocated. gst_rtsp_address_free()
+ * after usage.
+ */
+GstRTSPAddress *
+gst_rtsp_stream_get_multicast_address (GstRTSPStream * stream,
     GSocketFamily family)
 {
   GstRTSPStreamPrivate *priv;
@@ -923,8 +935,12 @@ gst_rtsp_stream_get_multicast_address_locked (GstRTSPStream * stream,
   GstRTSPAddress **addrp;
   GstRTSPAddressFlags flags;
 
+  g_return_val_if_fail (GST_IS_RTSP_STREAM (stream), NULL);
+
   priv = stream->priv;
 
+  g_mutex_lock (&stream->priv->lock);
+
   if (family == G_SOCKET_FAMILY_IPV6) {
     flags = GST_RTSP_ADDRESS_FLAG_IPV6;
     addrp = &priv->mcast_addr_v6;
@@ -951,50 +967,26 @@ gst_rtsp_stream_get_multicast_address_locked (GstRTSPStream * stream,
   }
   result = gst_rtsp_address_copy (*addrp);
 
+  g_mutex_unlock (&stream->priv->lock);
+
   return result;
 
   /* ERRORS */
 no_pool:
   {
     GST_ERROR_OBJECT (stream, "no address pool specified");
+    g_mutex_unlock (&stream->priv->lock);
     return NULL;
   }
 no_address:
   {
     GST_ERROR_OBJECT (stream, "failed to acquire address from pool");
+    g_mutex_unlock (&stream->priv->lock);
     return NULL;
   }
 }
 
 /**
- * gst_rtsp_stream_get_multicast_address:
- * @stream: a #GstRTSPStream
- * @family: the #GSocketFamily
- *
- * Get the multicast address of @stream for @family. The original
- * #GstRTSPAddress is cached and copy is returned, so freeing the return value
- * won't release the address from the pool.
- *
- * Returns: (transfer full) (nullable): the #GstRTSPAddress of @stream
- * or %NULL when no address could be allocated. gst_rtsp_address_free()
- * after usage.
- */
-GstRTSPAddress *
-gst_rtsp_stream_get_multicast_address (GstRTSPStream * stream,
-    GSocketFamily family)
-{
-  GstRTSPAddress *result;
-
-  g_return_val_if_fail (GST_IS_RTSP_STREAM (stream), NULL);
-
-  g_mutex_lock (&stream->priv->lock);
-  result = gst_rtsp_stream_get_multicast_address_locked (stream, family);
-  g_mutex_unlock (&stream->priv->lock);
-
-  return result;
-}
-
-/**
  * gst_rtsp_stream_reserve_address:
  * @stream: a #GstRTSPStream
  * @address: an address