Reviewers: Lasse Reichstein,

Description:
Fix new strtod on Windows platform, using OS::SNPrintF instead of snprintf.
Fixes error in r5600.

Please review this at http://codereview.chromium.org/3601018/show

SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/

Affected files:
  M     src/strtod.cc


Index: src/strtod.cc
===================================================================
--- src/strtod.cc       (revision 5601)
+++ src/strtod.cc       (working copy)
@@ -40,8 +40,9 @@

 double strtod(Vector<char> buffer, int exponent) {
   char gay_buffer[1024];
+  Vector<char> gay_buffer_vector(gay_buffer, sizeof(gay_buffer));
   buffer.start()[buffer.length()] = '\0';
-  snprintf(gay_buffer, 1024, "%se%d", buffer.start(), exponent);
+  OS::SNPrintF(gay_buffer_vector, "%se%d", buffer.start(), exponent);
   return gay_strtod(gay_buffer, NULL);
 }



--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to