oggdemux: do not warn when finding a non BOS page
authorVincent Penquerc'h <vincent.penquerch@collabora.co.uk>
Tue, 23 Aug 2011 09:47:53 +0000 (10:47 +0100)
committerSebastian Dröge <sebastian.droege@collabora.co.uk>
Wed, 24 Aug 2011 06:31:17 +0000 (08:31 +0200)
After all, we do hope to find actual data for these streams.
However, warn if we could not set up a chain when we find a
non BOS page, as that means we don't have a valid Ogg stream.

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

ext/ogg/gstoggdemux.c

index 5280270..f04b5cc 100644 (file)
@@ -2674,11 +2674,13 @@ gst_ogg_demux_read_chain (GstOggDemux * ogg, GstOggChain ** res_chain)
       break;
     }
     if (!ogg_page_bos (&og)) {
-      GST_WARNING_OBJECT (ogg, "page is not BOS page");
+      GST_INFO_OBJECT (ogg, "page is not BOS page, all streams identified");
       /* if we did not find a chain yet, assume this is a bogus stream and
        * ignore it */
-      if (!chain)
+      if (!chain) {
+        GST_WARNING_OBJECT (ogg, "No chain found, no Ogg data in stream ?");
         ret = GST_FLOW_UNEXPECTED;
+      }
       break;
     }