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

Unified Diff: vm/class_finalizer.cc

Issue 10827209: Unify class ids and snapshot object ids list so that we don't have disparate and sometimes confusin… (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
« no previous file with comments | « vm/assembler_macros_x64.cc ('k') | vm/class_table.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: vm/class_finalizer.cc
===================================================================
--- vm/class_finalizer.cc (revision 10409)
+++ vm/class_finalizer.cc (working copy)
@@ -340,41 +340,41 @@
(cls.library() != Library::CoreImplLibrary())) {
// Prevent extending core implementation classes.
switch (super_class.id()) {
- case kInteger:
- case kSmi:
- case kMint:
- case kBigint:
- case kDouble:
- case kOneByteString:
- case kTwoByteString:
- case kFourByteString:
- case kExternalOneByteString:
- case kExternalTwoByteString:
- case kExternalFourByteString:
- case kBool:
- case kArray:
- case kImmutableArray:
- case kGrowableObjectArray:
- case kInt8Array:
- case kExternalInt8Array:
- case kUint8Array:
- case kExternalUint8Array:
- case kInt16Array:
- case kExternalInt16Array:
- case kUint16Array:
- case kExternalUint16Array:
- case kInt32Array:
- case kExternalInt32Array:
- case kUint32Array:
- case kExternalUint32Array:
- case kInt64Array:
- case kExternalInt64Array:
- case kUint64Array:
- case kExternalUint64Array:
- case kFloat32Array:
- case kExternalFloat32Array:
- case kFloat64Array:
- case kExternalFloat64Array: {
+ case kIntegerCid:
+ case kSmiCid:
+ case kMintCid:
+ case kBigintCid:
+ case kDoubleCid:
+ case kOneByteStringCid:
+ case kTwoByteStringCid:
+ case kFourByteStringCid:
+ case kExternalOneByteStringCid:
+ case kExternalTwoByteStringCid:
+ case kExternalFourByteStringCid:
+ case kBoolCid:
+ case kArrayCid:
+ case kImmutableArrayCid:
+ case kGrowableObjectArrayCid:
+ case kInt8ArrayCid:
+ case kExternalInt8ArrayCid:
+ case kUint8ArrayCid:
+ case kExternalUint8ArrayCid:
+ case kInt16ArrayCid:
+ case kExternalInt16ArrayCid:
+ case kUint16ArrayCid:
+ case kExternalUint16ArrayCid:
+ case kInt32ArrayCid:
+ case kExternalInt32ArrayCid:
+ case kUint32ArrayCid:
+ case kExternalUint32ArrayCid:
+ case kInt64ArrayCid:
+ case kExternalInt64ArrayCid:
+ case kUint64ArrayCid:
+ case kExternalUint64ArrayCid:
+ case kFloat32ArrayCid:
+ case kExternalFloat32ArrayCid:
+ case kFloat64ArrayCid:
+ case kExternalFloat64ArrayCid: {
const Script& script = Script::Handle(cls.script());
ReportError(script, cls.token_pos(),
"'%s' is not allowed to extend '%s'",
« no previous file with comments | « vm/assembler_macros_x64.cc ('k') | vm/class_table.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698