On Sat, Jan 18, 2014 at 02:09:59PM +0100, Kostya Shishkov wrote: > On Sat, Jan 18, 2014 at 01:31:52PM +0100, Tim Walker wrote: > > From: Michael Niedermayer <[email protected]> > > > > --- > > Appears to be a functional change in this merge commit: > > > > http://git.videolan.org/?p=ffmpeg.git;a=commit;h=41f552 > > > > Fixes some interlaced VC-1 samples, e.g.: > > > > https://dl.dropboxusercontent.com/u/37259775/ivc1/ivc1-001.mkv > > https://dl.dropboxusercontent.com/u/37259775/ivc1/ivc1-004.mkv > > > > Can people who know VC-1 better please shed some light on this? > > [...] > > Looks harmless though I don't see how it fixes anything. > avconv -i ivc1-001.mkv -f md5 check.md5 > doesn't change with this patch applied
Ah, tested it on some old branch. For some B-frames coding mode switches from interlaced field to interlaced frame and numref should be reset for it, otherwise it leads to wrong decisions in motion compensation code. The patch is almost OK but it should also remove v->numref = 0 at line 1015 since it's redundant now. _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
