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

James Almer jamrial at gmail.com
Fri May 12 22:24:40 EEST 2023


On 5/12/2023 12:17 PM, Devin Heitmueller wrote:
> 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.

Thanks for spotting it. Adapted decklink as well, and pushed.


More information about the ffmpeg-devel mailing list