[vlc-devel] [PATCH 4/4] avcodec: remove a conditional compilation

Steve Lhomme robux4 at ycbcr.xyz
Fri Oct 2 08:54:30 CEST 2020


Why is it OK to be removed ? I assumed it was added at a time some 
ffmpeg/libav didn't have the define, which is probably not versions we 
support anymore. But a reason in the commit log would be helpful.

On 2020-10-01 18:37, Zhao Zhili wrote:
> ---
>   modules/codec/avcodec/avcodec.c | 2 --
>   1 file changed, 2 deletions(-)
> 
> diff --git a/modules/codec/avcodec/avcodec.c b/modules/codec/avcodec/avcodec.c
> index 75bb9337d7..2db8797d32 100644
> --- a/modules/codec/avcodec/avcodec.c
> +++ b/modules/codec/avcodec/avcodec.c
> @@ -133,10 +133,8 @@ vlc_module_begin ()
>       add_string( "avcodec-codec", NULL, CODEC_TEXT, CODEC_LONGTEXT, true )
>       add_obsolete_bool( "ffmpeg-hw" ) /* removed since 2.1.0 */
>       add_obsolete_string( "avcodec-hw" ) /* removed since 4.0.0 */
> -#if defined(FF_THREAD_FRAME)
>       add_obsolete_integer( "ffmpeg-threads" ) /* removed since 2.1.0 */
>       add_integer( "avcodec-threads", 0, THREADS_TEXT, THREADS_LONGTEXT, true );
> -#endif
>       add_string( "avcodec-options", NULL, AV_OPTIONS_TEXT, AV_OPTIONS_LONGTEXT, true )
>   
>   
> -- 
> 2.25.1
> 
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel
> 


More information about the vlc-devel mailing list