Revert "pad: Don't consider flushing pads as needing reconfiguration"
authorSebastian Dröge <slomo@circular-chaos.org>
Mon, 15 Jul 2013 09:32:54 +0000 (11:32 +0200)
committerSebastian Dröge <slomo@circular-chaos.org>
Mon, 15 Jul 2013 09:32:54 +0000 (11:32 +0200)
This reverts commit 948a9d2f2b728f5fb60be45d47a818cebeb60c7d.

This is racy and trying to reconfigure and fail is still better
than not trying to reconfigure at all.

https://bugzilla.gnome.org/show_bug.cgi?id=704100

gst/gstpad.c

index e1f386a..5c7be59 100644 (file)
@@ -1400,13 +1400,8 @@ gst_pad_needs_reconfigure (GstPad * pad)
   g_return_val_if_fail (GST_IS_PAD (pad), FALSE);
 
   GST_OBJECT_LOCK (pad);
-  if (GST_PAD_IS_FLUSHING (pad)) {
-    GST_DEBUG_OBJECT (pad, "pad is flushing");
-    reconfigure = FALSE;
-  } else {
-    reconfigure = GST_PAD_NEEDS_RECONFIGURE (pad);
-    GST_DEBUG_OBJECT (pad, "peeking RECONFIGURE flag %d", reconfigure);
-  }
+  reconfigure = GST_PAD_NEEDS_RECONFIGURE (pad);
+  GST_DEBUG_OBJECT (pad, "peeking RECONFIGURE flag %d", reconfigure);
   GST_OBJECT_UNLOCK (pad);
 
   return reconfigure;
@@ -1429,15 +1424,10 @@ gst_pad_check_reconfigure (GstPad * pad)
   g_return_val_if_fail (GST_IS_PAD (pad), FALSE);
 
   GST_OBJECT_LOCK (pad);
-  if (GST_PAD_IS_FLUSHING (pad)) {
-    GST_DEBUG_OBJECT (pad, "pad is flushing");
-    reconfigure = FALSE;
-  } else {
-    reconfigure = GST_PAD_NEEDS_RECONFIGURE (pad);
-    if (reconfigure) {
-      GST_DEBUG_OBJECT (pad, "remove RECONFIGURE flag");
-      GST_OBJECT_FLAG_UNSET (pad, GST_PAD_FLAG_NEED_RECONFIGURE);
-    }
+  reconfigure = GST_PAD_NEEDS_RECONFIGURE (pad);
+  if (reconfigure) {
+    GST_DEBUG_OBJECT (pad, "remove RECONFIGURE flag");
+    GST_OBJECT_FLAG_UNSET (pad, GST_PAD_FLAG_NEED_RECONFIGURE);
   }
   GST_OBJECT_UNLOCK (pad);