commit f6aade0adad9ae1bc880c9246e2854a5276045c0
author Joachim Reiersen <joachimr@meta.com> Tue Aug 05 21:03:48 2025
committer WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> Mon Aug 11 18:34:23 2025
tree 663a4b9fa63607fa0a7cdcaa70a1fe851664578e
parent 85642b2e0881e2dc8466dffd173a33873a5e4d97

Separate assignment statements in echo_canceller3

Two variable assignments had been combined into one statement as they
were separated by commas, not semicolons. Based on the formatting of
surrounding code this looks unintentional.

(While not a logical bug, fixing as this caused a compilation issue in
one particular internal build configuration)

Bug: None
Change-Id: I076209010e012bc59617439523edba637d54bb10
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/403186
Reviewed-by: Per Åhgren <peah@webrtc.org>
Reviewed-by: Gustaf Ullberg <gustaf@webrtc.org>
Commit-Queue: Joachim Reiersen <joachimr@meta.com>
Cr-Commit-Position: refs/heads/main@{#45318}

modules/audio_processing/aec3/echo_canceller3.cc

diff --git a/modules/audio_processing/aec3/echo_canceller3.cc b/modules/audio_processing/aec3/echo_canceller3.cc
index b6752ef..068a0fd 100644
--- a/modules/audio_processing/aec3/echo_canceller3.cc
+++ b/modules/audio_processing/aec3/echo_canceller3.cc
@@ -811,7 +811,7 @@
     linear_output_framer_.reset(
         new BlockFramer(/*num_bands=*/1, num_capture_channels_));
     linear_output_block_ =
-        std::make_unique<Block>(/*num_bands=*/1, num_capture_channels_),
+        std::make_unique<Block>(/*num_bands=*/1, num_capture_channels_);
     linear_output_sub_frame_view_ = std::vector<std::vector<ArrayView<float>>>(
         1, std::vector<ArrayView<float>>(num_capture_channels_));
   }