Reviewed by Geoff's rubber stamp
authorbeidson <beidson@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 4 Aug 2006 21:12:32 +0000 (21:12 +0000)
committerbeidson <beidson@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 4 Aug 2006 21:12:32 +0000 (21:12 +0000)
        Fix a build break on Intel hardware causes by adapting stricter
        compiler warnings (-Wshorten-64-to-32)

        * API/testapi.c:
        (assertEqualsAsNumber): manually cast some doubles to floats
        (main): ditto

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

JavaScriptCore/API/testapi.c
JavaScriptCore/ChangeLog

index 040c9244d224a5368541af39a509fe9aca633b9f..c11fc3f24f4b55fd19053dcd2facef7ff4377046 100644 (file)
@@ -46,7 +46,11 @@ static void assertEqualsAsBoolean(JSValueRef value, bool expectedValue)
 static void assertEqualsAsNumber(JSValueRef value, double expectedValue)
 {
     double number = JSValueToNumber(context, value, NULL);
-    if (number != expectedValue && !(isnan(number) && isnan(expectedValue)))
+
+    // FIXME <rdar://4668451> - On i386 the isnan(double) macro tries to map to the isnan(float) function,
+    // causing a build break with -Wshorten-64-to-32 enabled.  The issue is known by the appropriate team.
+    // After that's resolved, we can remove these casts
+    if (number != expectedValue && !(isnan((float)number) && isnan((float)expectedValue)))
         fprintf(stderr, "assertEqualsAsNumber failed: %p, %lf\n", value, expectedValue);
 }
 
@@ -560,7 +564,10 @@ int main(int argc, char* argv[])
     assert(exception);
     
     exception = NULL;
-    assert(isnan(JSValueToNumber(context, jsObjectNoProto, &exception)));
+    // FIXME <rdar://4668451> - On i386 the isnan(double) macro tries to map to the isnan(float) function,
+    // causing a build break with -Wshorten-64-to-32 enabled.  The issue is known by the appropriate team.
+    // After that's resolved, we can remove these casts
+    assert(isnan((float)JSValueToNumber(context, jsObjectNoProto, &exception)));
     assert(exception);
 
     exception = NULL;
index c3f12cf30ade3f3e0d712d07ab197844f5c289c7..e28587c7cbbe23d32792e18896cc86dbd6f9dfdf 100644 (file)
@@ -1,3 +1,14 @@
+2006-08-04  Brady Eidson  <beidson@apple.com>
+
+        Reviewed by Geoff's rubber stamp
+
+        Fix a build break on Intel hardware causes by adapting stricter
+        compiler warnings (-Wshorten-64-to-32)
+
+        * API/testapi.c:
+        (assertEqualsAsNumber): manually cast some doubles to floats
+        (main): ditto
+
 2006-08-04  Sam Weinig  <sam.weinig@gmail.com>
 
         Reviewed by Darin.