Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1406)

Unified Diff: vm/object.cc

Issue 10834284: - Split Dart core libraries into shared sources and patch sources. (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/runtime/
Patch Set: Created 8 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: vm/object.cc
===================================================================
--- vm/object.cc (revision 10547)
+++ vm/object.cc (working copy)
@@ -430,11 +430,9 @@
void Object::RegisterClass(const Class& cls,
const char* cname,
- const Script& script,
const Library& lib) {
const String& name = String::Handle(Symbols::New(cname));
cls.set_name(name);
- cls.set_script(script);
lib.AddClass(cls);
}
@@ -511,95 +509,96 @@
// Now that the symbol table is initialized and that the core dictionary as
// well as the core implementation dictionary have been setup, preallocate
// remaining classes and register them by name in the dictionaries.
- const Script& impl_script = Script::Handle(Bootstrap::LoadImplScript());
+ const Script& impl_script = Script::Handle(
+ Bootstrap::LoadCoreImplScript(false));
cls = Class::New<Integer>();
object_store->set_integer_implementation_class(cls);
- RegisterClass(cls, "IntegerImplementation", impl_script, core_impl_lib);
+ RegisterClass(cls, "IntegerImplementation", core_impl_lib);
pending_classes.Add(cls, Heap::kOld);
cls = Class::New<Smi>();
object_store->set_smi_class(cls);
- RegisterClass(cls, "Smi", impl_script, core_impl_lib);
+ RegisterClass(cls, "Smi", core_impl_lib);
pending_classes.Add(cls, Heap::kOld);
cls = Class::New<Mint>();
object_store->set_mint_class(cls);
- RegisterClass(cls, "Mint", impl_script, core_impl_lib);
+ RegisterClass(cls, "Mint", core_impl_lib);
pending_classes.Add(cls, Heap::kOld);
cls = Class::New<Bigint>();
object_store->set_bigint_class(cls);
- RegisterClass(cls, "Bigint", impl_script, core_impl_lib);
+ RegisterClass(cls, "Bigint", core_impl_lib);
pending_classes.Add(cls, Heap::kOld);
cls = Class::New<Double>();
object_store->set_double_class(cls);
- RegisterClass(cls, "Double", impl_script, core_impl_lib);
+ RegisterClass(cls, "Double", core_impl_lib);
pending_classes.Add(cls, Heap::kOld);
cls = Class::New<Bool>();
object_store->set_bool_class(cls);
- RegisterClass(cls, "Bool", impl_script, core_impl_lib);
+ RegisterClass(cls, "Bool", core_impl_lib);
pending_classes.Add(cls, Heap::kOld);
cls = object_store->array_class(); // Was allocated above.
- RegisterClass(cls, "ObjectArray", impl_script, core_impl_lib);
+ RegisterClass(cls, "ObjectArray", core_impl_lib);
pending_classes.Add(cls, Heap::kOld);
cls = object_store->growable_object_array_class(); // Was allocated above.
- RegisterClass(cls, "GrowableObjectArray", impl_script, core_impl_lib);
+ RegisterClass(cls, "GrowableObjectArray", core_impl_lib);
pending_classes.Add(cls, Heap::kOld);
cls = Class::New<ImmutableArray>();
object_store->set_immutable_array_class(cls);
cls.set_type_arguments_instance_field_offset(Array::type_arguments_offset());
ASSERT(object_store->immutable_array_class() != object_store->array_class());
- RegisterClass(cls, "ImmutableArray", impl_script, core_impl_lib);
+ RegisterClass(cls, "ImmutableArray", core_impl_lib);
pending_classes.Add(cls, Heap::kOld);
cls = object_store->one_byte_string_class(); // Was allocated above.
- RegisterClass(cls, "OneByteString", impl_script, core_impl_lib);
+ RegisterClass(cls, "OneByteString", core_impl_lib);
pending_classes.Add(cls, Heap::kOld);
cls = Class::New<TwoByteString>();
object_store->set_two_byte_string_class(cls);
- RegisterClass(cls, "TwoByteString", impl_script, core_impl_lib);
+ RegisterClass(cls, "TwoByteString", core_impl_lib);
pending_classes.Add(cls, Heap::kOld);
cls = Class::New<FourByteString>();
object_store->set_four_byte_string_class(cls);
- RegisterClass(cls, "FourByteString", impl_script, core_impl_lib);
+ RegisterClass(cls, "FourByteString", core_impl_lib);
pending_classes.Add(cls, Heap::kOld);
cls = Class::New<ExternalOneByteString>();
object_store->set_external_one_byte_string_class(cls);
- RegisterClass(cls, "ExternalOneByteString", impl_script, core_impl_lib);
+ RegisterClass(cls, "ExternalOneByteString", core_impl_lib);
pending_classes.Add(cls, Heap::kOld);
cls = Class::New<ExternalTwoByteString>();
object_store->set_external_two_byte_string_class(cls);
- RegisterClass(cls, "ExternalTwoByteString", impl_script, core_impl_lib);
+ RegisterClass(cls, "ExternalTwoByteString", core_impl_lib);
pending_classes.Add(cls, Heap::kOld);
cls = Class::New<ExternalFourByteString>();
object_store->set_external_four_byte_string_class(cls);
- RegisterClass(cls, "ExternalFourByteString", impl_script, core_impl_lib);
+ RegisterClass(cls, "ExternalFourByteString", core_impl_lib);
pending_classes.Add(cls, Heap::kOld);
cls = Class::New<Stacktrace>();
object_store->set_stacktrace_class(cls);
- RegisterClass(cls, "Stacktrace", impl_script, core_impl_lib);
+ RegisterClass(cls, "Stacktrace", core_impl_lib);
pending_classes.Add(cls, Heap::kOld);
// Super type set below, after Object is allocated.
cls = Class::New<JSRegExp>();
object_store->set_jsregexp_class(cls);
- RegisterClass(cls, "JSSyntaxRegExp", impl_script, core_impl_lib);
+ RegisterClass(cls, "JSSyntaxRegExp", core_impl_lib);
pending_classes.Add(cls, Heap::kOld);
// Initialize the base interfaces used by the core VM classes.
- const Script& script = Script::Handle(Bootstrap::LoadScript());
+ const Script& script = Script::Handle(Bootstrap::LoadCoreScript(false));
// Allocate and initialize the Object class and type. The Object
// class and ByteArray subclasses are the only pre-allocated,
@@ -778,11 +777,21 @@
if (!error.IsNull()) {
return error.raw();
}
+ Script& patch_script = Script::Handle(Bootstrap::LoadCoreScript(true));
+ error = core_lib.Patch(patch_script);
+ if (!error.IsNull()) {
+ return error.raw();
+ }
error = Bootstrap::Compile(core_impl_lib, impl_script);
if (!error.IsNull()) {
return error.raw();
}
- const Script& math_script = Script::Handle(Bootstrap::LoadMathScript());
+ patch_script = Bootstrap::LoadCoreImplScript(true);
+ error = core_impl_lib.Patch(patch_script);
+ if (!error.IsNull()) {
+ return error.raw();
+ }
+ const Script& math_script = Script::Handle(Bootstrap::LoadMathScript(false));
Library::InitMathLibrary(isolate);
const Library& math_lib = Library::Handle(Library::MathLibrary());
ASSERT(!math_lib.IsNull());
@@ -790,7 +799,13 @@
if (!error.IsNull()) {
return error.raw();
}
- const Script& isolate_script = Script::Handle(Bootstrap::LoadIsolateScript());
+ patch_script = Bootstrap::LoadMathScript(true);
+ error = math_lib.Patch(patch_script);
+ if (!error.IsNull()) {
+ return error.raw();
+ }
+ const Script& isolate_script = Script::Handle(
+ Bootstrap::LoadIsolateScript(false));
Library::InitIsolateLibrary(isolate);
const Library& isolate_lib = Library::Handle(Library::IsolateLibrary());
ASSERT(!isolate_lib.IsNull());
@@ -798,7 +813,13 @@
if (!error.IsNull()) {
return error.raw();
}
- const Script& mirrors_script = Script::Handle(Bootstrap::LoadMirrorsScript());
+ patch_script = Bootstrap::LoadIsolateScript(true);
+ error = isolate_lib.Patch(patch_script);
+ if (!error.IsNull()) {
+ return error.raw();
+ }
+ const Script& mirrors_script = Script::Handle(
+ Bootstrap::LoadMirrorsScript(false));
Library::InitMirrorsLibrary(isolate);
const Library& mirrors_lib = Library::Handle(Library::MirrorsLibrary());
ASSERT(!mirrors_lib.IsNull());
@@ -806,6 +827,11 @@
if (!error.IsNull()) {
return error.raw();
}
+ patch_script = Bootstrap::LoadMirrorsScript(true);
+ error = mirrors_lib.Patch(patch_script);
+ if (!error.IsNull()) {
+ return error.raw();
+ }
Bootstrap::SetupNativeResolver();
// Remove the Object superclass cycle by setting the super type to null (not
@@ -6169,9 +6195,8 @@
}
-RawError* Library::Patch(const String& url, const String& source) const {
- const Script& script = Script::Handle(
- Script::New(url, source, RawScript::kPatchTag));
+RawError* Library::Patch(const Script& script) const {
+ ASSERT(script.kind() == RawScript::kPatchTag);
return Compiler::Compile(*this, script);
}

Powered by Google App Engine
This is Rietveld 408576698