From 73ef48cf05b39b0d2a5c289c1a3c0776bf2b4008 Mon Sep 17 00:00:00 2001 From: Hans Wennborg Date: Wed, 30 Mar 2022 14:15:51 +0200 Subject: [PATCH] Remove set but otherwise unused variables Recent Clang versions have enhanced -Wunused-but-set-variable which now warns about this. Bug: chromium:1309955 Change-Id: Ie70df85f5a6d2cbabb4e10960bfd926ff7bf32cb Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/257162 Reviewed-by: Henrik Lundin Reviewed-by: Henrik Andreassson Commit-Queue: Hans Wennborg Cr-Commit-Position: refs/heads/main@{#36381} --- media/engine/unhandled_packets_buffer.cc | 2 -- modules/audio_coding/neteq/test/neteq_decoding_test.cc | 2 -- 2 files changed, 4 deletions(-) diff --git a/media/engine/unhandled_packets_buffer.cc b/media/engine/unhandled_packets_buffer.cc index cb6f0ec335..563712bdf3 100644 --- a/media/engine/unhandled_packets_buffer.cc +++ b/media/engine/unhandled_packets_buffer.cc @@ -46,7 +46,6 @@ void UnhandledPacketsBuffer::BackfillPackets( start = insert_pos_; } - size_t count = 0; std::vector remaining; remaining.reserve(kMaxStashedPackets); for (size_t i = 0; i < buffer_.size(); ++i) { @@ -56,7 +55,6 @@ void UnhandledPacketsBuffer::BackfillPackets( // scheme. const uint32_t ssrc = buffer_[pos].ssrc; if (absl::c_linear_search(ssrcs, ssrc)) { - ++count; consumer(ssrc, buffer_[pos].packet_time_us, buffer_[pos].packet); } else { remaining.push_back(buffer_[pos]); diff --git a/modules/audio_coding/neteq/test/neteq_decoding_test.cc b/modules/audio_coding/neteq/test/neteq_decoding_test.cc index 6f27cdad4f..fc49819cf9 100644 --- a/modules/audio_coding/neteq/test/neteq_decoding_test.cc +++ b/modules/audio_coding/neteq/test/neteq_decoding_test.cc @@ -225,7 +225,6 @@ void NetEqDecodingTest::WrapTest(uint16_t start_seq_no, // Insert speech for 2 seconds. const int kSpeechDurationMs = 2000; - int packets_inserted = 0; uint16_t last_seq_no; uint32_t last_timestamp; bool timestamp_wrapped = false; @@ -240,7 +239,6 @@ void NetEqDecodingTest::WrapTest(uint16_t start_seq_no, if (drop_seq_numbers.find(seq_no) == drop_seq_numbers.end()) { // This sequence number was not in the set to drop. Insert it. ASSERT_EQ(0, neteq_->InsertPacket(rtp_info, payload)); - ++packets_inserted; } NetEqNetworkStatistics network_stats; ASSERT_EQ(0, neteq_->NetworkStatistics(&network_stats));