Add ByteArray interface and provide internal and external implementations.

runtime/vm/object.cc

 // 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.
 
 #include "vm/object.h"
 
 #include "platform/assert.h"
 #include "vm/assembler.h"
 #include "vm/bigint_operations.h"
 #include "vm/bootstrap.h"
@@ skipping 521 matching lines @@
 
   cls = Class::New<JSRegExp>();
   object_store->set_jsregexp_class(cls);
   RegisterClass(cls, "JSSyntaxRegExp", impl_script, core_impl_lib);
   pending_classes.Add(&Class::ZoneHandle(cls.raw()));
 
   // Initialize the base interfaces used by the core VM classes.
   const Script& script = Script::Handle(Bootstrap::LoadScript());
 
   // Allocate and initialize the Object class and type.
-  // The Object and ByteBuffer classes are the only pre-allocated
+  // The Object and ExternalByteArray classes are the only pre-allocated
   // non-interface classes in the core library.
   cls = Class::New<Instance>();
   object_store->set_object_class(cls);
   cls.set_name(String::Handle(String::NewSymbol("Object")));
   cls.set_script(script);
   core_lib.AddClass(cls);
   pending_classes.Add(&Class::ZoneHandle(cls.raw()));
   type = Type::NewNonParameterizedType(cls);
   object_store->set_object_type(type);
 
-  cls = Class::New<ByteBuffer>();
-  object_store->set_byte_buffer_class(cls);
-  cls.set_name(String::Handle(String::NewSymbol("ByteBuffer")));
+  cls = Class::New<InternalByteArray>();
+  object_store->set_internal_byte_array_class(cls);
+  cls.set_name(String::Handle(String::NewSymbol("InternalByteArray")));
   cls.set_script(script);
   core_lib.AddClass(cls);
 
+  cls = Class::New<ExternalByteArray>();
+  object_store->set_external_byte_array_class(cls);
+  cls.set_name(String::Handle(String::NewSymbol("ExternalByteArray")));
+  cls.set_script(script);
+  core_lib.AddClass(cls);
+
   // Set the super type of class Stacktrace to Object type so that the
   // 'toString' method is implemented.
   cls = object_store->stacktrace_class();
   cls.set_super_type(type);
 
   cls = CreateAndRegisterInterface("Function", script, core_lib);
   pending_classes.Add(&Class::ZoneHandle(cls.raw()));
   type = Type::NewNonParameterizedType(cls);
   object_store->set_function_interface(type);
 
@@ skipping 20 matching lines @@
   cls = CreateAndRegisterInterface("bool", script, core_lib);
   pending_classes.Add(&Class::ZoneHandle(cls.raw()));
   type = Type::NewNonParameterizedType(cls);
   object_store->set_bool_interface(type);
 
   cls = CreateAndRegisterInterface("List", script, core_lib);
   pending_classes.Add(&Class::ZoneHandle(cls.raw()));
   type = Type::NewNonParameterizedType(cls);
   object_store->set_list_interface(type);
 
+  cls = CreateAndRegisterInterface("ByteArray", script, core_lib);
+  pending_classes.Add(&Class::ZoneHandle(cls.raw()));
+  type = Type::NewNonParameterizedType(cls);
+  object_store->set_byte_array_interface(type);
+
   // The classes 'Null' and 'void' are not registered in the class dictionary,
   // because their names are reserved keywords. Their names are not heap
   // allocated, because the classes reside in the VM isolate.
   // The corresponding types are stored in the object store.
   cls = null_class_;
   type = Type::NewNonParameterizedType(cls);
   object_store->set_null_type(type);
 
   cls = void_class_;
   type = Type::NewNonParameterizedType(cls);
@@ skipping 51 matching lines @@
   cls = Class::New<Array>();
   object_store->set_array_class(cls);
 
   Array& empty_array = Array::Handle();
   empty_array = Array::New(0);
   object_store->set_empty_array(empty_array);
 
   cls = Class::New<ImmutableArray>();
   object_store->set_immutable_array_class(cls);
 
-  cls = Class::New<ByteBuffer>();
-  object_store->set_byte_buffer_class(cls);
+  cls = Class::New<InternalByteArray>();
+  object_store->set_internal_byte_array_class(cls);
+
+  cls = Class::New<ExternalByteArray>();
+  object_store->set_external_byte_array_class(cls);
 
   cls = Class::New<Instance>();
   object_store->set_object_class(cls);
 
   cls = Class::New<Smi>();
   object_store->set_smi_class(cls);
 
   cls = Class::New<Mint>();
   object_store->set_mint_class(cls);
 
@@ skipping 537 matching lines @@
       return object_store->external_four_byte_string_class();
     case kBool:
       ASSERT(object_store->bool_class() != Class::null());
       return object_store->bool_class();
     case kArray:
       ASSERT(object_store->array_class() != Class::null());
       return object_store->array_class();
     case kImmutableArray:
       ASSERT(object_store->immutable_array_class() != Class::null());
       return object_store->immutable_array_class();
