This is an automated email from the git hooks/post-receive script. Git pushed a commit to branch release/8.0 in repository ffmpeg.
commit 0f43795c24d64f82090ab1183e672140b62625e1 Author: Jack Lau <[email protected]> AuthorDate: Wed Dec 10 16:11:43 2025 +0800 Commit: Jack Lau <[email protected]> CommitDate: Tue Feb 10 09:32:56 2026 +0000 avformat/whip: add new option ts_buffer_size to deprecate buffer_size See https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20965#issuecomment-15854 The old option name `buffer_size` maybe confusing because there're some protocols and muxer using this. Signed-off-by: Jack Lau <[email protected]> (cherry picked from commit cc8f3921367f562850bc3cd5049eeff8427f6356) --- doc/muxers.texi | 4 +++- libavformat/whip.c | 10 ++++++---- 2 files changed, 9 insertions(+), 5 deletions(-) diff --git a/doc/muxers.texi b/doc/muxers.texi index 18c99e57b6..c5935c0923 100644 --- a/doc/muxers.texi +++ b/doc/muxers.texi @@ -3955,10 +3955,12 @@ Default value is 5000. Set the maximum size, in bytes, of RTP packets that send out. Default value is 1500. -@item buffer_size @var{integer} +@item buffer_size, ts_buffer_size @var{integer} Set the buffer size, in bytes, of underlying protocol. Default value is -1(auto). The UDP auto selects a reasonable value. +Using the buffer_size option name is deprecated and should not be used. + @item authorization @var{string} The optional Bearer token for WHIP Authorization. diff --git a/libavformat/whip.c b/libavformat/whip.c index b914404f91..a45d9fe79f 100644 --- a/libavformat/whip.c +++ b/libavformat/whip.c @@ -295,7 +295,7 @@ typedef struct WHIPContext { * Note that pion requires a smaller value, for example, 1200. */ int pkt_size; - int buffer_size;/* Underlying protocol send/receive buffer size */ + int ts_buffer_size;/* Underlying protocol send/receive buffer size */ /** * The optional Bearer token for WHIP Authorization. * See https://www.ietf.org/archive/id/draft-ietf-wish-whip-08.html#name-authentication-and-authoriz @@ -1198,7 +1198,7 @@ static int udp_connect(AVFormatContext *s) av_dict_set_int(&opts, "fifo_size", 0, 0); /* Pass through the pkt_size and buffer_size to underling protocol */ av_dict_set_int(&opts, "pkt_size", whip->pkt_size, 0); - av_dict_set_int(&opts, "buffer_size", whip->buffer_size, 0); + av_dict_set_int(&opts, "buffer_size", whip->ts_buffer_size, 0); ret = ffurl_open_whitelist(&whip->udp, url, AVIO_FLAG_WRITE, &s->interrupt_callback, &opts, s->protocol_whitelist, s->protocol_blacklist, NULL); @@ -1818,7 +1818,7 @@ static int whip_write_packet(AVFormatContext *s, AVPacket *pkt) av_log(whip, AV_LOG_WARNING, "Ignore failed to write packet=%dB, ret=%d\n", pkt->size, ret); ret = 0; } else if (ret == AVERROR(EAGAIN)) { - av_log(whip, AV_LOG_ERROR, "UDP send blocked, please increase the buffer via -buffer_size\n"); + av_log(whip, AV_LOG_ERROR, "UDP send blocked, please increase the buffer via -ts_buffer_size\n"); } else av_log(whip, AV_LOG_ERROR, "Failed to write packet, size=%d, ret=%d\n", pkt->size, ret); goto end; @@ -1899,10 +1899,12 @@ static int whip_check_bitstream(AVFormatContext *s, AVStream *st, const AVPacket #define OFFSET(x) offsetof(WHIPContext, x) #define ENC AV_OPT_FLAG_ENCODING_PARAM +#define DEP AV_OPT_FLAG_DEPRECATED static const AVOption options[] = { { "handshake_timeout", "Timeout in milliseconds for ICE and DTLS handshake.", OFFSET(handshake_timeout), AV_OPT_TYPE_INT, { .i64 = 5000 }, -1, INT_MAX, ENC }, { "pkt_size", "The maximum size, in bytes, of RTP packets that send out", OFFSET(pkt_size), AV_OPT_TYPE_INT, { .i64 = 1200 }, -1, INT_MAX, ENC }, - { "buffer_size", "The buffer size, in bytes, of underlying protocol", OFFSET(buffer_size), AV_OPT_TYPE_INT, { .i64 = -1 }, -1, INT_MAX, ENC }, + { "buffer_size", "The buffer size, in bytes, of underlying protocol", OFFSET(ts_buffer_size), AV_OPT_TYPE_INT, { .i64 = -1 }, -1, INT_MAX, ENC | DEP }, + { "ts_buffer_size", "The buffer size, in bytes, of underlying protocol", OFFSET(ts_buffer_size), AV_OPT_TYPE_INT, { .i64 = -1 }, -1, INT_MAX, ENC }, { "authorization", "The optional Bearer token for WHIP Authorization", OFFSET(authorization), AV_OPT_TYPE_STRING, { .str = NULL }, 0, 0, ENC }, { "cert_file", "The optional certificate file path for DTLS", OFFSET(cert_file), AV_OPT_TYPE_STRING, { .str = NULL }, 0, 0, ENC }, { "key_file", "The optional private key file path for DTLS", OFFSET(key_file), AV_OPT_TYPE_STRING, { .str = NULL }, 0, 0, ENC }, _______________________________________________ ffmpeg-cvslog mailing list -- [email protected] To unsubscribe send an email to [email protected]
