gst/base/gstadapter.c: Flushing out 0 bytes is ok for this function.
authorWim Taymans <wim.taymans@gmail.com>
Tue, 30 Aug 2005 19:29:59 +0000 (19:29 +0000)
committerWim Taymans <wim.taymans@gmail.com>
Tue, 30 Aug 2005 19:29:59 +0000 (19:29 +0000)
Original commit message from CVS:
* gst/base/gstadapter.c: (gst_adapter_flush):
Flushing out 0 bytes is ok for this function.

* gst/base/gstbasesink.c: (gst_base_sink_handle_object):
no newsegment gives a warning and sets the start/stop to
invalid.

* gst/base/gstbasetransform.c: (gst_base_transform_change_state),
(gst_base_transform_set_passthrough):
Some debug info.

* gst/gstminiobject.c: (gst_mini_object_ref):
Check refcount here too.

* gst/gstpad.c: (gst_pad_init):
Pads are initially flushing and refusing data.

* gst/gstutils.c: (gst_element_link_pads_filtered):
When adding a capsfilter element make sure it has the
same state as the parent bin.

ChangeLog
common
gst/base/gstadapter.c
gst/base/gstbasesink.c
gst/base/gstbasetransform.c
gst/gstminiobject.c
gst/gstpad.c
gst/gstutils.c
libs/gst/base/gstadapter.c
libs/gst/base/gstbasesink.c
libs/gst/base/gstbasetransform.c

index a3a95c28a20f54fed1fca1609544e808e9964720..beb642755495e7f0eeae18c7c44a3cd4484995d6 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,26 @@
+2005-08-30  Wim Taymans  <wim@fluendo.com>
+
+       * gst/base/gstadapter.c: (gst_adapter_flush):
+       Flushing out 0 bytes is ok for this function.
+
+       * gst/base/gstbasesink.c: (gst_base_sink_handle_object):
+       no newsegment gives a warning and sets the start/stop to 
+       invalid.
+
+       * gst/base/gstbasetransform.c: (gst_base_transform_change_state),
+       (gst_base_transform_set_passthrough):
+       Some debug info.
+
+       * gst/gstminiobject.c: (gst_mini_object_ref):
+       Check refcount here too.
+
+       * gst/gstpad.c: (gst_pad_init):
+       Pads are initially flushing and refusing data.
+
+       * gst/gstutils.c: (gst_element_link_pads_filtered):
+       When adding a capsfilter element make sure it has the
+       same state as the parent bin.
+
 2005-08-30  Stefan Kost  <ensonic@users.sf.net>
 
        * docs/gst/tmpl/.cvsignore:
diff --git a/common b/common
index 89d82b662da00096b70e70a7a102b294acaae6dd..3fb3bedc9180bab9dc8c2dc784c9327bd1cc8109 160000 (submodule)
--- a/common
+++ b/common
@@ -1 +1 @@
-Subproject commit 89d82b662da00096b70e70a7a102b294acaae6dd
+Subproject commit 3fb3bedc9180bab9dc8c2dc784c9327bd1cc8109
index 94104de00979a2ea600ea5aafb7f0884d20dd42a..c8c64c26e83f32eabe872a8a498eb23bb5885b85 100644 (file)
@@ -236,7 +236,7 @@ gst_adapter_flush (GstAdapter * adapter, guint flush)
   GstBuffer *cur;
 
   g_return_if_fail (GST_IS_ADAPTER (adapter));
-  g_return_if_fail (flush > 0);
+  g_return_if_fail (flush >= 0);
   g_return_if_fail (flush <= adapter->size);
 
   GST_LOG_OBJECT (adapter, "flushing %u bytes", flush);
