[Webkit-unassigned] [Bug 40724] Check for extra spacing on a variable declaration line.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jun 17 21:43:02 PDT 2010


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


Shinichiro Hamaji <hamaji at chromium.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #59016|review?                     |review+
               Flag|                            |




--- Comment #5 from Shinichiro Hamaji <hamaji at chromium.org>  2010-06-17 21:43:02 PST ---
(From update of attachment 59016)
Looks great. Thanks for your quick update.

> > - You may have already done this, but please check your patch with existing code and check if there are new false positives. For example, I often run ./WebKitTools/Scripts/run-webkit-style WebCore/*/*.cpp | grep <the new error message> to see the change.
> 
> This seemed to pass for me, it even spotted a handful of existing errors :)

Nice! Thanks for your checking.

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



More information about the webkit-unassigned mailing list