[FFmpeg-devel] [PATCH 1/6] avutil/pixfmt: add Y216 pixel format

[martin schitter]

On 22.10.24 15:10, James Almer wrote:
> On 10/22/2024 11:26 AM, martin schitter wrote:
>>
>>
>> On 22.10.24 14:48, James Almer wrote:
>>> +    AV_PIX_FMT_Y216BE,      ///< packed YUV 4:2:2 like YUYV422, 
>>> 32bpp, big-endian
>>> +    AV_PIX_FMT_Y216LE,      ///< packed YUV 4:2:2 like YUYV422, 
>>> 32bpp, little-endian
>>
>> Why to you avoid any more verbose naming, where any developer would 
>> see the actual order of components by just looking at the actual 
>> letters to avoid wrong usage?
> 
> It's too late for that. This naming scheme goes back more than a decade. 
> It would require a big, API breaking and disruptive overhaul to switch 
> to one where the names follow the component order and bitdepth.

if you do a simple:

     ❯ rg "(YUYV|YVYU|UYVY|VYUY)" libavutil/pixfmt.h

you'll see, that the slightly more verbose variants were already 
invented and used by clever developers long before us. :)

They build a very simple help to prevent stupid errors caused by 
unavoidable presence of slightly different ordered but otherwise very 
similar variants of packed 4:2:2 data.

martin