JavaScriptCore:
authoraroben <aroben@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 8 Dec 2006 22:34:07 +0000 (22:34 +0000)
committeraroben <aroben@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 8 Dec 2006 22:34:07 +0000 (22:34 +0000)
        Reviewed by Anders.

        This is a mo' better fix for ensuring we don't use macro definitiones
        of min/max.

        * kjs/config.h:
        * wtf/Vector.h:

WebCore:

        Reviewed by Anders.

        This is a mo' better fix for ensuring we don't use macro definitiones
        of min/max.

        * config.h:

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

JavaScriptCore/ChangeLog
JavaScriptCore/kjs/config.h
JavaScriptCore/wtf/Vector.h
WebCore/ChangeLog
WebCore/config.h

index 874162c36da943977c487d4b7ed25f5a50391504..4ffad6fe268738cd3db2fbee78195f32daba1626 100644 (file)
@@ -1,3 +1,13 @@
+2006-12-08  Adam Roben  <aroben@apple.com>
+
+        Reviewed by Anders.
+
+        This is a mo' better fix for ensuring we don't use macro definitiones
+        of min/max.
+
+        * kjs/config.h:
+        * wtf/Vector.h:
+
 2006-12-07  Kevin Fyure  <digdog@macports.org>
 
         Reviewed by Darin.
index 134c2a9f55cdb4b735dfb7d8d8023e75b36227fd..a2cc02478307fcab672fd2cc76237843879287f8 100644 (file)
 
 #elif PLATFORM(WIN_OS)
 
-// We need to define this in order to use the numeric limit 
-// values provided by the C++ standard library.
-#define NOMINMAX
+// 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
 
 #define HAVE_FLOAT_H 1
 #define HAVE_FUNC__FINITE 1
 #define HAVE_SYS_TIMEB_H 1
 
+
 #define USE_SYSTEM_MALLOC 1
 
 #else
index bf8d94ae970165d5ce2421e52bbb42dec61de5d8..496f1c761ed69bded8790580dd0ef1afd953b194 100644 (file)
 #include <stdlib.h>
 #include <utility>
 
-// FIXME: For unknown reasons defining NOMINMAX is not preventing the
-// min and max macros from being defined on Win32.
-#ifdef max
-#undef max
-#endif
-
-#ifdef min
-#undef min
-#endif
-
 namespace WTF {
 
     using std::min;
index 57a6ed5364b33bb2f2e3b38dd0958957afdbb246..829669ae82c8d697cfd08822a6e2bf917cbe9e7f 100644 (file)
@@ -1,3 +1,12 @@
+2006-12-08  Adam Roben  <aroben@apple.com>
+
+        Reviewed by Anders.
+
+        This is a mo' better fix for ensuring we don't use macro definitiones
+        of min/max.
+
+        * config.h:
+
 2006-12-08  Don Gibson  <dgibson77@gmail.com>
 
         Reviewed and landed by Alexey.
index 0647394eca8bcd5b8ff01e57daa5fa10f4ab260f..e5bf68fb561388123cb400fb89ecc005e0f834e7 100644 (file)
@@ -49,9 +49,10 @@ typedef float CGFloat;
 #define WINVER 0x0500
 #endif
 
-// We need to define this in order to use the numeric limit 
-// values provided by the C++ standard library.
-#define NOMINMAX
+// 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
 
 // Hack to match configuration of JavaScriptCore.
 // Maybe there's a better way to do this.