Title: [268841] trunk/Source/WebCore
Revision
268841
Author
cdu...@apple.com
Date
2020-10-21 15:38:09 -0700 (Wed, 21 Oct 2020)

Log Message

Address post-landing review feedback from Sam Weinig for r268820.

* Modules/webaudio/AudioSummingJunction.cpp:
(WebCore::AudioSummingJunction::maximumNumberOfChannels const):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (268840 => 268841)


--- trunk/Source/WebCore/ChangeLog	2020-10-21 22:22:09 UTC (rev 268840)
+++ trunk/Source/WebCore/ChangeLog	2020-10-21 22:38:09 UTC (rev 268841)
@@ -1,5 +1,12 @@
 2020-10-21  Chris Dumez  <cdu...@apple.com>
 
+        Address post-landing review feedback from Sam Weinig for r268820.
+
+        * Modules/webaudio/AudioSummingJunction.cpp:
+        (WebCore::AudioSummingJunction::maximumNumberOfChannels const):
+
+2020-10-21  Chris Dumez  <cdu...@apple.com>
+
         Update FFTFrame::realData() / imagData() to return an AudioFloatArray
         https://bugs.webkit.org/show_bug.cgi?id=218048
 

Modified: trunk/Source/WebCore/Modules/webaudio/AudioSummingJunction.cpp (268840 => 268841)


--- trunk/Source/WebCore/Modules/webaudio/AudioSummingJunction.cpp	2020-10-21 22:22:09 UTC (rev 268840)
+++ trunk/Source/WebCore/Modules/webaudio/AudioSummingJunction.cpp	2020-10-21 22:38:09 UTC (rev 268841)
@@ -94,11 +94,11 @@
 unsigned AudioSummingJunction::maximumNumberOfChannels() const
 {
     unsigned maxChannels = 0;
-    std::for_each(m_outputs.begin(), m_outputs.end(), [&](auto* output) {
+    for (auto* output : m_outputs) {
         // Use output()->numberOfChannels() instead of output->bus()->numberOfChannels(),
         // because the calling of AudioNodeOutput::bus() is not safe here.
         maxChannels = std::max(maxChannels, output->numberOfChannels());
-    });
+    }
     return maxChannels;
 }
 
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to