Title: [268558] trunk/Source/WebCore
- Revision
- 268558
- Author
- [email protected]
- Date
- 2020-10-15 15:24:27 -0700 (Thu, 15 Oct 2020)
Log 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):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (268557 => 268558)
--- trunk/Source/WebCore/ChangeLog 2020-10-15 22:02:32 UTC (rev 268557)
+++ trunk/Source/WebCore/ChangeLog 2020-10-15 22:24:27 UTC (rev 268558)
@@ -1,5 +1,19 @@
2020-10-15 Chris Dumez <[email protected]>
+ 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 <[email protected]>
+
Avoid double zero'ing of AudioArray in DelayDSPKernel constructors
https://bugs.webkit.org/show_bug.cgi?id=217779
Modified: trunk/Source/WebCore/Modules/webaudio/ConstantSourceNode.cpp (268557 => 268558)
--- trunk/Source/WebCore/Modules/webaudio/ConstantSourceNode.cpp 2020-10-15 22:02:32 UTC (rev 268557)
+++ trunk/Source/WebCore/Modules/webaudio/ConstantSourceNode.cpp 2020-10-15 22:24:27 UTC (rev 268558)
@@ -98,9 +98,7 @@
outputBus.zero();
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();
}
}
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
