[CMake] Features list should print dots every other row
authormcatanzaro@igalia.com <mcatanzaro@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 22 Apr 2015 16:40:26 +0000 (16:40 +0000)
committermcatanzaro@igalia.com <mcatanzaro@igalia.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 22 Apr 2015 16:40:26 +0000 (16:40 +0000)
https://bugs.webkit.org/show_bug.cgi?id=143832

Reviewed by Martin Robinson.

Discount private options when determining whether to print dots on a given row of the
features list. Also, simplify the logic by using only one for loop, and fix a couple errors
(inverted use of SHOULD_PRINT_DOTS, one use of the renamed variable SHOULD_PRINT_POINTS that
wasn't noticed because it only affects the first line, and use of the variable name as a
string in a conditional.)

* Source/cmake/WebKitFeatures.cmake:

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

ChangeLog
Source/cmake/WebKitFeatures.cmake

index 9478356..e955bd0 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,20 @@
 2015-04-22  Michael Catanzaro  <mcatanzaro@igalia.com>
 
+        [CMake] Features list should print dots every other row
+        https://bugs.webkit.org/show_bug.cgi?id=143832
+
+        Reviewed by Martin Robinson.
+
+        Discount private options when determining whether to print dots on a given row of the
+        features list. Also, simplify the logic by using only one for loop, and fix a couple errors
+        (inverted use of SHOULD_PRINT_DOTS, one use of the renamed variable SHOULD_PRINT_POINTS that
+        wasn't noticed because it only affects the first line, and use of the variable name as a
+        string in a conditional.)
+
+        * Source/cmake/WebKitFeatures.cmake:
+
+2015-04-22  Michael Catanzaro  <mcatanzaro@igalia.com>
+
         [CMake] Require specifying visibility of WebKit options
         https://bugs.webkit.org/show_bug.cgi?id=143831
 
index ea87838..54fe327 100644 (file)
@@ -195,28 +195,24 @@ endmacro()
 macro(PRINT_WEBKIT_OPTIONS)
     message(STATUS "Enabled features:")
 
-    set(_SHOULD_PRINT_POINTS OFF)
+    set(_SHOULD_PRINT_DOTS ON)
     foreach (_name ${_WEBKIT_AVAILABLE_OPTIONS})
-        string(LENGTH ${_name} _NAME_LENGTH)
-
-        set(_MESSAGE " ${_name} ")
+        if (${_WEBKIT_AVAILABLE_OPTIONS_IS_PUBLIC_${_name}})
+            string(LENGTH ${_name} _NAME_LENGTH)
+            set(_MESSAGE " ${_name} ")
 
-        # Print dots on every other row, for readability.
-        if (_SHOULD_PRINT_DOTS)
+            # Print dots on every other row, for readability.
             foreach (IGNORE RANGE ${_NAME_LENGTH} ${_MAX_FEATURE_LENGTH})
-                set(_MESSAGE "${_MESSAGE} ")
+                if (${_SHOULD_PRINT_DOTS})
+                    set(_MESSAGE "${_MESSAGE}.")
+                else ()
+                    set(_MESSAGE "${_MESSAGE} ")
+                endif ()
             endforeach ()
-            set(_SHOULD_PRINT_DOTS OFF)
-        else ()
-            foreach (IGNORE RANGE ${_NAME_LENGTH} ${_MAX_FEATURE_LENGTH})
-                set(_MESSAGE "${_MESSAGE}.")
-            endforeach ()
-            set(_SHOULD_PRINT_DOTS ON)
-        endif ()
 
-        set(_MESSAGE "${_MESSAGE} ${${_name}}")
+            set(_SHOULD_PRINT_DOTS (NOT ${_SHOULD_PRINT_DOTS}))
 
-        if (${_WEBKIT_AVAILABLE_OPTIONS_IS_PUBLIC_${_name}})
+            set(_MESSAGE "${_MESSAGE} ${${_name}}")
             message(STATUS "${_MESSAGE}")
         endif ()
     endforeach ()