[FFmpeg-devel] [PATCH v2] avfilter/ccfifo: remove unnecessary context allocations

Devin Heitmueller devin.heitmueller at ltnglobal.com
Fri May 12 18:17:18 EEST 2023


On Thu, May 11, 2023 at 2:20 PM James Almer <jamrial at gmail.com> wrote:
>
> This is not public API, no it has no need for an alloc() and free()
> functions. The struct can reside on stack.

I don't have any objection to this patch in principle, assuming this
is the standard convention that ffmpeg uses for such things.

I will point out though that the patch doesn't include the usage in
libavdevice/decklink_enc.cpp, so it would cause build breakage if the
proposed changes are not reflected in that file as well.

Devin

-- 
Devin Heitmueller, Senior Software Engineer
LTN Global Communications
o: +1 (301) 363-1001
w: https://ltnglobal.com  e: devin.heitmueller at ltnglobal.com


More information about the ffmpeg-devel mailing list