[webkit-reviews] review granted: [Bug 196922] [WTF] holdLock should be marked WARN_UNUSED_RETURN : [Attachment 367476] Patch

[bugzilla-daemon at webkit.org]

Keith Miller <keith_miller at apple.com> has granted Robin Morisset
<rmorisset at apple.com>'s request for review:
Bug 196922: [WTF] holdLock should be marked WARN_UNUSED_RETURN
https://bugs.webkit.org/show_bug.cgi?id=196922

Attachment 367476: Patch

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




--- Comment #4 from Keith Miller <keith_miller at apple.com> ---
Comment on attachment 367476
  --> https://bugs.webkit.org/attachment.cgi?id=367476
Patch

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

r=me

> Source/JavaScriptCore/heap/BlockDirectory.cpp:66
> +	       // FIXME: should we instead do a cheaper thing like a volatile
load in the page?

Wow, that’s weird. What’s this function used for?