[webkit-reviews] review requested: [Bug 31618] "using" statement coding style guidelines need clarification : [Attachment 43413] Proposed patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Wed Nov 18 01:25:24 PST 2009
Chris Jerdonek <chris.jerdonek at gmail.com> has asked for review:
Bug 31618: "using" statement coding style guidelines need clarification
https://bugs.webkit.org/show_bug.cgi?id=31618
Attachment 43413: Proposed patch
https://bugs.webkit.org/attachment.cgi?id=43413&action=review
------- Additional Comments from Chris Jerdonek <chris.jerdonek at gmail.com>
(1) This patch corrects and clarifies the style guidelines re: "using"
statements, per the report.
(2) It also clarifies a part of the web page about code cleanup:
http://webkit.org/projects/cleanup/index.html
The web site currently gives the impression that the WebKit project wants
patches whose sole purpose is to clean up the code, but Alexey and Mark
clarified otherwise--
https://bugs.webkit.org/show_bug.cgi?id=31526
(3) The style page now validates.
(4) The patch clarifies on the web site that JavaScriptGlue does not conform
well to the guidelines (per Darin's e-mail on webkit-dev).
(I am still fuzzy on whether JSG does not conform because it is exempt from the
guidelines -- so that it will never conform, or because less work has gone into
it. If the former, the site should probably state that somewhere so people
don't invest unnecessary work in trying to clean it up.)
More information about the webkit-reviews
mailing list