[Bug 730039] rtpjitterbuffer: jitter buffer stalls if first seqnum is different than seqnum-base

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=730039
  GStreamer | gst-plugins-good | git

Wim Taymans <wim.taymans> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |wim.taymans at gmail.com

--- Comment #3 from Wim Taymans <wim.taymans at gmail.com> 2014-05-13 14:43:24 UTC ---
(In reply to comment #2)
> Not sure if the following is a realistic case, but I think it could happen.
> With the new approach, what would happen in this case?
> 
> 10345 10346 -1
> h            t
> 
> 10344 is to be inserted. But because of this:
> 
>   for (; list; list = g_list_previous (list)) {
>     ...
> 
>     if (qitem->seqnum == -1)
>       break;
> 
> It seems it will be inserted in the tail:
> 
> 10345 10346 -1 10344
> h                  t
> 
> instead of:
> 
> 10344 10345 10346 -1
> h                  t

It looks like that break; should be a continue;

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.