Changeset 31948 in webkit for trunk/JavaScriptCore/kjs/ustring.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/ustring.cpp (modified) (3 diffs)

trunk/JavaScriptCore/kjs/ustring.cpp

@@ -617 +617 @@
   int sign;
   
-  char *result = kjs_dtoa(d, 0, 0, &decimalPoint, &sign, NULL);
+  char *result = dtoa(d, 0, &decimalPoint, &sign, NULL);
   int length = static_cast<int>(strlen(result));
   
@@ -670 +670 @@
     buf[i++] = '\0';
   }
-  
-  kjs_freedtoa(result);
-  
+
+  freedtoa(result);
+
   return UString(buf);
 }
@@ -992 +992 @@
     // regular number ?
     char *end;
-    d = kjs_strtod(c, &end);
+    d = strtod(c, &end);
     if ((d != 0.0 || end != c) && d != Inf && d != -Inf) {
       c = end;