Changeset 268558 in webkit

Timestamp:

Oct 15, 2020 3:24:27 PM (4 years ago)

Author:

Chris Dumez

Message:

Use std::fill_n() instead of for loops in ConstantSourceNode
​https://bugs.webkit.org/show_bug.cgi?id=217777

Reviewed by Eric Carlson.

Use std::fill_n() instead of for loops in ConstantSourceNode.

No new tests, no Web-facing behavior change.

• Modules/webaudio/ConstantSourceNode.cpp:

(WebCore::ConstantSourceNode::process):

Location:

trunk/Source/WebCore

Files:

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

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2020-10-15  Chris Dumez  <cdumez@apple.com>
+
+        Use std::fill_n() instead of for loops in ConstantSourceNode
+        https://bugs.webkit.org/show_bug.cgi?id=217777
+
+        Reviewed by Eric Carlson.
+
+        Use std::fill_n() instead of for loops in ConstantSourceNode.
+
+        No new tests, no Web-facing behavior change.
+
+        * Modules/webaudio/ConstantSourceNode.cpp:
+        (WebCore::ConstantSourceNode::process):
+
 2020-10-15  Chris Dumez  <cdumez@apple.com>
 

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

@@ -99 +99 @@
     else {
         float* dest = outputBus.channel(0)->mutableData();
-        dest += quantumFrameOffset;
-        for (unsigned i = 0; i < nonSilentFramesToProcess; ++i)
-            dest[i] = value;
+        std::fill_n(dest + quantumFrameOffset, nonSilentFramesToProcess, value);
         outputBus.clearSilentFlag();
     }