[Webkit-unassigned] [Bug 25884] WebKit needs a style linting tool

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=25884


Albert J. Wong <ajwong at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |ajwong at chromium.org




--- Comment #10 from Albert J. Wong <ajwong at chromium.org>  2009-07-07 18:49:36 PDT ---
>From the peanut gallery...

You'll never get rid of all false positives, especially since cpplint is
basically based off a list of regexes.  However, tilting it to be overly
permissive would be a good starting point.

As for making the linter understand just the diff, I suppose if you tone down
what it can check enough, you'd be able to do this.  However, it's likely to
become much more flakey.  (eg., how does it know what indentation level to
expect if the context for the diff starts mid-block?)

If the main behavior you want is to be able to filter out already existing lint
issues, that can be done as a post-process step for the lint output.  Write a
wrapper that takes the base file, the diff, produces the output file to lint,
and then only returns lint errors for changed lines.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.