On Fri, Sep 28, 2012 at 01:31:23PM +0200, Jordi Ortiz wrote:
> 
> --- a/libavcodec/Makefile
> +++ b/libavcodec/Makefile
> @@ -339,8 +339,8 @@ OBJS-$(CONFIG_SIPR_DECODER)            += sipr.o 
> acelp_pitch_delay.o \
>  OBJS-$(CONFIG_SMC_DECODER)             += smc.o
> -OBJS-$(CONFIG_SNOW_DECODER)            += snowdec.o snow.o
> -OBJS-$(CONFIG_SNOW_ENCODER)            += snowenc.o snow.o              \
> +OBJS-$(CONFIG_SNOW_DECODER)            += snowdec.o snow.o snow_dwt.o
> +OBJS-$(CONFIG_SNOW_ENCODER)            += snowenc.o snow.o snow_dwt.o   \

So this code is used for both the decoder and the encoder, but ...

> --- a/libavcodec/x86/Makefile
> +++ b/libavcodec/x86/Makefile
> @@ -16,7 +16,6 @@ MMX-OBJS                               += x86/dsputil_mmx.o 
>             \
>  MMX-OBJS-$(CONFIG_CAVS_DECODER)        += x86/cavsdsp.o
> -MMX-OBJS-$(CONFIG_DWT)                 += x86/snowdsp.o
>  MMX-OBJS-$(CONFIG_ENCODERS)            += x86/dsputilenc_mmx.o          \
> @@ -29,6 +28,7 @@ MMX-OBJS-$(CONFIG_PRORES_DECODER)      += 
> x86/proresdsp_init.o
>                                            x86/rv40dsp_init.o
> +MMX-OBJS-$(CONFIG_SNOW_DECODER)        += x86/snowdsp.o
>  MMX-OBJS-$(CONFIG_VC1_DECODER)         += x86/vc1dsp_mmx.o

... here it's just used by the decoder?

> --- a/libavcodec/dwt.c
> +++ b/libavcodec/dwt.c

This file contains references to snow functions w/o any ifdefs.
Doesn't this fail to compile with --disable-decoder=snow
--disable-encoder=snow and --enable-dwt?

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

Reply via email to