[vlc-devel] [PATCH 5/6] transcode: video: use i_chroma instead of i_codec

[Romain Vimont]

On Tue, Jun 09, 2020 at 07:19:03PM +0200, Francois Cartegnie wrote:
> Le 09/06/2020 à 18:43, Alexandre Janniaux a écrit :
> > ---
> >  modules/stream_out/transcode/video.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/modules/stream_out/transcode/video.c b/modules/stream_out/transcode/video.c
> > index 14282ae864..e94588c978 100644
> > --- a/modules/stream_out/transcode/video.c
> > +++ b/modules/stream_out/transcode/video.c
> > @@ -627,7 +627,7 @@ int transcode_video_process( sout_stream_t *p_stream, sout_stream_id_sys_t *id,
> >              /* In case the encoder wasn't open yet, check if we need to add
> >               * a converter between last user filter and encoder. */
> >              if( !is_encoder_open &&
> > -                filter_fmt_out.i_codec != encoder_fmt_in->i_codec )
> > +                filter_fmt_out.video.i_chroma != encoder_fmt_in->video.i_chroma )
> 
> Not sure about this. That's only equal with uncompressed formats.
> 
> Codec could be H264 and chroma i420.

Btw, couldn't we split VLC_CODEC_* into something like VLC_CHROMA_* and
VLC_CODEC_*?

This would avoid the need for combinations like VLC_CODEC_CVPX_I420: the
codec would be VLC_CODEC_CVPX and the chroma would be VLC_CHROMA_I420
(or VLC_CHROMA_OPAQUE).

> 
> 
> -- 
> Francois Cartegnie
> VideoLAN - VLC Developer
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel