On Thu, May 11, 2023 at 2:20 PM James Almer <[email protected]> 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: [email protected] _______________________________________________ ffmpeg-devel mailing list [email protected] https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email [email protected] with subject "unsubscribe".
