Created attachment 48617 [details] [review]
Patch for AVC support in matroska

This Patch adds AVC support to matroska plugin. AVC header is stored in caps
structure codec_data what is compatible with gst-ffmpeg. 

The code was inspired by libavcodec and muxed files play perfectly using
mplayer.

The demuxer already supports AVC in gst-plugins cvs. As far as I know the
matroska spec muxing isn't that easy, the files may play correctly in most
players but wouldn't be spec compliant as native mpeg4 muxing needs to add
information about reference frames for P/B frames.

I will ask the people from matroska aboout this issue.

any news?

Referencing the frames (ReferenceBlock element) is no longer needed in the
Matroska specs. What seems to be required is ReferencePriority (non-zero for
frames which are referenced and 0 for the others).

The simplest implementation would be to set 1 for all frames except the last one
(in decoding order) frame in the BlockGroup.

Anyway, I was looking at mkvtoolnix source and I have not found referencing
there either.

In the moment, I am focusing on 0.9 version of the plugin so I can not help much
with 0.8.

I suggest to close this as "OBSOLETE"

Given that Michal is the matroska fellow that I know, I will follow what he says ;-)