On 5/12/2023 12:17 PM, Devin Heitmueller wrote:
On Thu, May 11, 2023 at 2:20 PM James Almer <jamr...@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.
_______________________________________________
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