[FFmpeg-devel] [PATCH 05/18] lavc/dv: remove ff_dvvideo_init()

Anton Khirnov anton at khirnov.net
Thu Aug 25 12:48:27 EEST 2022


Quoting Andreas Rheinhardt (2022-08-24 14:27:43)
> There is actually another issue here: Encoders are not supposed to set
> chroma_sample_location at all according to the documentation. I sent a
> patch to implement this
> (https://ffmpeg.org/pipermail/ffmpeg-devel/2022-July/298518.html -- it
> also inlines and removes ff_dvvideo_init()), but I am unsure whether it
> is not the documentation that needs to be updated.

I would say update the documentation. It does make sense for encoders to
set this field.

-- 
Anton Khirnov


More information about the ffmpeg-devel mailing list