[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