use NOMINMAX instead of #define min min
authorparoga@webkit.org <paroga@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 23 Sep 2013 22:25:25 +0000 (22:25 +0000)
committerparoga@webkit.org <paroga@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 23 Sep 2013 22:25:25 +0000 (22:25 +0000)
https://bugs.webkit.org/show_bug.cgi?id=73563

Reviewed by Brent Fulgham.

Use NOMINMAX instead of #define min/max as a cleaner
way of ensuring that Windows system header files don't
define min/max as macro in the first place.

Source/JavaScriptCore:

* config.h:

Source/WebCore:

* WebCorePrefix.h:
* config.h:

Source/WebKit/win:

* WebKitPrefix.h:

Source/WebKit2:

* config.h:

Source/WTF:

* config.h:

Tools:

* DumpRenderTree/DumpRenderTreePrefix.h:
* DumpRenderTree/cg/ImageDiffCG.cpp:
* DumpRenderTree/config.h:
* DumpRenderTree/win/ImageDiffCairo.cpp:
* WebKitTestRunner/WebKitTestRunnerPrefix.h:
* WebKitTestRunner/config.h:

WebKitLibraries:

* win/tools/vsprops/common.props:

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

20 files changed:
Source/JavaScriptCore/ChangeLog
Source/JavaScriptCore/config.h
Source/WTF/ChangeLog
Source/WTF/config.h
Source/WebCore/ChangeLog
Source/WebCore/WebCorePrefix.h
Source/WebCore/config.h
Source/WebKit/win/ChangeLog
Source/WebKit/win/WebKitPrefix.h
Source/WebKit2/ChangeLog
Source/WebKit2/config.h
Tools/ChangeLog
Tools/DumpRenderTree/DumpRenderTreePrefix.h
Tools/DumpRenderTree/cg/ImageDiffCG.cpp
Tools/DumpRenderTree/config.h
Tools/DumpRenderTree/win/ImageDiffCairo.cpp
Tools/WebKitTestRunner/WebKitTestRunnerPrefix.h
Tools/WebKitTestRunner/config.h
WebKitLibraries/ChangeLog
WebKitLibraries/win/tools/vsprops/common.props

index fbd7c29..2ca46b4 100644 (file)
@@ -1,3 +1,16 @@
+2013-09-23  Patrick Gansterer  <paroga@webkit.org>
+
+        use NOMINMAX instead of #define min min
+        https://bugs.webkit.org/show_bug.cgi?id=73563
+
+        Reviewed by Brent Fulgham.
+
+        Use NOMINMAX instead of #define min/max as a cleaner
+        way of ensuring that Windows system header files don't
+        define min/max as macro in the first place.
+
+        * config.h:
+
 2013-09-23  Filip Pizlo  <fpizlo@apple.com>
 
         Never use ReturnPC for exception handling and quit using exception check indices as a lame replica of the CodeOrigin index
index 0142a9c..beebec6 100644 (file)
 #define WINVER 0x0502
 #endif
 
-// If we don't define these, they get defined in windef.h. 
-// We want to use std::min and std::max
-#define max max
-#define min min
-
 #if !COMPILER(MSVC7_OR_LOWER) && !OS(WINCE)
 // We need to define this before the first #include of stdlib.h or it won't contain rand_s.
 #ifndef _CRT_RAND_S
index 3f5fc61..ac94769 100644 (file)
@@ -1,3 +1,16 @@
+2013-09-23  Patrick Gansterer  <paroga@webkit.org>
+
+        use NOMINMAX instead of #define min min
+        https://bugs.webkit.org/show_bug.cgi?id=73563
+
+        Reviewed by Brent Fulgham.
+
+        Use NOMINMAX instead of #define min/max as a cleaner
+        way of ensuring that Windows system header files don't
+        define min/max as macro in the first place.
+
+        * config.h:
+
 2013-09-23  Anders Carlsson  <andersca@apple.com>
 
         Remove PassTraits.h
index 5552be3..8476328 100644 (file)
 #define WINVER 0x0502
 #endif
 
-// If we don't define these, they get defined in windef.h. 
-// We want to use std::min and std::max
-#define max max
-#define min min
-
 #if !COMPILER(MSVC7_OR_LOWER) && !OS(WINCE)
 // We need to define this before the first #include of stdlib.h or it won't contain rand_s.
 #ifndef _CRT_RAND_S
