[Bug 753323] mpegts: add support for JPEG 2000 to mpegtsmux and tsdemux

[GStreamer (GNOME Bugzilla)]

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

Edward Hervey <bilboed at bilboed.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |bilboed at bilboed.com

--- Comment #18 from Edward Hervey <bilboed at bilboed.com> ---
Some comments on the patches:
* Just make two patches, one for the demuxer, one for the muxer.
* Don't make "fix the previous patch", make them patches which are
self-contained
* The demuxer part has comments/todo, I'll assume it's definitely not ready
* Use C comments (/* .... */) and not c++ ones (//...)
* You should get the profile/level/framerate/size/... from the jp2k descriptor
(J2k_video_descriptor, 2.6.80 in the mpeg-ts specifications). That would
require creating a 3rd patch to add support for parsing/creating that
descriptor in the mpeg-ts library (gst-libs/gst/mpegts/..).
* Since elsm is part of the official jpeg2000 specification (Annex M of Rec.
ITU-T T.800), I don't see why you would need to parse it in the demuxer, just
push it as-is, decoders *should* be able to handle it, no ?
* Please provide some sample streams to test the code with. They will most
likely be too big to post here, so provide links to them.

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