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

Unified Diff: runtime/vm/kernel_binary.cc

Issue 2781893004: Remove definitions from Kernel canonical names. (Closed)
Patch Set: Update test expectations. Created 3 years, 9 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
« no previous file with comments | « runtime/vm/kernel.cc ('k') | runtime/vm/kernel_reader.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/kernel_binary.cc
diff --git a/runtime/vm/kernel_binary.cc b/runtime/vm/kernel_binary.cc
index a1cad8377e2c09b8d4a288acc9e78b0380914b6b..44980724fba3ac957b4b70827f9e7852f2ab0e22 100644
--- a/runtime/vm/kernel_binary.cc
+++ b/runtime/vm/kernel_binary.cc
@@ -494,13 +494,6 @@ class Reader {
return name;
}
- CanonicalName* ReadDefiningCanonicalNameReference(LinkedNode* node_to_link) {
- CanonicalName* name = ReadCanonicalNameReference();
- ASSERT(name != NULL);
- name->BindTo(node_to_link);
- return name;
- }
-
intptr_t offset() { return offset_; }
private:
@@ -687,11 +680,9 @@ Library* Library::ReadFrom(Reader* reader) {
int flags = reader->ReadFlags();
ASSERT(flags == 0); // external libraries not supported
- CanonicalName* canonical_name =
- reader->ReadDefiningCanonicalNameReference(this);
-
+ canonical_name_ = reader->ReadCanonicalNameReference();
name_ = Reference::ReadStringFrom(reader);
- import_uri_ = canonical_name->name();
+ import_uri_ = canonical_name_->name();
source_uri_index_ = reader->ReadUInt();
reader->set_current_script_id(source_uri_index_);
@@ -717,7 +708,7 @@ Library* Library::ReadFrom(Reader* reader) {
Class* Class::ReadFrom(Reader* reader) {
TRACE_READ_OFFSET();
- reader->ReadDefiningCanonicalNameReference(this);
+ canonical_name_ = reader->ReadCanonicalNameReference();
position_ = reader->ReadPosition(false);
is_abstract_ = reader->ReadBool();
name_ = Reference::ReadStringFrom(reader);
@@ -809,7 +800,7 @@ Field* Field::ReadFrom(Reader* reader) {
Tag tag = reader->ReadTag();
ASSERT(tag == kField);
- reader->ReadDefiningCanonicalNameReference(this);
+ canonical_name_ = reader->ReadCanonicalNameReference();
position_ = reader->ReadPosition(false);
end_position_ = reader->ReadPosition(false);
flags_ = reader->ReadFlags();
@@ -830,7 +821,7 @@ Constructor* Constructor::ReadFrom(Reader* reader) {
Tag tag = reader->ReadTag();
ASSERT(tag == kConstructor);
- reader->ReadDefiningCanonicalNameReference(this);
+ canonical_name_ = reader->ReadCanonicalNameReference();
VariableScope<ReaderHelper> parameters(reader->helper());
position_ = reader->ReadPosition();
end_position_ = reader->ReadPosition();
@@ -848,7 +839,7 @@ Procedure* Procedure::ReadFrom(Reader* reader) {
Tag tag = reader->ReadTag();
ASSERT(tag == kProcedure);
- reader->ReadDefiningCanonicalNameReference(this);
+ canonical_name_ = reader->ReadCanonicalNameReference();
VariableScope<ReaderHelper> parameters(reader->helper());
position_ = reader->ReadPosition(false);
end_position_ = reader->ReadPosition(false);
@@ -1890,15 +1881,6 @@ Program* Program::ReadFrom(Reader* reader) {
program->main_method_reference_ = Reference::ReadMemberFrom(reader);
-#ifdef DEBUG
- for (intptr_t i = 0; i < canonical_names; ++i) {
- CanonicalName* name = reader->helper()->GetCanonicalName(i);
- if (name->is_referenced() && name->definition() == NULL) {
- FATAL("Missing definition for canonical name");
- }
- }
-#endif
-
return program;
}
« no previous file with comments | « runtime/vm/kernel.cc ('k') | runtime/vm/kernel_reader.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698