[webkit-reviews] review denied: [Bug 51695] check-webkit-style treated some macros with parentheses after #elif as function calls : [Attachment 77947] proposed patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Jan 4 17:26:20 PST 2011


David Levin <levin at chromium.org> has denied Koan-Sin Tan
<koansin.tan at gmail.com>'s request for review:
Bug 51695: check-webkit-style treated some macros with parentheses after #elif
as function calls
https://bugs.webkit.org/show_bug.cgi?id=51695

Attachment 77947: proposed patch
https://bugs.webkit.org/attachment.cgi?id=77947&action=review

------- Additional Comments from David Levin <levin at chromium.org>
Really close!

It looks good but it needs a simple test to verify it. See cpp_unittest.py.
Look for the error "Extra space before (" to see where similar tests are and
add one which verifies that you don't get the error for the false positive
(#elif if I remember correctly).

Run the test using "test-webkitpy" to verify that your test passes.


More information about the webkit-reviews mailing list