Changeset 128900 in webkit for trunk/Source/JavaScriptCore/runtime/JSCell.h

Timestamp:

Sep 18, 2012, 9:13:11 AM (13 years ago)

Author:

[email protected]

Message:

Use WTF::HasTrivialDestructor instead of compiler-specific versions in JSC::NeedsDestructor
​https://bugs.webkit.org/show_bug.cgi?id=96980

Reviewed by Benjamin Poulain.

Source/JavaScriptCore:

• runtime/JSCell.h:

(JSC):
(NeedsDestructor):

Source/WTF:

• wtf/TypeTraits.h:

(WTF):

File:

• trunk/Source/JavaScriptCore/runtime/JSCell.h (modified) (2 diffs)

trunk/Source/JavaScriptCore/runtime/JSCell.h

@@ -34 +34 @@
 #include "WriteBarrier.h"
 #include <wtf/Noncopyable.h>
+#include <wtf/TypeTraits.h>
 
 namespace JSC {
@@ -309 +310 @@
     }
 
-#if COMPILER(CLANG)
     template<class T>
     struct NeedsDestructor {
-        static const bool value = !__has_trivial_destructor(T);
+        static const bool value = !WTF::HasTrivialDestructor<T>::value;
     };
-#else
-    // Write manual specializations for this struct template if you care about non-clang compilers.
-    template<class T>
-    struct NeedsDestructor {
-        static const bool value = true;
-    };
-#endif
 
     template<typename T>