Changeset 31948 in webkit for trunk/JavaScriptCore/kjs/number_object.cpp

Timestamp:

Apr 16, 2008, 11:41:54 AM (17 years ago)

Author:

[email protected]

Message:

JavaScriptCore:

2008-04-16 Sam Weinig <​[email protected]>

Reviewed by Darin Adler.

• Remove kjs_ prefix from strtod, dtoa, and freedtoa and put it in the KJS namespace.
• Make strtod, dtoa, and freedtoa c++ functions instead of extern "C".
• Remove mode switching from dtoa. ~2% improvement on test 26.
• Removes all unnecessary #defines from dtoa code.

• JavaScriptCore.exp:
• kjs/dtoa.cpp: (KJS::ulp): (KJS::b2d): (KJS::d2b): (KJS::ratio): (KJS::): (KJS::strtod): (KJS::freedtoa): (KJS::dtoa):
• kjs/dtoa.h:
• kjs/function.cpp: (KJS::parseInt):
• kjs/lexer.cpp: (KJS::Lexer::lex):
• kjs/number_object.cpp: (KJS::integer_part_noexp): (KJS::numberProtoFuncToExponential):
• kjs/ustring.cpp: (KJS::UString::from): (KJS::UString::toDouble):

WebCore:

2008-04-16 Sam Weinig <​[email protected]>

Reviewed by Darin Adler.

Rename kjs_strtod to KJS::strtod.

• platform/text/String.cpp: (WebCore::charactersToDouble):

File:

• trunk/JavaScriptCore/kjs/number_object.cpp (modified) (4 diffs)

trunk/JavaScriptCore/kjs/number_object.cpp

@@ -76 +76 @@
     int decimalPoint;
     int sign;
-    char* result = kjs_dtoa(d, 0, 0, &decimalPoint, &sign, NULL);
+    char* result = dtoa(d, 0, &decimalPoint, &sign, NULL);
     bool resultIsInfOrNan = (decimalPoint == 9999);
     size_t length = strlen(result);
@@ -98 +98 @@
     }
 
-    kjs_freedtoa(result);
+    freedtoa(result);
 
     return str;
@@ -354 +354 @@
     int decimalPoint;
     int sign;
-    char* result = kjs_dtoa(x, 0, 0, &decimalPoint, &sign, NULL);
+    char* result = dtoa(x, 0, &decimalPoint, &sign, NULL);
     size_t resultLength = strlen(result);
     decimalPoint += decimalAdjust;
@@ -377 +377 @@
     ASSERT(i <= 80);
 
-    kjs_freedtoa(result);
+    freedtoa(result);
 
     return jsString(buf);