[Bug 728017] [regression]eos event could not be send out from gstrtpjitterbuffer.

[GStreamer (bugzilla.gnome.org)]

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

Wim Taymans <wim.taymans> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |FIXED

--- Comment #3 from Wim Taymans <wim.taymans at gmail.com> 2014-04-18 07:47:35 UTC ---
(In reply to comment #2)
> Hi:
> I merged the commit to 1.2.3 branch but it does not resolved the issue.

Be careful with porting to 1.2, it causes conflicts. The 1.2 merged patch is
here:
http://cgit.freedesktop.org/gstreamer/gst-plugins-good/commit/?h=1.2&id=498e2e082261d16f285fd8c34852fe862b33fdd3

I tested with rtsp-server streaming sample_h264_1mbit.mp4 and it worked
correctly for me.

> Eos event was found about 5*5 seconds behind the last buffer.
> However, rtp_jitter_buffer_is_buffering always returns TRUE and it will keeps
> waiting the event.

It sounds like you are not testing the right patch. With the patch, In the EOS
case, check_buffering_percent() will set buffering to FALSE.

> 
> Here is my solution:

The problem with your solution is that EOS is sent too soon, you first need to
push out all the buffers from the jitterbuffer and then check EOS.

If you believe you tried the right patch, can you make a new debug log and
reopen this bug?

-- 
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.