On Wed, 13 Apr 2022, Martin Storsjö wrote:

On Mon, 11 Apr 2022, Tristan Matthews wrote:

 This avoids having to do one pass to calculate the full length to allocate
 followed by a second pass to actually append values.
 ---
 libavformat/librtmp.c | 123 +++++++++++-------------------------------
 1 file changed, 32 insertions(+), 91 deletions(-)

Thanks, this patch looks good to me. I'll wait for a little while still if Marton wants to comment on it, before I go ahead and push it.

According to commit 865461099e062de5a3a109c2a5be98004c11d8bd the buffer passed to RTMP_SetupURL has to be kept as long as the RTMP context is alive.

Regards,
Marton
_______________________________________________
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