Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(916)

Unified Diff: webrtc/modules/rtp_rtcp/source/rtp_packet.cc

Issue 2535593002: RTC_[D]CHECK_op: Remove "u" suffix on integer constants (Closed)
Patch Set: Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: webrtc/modules/rtp_rtcp/source/rtp_packet.cc
diff --git a/webrtc/modules/rtp_rtcp/source/rtp_packet.cc b/webrtc/modules/rtp_rtcp/source/rtp_packet.cc
index 245ea8b3520b69cd01ef16b3d8637217ad4d0b15..72fd7892b235f1c1118b07457d18f87f9e83e1e7 100644
--- a/webrtc/modules/rtp_rtcp/source/rtp_packet.cc
+++ b/webrtc/modules/rtp_rtcp/source/rtp_packet.cc
@@ -253,9 +253,9 @@ void Packet::SetSsrc(uint32_t ssrc) {
}
void Packet::SetCsrcs(const std::vector<uint32_t>& csrcs) {
- RTC_DCHECK_EQ(num_extensions_, 0u);
- RTC_DCHECK_EQ(payload_size_, 0u);
- RTC_DCHECK_EQ(padding_size_, 0u);
+ RTC_DCHECK_EQ(num_extensions_, 0);
+ RTC_DCHECK_EQ(payload_size_, 0);
+ RTC_DCHECK_EQ(padding_size_, 0);
RTC_DCHECK_LE(csrcs.size(), 0x0fu);
RTC_DCHECK_LE(kFixedHeaderSize + 4 * csrcs.size(), capacity());
payload_offset_ = kFixedHeaderSize + 4 * csrcs.size();
@@ -269,7 +269,7 @@ void Packet::SetCsrcs(const std::vector<uint32_t>& csrcs) {
}
uint8_t* Packet::AllocatePayload(size_t size_bytes) {
- RTC_DCHECK_EQ(padding_size_, 0u);
+ RTC_DCHECK_EQ(padding_size_, 0);
if (payload_offset_ + size_bytes > capacity()) {
LOG(LS_WARNING) << "Cannot set payload, not enough space in buffer.";
return nullptr;
@@ -283,7 +283,7 @@ uint8_t* Packet::AllocatePayload(size_t size_bytes) {
}
void Packet::SetPayloadSize(size_t size_bytes) {
- RTC_DCHECK_EQ(padding_size_, 0u);
+ RTC_DCHECK_EQ(padding_size_, 0);
RTC_DCHECK_LE(size_bytes, payload_size_);
payload_size_ = size_bytes;
buffer_.SetSize(payload_offset_ + payload_size_);
@@ -473,8 +473,8 @@ bool Packet::AllocateExtension(ExtensionType type,
if (extension_id == ExtensionManager::kInvalidId) {
return false;
}
- RTC_DCHECK_GT(length, 0u);
- RTC_DCHECK_LE(length, 16u);
+ RTC_DCHECK_GT(length, 0);
+ RTC_DCHECK_LE(length, 16);
size_t num_csrc = data()[0] & 0x0F;
size_t extensions_offset = kFixedHeaderSize + (num_csrc * 4) + 4;
« no previous file with comments | « webrtc/modules/rtp_rtcp/source/rtp_format_h264.cc ('k') | webrtc/modules/rtp_rtcp/source/rtp_packet_history.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698