[Bug 702004] qtdemux: add support for the avc3 sample entry format of the AVC file format

[GStreamer (bugzilla.gnome.org)]

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

--- Comment #18 from Tim-Philipp Müller <t.i.m at zen.co.uk> 2013-06-18 15:39:30 UTC ---
I'm not saying one is normal and one is abnormal, but avc3 is basically a mix
between those two modes, which will make everything even more confusing, we
barely handle those two modes properly. We have now stream-format=avc which
means sps/pps are sent out-of-band and nals have no sync code and everything is
framed in access units, and stream-format=byte-stream where sps/pps are in the
bitstream and nals have sync codes.

If I understood comment #12 correctly, we don't have to parse the sample data
(not much anyway), we can assume the sps/pps are in the first sample. It
shouldn't be too difficult, even if it's a bit annoying.

It is true that we already have h264 parsing spread across multiple modules. We
will eventually consolidate that when the codecparser library moves to -base.
It's not clear to me whether that's really an argument for or against doing
things as I suggest though. I think what has to be weighed is the complexity of
making h264parse handle yet another mode, namely this mixed mode, and the
complexity of parsing out the sps/pps and putting them into caps. The latter is
considerably simpler, and easier to maintain in the long run IMHO. If there are
opportunities for utility functions, they can be put into pbutils with the
codec utils.

-- 
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.