[Bug 664257] [mpegtsparse] Support UTF-16BE text encoding

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Mon Nov 21 09:11:52 PST 2011


https://bugzilla.gnome.org/show_bug.cgi?id=664257
  GStreamer | gst-plugins-bad | git

Sebastian Pölsterl <sebp> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #201583|0                           |1
        is obsolete|                            |

--- Comment #2 from Sebastian Pölsterl <sebp at k-d-w.org> 2011-11-21 17:11:48 UTC ---
Created an attachment (id=201830)
 View: https://bugzilla.gnome.org/attachment.cgi?id=201830
 Review: https://bugzilla.gnome.org/review?bug=664257&attachment=201830

Support additional encodings

Apparently, I used an outdated version of the spec. I updated support for
additional encodings based on V1.11.1 of this spec.

The whole encoding/decoding was written by myself a couple of years back. The
is_multibyte part is only responsible to remove control codes from the text. I
did not have any samples with multi-byte encoding back then, therefore it might
be wrong all along. In addition, the spec doesn't mention anything about the
byte order. All I know is that it works with the sample file.

Regarding "Big5 subset of ISO/IEC 10646". I tried to find out what exactly this
means, but could not find any information that links Big5 and ISO/IEC 10646.

I suggest to accept this patch since the spec is very vague about the details
and it works correctly with the sample file. If someone else comes along later
with a sample that does not work, we can look into it again.

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- 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