[webkit-reviews] review granted: [Bug 180691] Symbol not found: __ZN3WTF8LockBase10unlockSlowEv : [Attachment 329096] [PATCH] Proposed Fix

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Dec 12 02:24:38 PST 2017

Yusuke Suzuki <utatane.tea at gmail.com> has granted Joseph Pecoraro
<joepeck at webkit.org>'s request for review:
Bug 180691: Symbol not found: __ZN3WTF8LockBase10unlockSlowEv

Attachment 329096: [PATCH] Proposed Fix


--- Comment #4 from Yusuke Suzuki <utatane.tea at gmail.com> ---
Comment on attachment 329096
  --> https://bugs.webkit.org/attachment.cgi?id=329096
[PATCH] Proposed Fix

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

>> Source/WTF/wtf/mac/DeprecatedSymbolsUsedBySafari.mm:132
> Not sure if it is worth adding a restriction here, such as:
>     && __MAC_OS_X_VERSION_MIN_REQUIRED < 101340
> But reviewers let me know if you think that would be worth doing.

I think this restriction is useful since it can be a marker for dropping this
in the future.
Adding `FIXME` comment is also nice.

More information about the webkit-reviews mailing list