[Ffmpeg-cvslog] Re: r7128 - trunk/libavformat/matroska.c

[Steve Lhomme]

Aurelien Jacobs wrote:
> On Mon, 20 Nov 2006 21:19:41 +0100
> Steve Lhomme <steve.lhomme at free.fr> wrote:
> 
>> Aurelien Jacobs wrote:
>>>> If you really wanted an answer you would post it to matroska-devel.
>>>>
>>>> Anyway, I've had RealVideo decoding in DrFFMPEG working fine for about a 
>>>> year without that hack. So I don't know why it has been added there but 
>>>> it's probably not the right way to do it.
>>> Really ? Do you have anything better to propose ?
>>> Try to play RV20 in mkv with ffplay before and after this set of patch.
>> Do you have such a file I can test with ? And where did you get such a 
>> file ? Because I don't think anyone ever used RV20 in matroska apart 
>> from testing.
> 
> http://gnuage.org/pub/rv20.mkv
> I simply built it using mkvmerge without doing anything special.

Indeed that file plays fine in the DirectShow player but not with 
DrFFMPEG. I'll give your patch a try. But not for RV40 as it works fine 
without the patch (for me). It may mean that the extradata size is 
expected to be different between the versions of RealVideo. And I don't 
know what is the correct size either from DirectShow or libavcodec, 
since none of them are the official Real code (but the DirectShow uses 
the Real codecs).

Steve