On 08.06.2025 17:45, James Almer wrote:
On 6/8/2025 11:29 AM, Andreas Rheinhardt wrote:
Timo Rothenpieler:
From: James Almer <jamr...@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.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to