Change Bool::Get to return a handle instead of a pointer to a raw object.

runtime/vm/object.h

 // Copyright (c) 2012, 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.
 
 #ifndef VM_OBJECT_H_
 #define VM_OBJECT_H_
 
 #include "include/dart_api.h"
 #include "platform/assert.h"
 #include "platform/utils.h"
@@ skipping 5083 matching lines @@
   }
 
   static const Bool& True() {
     return Object::bool_true();
   }
 
   static const Bool& False() {
     return Object::bool_false();
   }
 
-  static RawBool* Get(bool value) {
-    return value ? Bool::True().raw() : Bool::False().raw();
+  static const Bool& Get(bool value) {
+    return value ? Bool::True() : Bool::False();
   }

[srdjan, 2013/08/27 15:32:44]

Best from both worlds?
add:
static RawBool* GetRaw(bool value)  {
  return Get(value).raw();
}

But I do not think it is absolutely necessary.

[Ivan Posva, 2013/08/27 15:51:25]

I would think that is overkill.

 
  private:
   void set_value(bool value) const { raw_ptr()->value_ = value; }
 
   // New should only be called to initialize the two legal bool values.
   static RawBool* New(bool value);
 
   FINAL_HEAP_OBJECT_IMPLEMENTATION(Bool, Instance);
   friend class Class;
   friend class Object;  // To initialize the true and false values.
@@ skipping 1014 matching lines @@
 
 
 RawObject* MegamorphicCache::GetTargetFunction(const Array& array,
                                                intptr_t index) {
   return array.At((index * kEntryLength) + kTargetFunctionIndex);
 }
 
 }  // namespace dart
 
 #endif  // VM_OBJECT_H_