[webkit-reviews] review denied: [Bug 227436] Change static funtion to extern in KeywordLookup.h. : [Attachment 432366] Patch

[bugzilla-daemon at webkit.org]

Michael Catanzaro <mcatanzaro at gnome.org> has denied Joonghun Park
<jh718.park at samsung.com>'s request for review:
Bug 227436: Change static funtion to extern in KeywordLookup.h.
https://bugs.webkit.org/show_bug.cgi?id=227436

Attachment 432366: Patch

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




--- Comment #5 from Michael Catanzaro <mcatanzaro at gnome.org> ---
Comment on attachment 432366
  --> https://bugs.webkit.org/attachment.cgi?id=432366
Patch

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

>>> Source/JavaScriptCore/parser/Lexer.cpp:788
>>> +ALWAYS_INLINE bool cannotBeIdentPartOrEscapeStart(LChar c)
>> 
>> Problem is using ALWAYS_INLINE for a function that's not defined in the
header doesn't make sense. We need to either:
>> 
>>  (a) Also remove ALWAYS_INLINE, or
>>  (b) Move the definitions of these functions to KeywordLookup.h (which is
generated by KeywordLookupGenerator.py)
> 
> Please do not remove ALWAYS_INLINE. They are *super* critical
performance-sensitive function, and removing that can cause severe perf impact
on several benchmarks.

Let's do (b) then.