On 2014-01-22 16:59:55 +0200, Martin Storsjö wrote:
> mpegvideo_enc.c unconditionally calls ff_h263dsp_init, therefore
> mpegvideoenc should depend on h263dsp.
> 
> This fixes builds with e.g. --disable-decoders --disable-encoders
> --enable-encoder=mjpeg.
> ---
>  configure | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/configure b/configure
> index 98ac566..1977a82 100755
> --- a/configure
> +++ b/configure
> @@ -1612,7 +1612,7 @@ rdft_select="fft"
>  mpegaudio_select="mpegaudiodsp"
>  mpegaudiodsp_select="dct"
>  mpegvideo_select="dsputil hpeldsp videodsp"
> -mpegvideoenc_select="mpegvideo"
> +mpegvideoenc_select="h263dsp mpegvideo"
>  
>  # decoders / encoders
>  aac_decoder_select="mdct sinewin"

That call to ff_h263dsp_init() could be safely placed under 'if
(CONFIG_H263_ENCODER)' and optimized away with dead code elimination.
The only use is the call to ff_h263_loop_filter which is handled in the
same way.

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

Reply via email to