[MPlayer-dev-eng] [PATCH] make lavf demuxer and mplayer demuxerreturn same format extradata.

elupus elupus at ecce.se
Sun Nov 5 13:50:41 CET 2006


> > Hi, 
> > 
> > Here's a patch that partially fixes 
> invalid_fps_broken_video_lavc.rm in
> > incomming. 
> > 
> 
> Attached a minimal patch (based on your code) to fix your sample, and
> all other RV20 files (except for rv20errors.rm). This was 
> broken mainly
> by fixing the biSize value passed by real demuxer.
> 
> This patch does not pass the cmsg24 data to lavc RV20 codec, 
> it was not
> done even before RV20 got broken (the diff2 patch does it in case
> anybody wants to experiment with it, but rv20errors still has broken
> size).
> 
> 
> I'll check the other parts of your patch later, changing how the
> extradata is passed to vd_realvid may break the binary codecs on
> different endianness systems, and require changing all other demuxers
> that may use real codecs.
> 

Another idea wich would be less intrusive would be to just fixup the extra
data as returned from lavf and byteswap the two first dwords of it when it
is demuxing a realfile. But imho, it makes less sence, you'd then also have
to undo it when lavc is used to decode.

/Joakim




More information about the MPlayer-dev-eng mailing list