Changeset 190719 in webkit

Timestamp:

Oct 8, 2015 3:37:43 AM (9 years ago)

Author:

commit-queue@webkit.org

Message:

Remove PageReservation.h clang fixme that has been fixed for a while
​https://bugs.webkit.org/show_bug.cgi?id=149908

Patch by Joseph Pecoraro <Joseph Pecoraro> on 2015-10-08
Reviewed by Csaba Osztrogonác.

• wtf/PageReservation.h:

(WTF::PageReservation::operator bool): Deleted.

Location:

trunk/Source/WTF

Files:

• ChangeLog (modified) (1 diff)
• wtf/PageReservation.h (modified) (1 diff)

trunk/Source/WTF/ChangeLog

@@ +1 @@
+2015-10-08  Joseph Pecoraro  <pecoraro@apple.com>
+
+        Remove PageReservation.h clang fixme that has been fixed for a while
+        https://bugs.webkit.org/show_bug.cgi?id=149908
+
+        Reviewed by Csaba Osztrogonác.
+
+        * wtf/PageReservation.h:
+        (WTF::PageReservation::operator bool): Deleted.
+
 2015-10-06  Daniel Bates  <dbates@webkit.org>
 

trunk/Source/WTF/wtf/PageReservation.h

@@ -66 +66 @@
     using PageBlock::base;
     using PageBlock::size;
-
-#ifndef __clang__
     using PageBlock::operator bool;
-#else
-    // FIXME: This is a workaround for <rdar://problem/8876150>, wherein Clang incorrectly emits an access
-    // control warning when a client tries to use operator bool exposed above via "using PageBlock::operator bool".
-    operator bool() const { return PageBlock::operator bool(); }
-#endif
 
     void commit(void* start, size_t size)