[webkit-reviews] review granted: [Bug 73789] Need SSE optimization for SincResampler::Process() : [Attachment 118305] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Dec 7 20:08:33 PST 2011


Benjamin Poulain <benjamin at webkit.org> has granted xingnan.wang at intel.com's
request for review:
Bug 73789: Need SSE optimization for SincResampler::Process()
https://bugs.webkit.org/show_bug.cgi?id=73789

Attachment 118305: Patch
https://bugs.webkit.org/attachment.cgi?id=118305&action=review

------- Additional Comments from Benjamin Poulain <benjamin at webkit.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=118305&action=review


> Source/WebCore/platform/audio/SincResampler.cpp:319
> +		   // Summarize the SSE results to sum1 and sum2

Make comments look like sentences by starting with a capital letter and ending
with a period (punctation). One exception may be end of line comments like this
"if (x == y) // false for NaN".


More information about the webkit-reviews mailing list