index 3e460eb..649f2ee 100644 (file)
@@ -1,3 +1,17 @@
+2013-09-23  Patrick Gansterer  <paroga@webkit.org>
+
+        use NOMINMAX instead of #define min min
+        https://bugs.webkit.org/show_bug.cgi?id=73563
+
+        Reviewed by Brent Fulgham.
+
+        Use NOMINMAX instead of #define min/max as a cleaner
+        way of ensuring that Windows system header files don't
+        define min/max as macro in the first place.
+
+        * WebCorePrefix.h:
+        * config.h:
+
 2013-09-23  Simon Fraser  <simon.fraser@apple.com>
 
         REGRESSION (r155998): when zooming in, tiles are too small
index 543f50f..c15a89f 100644 (file)
 #endif
 #endif
 
-// If we don't define these, they get defined in windef.h. 
-// We want to use std::min and std::max
-#ifdef __cplusplus
-#define max max
-#define min min
-#endif
-
 #else
 
 #include <pthread.h>
index 30bed8e..653731c 100644 (file)
 #define WINVER 0x0502
 #endif
 
-// If we don't define these, they get defined in windef.h.
-// We want to use std::min and std::max.
-#ifndef max
-#define max max
-#endif
-#ifndef min
-#define min min
-#endif
-
 // CURL needs winsock, so don't prevent inclusion of it
 #if !USE(CURL)
 #ifndef _WINSOCKAPI_
index 1a03545..d94dbd0 100644 (file)
@@ -1,3 +1,16 @@
+2013-09-23  Patrick Gansterer  <paroga@webkit.org>
+
+        use NOMINMAX instead of #define min min
+        https://bugs.webkit.org/show_bug.cgi?id=73563
+
+        Reviewed by Brent Fulgham.
+
+        Use NOMINMAX instead of #define min/max as a cleaner
+        way of ensuring that Windows system header files don't
+        define min/max as macro in the first place.
+
+        * WebKitPrefix.h:
+
 2013-09-22  Sam Weinig  <sam@webkit.org>
 
         Try to fix the windows build (take 2).
index 92af4d0..62cfe23 100644 (file)
 #define WINVER 0x0502
 #endif
 
-// If we don't define these, they get defined in windef.h.
-// We want to use std::min and std::max.
-#ifndef max
-#define max max
-#endif
-#ifndef min
-#define min min
-#endif
-
 #ifndef _WINSOCKAPI_
 #define _WINSOCKAPI_ // Prevent inclusion of winsock.h in windows.h
 #endif
index e178a70..829d565 100644 (file)
@@ -1,3 +1,16 @@
+2013-09-23  Patrick Gansterer  <paroga@webkit.org>
+
+        use NOMINMAX instead of #define min min
+        https://bugs.webkit.org/show_bug.cgi?id=73563
+
+        Reviewed by Brent Fulgham.
+
+        Use NOMINMAX instead of #define min/max as a cleaner
+        way of ensuring that Windows system header files don't
+        define min/max as macro in the first place.
+
+        * config.h:
+
 2013-09-23  ChangSeok Oh  <changseok.oh@collabora.com>
 
         [GTK][WK2] A plugin dirtyRect within iframe remains after moving to a different page.
index caaed63..ef41dc2 100644 (file)
@@ -65,15 +65,6 @@ static const type& name() \
 #define WINVER 0x0502
 #endif
 
-/* If we don't define these, they get defined in windef.h. */
-/* We want to use std::min and std::max. */
-#ifndef max
-#define max max
-#endif
-#ifndef min
-#define min min
-#endif
-
 #ifndef _WINSOCKAPI_
 #define _WINSOCKAPI_ /* Prevent inclusion of winsock.h in windows.h */
 #endif
index bec6511..b5640fe 100644 (file)
@@ -1,3 +1,21 @@
+2013-09-23  Patrick Gansterer  <paroga@webkit.org>
+
+        use NOMINMAX instead of #define min min
+        https://bugs.webkit.org/show_bug.cgi?id=73563
+
+        Reviewed by Brent Fulgham.
+
+        Use NOMINMAX instead of #define min/max as a cleaner
+        way of ensuring that Windows system header files don't
+        define min/max as macro in the first place.
+
+        * DumpRenderTree/DumpRenderTreePrefix.h:
+        * DumpRenderTree/cg/ImageDiffCG.cpp:
+        * DumpRenderTree/config.h:
+        * DumpRenderTree/win/ImageDiffCairo.cpp:
+        * WebKitTestRunner/WebKitTestRunnerPrefix.h:
+        * WebKitTestRunner/config.h:
+
 2013-09-23  Benjamin Poulain  <benjamin@webkit.org>
 
         Delete TestWebKitAPI domain or some keys before running TestWebKitAPI
