gst/ffmpegcolorspace/gstffmpegcolorspace.c: No need to take the STREAM lock anymore.

Original commit message from CVS:
* gst/ffmpegcolorspace/gstffmpegcolorspace.c:
(gst_ffmpegcsp_getcaps), (gst_ffmpegcsp_configure_context),
(gst_ffmpegcsp_setcaps), (gst_ffmpegcsp_bufferalloc),
(gst_ffmpegcsp_chain), (gst_ffmpegcsp_change_state):
No need to take the STREAM lock anymore.

diff --git a/ChangeLog b/ChangeLog
index 7e2c0c191ec9f4d444abcddfb734a86920c35d92..8e85eec5f6afacd684cc2de67b6d45cddd13958e 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2005-05-25  Wim Taymans  <wim@fluendo.com>
+
+       * gst/ffmpegcolorspace/gstffmpegcolorspace.c:
+       (gst_ffmpegcsp_getcaps), (gst_ffmpegcsp_configure_context),
+       (gst_ffmpegcsp_setcaps), (gst_ffmpegcsp_bufferalloc),
+       (gst_ffmpegcsp_chain), (gst_ffmpegcsp_change_state):
+       No need to take the STREAM lock anymore.
+
 2005-05-25  Wim Taymans  <wim@fluendo.com>
 
        * ext/ogg/gstoggdemux.c: (gst_ogg_pad_dispose),

diff --git a/gst/ffmpegcolorspace/gstffmpegcolorspace.c b/gst/ffmpegcolorspace/gstffmpegcolorspace.c
index 9c247a5d951a8866dfc0bd51b3c41a7402a73c20..3778d873725e403b579b91430bf70ea15d9f6448 100644 (file)
--- a/gst/ffmpegcolorspace/gstffmpegcolorspace.c
+++ b/gst/ffmpegcolorspace/gstffmpegcolorspace.c
@@ -412,8 +412,6 @@ gst_ffmpegcsp_chain (GstPad * pad, GstBuffer * buffer)
 
   space = GST_FFMPEGCSP (GST_PAD_PARENT (pad));
 
-  GST_STREAM_LOCK (pad);
-
   GST_DEBUG ("from %d -> to %d", space->from_pixfmt, space->to_pixfmt);
   if (space->from_pixfmt == PIX_FMT_NB || space->to_pixfmt == PIX_FMT_NB)
     goto unkown_format;
@@ -460,20 +458,17 @@ gst_ffmpegcsp_chain (GstPad * pad, GstBuffer * buffer)
   }
 
   res = gst_pad_push (space->srcpad, outbuf);
-  GST_STREAM_UNLOCK (pad);
 
   return res;
 
   /* ERRORS */
 no_buffer:
   {
-    GST_STREAM_UNLOCK (pad);
     gst_buffer_unref (buffer);
     return GST_FLOW_ERROR;
   }
 unkown_format:
   {
-    GST_STREAM_UNLOCK (pad);
     GST_ELEMENT_ERROR (space, CORE, NOT_IMPLEMENTED, (NULL),
         ("attempting to convert colorspaces between unknown formats"));
     gst_buffer_unref (buffer);
@@ -500,11 +495,9 @@ gst_ffmpegcsp_change_state (GstElement * element)
 
   switch (transition) {
     case GST_STATE_PAUSED_TO_READY:
-      GST_STREAM_LOCK (space->sinkpad);
       if (space->palette)
         av_free (space->palette);
       space->palette = NULL;
-      GST_STREAM_UNLOCK (space->sinkpad);
       break;
     default:
       break;