Use specialized unwrapping methods for reading Double and Bool values.

runtime/vm/dart_api_impl.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 "include/dart_api.h"
 
 #include "vm/bigint_operations.h"
 #include "vm/class_finalizer.h"
 #include "vm/compiler.h"
 #include "vm/dart.h"
@@ skipping 1036 matching lines @@
   return obj.IsBool();
 }
 
 
 DART_EXPORT Dart_Handle Dart_NewBoolean(bool value) {
   CHECK_ISOLATE_SCOPE(Isolate::Current());
   return value ? Api::True() : Api::False();
 }
 
 
-DART_EXPORT Dart_Handle Dart_BooleanValue(Dart_Handle bool_object,
+DART_EXPORT Dart_Handle Dart_BooleanValue(Dart_Handle boolean_obj,
                                           bool* value) {
   DARTSCOPE(Isolate::Current());
-  const Object& obj = Object::Handle(Api::UnwrapHandle(bool_object));
-  if (obj.IsBool()) {
-    Bool& bool_obj = Bool::Handle();
-    bool_obj ^= obj.raw();
-    *value = bool_obj.value();
-    return Api::Success();
+  const Bool& obj = Api::UnwrapBoolHandle(boolean_obj);
+  if (obj.IsNull()) {
+    RETURN_TYPE_ERROR(boolean_obj, Bool);
   }
-  return Api::Error("Object is not a Boolean");
+  *value = obj.value();
+  return Api::Success();
 }
 
 
 // --- Doubles ---
 
 
 DART_EXPORT bool Dart_IsDouble(Dart_Handle object) {
   DARTSCOPE(Isolate::Current());
   const Object& obj = Object::Handle(Api::UnwrapHandle(object));
   return obj.IsDouble();
 }
 
 
 DART_EXPORT Dart_Handle Dart_NewDouble(double value) {
   DARTSCOPE(Isolate::Current());
   const Double& obj = Double::Handle(Double::New(value));
   return Api::NewLocalHandle(obj);
 }
 
 
-DART_EXPORT Dart_Handle Dart_DoubleValue(Dart_Handle integer, double* result) {
+DART_EXPORT Dart_Handle Dart_DoubleValue(Dart_Handle double_obj,
+                                         double* value) {
   DARTSCOPE(Isolate::Current());
-  const Object& obj = Object::Handle(Api::UnwrapHandle(integer));
-  if (obj.IsDouble()) {
-    Double& double_obj = Double::Handle();
-    double_obj ^= obj.raw();
-    *result = double_obj.value();
-    return Api::Success();
+  const Double& obj = Api::UnwrapDoubleHandle(double_obj);
+  if (obj.IsNull()) {
+    RETURN_TYPE_ERROR(double_obj, Double);
   }
-  return Api::Error("Object is not a Double");
+  *value = obj.value();
+  return Api::Success();
 }
 
 
 // --- Strings ---
 
 
 DART_EXPORT bool Dart_IsString(Dart_Handle object) {
   DARTSCOPE(Isolate::Current());
   const Object& obj = Object::Handle(Api::UnwrapHandle(object));
   return obj.IsString();
@@ skipping 1362 matching lines @@
     }
     delete debug_region;
   } else {
     *buffer = NULL;
     *buffer_size = 0;
   }
 }
 
 
 }  // namespace dart