[webkit-reviews] review granted: [Bug 223452] Avoid heap allocation under AudioWorkletNode::process() : [Attachment 423650] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Mar 18 14:33:07 PDT 2021
Darin Adler <darin at apple.com> has granted Chris Dumez <cdumez at apple.com>'s
request for review:
Bug 223452: Avoid heap allocation under AudioWorkletNode::process()
https://bugs.webkit.org/show_bug.cgi?id=223452
Attachment 423650: Patch
https://bugs.webkit.org/attachment.cgi?id=423650&action=review
--- Comment #4 from Darin Adler <darin at apple.com> ---
Comment on attachment 423650
--> https://bugs.webkit.org/attachment.cgi?id=423650
Patch
View in context: https://bugs.webkit.org/attachment.cgi?id=423650&action=review
> Source/WebCore/ChangeLog:26
> + behavior is thus consistant with Blink.
consistent
> Source/WebCore/Modules/webaudio/AudioWorkletNode.cpp:215
> + std::fill(paramValues->data(), paramValues->data() +
framesToProcess, audioParam->finalValue());
How about using std::fill_n instead?
More information about the webkit-reviews
mailing list