Changeset 268841 in webkit

Timestamp:

Oct 21, 2020, 3:38:09 PM (6 years ago)

Author:

Chris Dumez

Message:

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

• Modules/webaudio/AudioSummingJunction.cpp:

(WebCore::AudioSummingJunction::maximumNumberOfChannels const):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• Modules/webaudio/AudioSummingJunction.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2020-10-21  Chris Dumez  <cdumez@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  <cdumez@apple.com>
 

trunk/Source/WebCore/Modules/webaudio/AudioSummingJunction.cpp

@@ -95 +95 @@
 {
     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;
 }