rtpbin: Constify function parameters and use correct types

Previously these parameters were randomly changed in the body of the
function to avoid having to declare a new variable, which made the code
very hard to follow. By marking them as const this won't be possible
anymore in the future.

Also the RTP clock-base (RTP time from RTSP RTP-Info) is an unsigned
64 bit integer as it's an extended RTP timestamp.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/6543>
This commit is contained in:
Sebastian Dröge 2024-04-17 16:50:30 +03:00 committed by GStreamer Marge Bot
parent 155c3fb3b2
commit d8dabf142f

View file

@ -1468,10 +1468,11 @@ gst_rtp_bin_send_sync_event (GstRtpBinStream * stream)
* *
* Must be called with GST_RTP_BIN_LOCK */ * Must be called with GST_RTP_BIN_LOCK */
static void static void
gst_rtp_bin_associate (GstRtpBin * bin, GstRtpBinStream * stream, guint8 len, gst_rtp_bin_associate (GstRtpBin * bin, GstRtpBinStream * stream,
const guint8 * data, guint64 ntpnstime, guint64 extrtptime, const guint8 len, const guint8 * data, const guint64 ntpnstime,
guint64 base_rtptime, guint64 base_time, guint clock_rate, const guint64 extrtptime, const guint64 base_rtptime,
gint64 rtp_clock_base) const guint64 base_time, const guint clock_rate,
const guint64 rtp_clock_base)
{ {
/* Don't do any stream offsetting in RFC7273 sync mode. Everything is /* Don't do any stream offsetting in RFC7273 sync mode. Everything is
* handled inside rtpjitterbuffer for this case. */ * handled inside rtpjitterbuffer for this case. */