[FFmpeg-devel] [RFC] move wmv2.c to its own file

Ivan Kalvachev ikalvachev
Tue Oct 30 22:35:14 CET 2007


2007/10/30, Aurelien Jacobs <aurel at gnuage.org>:
> Diego Biurrun wrote:
>
> > Here's another attempt at $subject that builds upon previous work.  It
> > does compile, but it slightly increases the size of libavcodec.a,
> > probably because I currently duplicate
> >
> > static VLC mb_non_intra_vlc[4];
> > static VLC inter_intra_vlc;
>
> This is wrong...
> Those vars are initialized in msmpeg4.c. The duplicated ones in wmv2.c
> are never initialized !

Good catch. I wonder how such serious bug have not been found for so long time.

> Another problem is the usage of ff_wmv2_decode_mb in msmpeg4.c.
> It shouldn't be used when wmv2 is not compiled in.
>
> I fixed those issues in the attached updated patch.
> It may be good enough to be committed.

Why are you changing ff_code012() to ff_msmpeg_code012() ?
Do you know smaller patches are reviewed faster ;)




More information about the ffmpeg-devel mailing list