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

[James Almer]

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.

> 
> IMHO this would make anything much more handy and less error-prone than 
> this old-fashioned FOURCC codes, which are hard to remember and very 
> uncomfortable to grep in source code.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature.asc
Type: application/pgp-signature
Size: 495 bytes
Desc: OpenPGP digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20241022/c7a09e29/attachment.sig>