rtpjitterbuffer: fix gap-time calculation and remove "late"
authorHavard Graff <havard.graff@gmail.com>
Wed, 8 Jul 2015 19:08:36 +0000 (21:08 +0200)
committerSebastian Dröge <sebastian@centricular.com>
Wed, 8 Jul 2015 20:18:48 +0000 (23:18 +0300)
commitddd032f56b64316d5a1aa5b267ad34df4e774373
tree873f577a1f547db0405ef43ff9b57bb69928fcf7
parent40524e5a493702fa9ff67fde43f94a9da2f16233
rtpjitterbuffer: fix gap-time calculation and remove "late"

The amount of time that is completely expired and not worth waiting for,
is the duration of the packets in the gap (gap * duration) - the
latency (size) of the jitterbuffer (priv->latency_ns). This is the duration
that we make a "multi-lost" packet for.

The "late" concept made some sense in 0.10 as it reflected that a buffer
coming in had not been waited for at all, but had a timestamp that was
outside the jitterbuffer to wait for. With the rewrite of the waiting
(timeout) mechanism in 1.0, this no longer makes any sense, and the
variable no longer reflects anything meaningful (num > 0 is useless,
the duration is what matters)

Fixed up the tests that had been slightly modified in 1.0 to allow faulty
behavior to sneak in, and port some of them to use GstHarness.

https://bugzilla.gnome.org/show_bug.cgi?id=738363
gst/rtpmanager/gstrtpjitterbuffer.c
tests/check/elements/rtpjitterbuffer.c