[webkit-reviews] review requested: [Bug 25884] WebKit needs a style linting tool : [Attachment 32509] Add cpplint w/ some autofix

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jul 9 03:08:59 PDT 2009


Shinichiro Hamaji <hamaji at chromium.org> has asked  for review:
Bug 25884: WebKit needs a style linting tool
https://bugs.webkit.org/show_bug.cgi?id=25884

Attachment 32509: Add cpplint w/ some autofix
https://bugs.webkit.org/attachment.cgi?id=32509&action=review

------- Additional Comments from Shinichiro Hamaji <hamaji at chromium.org>
As most common kinds of lint errors seem to be trivial ones, I implemented
autofix feature for most common 10 kinds of warnings. I applied the tool for
some components in WebCore (css, dom, editing, rendering, history, storage,
html, svg, wml, inspector, workers, xml, and page) and it seems to be working
good. It found 298 lint errors and fixed 247 of them automatically. There were
no false-positives in the 247 error (if my investigation on the git-diff was
correct).


More information about the webkit-reviews mailing list