[webkit-reviews] review granted: [Bug 71647] DFG operationCreateThis slow path may get the wrong callee in case of inlining : [Attachment 113820] the patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Sun Nov 6 22:19:51 PST 2011
Oliver Hunt <oliver at apple.com> has granted Filip Pizlo <fpizlo at apple.com>'s
request for review:
Bug 71647: DFG operationCreateThis slow path may get the wrong callee in case
of inlining
https://bugs.webkit.org/show_bug.cgi?id=71647
Attachment 113820: the patch
https://bugs.webkit.org/attachment.cgi?id=113820&action=review
------- Additional Comments from Oliver Hunt <oliver at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=113820&action=review
>> Source/JavaScriptCore/dfg/DFGOperations.h:67
>> +typedef JSCell* DFG_OPERATION (*C_DFGOperation_ECC)(ExecState*, JSCell*,
JSCell*);
>
> DFG_OPERATION is incorrectly named. Don't use underscores in your identifier
names. [readability/naming] [4]
We really should beat the stylebot into not producing these warnings
More information about the webkit-reviews
mailing list