[Bug 764889] New: rtp_jitter_buffer_get_ts_diff returns incorrect values

[GStreamer (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=764889

            Bug ID: 764889
           Summary: rtp_jitter_buffer_get_ts_diff returns incorrect values
    Classification: Platform
           Product: GStreamer
           Version: git master
                OS: All
            Status: NEW
          Severity: normal
          Priority: Normal
         Component: gst-plugins-good
          Assignee: gstreamer-bugs at lists.freedesktop.org
          Reporter: dyudaken at gmail.com
        QA Contact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---

rtp_jitter_buffer_get_ts_diff should return the difference in rtptime from the
oldest -> newest packet.
In the current implementation, it returns the opposite.

To test this you can do:
gst-launch-1.0 audiotestsrc num-buffers=1 ! audioconvert ! avenc_g722  !
rtpg722pay mtu=9999 ! rtpjitterbuffer drop-on-latency=true latency=500 !
fakesink dump=true
AND
gst-launch-1.0 audiotestsrc num-buffers=1 ! audioconvert ! avenc_g722  !
rtpg722pay mtu=9999 ! rtpjitterbuffer drop-on-latency=false latency=500 !
fakesink dump=true

Since there is no latency, these should both dump the same number of buffers (4
locally), but with this bug and drop-on-latency=true, it dumps 3.

I suspect in the code that the g_queue_peek_head_link/g_queue_peek_tail_link
should be reversed

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.