admin/webrtc_m130
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix VideoSendStreamImpl::OnPacketFeedbackError: operator precedence.

Browse Source 
in:
if (auto it = some_set.find(some_value) != some_set.end()) {}

"it" is bool, not an iterator.

Bug: webrtc:9652
Change-Id: Icfaab685f4e2f4456e24d5a14b11309dddfdc0fe
Reviewed-on: https://webrtc-review.googlesource.com/84420
Commit-Queue: Benoit Lize <lizeb@chromium.org>
Reviewed-by: Stefan Holmer <stefan@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24389}
This commit is contained in:
Benoît Lizé 2018-06-20 14:00:34 +02:00 committed by Commit Bot
parent ec93075c00
commit 86fbea10a5
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
video/video_send_stream_impl.cc
View File

@ -709,8 +709,8 @@ void VideoSendStreamImpl::OnPacketFeedbackVector(
}
// Lost feedbacks are not considered to be lost packets.
for (const PacketFeedback& packet : packet_feedback_vector) {
if (auto it = feedback_packet_seq_num_set_.find(packet.sequence_number) !=
feedback_packet_seq_num_set_.end()) {
auto it = feedback_packet_seq_num_set_.find(packet.sequence_number);
if (it != feedback_packet_seq_num_set_.end()) {
const bool lost = packet.arrival_time_ms == PacketFeedback::kNotReceived;
loss_mask_vector_.push_back(lost);
feedback_packet_seq_num_set_.erase(it);