[Bug 747774] dashdemux: text streams support

[GStreamer (GNOME Bugzilla)]

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

Sebastian Dröge (slomo) <slomo at coaxion.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #309473|none                        |accepted-commit_after_freez
             status|                            |e

--- Comment #13 from Sebastian Dröge (slomo) <slomo at coaxion.net> ---
Review of attachment 309473:
 --> (https://bugzilla.gnome.org/review?bug=747774&attachment=309473)

Looks good to me

::: ext/dash/gstmpdparser.c
@@ +4426,3 @@
   }

+  caps_string = gst_mpdparser_mimetype_to_caps (mimeType);

Maybe we should be more restrictive in that function, and only whitelist some
mimetypes instead of converting every unknown mimetype directly to caps? But if
so, in a follow-up patch.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.