index 78dc662e14e9362615f72caaada56958342cdb1f..0b25128ae8de58ea519efe6343dcda2e2fff29c6 100644 (file)
@@ -483,13 +483,13 @@ gst_base_sink_handle_object (GstBaseSink * basesink, GstPad * pad,
     GstBuffer *buf = GST_BUFFER (obj);
 
     if (!basesink->have_newsegment) {
-      GST_ELEMENT_ERROR (basesink, STREAM, STOPPED,
+      GST_ELEMENT_WARNING (basesink, STREAM, STOPPED,
           ("Received buffer without a new-segment. Cannot sync to clock."),
           ("Received buffer without a new-segment. Cannot sync to clock."));
       basesink->have_newsegment = TRUE;
       /* this means this sink will not be able to sync to the clock */
-      basesink->segment_start = 0;
-      basesink->segment_stop = 0;
+      basesink->segment_start = -1;
+      basesink->segment_stop = -1;
     }
 
     /* check if the buffer needs to be dropped */
index 300fe87b9cbcbcd725b3c1d49f4909b931226b4b..2294853a6bf251867c82d8553b2d74144bc7f4ff 100644 (file)
@@ -1046,6 +1046,7 @@ gst_base_transform_change_state (GstElement * element)
             GST_PAD_CAPS (trans->srcpad)) || trans->passthrough;
       else
         trans->in_place = trans->passthrough;
+      GST_DEBUG_OBJECT (trans, "in_place %d", trans->in_place);
       gst_caps_replace (&trans->cache_caps1, NULL);
       gst_caps_replace (&trans->cache_caps2, NULL);
       GST_UNLOCK (trans);
@@ -1090,6 +1091,8 @@ gst_base_transform_set_passthrough (GstBaseTransform * trans,
 {
   g_return_if_fail (trans != NULL);
 
+  GST_DEBUG_OBJECT (trans, "setting passthrough %d", passthrough);
+
   GST_LOCK (trans);
   trans->passthrough = passthrough;
   GST_UNLOCK (trans);
index 02ba2396686c7ac207f7b2d690680dad3efde150..57a8bf444599ef0cec8220886807ad0b381d9448 100644 (file)
@@ -190,6 +190,7 @@ GstMiniObject *
 gst_mini_object_ref (GstMiniObject * mini_object)
 {
   g_return_val_if_fail (mini_object != NULL, NULL);
+  g_return_val_if_fail (mini_object->refcount > 0, NULL);
 
 #ifdef DEBUG_REFCOUNT
   GST_CAT_LOG (GST_CAT_REFCOUNTING, "%p ref %d->%d",
index 20a654d62d3c3084db79c575d7bcf444ca0313de..04bd49412f01f73baed127957ec9c7d4bf639a50 100644 (file)
@@ -247,7 +247,7 @@ gst_pad_init (GstPad * pad)
   pad->do_buffer_signals = 0;
   pad->do_event_signals = 0;
 
-  GST_PAD_UNSET_FLUSHING (pad);
+  GST_PAD_SET_FLUSHING (pad);
 
   pad->preroll_lock = g_mutex_new ();
   pad->preroll_cond = g_cond_new ();
index 3a8561e10e40d21e3c4d58464574e8cfcb014b05..a3454feae21b601878015c3b1edbac3e1f6acac0 100644 (file)
@@ -1272,6 +1272,8 @@ gst_element_link_pads_filtered (GstElement * src, const gchar * srcpadname,
   if (filter) {
     GstElement *capsfilter;
     GstObject *parent;
+    GstElementState state, pending;
+    GTimeVal tv;
 
     capsfilter = gst_element_factory_make ("capsfilter", NULL);
     if (!capsfilter) {
@@ -1282,6 +1284,9 @@ gst_element_link_pads_filtered (GstElement * src, const gchar * srcpadname,
     parent = gst_object_get_parent (GST_OBJECT (src));
     g_return_val_if_fail (GST_IS_BIN (parent), FALSE);
 
+    GST_TIME_TO_TIMEVAL (0, tv);
+    gst_element_get_state (GST_ELEMENT_CAST (parent), &state, &pending, &tv);
+
     if (!gst_bin_add (GST_BIN (parent), capsfilter)) {
       GST_ERROR ("Could not add capsfilter");
       gst_object_unref (capsfilter);
@@ -1289,6 +1294,11 @@ gst_element_link_pads_filtered (GstElement * src, const gchar * srcpadname,
       return FALSE;
     }
 
+    if (pending != GST_STATE_VOID_PENDING)
+      state = pending;
+
+    gst_element_set_state (capsfilter, state);
+
     gst_object_unref (parent);
 
     g_object_set (capsfilter, "filter-caps", filter, NULL);
index 94104de00979a2ea600ea5aafb7f0884d20dd42a..c8c64c26e83f32eabe872a8a498eb23bb5885b85 100644 (file)
@@ -236,7 +236,7 @@ gst_adapter_flush (GstAdapter * adapter, guint flush)
   GstBuffer *cur;
 
   g_return_if_fail (GST_IS_ADAPTER (adapter));
-  g_return_if_fail (flush > 0);
+  g_return_if_fail (flush >= 0);
   g_return_if_fail (flush <= adapter->size);
 
   GST_LOG_OBJECT (adapter, "flushing %u bytes", flush);
index 78dc662e14e9362615f72caaada56958342cdb1f..0b25128ae8de58ea519efe6343dcda2e2fff29c6 100644 (file)
@@ -483,13 +483,13 @@ gst_base_sink_handle_object (GstBaseSink * basesink, GstPad * pad,
     GstBuffer *buf = GST_BUFFER (obj);
 
     if (!basesink->have_newsegment) {
-      GST_ELEMENT_ERROR (basesink, STREAM, STOPPED,
+      GST_ELEMENT_WARNING (basesink, STREAM, STOPPED,
           ("Received buffer without a new-segment. Cannot sync to clock."),
           ("Received buffer without a new-segment. Cannot sync to clock."));
       basesink->have_newsegment = TRUE;
       /* this means this sink will not be able to sync to the clock */
-      basesink->segment_start = 0;
-      basesink->segment_stop = 0;
+      basesink->segment_start = -1;
+      basesink->segment_stop = -1;
     }
 
     /* check if the buffer needs to be dropped */
index 300fe87b9cbcbcd725b3c1d49f4909b931226b4b..2294853a6bf251867c82d8553b2d74144bc7f4ff 100644 (file)
@@ -1046,6 +1046,7 @@ gst_base_transform_change_state (GstElement * element)
             GST_PAD_CAPS (trans->srcpad)) || trans->passthrough;
       else
         trans->in_place = trans->passthrough;
+      GST_DEBUG_OBJECT (trans, "in_place %d", trans->in_place);
       gst_caps_replace (&trans->cache_caps1, NULL);
       gst_caps_replace (&trans->cache_caps2, NULL);
       GST_UNLOCK (trans);
@@ -1090,6 +1091,8 @@ gst_base_transform_set_passthrough (GstBaseTransform * trans,
 {
   g_return_if_fail (trans != NULL);
 
+  GST_DEBUG_OBJECT (trans, "setting passthrough %d", passthrough);
+
   GST_LOCK (trans);
   trans->passthrough = passthrough;
   GST_UNLOCK (trans);