check-webkit-style needs to respect Qt API coding style
authorzoltan@webkit.org <zoltan@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 13 Jun 2012 15:15:44 +0000 (15:15 +0000)
committerzoltan@webkit.org <zoltan@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 13 Jun 2012 15:15:44 +0000 (15:15 +0000)
https://bugs.webkit.org/show_bug.cgi?id=88995

Reviewed by Csaba Osztrogon√°c.

Add -whitespace/declaration exception to Qt API files.

* Scripts/webkitpy/style/checker.py:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@120212 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/Scripts/webkitpy/style/checker.py

index 5d90309..a93b90d 100644 (file)
@@ -1,3 +1,14 @@
+2012-06-13  Zoltan Horvath  <zoltan@webkit.org>
+
+        check-webkit-style needs to respect Qt API coding style
+        https://bugs.webkit.org/show_bug.cgi?id=88995
+
+        Reviewed by Csaba Osztrogon√°c.
+
+        Add -whitespace/declaration exception to Qt API files.
+
+        * Scripts/webkitpy/style/checker.py:
+
 2012-06-13  Christophe Dumez  <christophe.dumez@intel.com>
 
         [WK2] Add implementation for dispatchIntent in WebFrameLoaderClient
index cd8f565..4d29684 100644 (file)
@@ -140,11 +140,17 @@ _PATH_RULES_SPECIFIER = [
       # Qt code uses '_' in some places (such as private slots
       # and on test xxx_data methos on tests)
       "Source/JavaScriptCore/qt/",
-      "Source/WebKit/qt/Api/",
       "Source/WebKit/qt/tests/",
       "Source/WebKit/qt/declarative/",
       "Source/WebKit/qt/examples/"],
      ["-readability/naming"]),
+
+    ([# The Qt APIs use Qt declaration style, it puts the * to
+      # the variable name, not to the class.
+      "Source/WebKit/qt/Api/"],
+     ["-readability/naming",
+      "-whitespace/declaration"]),
+
      ([# Qt's MiniBrowser has no config.h
        "Tools/MiniBrowser/qt"],
       ["-build/include"]),