[Bug 753480] matroskademux: text/x-raw subtitle tracks ouputs are escaped

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Mon Dec 14 03:09:54 PST 2015


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

Emmauel Bouillot <emmanuel.bouillot at openwide.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |emmanuel.bouillot at openwide.
                   |                            |fr

--- Comment #2 from Emmauel Bouillot <emmanuel.bouillot at openwide.fr> ---
Created attachment 317344
  --> https://bugzilla.gnome.org/attachment.cgi?id=317344&action=edit
patch for matroskademux : use utf8 format instead of pango-markup

The matroska demux force the pango-markup format output of text/x-raw
subutitles and modify the content of the raw text (originaly in utf8 format).
This may cause problems when trying to use it as non-pango text. It seems more
logical that the demuxer gives the same format on output the original format it
contains and so provide the same format as matroskamux. 

Modifying matroskademux to ouput utf8 format instead of pango-markup does not
require a lot of modifications and will not break compatibility with other
plugins since all plugins using pango-markup as input (good/bad/ugly) also
accept utf8 and convert it internally when necessary (textrender, textoverlay,
subparse, srtenc, kateenc, webvttenc)

We studied the possibility to use both format on the demuxer, but matroksademux
use static caps for the sinks, determinated by the source streams format.
Turning it into dynamic and use negociation to determine the sink format only
for text/x-raw may lead to deep change on the way the demuxer works and seems
to be an overkill solution.

This is why using "text/x-raw, format=utf8" instead of "text/x-raw,
format=pango-markup" option seems to be more adventageous.

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


More information about the gstreamer-bugs mailing list