Fix isValidEnum to work with older versions of GCC
authorandersca@apple.com <andersca@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 16 Sep 2016 17:46:51 +0000 (17:46 +0000)
committerandersca@apple.com <andersca@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 16 Sep 2016 17:46:51 +0000 (17:46 +0000)
https://bugs.webkit.org/show_bug.cgi?id=162065

Reviewed by Michael Catanzaro.

Source/WTF:

Use the ternary operator instead of a more complex function body.

* wtf/EnumTraits.h:

Tools:

Add EnumTraits.cpp to the CMake build.

* TestWebKitAPI/CMakeLists.txt:

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

Source/WTF/ChangeLog
Source/WTF/wtf/EnumTraits.h
Tools/ChangeLog
Tools/TestWebKitAPI/CMakeLists.txt

index 85b500b..9e2fc7c 100644 (file)
@@ -1,3 +1,14 @@
+2016-09-16  Anders Carlsson  <andersca@apple.com>
+
+        Fix isValidEnum to work with older versions of GCC
+        https://bugs.webkit.org/show_bug.cgi?id=162065
+
+        Reviewed by Michael Catanzaro.
+
+        Use the ternary operator instead of a more complex function body.
+
+        * wtf/EnumTraits.h:
+
 2016-09-15  JF Bastien  <jfbastien@apple.com>
 
         TextBreakIterator: unconvolute character break cache
index 09295ee..c6bde9f 100644 (file)
@@ -39,10 +39,7 @@ template<typename T, typename E, E e, E... es>
 struct EnumValueChecker<T, EnumValues<E, e, es...>> {
     static constexpr bool isValidEnum(T t)
     {
-        if (static_cast<T>(e) == t)
-            return true;
-
-        return EnumValueChecker<T, EnumValues<E, es...>>::isValidEnum(t);
+        return (static_cast<T>(e) == t) ? true : EnumValueChecker<T, EnumValues<E, es...>>::isValidEnum(t);
     }
 };
 
index 0031b1f..74ce380 100644 (file)
@@ -1,3 +1,14 @@
+2016-09-16  Anders Carlsson  <andersca@apple.com>
+
+        Fix isValidEnum to work with older versions of GCC
+        https://bugs.webkit.org/show_bug.cgi?id=162065
+
+        Reviewed by Michael Catanzaro.
+
+        Add EnumTraits.cpp to the CMake build.
+
+        * TestWebKitAPI/CMakeLists.txt:
+
 2016-09-16  Nael Ouedraogo  <nael.ouedraogo@crf.canon.fr>
 
         Unreviewed: Change my primary address for auto-complete in bugzilla.
index 220af77..2da4c8d 100644 (file)
@@ -48,6 +48,7 @@ set(TestWTF_SOURCES
     ${TESTWEBKITAPI_DIR}/Tests/WTF/Condition.cpp
     ${TESTWEBKITAPI_DIR}/Tests/WTF/DateMath.cpp
     ${TESTWEBKITAPI_DIR}/Tests/WTF/Deque.cpp
+    ${TESTWEBKITAPI_DIR}/Tests/WTF/EnumTraits.cpp
     ${TESTWEBKITAPI_DIR}/Tests/WTF/HashCountedSet.cpp
     ${TESTWEBKITAPI_DIR}/Tests/WTF/HashMap.cpp
     ${TESTWEBKITAPI_DIR}/Tests/WTF/HashSet.cpp