[vlc-devel] [PATCH 1/6] videotoolbox: remove early chroma configuration

Marvin Scholz epirat07 at gmail.com
Fri Jan 24 11:17:21 CET 2020


LGTM for the VT changes.

On 24 Jan 2020, at 11:03, Thomas Guillem wrote:

> It was used to fix decoder device creation from the core but it's not 
> needed
> since 2210c192db61ca479024e4959c386a2c31ce693f
> ---
>  modules/codec/videotoolbox.c | 4 ----
>  1 file changed, 4 deletions(-)
>
> diff --git a/modules/codec/videotoolbox.c 
> b/modules/codec/videotoolbox.c
> index 873f28e6921..85bc7064f27 100644
> --- a/modules/codec/videotoolbox.c
> +++ b/modules/codec/videotoolbox.c
> @@ -1343,10 +1343,6 @@ CreateVideoContext(decoder_t *p_dec)
>      p_dec->fmt_out.video = p_dec->fmt_in.video;
>      p_dec->fmt_out.video.p_palette = NULL;
>
> -    /* Most likely used chroma but can be reconfigured in the future 
> */
> -    p_dec->fmt_out.i_codec = VLC_CODEC_CVPX_NV12;
> -    p_dec->fmt_out.video.i_chroma = p_dec->fmt_out.i_codec;
> -
>      if (!p_dec->fmt_out.video.i_sar_num || 
> !p_dec->fmt_out.video.i_sar_den)
>      {
>          p_dec->fmt_out.video.i_sar_num = 1;
> -- 
> 2.20.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