Changeset 206580 in webkit

Timestamp:

Sep 29, 2016, 8:12:54 AM (9 years ago)

Author:

commit-queue@webkit.org

Message:

Clang 3.9 reports a compilation warning about ENABLE_EXCEPTION_SCOPE_VERIFICATION
​https://bugs.webkit.org/show_bug.cgi?id=162718

Patch by Fujii Hironori <Fujii Hironori> on 2016-09-29
Reviewed by Alex Christensen.

Clang 3.9 reports a following compilation warning:

Source/JavaScriptCore/runtime/VM.h:656:5: warning: macro expansion producing 'defined' has undefined behavior [-Wexpansion-to-defined]

• wtf/Platform.h: Changed the definition of ENABLE_EXCEPTION_SCOPE_VERIFICATION not to use 'defined'.

Location:

trunk/Source/WTF

Files:

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

trunk/Source/WTF/ChangeLog

@@ +1 @@
+2016-09-29  Fujii Hironori  <Hironori.Fujii@sony.com>
+
+        Clang 3.9 reports a compilation warning about ENABLE_EXCEPTION_SCOPE_VERIFICATION
+        https://bugs.webkit.org/show_bug.cgi?id=162718
+
+        Reviewed by Alex Christensen.
+
+        Clang 3.9 reports a following compilation warning:
+          Source/JavaScriptCore/runtime/VM.h:656:5: warning: macro expansion producing 'defined' has undefined behavior [-Wexpansion-to-defined]
+
+        * wtf/Platform.h: Changed the definition of ENABLE_EXCEPTION_SCOPE_VERIFICATION not to use 'defined'.
+
 2016-09-28  Mark Lam  <mark.lam@apple.com>
 

trunk/Source/WTF/wtf/Platform.h

@@ -897 +897 @@
 
 #ifndef ENABLE_EXCEPTION_SCOPE_VERIFICATION
-#define ENABLE_EXCEPTION_SCOPE_VERIFICATION (!defined(NDEBUG))
+#ifdef NDEBUG
+#define ENABLE_EXCEPTION_SCOPE_VERIFICATION 0
+#else
+#define ENABLE_EXCEPTION_SCOPE_VERIFICATION 1
+#endif
 #endif