-    case kByteBuffer:
-      ASSERT(object_store->byte_buffer_class() != Class::null());
-      return object_store->byte_buffer_class();
+    case kInternalByteArray:
+      ASSERT(object_store->internal_byte_array_class() != Class::null());
+      return object_store->internal_byte_array_class();
+    case kExternalByteArray:
+      ASSERT(object_store->external_byte_array_class() != Class::null());
+      return object_store->external_byte_array_class();
     case kStacktrace:
       ASSERT(object_store->stacktrace_class() != Class::null());
       return object_store->stacktrace_class();
     case kJSRegExp:
       ASSERT(object_store->jsregexp_class() != Class::null());
       return object_store->jsregexp_class();
     case kClosure:
       return Class::New<Closure>();
     case kInstance:
       return Class::New<Instance>();
@@ skipping 6184 matching lines @@
   Class& cls = Class::Handle(object_store->immutable_array_class());
   return reinterpret_cast<RawImmutableArray*>(Array::New(cls, len, space));
 }
 
 
 const char* ImmutableArray::ToCString() const {
   return "ImmutableArray";
 }
 
 
-RawByteBuffer* ByteBuffer::New(uint8_t* data,
-                               intptr_t len,
-                               Heap::Space space) {
+intptr_t ByteArray::Length() const {
+  // ByteArray is an abstract class.
+  UNREACHABLE();
+  return 0;
+}
+
+
+const char* ByteArray::ToCString() const {
+  // ByteArray is an abstract class.
+  UNREACHABLE();
+  return "ByteArray";
+}
+
+
+RawInternalByteArray* InternalByteArray::New(intptr_t len,
+                                             Heap::Space space) {
   Isolate* isolate = Isolate::Current();
-  const Class& byte_buffer_class =
-      Class::Handle(isolate->object_store()->byte_buffer_class());
-  ByteBuffer& result = ByteBuffer::Handle();
+  const Class& internal_byte_array_class =
+      Class::Handle(isolate->object_store()->internal_byte_array_class());
+  InternalByteArray& result = InternalByteArray::Handle();
   {
-    RawObject* raw = Object::Allocate(byte_buffer_class,
-                                      ByteBuffer::InstanceSize(),
+    RawObject* raw = Object::Allocate(internal_byte_array_class,
+                                      InternalByteArray::InstanceSize(len),
+                                      space);
+    NoGCScope no_gc;
+    result ^= raw;
+    result.SetLength(len);

[Ivan Posva, 2012/01/25 23:49:57]

Not calling bzero here will cause you to leak the byte values of the null
object.

[cshapiro, 2012/01/26 02:25:42]

Right.  I assumed we zero initialized.  Fixed.

+  }
+  return result.raw();
+}
+
+
+RawInternalByteArray* InternalByteArray::New(const uint8_t* data,
+                                             intptr_t len,
+                                             Heap::Space space) {
+  InternalByteArray& result =
+      InternalByteArray::Handle(InternalByteArray::New(len, space));
+  {
+    NoGCScope no_gc;
+    memmove(result.Addr<uint8_t>(0), data, len);
+  }
+  return result.raw();
+}
+
+
+const char* InternalByteArray::ToCString() const {
+  return "InternalByteArray";
+}
+
+
+RawExternalByteArray* ExternalByteArray::New(uint8_t* data,
+                                             intptr_t len,
+                                             Heap::Space space) {
+  Isolate* isolate = Isolate::Current();
+  const Class& external_byte_array_class =
+      Class::Handle(isolate->object_store()->external_byte_array_class());
+  ExternalByteArray& result = ExternalByteArray::Handle();
+  {
+    RawObject* raw = Object::Allocate(external_byte_array_class,
+                                      ExternalByteArray::InstanceSize(),
                                       space);
     NoGCScope no_gc;
     result ^= raw;
     result.SetLength(len);
     result.SetData(data);
   }
   return result.raw();
 }
 
 
-bool ByteBuffer::Equals(const Instance& other) const {
-  if (this->raw() == other.raw()) {
-    // Both handles point to the same raw instance.
-    return true;
-  }
-
-  if (!other.IsByteBuffer() || other.IsNull()) {
-    return false;
-  }
-
-  ByteBuffer& other_array = ByteBuffer::Handle();
-  other_array ^= other.raw();
-
-  intptr_t len = this->Length();
-  if (len != other_array.Length()) {
-    return false;
-  }
-
-  return memcmp(this->Addr<uint8_t>(0), other_array.Addr<uint8_t>(0), len) == 0;
+const char* ExternalByteArray::ToCString() const {
+  return "ExternalByteArray";
 }
 
 
-const char* ByteBuffer::ToCString() const {
-  return "ByteBuffer";
-}
-
-
 RawClosure* Closure::New(const Function& function,
                          const Context& context,
                          Heap::Space space) {
   Isolate* isolate = Isolate::Current();
   ASSERT(context.isolate() == isolate);
 
   const Class& cls = Class::Handle(function.signature_class());
   Closure& result = Closure::Handle();
   {
     RawObject* raw = Object::Allocate(cls, Closure::InstanceSize(), space);
@@ skipping 289 matching lines @@
   const String& str = String::Handle(pattern());
   const char* format = "JSRegExp: pattern=%s flags=%s";
   intptr_t len = OS::SNPrint(NULL, 0, format, str.ToCString(), Flags());
   char* chars = reinterpret_cast<char*>(
       Isolate::Current()->current_zone()->Allocate(len + 1));
   OS::SNPrint(chars, (len + 1), format, str.ToCString(), Flags());
   return chars;
 }
 
 }  // namespace dart