[FFmpeg-devel] [PATCH 1/7] avutil: add an API to handle 3D Reference Displays Information

[Timo Rothenpieler]

On 08.06.2025 17:45, James Almer wrote:
> On 6/8/2025 11:29 AM, Andreas Rheinhardt wrote:
>> Timo Rothenpieler:
>>> From: James Almer <jamrial at gmail.com>
>>>
>> I don't like that you add another allocator for this; instead we should
>> add a generic allocator for the frame side-data types.
> 
> Wont work for packet side data. And i purposely didn't add yet another 
> allocator that inserts the result into a frame, like there's in so many 
> other modules, because eventually the generic one would be introduced.
> 
> You said you wanted to take over my work on the generic allocator, but 
> not sure if you did anything with it. The core issue was handling more 
> complex types that didn't just have an extra nb_blocks argument.

So, what is the conclusion here?
I'd like to push this set if you can come to an agreement.

I haven't looked into it much, but implementing av_tdrdi_alloc() in a 
generic way does seem a bit hacky. And other types might need even more 
info for the allocation.