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

Side by Side Diff: webrtc/modules/rtp_rtcp/source/rtp_format_h264.cc

Issue 2535593002: RTC_[D]CHECK_op: Remove "u" suffix on integer constants (Closed)
Patch Set: Created 4 years 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2014 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2014 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
(...skipping 176 matching lines...) Expand 10 before | Expand all | Expand 10 after
187 size_t packet_length = avg_size; 187 size_t packet_length = avg_size;
188 if (fragment_length < avg_size) 188 if (fragment_length < avg_size)
189 packet_length = fragment_length; 189 packet_length = fragment_length;
190 packets_.push(PacketUnit(Fragment(fragment.buffer + offset, packet_length), 190 packets_.push(PacketUnit(Fragment(fragment.buffer + offset, packet_length),
191 offset - kNalHeaderSize == 0, 191 offset - kNalHeaderSize == 0,
192 fragment_length == packet_length, false, 192 fragment_length == packet_length, false,
193 fragment.buffer[0])); 193 fragment.buffer[0]));
194 offset += packet_length; 194 offset += packet_length;
195 fragment_length -= packet_length; 195 fragment_length -= packet_length;
196 } 196 }
197 RTC_CHECK_EQ(0u, fragment_length); 197 RTC_CHECK_EQ(0, fragment_length);
198 } 198 }
199 199
200 size_t RtpPacketizerH264::PacketizeStapA(size_t fragment_index) { 200 size_t RtpPacketizerH264::PacketizeStapA(size_t fragment_index) {
201 // Aggregate fragments into one packet (STAP-A). 201 // Aggregate fragments into one packet (STAP-A).
202 size_t payload_size_left = max_payload_len_; 202 size_t payload_size_left = max_payload_len_;
203 int aggregated_fragments = 0; 203 int aggregated_fragments = 0;
204 size_t fragment_headers_length = 0; 204 size_t fragment_headers_length = 0;
205 const Fragment* fragment = &input_fragments_[fragment_index]; 205 const Fragment* fragment = &input_fragments_[fragment_index];
206 RTC_CHECK_GE(payload_size_left, fragment->length); 206 RTC_CHECK_GE(payload_size_left, fragment->length);
207 while (payload_size_left >= fragment->length + fragment_headers_length) { 207 while (payload_size_left >= fragment->length + fragment_headers_length) {
208 RTC_CHECK_GT(fragment->length, 0u); 208 RTC_CHECK_GT(fragment->length, 0);
209 packets_.push(PacketUnit(*fragment, aggregated_fragments == 0, false, true, 209 packets_.push(PacketUnit(*fragment, aggregated_fragments == 0, false, true,
210 fragment->buffer[0])); 210 fragment->buffer[0]));
211 payload_size_left -= fragment->length; 211 payload_size_left -= fragment->length;
212 payload_size_left -= fragment_headers_length; 212 payload_size_left -= fragment_headers_length;
213 213
214 // Next fragment. 214 // Next fragment.
215 ++fragment_index; 215 ++fragment_index;
216 if (fragment_index == input_fragments_.size()) 216 if (fragment_index == input_fragments_.size())
217 break; 217 break;
218 fragment = &input_fragments_[fragment_index]; 218 fragment = &input_fragments_[fragment_index];
(...skipping 376 matching lines...) Expand 10 before | Expand all | Expand 10 after
595 h264->packetization_type = kH264FuA; 595 h264->packetization_type = kH264FuA;
596 h264->nalu_type = original_nal_type; 596 h264->nalu_type = original_nal_type;
597 if (first_fragment) { 597 if (first_fragment) {
598 h264->nalus[h264->nalus_length] = nalu; 598 h264->nalus[h264->nalus_length] = nalu;
599 h264->nalus_length = 1; 599 h264->nalus_length = 1;
600 } 600 }
601 return true; 601 return true;
602 } 602 }
603 603
604 } // namespace webrtc 604 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/modules/rtp_rtcp/source/rtcp_packet/transport_feedback.cc ('k') | webrtc/modules/rtp_rtcp/source/rtp_packet.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698