[FFmpeg-devel] [PATCH v2 00/13] YUVJ removal

Lynne dev at lynne.ee
Sat Oct 14 00:14:34 EEST 2023


Oct 13, 2023, 20:33 by vittorio.giovara at gmail.com:

> On Fri, Oct 13, 2023 at 10:27 AM Niklas Haas <ffmpeg at haasn.xyz> wrote:
>
>> Changes since v1:
>>
>> - Remove unneeded patch (AVCodecContext.colorspace init)
>> - Merge auto-range conversion into auto-scale filter
>> - Replace vf_zscale by vf_colorspace in fftools
>>
>
> Why is this? I haven't checked what vf_colorspace supports in a hot second,
> but iirc zscale can handle non linear spaces better and hdr conversion
> If it's because it's a built in filter, do you think we could first check
> for zscale presence and fallback to colorspace?
>

vf_colorspace != swscale

Relying on external library for basic functionality that we have
no control over, which may break its ABI or API at any moment,
when we have a built-in one is a no.
I wouldn't agree to having it optional in this case either. Users
can explicitly request it as a filter and use it, which fits in better
with its very explicit programming model too.


More information about the ffmpeg-devel mailing list