Changeset 27152 in webkit for trunk/JavaScriptCore/wtf/MathExtras.h

Timestamp:

Oct 27, 2007, 12:10:40 PM (18 years ago)

Author:

darin

Message:

Reviewed by Eric.

• a couple of Windows fixes

• wtf/MathExtras.h: (wtf_pow): Add a special case for MSVC, which has a "pow" function that does not properly handle the case where arg1 is NaN and arg2 is 0.

• kjs/math_object.cpp: (MathFuncImp::callAsFunction): Don't explicity specify "::pow" -- just "pow" is fine.

• wtf/ASCIICType.h: Check the compiler, not the OS, since it's the compiler/library that has the wchar_t that is just a typedef.

File:

• trunk/JavaScriptCore/wtf/MathExtras.h (modified) (1 diff)

trunk/JavaScriptCore/wtf/MathExtras.h

@@ -101 +101 @@
 inline double wtf_fmod(double x, double y) { return (!isinf(x) && isinf(y)) ? x : fmod(x, y); }
 
-#define fmod(x, y) wtf_fmod(x, y)
+// Work around a bug in the Microsoft CRT, where pow(NaN, 0) yields NaN instead of 1.
+inline double wtf_pow(double x, double y) { return y == 0 ? 1 : pow(x, y); }
 
 #define atan2(x, y) wtf_atan2(x, y)
+#define fmod(x, y) wtf_fmod(x, y)
+#define pow(x, y) wtf_pow(x, y)
 
 #if defined(_CRT_RAND_S)