[webkit-reviews] review granted: [Bug 170242] WebAssembly: NFC s/goto/lambda/g : [Attachment 305740] patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Mar 29 09:17:01 PDT 2017


Mark Lam <mark.lam at apple.com> has granted JF Bastien <jfbastien at apple.com>'s
request for review:
Bug 170242: WebAssembly: NFC s/goto/lambda/g
https://bugs.webkit.org/show_bug.cgi?id=170242

Attachment 305740: patch

https://bugs.webkit.org/attachment.cgi?id=305740&action=review




--- Comment #2 from Mark Lam <mark.lam at apple.com> ---
Comment on attachment 305740
  --> https://bugs.webkit.org/attachment.cgi?id=305740
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=305740&action=review

r=me with suggested fix.

> Source/JavaScriptCore/wasm/WasmMemory.cpp:121
> +    auto fail = [&] () -> bool {

This does not need to be a [&] lambda, right?  Just [] should do.


More information about the webkit-reviews mailing list