[Bug 747774] dashdemux: text streams support

[GStreamer (GNOME Bugzilla)]

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

--- Comment #3 from Chris Bass <floobleflam at gmail.com> ---
> How about moving this stream_contains_subtitles to mpdparser as the other
> mimetype to caps conversions are being done there. The difference here is
> that you need also to check the codec field. What do you think of adding a
> new gst_mpd_client_get_stream_caps() that would handle the previous and this
> new case?

Sure. Do you think it's best to (a) add a new function
gst_mpd_client_get_stream_caps(), or (b) to rename
gst_mpd_client_get_stream_mimeType() to gst_mpd_client_get_stream_caps(),
extending it to handle the subtitle cases and having it return a GstCaps rather
than gchar*?

At the moment, gst_mpd_client_get_stream_mimeType() isn't really returning the
mimeType of the streams - it calls gst_mpdparser_mimetype_to_caps to convert
the mimeTypes to caps strings, so its name is perhaps a little bit misleading
at the moment.

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