[webkit-reviews] review granted: [Bug 170644] [Cocoa] WebKit unnecessarily soft-links CorePrediction : [Attachment 306772] Link CorePrediction at build time

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Apr 11 10:34:59 PDT 2017

Sam Weinig <sam at webkit.org> has granted mitz at webkit.org's request for review:
Bug 170644: [Cocoa] WebKit unnecessarily soft-links CorePrediction

Attachment 306772: Link CorePrediction at build time


--- Comment #16 from Sam Weinig <sam at webkit.org> ---
Comment on attachment 306772
  --> https://bugs.webkit.org/attachment.cgi?id=306772
Link CorePrediction at build time

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

> +static inline bool isNullFunctionPointer(void* functionPointer)

This surprising function should probably have a comment explaining why it needs
inline-asm.  Also, is this something that can be lifted into WTF somewhere, as
it seems like something that could be generally useful for other weak linking.

> Source/WebKit2/UIProcess/WebResourceLoadStatisticsStore.h:36

Is it really necessary to have both predicates. Given that CORE_PREDICTION is
only on COCOA systems, it seems like just specifying it would be enough.

More information about the webkit-reviews mailing list