On 16/09/15 08:55, Martin Storsjö wrote: > On Tue, 15 Sep 2015, Eloi BAIL wrote: > >> This commit adds a warning trace when jitter buffer >> is full. It helps to understand leading decoding issues. >> >> Signed-off-by: Eloi BAIL <[email protected]> >> --- >> libavformat/rtpdec.c | 3 +++ >> 1 file changed, 3 insertions(+) >> >> diff --git a/libavformat/rtpdec.c b/libavformat/rtpdec.c >> index 0efcdb9..1d4b06c 100644 >> --- a/libavformat/rtpdec.c >> +++ b/libavformat/rtpdec.c >> @@ -703,6 +703,9 @@ static void enqueue_packet(RTPDemuxContext *s, >> uint8_t *buf, int len) >> packet->next = *cur; >> *cur = packet; >> s->queue_len++; >> + if (s->queue_len >= s->queue_size) >> + av_log(s->st ? s->st->codec : NULL, AV_LOG_WARNING, >> + "jitter buffer full\n"); >> } >> >> static int has_next_packet(RTPDemuxContext *s) >> -- >> 2.1.4 > > Ok with me, although I'll move it into the existing "buffer full" check > in rtp_parse_one_packet as suggested by Michael Niedermayer. >
If enqueue_packet is used only there it surely makes sense. (patch to address another issue while at it sent.) lu _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
