Implement Double.{toString, toStringAsExponential, toStringAsPrecision}

runtime/lib/double.cc

 // Copyright (c) 2011, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #include <math.h>
 
 #include "vm/bootstrap_natives.h"
 
 #include "vm/bigint_operations.h"
 #include "vm/double_conversion.h"
@@ skipping 168 matching lines @@
   } else if ((Mint::kMinValue <= result) && (result <= Mint::kMaxValue)) {
     arguments->SetReturn(Mint::Handle(Mint::New(static_cast<int64_t>(result))));
   } else {
     arguments->SetReturn(
         Bigint::Handle(BigintOperations::NewFromDouble(result)));
   }
 }
 
 
 DEFINE_NATIVE_ENTRY(Double_toStringAsFixed, 2) {
+  // The boundaries are exclusive.
+  static const double kLowerBoundary = -1e21;
+  static const double kUpperBoundary = 1e21;
+
   const Double& arg = Double::CheckedHandle(arguments->At(0));
   GET_NATIVE_ARGUMENT(Smi, fraction_digits, arguments->At(1));
   double d = arg.value();
-  int fraction_digits_value = fraction_digits.Value();
   String& result = String::Handle();
-  bool succeeded = DoubleToStringAsFixed(d, fraction_digits_value, result);
-  if (!succeeded) {
+  intptr_t fraction_digits_value = fraction_digits.Value();
+  if (0 <= fraction_digits_value && fraction_digits_value <= 20
+      && kLowerBoundary < d && d < kUpperBoundary) {
+    result = DoubleToStringAsFixed(d, static_cast<int>(fraction_digits_value));
+  }
+  if (result.IsNull()) {

[Ivan Posva, 2012/02/27 14:47:14]

I assume this test is here in case the DoubleToStringAsFixed failed for some
reason, otherwise a simple else would do.

[floitsch, 2012/02/27 19:55:40]

Done.

     GrowableArray<const Object*> args;
     args.Add(&String::ZoneHandle(String::New(
         "Illegal arguments to double.toStringAsFixed")));
     Exceptions::ThrowByType(Exceptions::kIllegalArgument, args);
   }
   arguments->SetReturn(result);
 }
 
 
 DEFINE_NATIVE_ENTRY(Double_toStringAsExponential, 2) {
-  UNIMPLEMENTED();
+  const Double& arg = Double::CheckedHandle(arguments->At(0));
+  GET_NATIVE_ARGUMENT(Smi, fraction_digits, arguments->At(1));
+  double d = arg.value();
+  String& result = String::Handle();
+  intptr_t fraction_digits_value = fraction_digits.Value();
+  if (-1 <= fraction_digits_value && fraction_digits_value <= 20) {
+    result =
+        DoubleToStringAsExponential(d, static_cast<int>(fraction_digits_value));

[Ivan Posva, 2012/02/27 14:47:14]

Style nit: Generally we break the line after the call and before the params, see
your code above on line 204.

[floitsch, 2012/02/27 19:55:40]

Done.

+  }
+  if (result.IsNull()) {
+    GrowableArray<const Object*> args;
+    args.Add(&String::ZoneHandle(String::New(
+        "Illegal arguments to double.toStringAsExponential")));
+    Exceptions::ThrowByType(Exceptions::kIllegalArgument, args);
+  }
+  arguments->SetReturn(result);
 }
 
 
 DEFINE_NATIVE_ENTRY(Double_toStringAsPrecision, 2) {
-  UNIMPLEMENTED();
+  const Double& arg = Double::CheckedHandle(arguments->At(0));
+  GET_NATIVE_ARGUMENT(Smi, precision, arguments->At(1));
+  double d = arg.value();
+  String& result = String::Handle();
+  intptr_t precision_value = precision.Value();
+  if (1 <= precision_value && precision_value <= 21) {
+    result = DoubleToStringAsPrecision(d, static_cast<int>(precision_value));
+  }
+  if (result.IsNull()) {
+    GrowableArray<const Object*> args;
+    args.Add(&String::ZoneHandle(String::New(
+        "Illegal arguments to double.toStringAsPrecision")));
+    Exceptions::ThrowByType(Exceptions::kIllegalArgument, args);
+  }
+  arguments->SetReturn(result);
 }
 
 
 DEFINE_NATIVE_ENTRY(Double_isInfinite, 1) {
   const Double& arg = Double::CheckedHandle(arguments->At(0));
   if (isinf(arg.value())) {
     arguments->SetReturn(Bool::Handle(Bool::True()));
   } else {
     arguments->SetReturn(Bool::Handle(Bool::False()));
   }
@@ skipping 17 matching lines @@
     arguments->SetReturn(Bool::Handle(Bool::True()));
   } else {
     arguments->SetReturn(Bool::Handle(Bool::False()));
   }
 }
 
 // Add here only functions using/referring to old-style casts.
 
 }  // namespace dart