[FFmpeg-devel] [PATCH] mpeg2 non linear quantizer full support
Fri Mar 14 13:42:45 CET 2008
Baptiste Coudurier wrote:
> Diff for some files is to show possible simplifications of existing
> code. I'll of course split the commits. Only mpeg-2 unquantize
> functions are affected.
> Index: libavcodec/mpegvideo.c
> --- libavcodec/mpegvideo.c (revision 12407)
> +++ libavcodec/mpegvideo.c (working copy)
> @@ -33,6 +33,7 @@
> #include "mpegvideo_common.h"
> #include "mjpegenc.h"
> #include "msmpeg4.h"
> +#include "mpeg12.h"
> #include "faandct.h"
> #include <limits.h>
I'm curious to know why you include mpeg12.h (here at at some other
places in this patch) ? I can't see anything which would require it.
Moreover, in case you add a dependency to mpeg12.o, you also need to
adjust the Makefile (but I guess this would add lots of ugly, almost
Maybe you could find a better place to put the new *_qscale table to
reduce artificial dependencies.
More information about the ffmpeg-devel