index 010ca01..cb40968 100644 (file)
 #import <Foundation/Foundation.h>
 
 #endif
-
-#if defined(WIN32) || defined(_WIN32)
-
-// If we don't define these, they get defined in windef.h. 
-// We want to use std::min and std::max
-#define max max
-#define min min
-
-#endif
index 900f1ba..7b0db88 100644 (file)
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-#if defined(WIN32) || defined(_WIN32)
-#define max max
-#define min min
-#endif
-
 // FIXME: We need to be able to include these defines from a config.h somewhere.
 #define JS_EXPORT_PRIVATE
 #define WTF_EXPORT_PRIVATE
index 6424465..ddb1d51 100644 (file)
 
 #endif // PLATFORM(MAC)
 
-#if OS(WINDOWS)
-// If we don't define these, they get defined in windef.h. 
-// We want to use std::min and std::max
-#undef max
-#define max max
-#undef min
-#define min min
-#endif
-
 #if PLATFORM(WIN)
 #define WTF_USE_CF 1 
 #if PLATFORM(WIN_CAIRO)
index d5a81b4..0ea171b 100644 (file)
@@ -50,9 +50,6 @@ static cairo_user_data_key_t s_imageDataKey;
 
 
 #if PLATFORM(WIN)
-#undef min
-#undef max
-
 static inline float strtof(const char* inputString, char** endptr)
 {
     return strtod(inputString, endptr);
index ac61ad7..d83143b 100644 (file)
 #include <Cocoa/Cocoa.h>
 #endif
 
-#if defined(WIN32) || defined(_WIN32)
-// If we don't define these, they get defined in windef.h. 
-// We want to use std::min and std::max
-#define max max
-#define min min
-#endif
-
 #if defined(BUILDING_GTK__)
 #include "autotoolsconfig.h"
 #endif /* defined (BUILDING_GTK__) */
index d5cd2f9..f6c3dd2 100644 (file)
 #endif
 #endif
 
-#if OS(WINDOWS)
-/* If we don't define these, they get defined in windef.h. */
-/* We want to use std::min and std::max. */
-#ifndef max
-#define max max
-#endif
-#ifndef min
-#define min min
-#endif
-#endif
-
 #endif
index 96be084..ea2e88e 100644 (file)
@@ -1,3 +1,16 @@
+2013-09-23  Patrick Gansterer  <paroga@webkit.org>
+
+        use NOMINMAX instead of #define min min
+        https://bugs.webkit.org/show_bug.cgi?id=73563
+
+        Reviewed by Brent Fulgham.
+
+        Use NOMINMAX instead of #define min/max as a cleaner
+        way of ensuring that Windows system header files don't
+        define min/max as macro in the first place.
+
+        * win/tools/vsprops/common.props:
+
 2013-09-20  Mark Rowe  <mrowe@apple.com>
 
         Fix link errors for external users when building WebKit with Xcode 5.
index 4a919f6..0226968 100644 (file)
@@ -33,7 +33,7 @@ nmake /nologo -f $(ProjectName).make clean</NMakeCleanCommandLine>
   <ItemDefinitionGroup>\r
     <ClCompile>\r
       <AdditionalOptions>/bigobj /GS %(AdditionalOptions)</AdditionalOptions>\r
-      <PreprocessorDefinitions>_WINDOWS;WINVER=0x502;_CRT_SECURE_CPP_OVERLOAD_STANDARD_NAMES=1;_HAS_EXCEPTIONS=0;BUILDING_$(ProjectName);WIN32;%(PreprocessorDefinitions)</PreprocessorDefinitions>\r
+      <PreprocessorDefinitions>_WINDOWS;WINVER=0x502;_CRT_SECURE_CPP_OVERLOAD_STANDARD_NAMES=1;_HAS_EXCEPTIONS=0;BUILDING_$(ProjectName);NOMINMAX;WIN32;%(PreprocessorDefinitions)</PreprocessorDefinitions>\r
       <PreprocessorDefinitions Condition="$(Platform)=='Win32'">WIN32;%(PreprocessorDefinitions)</PreprocessorDefinitions>\r
       <PreprocessorDefinitions Condition="$(Platform)=='Win64'">x64</PreprocessorDefinitions>\r
       <ExceptionHandling>false</ExceptionHandling>\r