Assertions.h should include ExportMacros.h
authorandersca@apple.com <andersca@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 14 Jul 2015 23:50:39 +0000 (23:50 +0000)
committerandersca@apple.com <andersca@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 14 Jul 2015 23:50:39 +0000 (23:50 +0000)
https://bugs.webkit.org/show_bug.cgi?id=146948

Reviewed by Tim Horton.

Source/JavaScriptCore:

Remove now unneeded WTF_EXPORT_PRIVATE define.

* API/JSBase.h:

Source/WTF:

* wtf/Assertions.h:

Tools:

Remove now unneeded WTF_EXPORT_PRIVATE define.

* DumpRenderTree/cg/ImageDiffCG.cpp:

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

Source/JavaScriptCore/API/JSBase.h
Source/JavaScriptCore/ChangeLog
Source/WTF/ChangeLog
Source/WTF/wtf/Assertions.h
Tools/ChangeLog
Tools/DumpRenderTree/cg/ImageDiffCG.cpp

index d68bd53..4c96088 100644 (file)
@@ -84,11 +84,6 @@ typedef struct OpaqueJSValue* JSObjectRef;
 #define JS_EXPORT
 #endif /* defined(JS_NO_EXPORT) */
 
-/* JS tests uses WTF but has no config.h, so we need to set the export defines here. */
-#ifndef WTF_EXPORT_PRIVATE
-#define WTF_EXPORT_PRIVATE JS_EXPORT
-#endif
-
 #ifdef __cplusplus
 extern "C" {
 #endif
index ef43040..ded3c81 100644 (file)
@@ -1,3 +1,14 @@
+2015-07-14  Anders Carlsson  <andersca@apple.com>
+
+        Assertions.h should include ExportMacros.h
+        https://bugs.webkit.org/show_bug.cgi?id=146948
+
+        Reviewed by Tim Horton.
+
+        Remove now unneeded WTF_EXPORT_PRIVATE define.
+
+        * API/JSBase.h:
+
 2015-07-14  Matthew Mirman  <mmirman@apple.com>
 
         Repatch. Makes compileArithSub in the DFG ensure that the constant is an int32.
index 22a6c09..f328412 100644 (file)
@@ -1,3 +1,12 @@
+2015-07-14  Anders Carlsson  <andersca@apple.com>
+
+        Assertions.h should include ExportMacros.h
+        https://bugs.webkit.org/show_bug.cgi?id=146948
+
+        Reviewed by Tim Horton.
+
+        * wtf/Assertions.h:
+
 2015-07-11  Joseph Pecoraro  <pecoraro@apple.com>
 
         Unreviewed build fix, restrict APP_LINKS to just iOS.
index d99a0d5..11d6120 100644 (file)
@@ -42,6 +42,7 @@
 #include <stdarg.h>
 #include <stdbool.h>
 #include <stddef.h>
+#include <wtf/ExportMacros.h>
 
 #ifdef NDEBUG
 /* Disable ASSERT* macros in release mode. */
index 6a66f49..aa50791 100644 (file)
@@ -1,3 +1,14 @@
+2015-07-14  Anders Carlsson  <andersca@apple.com>
+
+        Assertions.h should include ExportMacros.h
+        https://bugs.webkit.org/show_bug.cgi?id=146948
+
+        Reviewed by Tim Horton.
+
+        Remove now unneeded WTF_EXPORT_PRIVATE define.
+
+        * DumpRenderTree/cg/ImageDiffCG.cpp:
+
 2015-07-14  Carlos Alberto Lopez Perez  <clopez@igalia.com>
 
         [GTK] [Wayland] Build by default the X11 and Wayland targets.
index e285fb0..cdedd6d 100644 (file)
@@ -26,7 +26,6 @@
 
 // FIXME: We need to be able to include these defines from a config.h somewhere.
 #define JS_EXPORT_PRIVATE
-#define WTF_EXPORT_PRIVATE
 
 #include <stdio.h>
 #include <wtf/Platform.h>