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

Unified Diff: runtime/vm/class_finalizer.cc

Issue 269253005: Put metadata fields in their owner class. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 years, 7 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: runtime/vm/class_finalizer.cc
diff --git a/runtime/vm/class_finalizer.cc b/runtime/vm/class_finalizer.cc
index 9e90e410515ed048e2e481eb1ff739f91e3d5c29..d5b2fdb662431f06116974f0a7d8510171264f13 100644
--- a/runtime/vm/class_finalizer.cc
+++ b/runtime/vm/class_finalizer.cc
@@ -2289,7 +2289,6 @@ void ClassFinalizer::FinalizeTypesInClass(const Class& cls) {
// This class should not contain any fields or functions yet, because it has
// not been compiled yet. Since 'ResolveAndFinalizeMemberTypes(cls)' has not
// been called yet, unfinalized member types could choke the snapshotter.
- ASSERT(Array::Handle(cls.fields()).Length() == 0);
rmacnak 2014/05/08 01:35:21 This assert no longer holds because we can have me
ASSERT(Array::Handle(cls.functions()).Length() == 0);
}
}
« no previous file with comments | « runtime/tests/vm/vm.status ('k') | runtime/vm/object.h » ('j') | runtime/vm/object.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698