On Mon, Feb 18, 2013 at 10:29:18AM -0500, Daniel Kang wrote:
> This was caused by referencing a conditionally compiled table. Now the
> code is also compiled conditionally.
> ---
>  libavcodec/x86/Makefile    |    2 +
>  libavcodec/x86/dsputil.asm |  162 --------------------------------------
>  libavcodec/x86/h263_lf.asm |  187 
> ++++++++++++++++++++++++++++++++++++++++++++
>  3 files changed, 189 insertions(+), 162 deletions(-)
>  create mode 100644 libavcodec/x86/h263_lf.asm

Mention that this fixes Bugzilla #447.

> --- a/libavcodec/x86/Makefile
> +++ b/libavcodec/x86/Makefile
> @@ -44,6 +44,8 @@ YASM-OBJS-$(CONFIG_AC3DSP)             += x86/ac3dsp.o
>  YASM-OBJS-$(CONFIG_DCT)                += x86/dct32.o
>  YASM-OBJS-$(CONFIG_ENCODERS)           += x86/dsputilenc.o
>  YASM-OBJS-$(CONFIG_FFT)                += x86/fft.o
> +YASM-OBJS-$(CONFIG_H263_ENCODER)       += x86/h263_lf.o
> +YASM-OBJS-$(CONFIG_H263_DECODER)       += x86/h263_lf.o

order

Please call the file h263_loopfilter.asm to match h264_loopfilter.c in
libavcodec.  Otherwise LGTM.

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to