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

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

Issue 2080553003: Style updates for ForwardErrorCorrection and related classes (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Re-add explicit constructors. Created 4 years, 5 months 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) 2012 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2012 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 220 matching lines...) Expand 10 before | Expand all | Expand 10 after
231 } 231 }
232 crit_sect_.Enter(); 232 crit_sect_.Enter();
233 } 233 }
234 if (fec_->DecodeFEC(&received_packet_list_, &recovered_packet_list_) != 0) { 234 if (fec_->DecodeFEC(&received_packet_list_, &recovered_packet_list_) != 0) {
235 crit_sect_.Leave(); 235 crit_sect_.Leave();
236 return -1; 236 return -1;
237 } 237 }
238 assert(received_packet_list_.empty()); 238 assert(received_packet_list_.empty());
239 } 239 }
240 // Send any recovered media packets to VCM. 240 // Send any recovered media packets to VCM.
241 ForwardErrorCorrection::RecoveredPacketList::iterator it = 241 auto it = recovered_packet_list_.begin();
danilchap 2016/06/29 10:31:52 make one more step and use c++11 loop: for (auto*
brandtr 2016/06/29 14:24:10 Done.
242 recovered_packet_list_.begin();
243 for (; it != recovered_packet_list_.end(); ++it) { 242 for (; it != recovered_packet_list_.end(); ++it) {
244 if ((*it)->returned) // Already sent to the VCM and the jitter buffer. 243 if ((*it)->returned) // Already sent to the VCM and the jitter buffer.
245 continue; 244 continue;
246 ForwardErrorCorrection::Packet* packet = (*it)->pkt; 245 ForwardErrorCorrection::Packet* packet = (*it)->pkt;
247 ++packet_counter_.num_recovered_packets; 246 ++packet_counter_.num_recovered_packets;
248 crit_sect_.Leave(); 247 crit_sect_.Leave();
249 if (!recovered_packet_callback_->OnRecoveredPacket(packet->data, 248 if (!recovered_packet_callback_->OnRecoveredPacket(packet->data,
250 packet->length)) { 249 packet->length)) {
251 return -1; 250 return -1;
252 } 251 }
253 crit_sect_.Enter(); 252 crit_sect_.Enter();
254 (*it)->returned = true; 253 (*it)->returned = true;
255 } 254 }
256 crit_sect_.Leave(); 255 crit_sect_.Leave();
257 return 0; 256 return 0;
258 } 257 }
259 258
260 } // namespace webrtc 259 } // namespace webrtc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698