[webkit-reviews] review granted: [Bug 197294] [bmalloc] Follow-up and fixing bug after r244481 : [Attachment 368268] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Apr 25 14:20:38 PDT 2019
Saam Barati <sbarati at apple.com> has granted Yusuke Suzuki <ysuzuki at apple.com>'s
request for review:
Bug 197294: [bmalloc] Follow-up and fixing bug after r244481
https://bugs.webkit.org/show_bug.cgi?id=197294
Attachment 368268: Patch
https://bugs.webkit.org/attachment.cgi?id=368268&action=review
--- Comment #2 from Saam Barati <sbarati at apple.com> ---
Comment on attachment 368268
--> https://bugs.webkit.org/attachment.cgi?id=368268
Patch
View in context: https://bugs.webkit.org/attachment.cgi?id=368268&action=review
r=me
> Source/bmalloc/bmalloc/IsoHeapImplInlines.h:234
> + auto getNewAllocationMode = [&] {
you can also make this cleaner by doing:
- removing all assignments to m_lastSlowPathTime in the lambda
- "auto now = std::chrono::steady_clock::now();" outside the lambda.
- then below, you could do: m_lastSlowPathTime = now;
More information about the webkit-reviews
mailing list