Chromium Code Reviews| Index: webrtc/modules/audio_coding/neteq/rtcp.cc |
| diff --git a/webrtc/modules/audio_coding/neteq/rtcp.cc b/webrtc/modules/audio_coding/neteq/rtcp.cc |
| index 7ef40bc814548de74cc86df77a3f4633a76c8f54..0263e763efb56dc377d4ef8308a0cb1185409297 100644 |
| --- a/webrtc/modules/audio_coding/neteq/rtcp.cc |
| +++ b/webrtc/modules/audio_coding/neteq/rtcp.cc |
| @@ -10,11 +10,11 @@ |
| #include "webrtc/modules/audio_coding/neteq/rtcp.h" |
| +#include <stdlib.h> |
| #include <string.h> |
| #include <algorithm> |
| -#include "webrtc/common_audio/signal_processing/include/signal_processing_library.h" |
| #include "webrtc/modules/include/module_common_types.h" |
| namespace webrtc { |
| @@ -46,10 +46,10 @@ void Rtcp::Update(const RTPHeader& rtp_header, uint32_t receive_timestamp) { |
| // Note that the value in |jitter_| is in Q4. |
| if (received_packets_ > 1) { |
| int32_t ts_diff = receive_timestamp - (rtp_header.timestamp - transit_); |
| - ts_diff = WEBRTC_SPL_ABS_W32(ts_diff); |
| - int32_t jitter_diff = (ts_diff << 4) - jitter_; |
| + int64_t jitter_diff = (std::abs(int64_t{ts_diff}) << 4) - jitter_; |
| // Calculate 15 * jitter_ / 16 + jitter_diff / 16 (with proper rounding). |
| jitter_ = jitter_ + ((jitter_diff + 8) >> 4); |
| + RTC_DCHECK_GE(jitter_, 0); |
|
ossu
2016/10/31 14:53:48
This, really, should be impossible: jitter_diff sh
|
| } |
| transit_ = rtp_header.timestamp - receive_timestamp; |
| } |