rtpjitterbuffer: don't log all clock_rate changes as warnings.

We never initialize clock_rate explicitly, therefore it is 0 by default. The
parameter is a uint32 and the only caller ensure that it is >0, therefore it
won't become -1 ever.
This commit is contained in:
Stefan Sauer 2014-10-04 17:17:13 +02:00
parent f35f3ccf7c
commit 98222a67ff

View file

@ -170,6 +170,7 @@ rtp_jitter_buffer_set_delay (RTPJitterBuffer * jbuf, GstClockTime delay)
/** /**
* rtp_jitter_buffer_set_clock_rate: * rtp_jitter_buffer_set_clock_rate:
* @jbuf: an #RTPJitterBuffer * @jbuf: an #RTPJitterBuffer
* @clock_rate: the new clock rate
* *
* Set the clock rate in the jitterbuffer. * Set the clock rate in the jitterbuffer.
*/ */
@ -177,13 +178,8 @@ void
rtp_jitter_buffer_set_clock_rate (RTPJitterBuffer * jbuf, guint32 clock_rate) rtp_jitter_buffer_set_clock_rate (RTPJitterBuffer * jbuf, guint32 clock_rate)
{ {
if (jbuf->clock_rate != clock_rate) { if (jbuf->clock_rate != clock_rate) {
if (jbuf->clock_rate == -1) { GST_DEBUG ("Clock rate changed from %" G_GUINT32_FORMAT " to %"
GST_DEBUG ("Clock rate changed from %" G_GUINT32_FORMAT " to %" G_GUINT32_FORMAT, jbuf->clock_rate, clock_rate);
G_GUINT32_FORMAT, jbuf->clock_rate, clock_rate);
} else {
GST_WARNING ("Clock rate changed from %" G_GUINT32_FORMAT " to %"
G_GUINT32_FORMAT, jbuf->clock_rate, clock_rate);
}
jbuf->clock_rate = clock_rate; jbuf->clock_rate = clock_rate;
rtp_jitter_buffer_reset_skew (jbuf); rtp_jitter_buffer_reset_skew (jbuf);
} }