Chromium Code Reviews| Index: runtime/vm/class_finalizer.h |
| =================================================================== |
| --- runtime/vm/class_finalizer.h (revision 1040) |
| +++ runtime/vm/class_finalizer.h (working copy) |
| @@ -12,9 +12,9 @@ |
| class Class; |
| class Function; |
| -class RawString; |
| class RawType; |
| class Script; |
| +class String; |
| class Type; |
| class TypeArguments; |
| @@ -32,9 +32,9 @@ |
| Type* interface, |
| Type* conflicting); |
| - // Finalize type. Used for local variable types. |
| - static void FinalizeType(const Type& type); |
| - static RawString* FinalizeTypeWhileParsing(const Type& type); |
| + // Finalize and canonicalize type while parsing. |
| + // Set the error message on failure. |
|
srdjan
2011/11/01 20:51:43
What if no failure? errmsg.IsNull or empty?
regis
2011/11/01 21:54:50
errmsg.IsNull() if no error. Added comment.
|
| + static RawType* FinalizeAndCanonicalizeType(const Type& type, String& errmsg); |
|
srdjan
2011/11/01 20:51:43
String* instead of String& (reference passed with
regis
2011/11/01 21:54:50
Done.
|
| // Pending classes are classes that need to be finalized. |
| static void AddPendingClasses(const GrowableArray<const Class*>& classes); |
| @@ -63,6 +63,7 @@ |
| static void FinalizeTypeArguments(const Class& cls, |
| const TypeArguments& arguments); |
| static RawType* ResolveType(const Class& cls, const Type& type); |
| + static void FinalizeType(const Type& type); |
| static void ResolveAndFinalizeSignature(const Class& cls, |
| const Function& function); |
| static void ResolveAndFinalizeMemberTypes(const Class& cls); |