Title: [220567] trunk
Revision
220567
Author
utatane....@gmail.com
Date
2017-08-10 17:36:24 -0700 (Thu, 10 Aug 2017)

Log Message

Unreviewed, attempt to fix build failure with VC2017

Source/WTF:

* wtf/PriorityQueue.h:

Tools:

* TestWebKitAPI/Tests/WTF/PriorityQueue.cpp:
(TEST):

Modified Paths

Diff

Modified: trunk/Source/WTF/ChangeLog (220566 => 220567)


--- trunk/Source/WTF/ChangeLog	2017-08-11 00:36:23 UTC (rev 220566)
+++ trunk/Source/WTF/ChangeLog	2017-08-11 00:36:24 UTC (rev 220567)
@@ -1,5 +1,11 @@
 2017-08-10  Yusuke Suzuki  <utatane....@gmail.com>
 
+        Unreviewed, attempt to fix build failure with VC2017
+
+        * wtf/PriorityQueue.h:
+
+2017-08-10  Yusuke Suzuki  <utatane....@gmail.com>
+
         Run more WTF tests
         https://bugs.webkit.org/show_bug.cgi?id=174970
 

Modified: trunk/Source/WTF/wtf/PriorityQueue.h (220566 => 220567)


--- trunk/Source/WTF/wtf/PriorityQueue.h	2017-08-11 00:36:23 UTC (rev 220566)
+++ trunk/Source/WTF/wtf/PriorityQueue.h	2017-08-11 00:36:24 UTC (rev 220567)
@@ -37,7 +37,7 @@
 // 2) You can iterate the elements.
 // 3) It has in-place decrease/increaseKey methods, although they are still O(n) rather than O(log(n)).
 
-template<typename T, bool (*isHigherPriority)(const T&, const T&) = isLessThan<T>, size_t inlineCapacity = 0>
+template<typename T, bool (*isHigherPriority)(const T&, const T&) = &isLessThan<T>, size_t inlineCapacity = 0>
 class PriorityQueue {
     using BufferType = Vector<T, inlineCapacity>;
     using const_iterator = typename BufferType::const_iterator;

Modified: trunk/Tools/ChangeLog (220566 => 220567)


--- trunk/Tools/ChangeLog	2017-08-11 00:36:23 UTC (rev 220566)
+++ trunk/Tools/ChangeLog	2017-08-11 00:36:24 UTC (rev 220567)
@@ -1,5 +1,12 @@
 2017-08-10  Yusuke Suzuki  <utatane....@gmail.com>
 
+        Unreviewed, attempt to fix build failure with VC2017
+
+        * TestWebKitAPI/Tests/WTF/PriorityQueue.cpp:
+        (TEST):
+
+2017-08-10  Yusuke Suzuki  <utatane....@gmail.com>
+
         Unreviewed, suppress warnings on GCC
 
         * TestWebKitAPI/Tests/WTF/CheckedArithmeticOperations.cpp:

Modified: trunk/Tools/TestWebKitAPI/Tests/WTF/PriorityQueue.cpp (220566 => 220567)


--- trunk/Tools/TestWebKitAPI/Tests/WTF/PriorityQueue.cpp	2017-08-11 00:36:23 UTC (rev 220566)
+++ trunk/Tools/TestWebKitAPI/Tests/WTF/PriorityQueue.cpp	2017-08-11 00:36:24 UTC (rev 220567)
@@ -76,7 +76,7 @@
 TEST(WTF_PriorityQueue, CustomPriorityFunction)
 {
     const unsigned numElements = 10;
-    PriorityQueue<unsigned, isGreaterThan<unsigned>> queue;
+    PriorityQueue<unsigned, &isGreaterThan<unsigned>> queue;
 
     EXPECT_EQ(0_z, queue.size());
     EXPECT_TRUE(queue.isEmpty());
@@ -106,7 +106,7 @@
 
 TEST(WTF_PriorityQueue, MoveOnly)
 {
-    PriorityQueue<MoveOnly, CompareMove<isLessThan<unsigned>>::compare> queue;
+    PriorityQueue<MoveOnly, &CompareMove<&isLessThan<unsigned>>::compare> queue;
 
     Vector<unsigned> values = { 23, 54, 4, 8, 1, 2, 4, 0 };
     Vector<unsigned> sorted = values;
@@ -123,7 +123,7 @@
 
 TEST(WTF_PriorityQueue, DecreaseKey)
 {
-    PriorityQueue<MoveOnly, CompareMove<isLessThan<unsigned>>::compare> queue;
+    PriorityQueue<MoveOnly, &CompareMove<&isLessThan<unsigned>>::compare> queue;
 
     Vector<unsigned> values = { 23, 54, 4, 8, 1, 2, 4, 0 };
     Vector<unsigned> sorted = values;
@@ -149,7 +149,7 @@
 
 TEST(WTF_PriorityQueue, IncreaseKey)
 {
-    PriorityQueue<MoveOnly, CompareMove<isGreaterThan<unsigned>>::compare> queue;
+    PriorityQueue<MoveOnly, &CompareMove<&isGreaterThan<unsigned>>::compare> queue;
 
     Vector<unsigned> values = { 23, 54, 4, 8, 1, 2, 4, 0 };
     Vector<unsigned> sorted = values;
@@ -175,7 +175,7 @@
 
 TEST(WTF_PriorityQueue, Iteration)
 {
-    PriorityQueue<MoveOnly, CompareMove<isGreaterThan<unsigned>>::compare> queue;
+    PriorityQueue<MoveOnly, &CompareMove<&isGreaterThan<unsigned>>::compare> queue;
 
     Vector<unsigned> values = { 23, 54, 4, 8, 1, 2, 4, 0 };
     Vector<unsigned> sorted = values;
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to