Sebastian Dröge
773b7f61f2
rtpjitterbuffer: Make it more explicit that update_rtx_timers() takes ownership of the passed in timer
...
It is not valid anymore afterwards and must not be used, otherwise an
already freed pointer might be used.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2996 >
2022-09-07 11:26:41 +00:00
Sebastian Dröge
646766629f
rtpjitterbuffer: Don't shadow variable
...
While this didn't cause any problems in this context it is simply
confusing.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2996 >
2022-09-07 11:26:41 +00:00
Sebastian Dröge
94122ba11b
rtpjitterbuffer: Change RTX timer availability checks to assertions
...
It's impossible to end up in the corresponding code without a timer for
RTX packets because otherwise it would be an unsolicited RTX packet and
we would've already returned early.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2996 >
2022-09-07 11:26:41 +00:00
Sebastian Dröge
dc408d56c5
rtpjitterbuffer: Only unschedule timers for late packets if they're not RTX packets and only once
...
Timers for RTX packets are dealt with later in update_rtx_timers(), and
timers for non-RTX packets would potentially also be unscheduled a
second time from there so avoid that.
Also don't shadow the timer variable from the outer scope but instead
make use of it directly.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2996 >
2022-09-07 11:26:40 +00:00
Raul Tambre
8bb230b72a
rtpjitterbuffer: remove lost timer for out of order packets
...
When receiving old packets remove the running lost timer if present.
This fixes incorrect reporting of a lost packet even if it arrived in time.
Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/2995 >
2022-09-07 11:17:54 +01:00
Thibault Saunier
5ff769d731
Move files from gst-plugins-good into the "subprojects/gst-plugins-good/" subdir
2021-09-24 16:13:50 -03:00
