Reflection on generics.

runtime/lib/mirrors.cc

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #include "lib/invocation_mirror.h"
 #include "vm/bootstrap_natives.h"
+#include "vm/class_finalizer.h"
 #include "vm/dart_entry.h"
 #include "vm/exceptions.h"
 #include "vm/object_store.h"
 #include "vm/port.h"
 #include "vm/symbols.h"
 
 namespace dart {
 
 static RawInstance* CreateMirror(const String& mirror_class_name,
                                  const Array& constructor_arguments) {
@@ skipping 75 matching lines @@
     ASSERT(type.IsTypeParameter());
     name ^= type.name();
     result.SetAt(2 * i, name);
     result.SetAt(2 * i + 1, type);
   }
   return result.raw();
 }
 
 
 static RawInstance* CreateTypedefMirror(const Class& cls,
+                                        const AbstractType& type,

[regis, 2013/08/08 17:06:34]

Parameter 'type' added but not used?
If it is for future work, add a TODO. Or wait before adding the extra parameter.

[rmacnak, 2013/08/08 20:04:06]

Leftover from intermediate implementation that completely switched the reflectee
to a Type. Would only work if there was a distinct Type for the declaration, but
we decided against this. Removed.

                                         const Instance& owner_mirror) {
   const Array& args = Array::Handle(Array::New(3));
   args.SetAt(0, MirrorReference::Handle(MirrorReference::New(cls)));
   args.SetAt(1, String::Handle(cls.UserVisibleName()));
   args.SetAt(2, owner_mirror);
   return CreateMirror(Symbols::_LocalTypedefMirrorImpl(), args);
 }
 
 
-static RawInstance* CreateFunctionTypeMirror(const Class& cls) {
-  const Array& args = Array::Handle(Array::New(1));
+static RawInstance* CreateFunctionTypeMirror(const Class& cls,
+                                             const AbstractType& type) {
+  const Array& args = Array::Handle(Array::New(2));
   args.SetAt(0, MirrorReference::Handle(MirrorReference::New(cls)));
+  args.SetAt(1, type);
   return CreateMirror(Symbols::_LocalFunctionTypeMirrorImpl(), args);
 }
 
 
 static RawInstance* CreateMethodMirror(const Function& func,
                                        const Instance& owner_mirror) {
   const Array& args = Array::Handle(Array::New(12));
   args.SetAt(0, MirrorReference::Handle(MirrorReference::New(func)));
   args.SetAt(1, String::Handle(func.UserVisibleName()));
   args.SetAt(2, owner_mirror);
@@ skipping 24 matching lines @@
   args.SetAt(2, owner_mirror);
   args.SetAt(3, Instance::Handle());  // Null for type.
   args.SetAt(4, field.is_static() ? Bool::True() : Bool::False());
   args.SetAt(5, field.is_final()  ? Bool::True() : Bool::False());
 
   return CreateMirror(Symbols::_LocalVariableMirrorImpl(), args);
 }
 
 
 static RawInstance* CreateClassMirror(const Class& cls,
+                                      const AbstractType& type,
                                       const Instance& owner_mirror) {
   if (cls.IsSignatureClass()) {
     if (cls.IsCanonicalSignatureClass()) {
       // We represent function types as canonical signature classes.
-      return CreateFunctionTypeMirror(cls);
+      return CreateFunctionTypeMirror(cls, type);
     } else {
       // We represent typedefs as non-canonical signature classes.
-      return CreateTypedefMirror(cls, owner_mirror);
+      return CreateTypedefMirror(cls, type, owner_mirror);
     }
   }
 
-  const Array& args = Array::Handle(Array::New(2));
+  AbstractTypeArguments& type_parameters =
+      AbstractTypeArguments::Handle(cls.type_parameters());
+  const Bool& is_generic =
+      type_parameters.IsNull() || type_parameters.Length() == 0

[regis, 2013/08/08 17:06:34]

We like parenthesis (a lot!).
In any case, this would be simpler:
const Bool& is_generic = (cls.NumTypeParameters() == 0) ? ...
And you can spare the type_parameters handle.

[rmacnak, 2013/08/08 20:04:06]

Done.

+          ? Bool::False()
+          : Bool::True();
+
+  const Array& args = Array::Handle(Array::New(4));
   args.SetAt(0, MirrorReference::Handle(MirrorReference::New(cls)));
-  args.SetAt(1, String::Handle(cls.UserVisibleName()));
+  args.SetAt(1, type);
+  args.SetAt(2, String::Handle(cls.UserVisibleName()));
+  args.SetAt(3, is_generic);
   return CreateMirror(Symbols::_LocalClassMirrorImpl(), args);
 }
 
 
+// Note a "raw type" is not the same as a RawType.
+static RawAbstractType* RawTypeOfClass(const Class& klass) {

[regis, 2013/08/08 17:06:34]

Why 'klass' and 'cls' elsewhere? I think 'cls' is preferred.

[rmacnak, 2013/08/08 20:04:06]

Different authors. Switched to cls, as its use is in the majority.

+  AbstractTypeArguments& type_arguments = AbstractTypeArguments::Handle();

[regis, 2013/08/08 17:06:34]

You can save a handle and pass directly Object::null_abstract_type_arguments().

[rmacnak, 2013/08/08 20:04:06]

Done.

+  Type& type =
+      Type::Handle(Type::New(klass, type_arguments, Scanner::kDummyTokenIndex));
+  return ClassFinalizer::FinalizeType(klass,
+                                      type,
+                                      ClassFinalizer::kCanonicalize);
+}
+
+
 static RawInstance* CreateLibraryMirror(const Library& lib) {
   const Array& args = Array::Handle(Array::New(3));
   args.SetAt(0, MirrorReference::Handle(MirrorReference::New(lib)));
   String& str = String::Handle();
   str = lib.name();
   args.SetAt(1, str);
   str = lib.url();
   args.SetAt(2, str);
   return CreateMirror(Symbols::_LocalLibraryMirrorImpl(), args);
 }
 
 
 static RawInstance* CreateTypeMirror(const AbstractType& type) {
   ASSERT(!type.IsMalformed());
   if (type.HasResolvedTypeClass()) {
     const Class& cls = Class::Handle(type.type_class());
     // Handle void and dynamic types.
     if (cls.IsVoidClass()) {
       Array& args = Array::Handle(Array::New(1));
       args.SetAt(0, Symbols::Void());
       // TODO(mlippautz): Create once in the VM isolate and retrieve from there.
       return CreateMirror(Symbols::_SpecialTypeMirrorImpl(), args);
     } else if (cls.IsDynamicClass()) {
       Array& args = Array::Handle(Array::New(1));
       args.SetAt(0, Symbols::Dynamic());
       // TODO(mlippautz): Create once in the VM isolate and retrieve from there.
       return CreateMirror(Symbols::_SpecialTypeMirrorImpl(), args);
     }
-    return CreateClassMirror(cls, Object::null_instance());
+    return CreateClassMirror(cls, type, Object::null_instance());
   } else if (type.IsTypeParameter()) {
     return CreateTypeVariableMirror(TypeParameter::Cast(type),
                                     Object::null_instance());
   }
   UNREACHABLE();
   return Instance::null();
 }
 
 
 static RawInstance* CreateIsolateMirror() {
@@ skipping 37 matching lines @@
 
 
 DEFINE_NATIVE_ENTRY(Mirrors_makeLocalMirrorSystem, 0) {
   return CreateMirrorSystem();
 }
 
 
 DEFINE_NATIVE_ENTRY(Mirrors_makeLocalClassMirror, 1) {
   GET_NON_NULL_NATIVE_ARGUMENT(Type, type, arguments->NativeArgAt(0));
   const Class& cls = Class::Handle(type.type_class());
-  return CreateClassMirror(cls, Object::null_instance());
+  ASSERT(!cls.IsNull());
+  return CreateClassMirror(cls,
+                           AbstractType::Handle(),
+                           Instance::null_instance());
 }
 
 
+DEFINE_NATIVE_ENTRY(Mirrors_makeLocalTypeMirror, 1) {
+  GET_NON_NULL_NATIVE_ARGUMENT(AbstractType, type, arguments->NativeArgAt(0));
+  return CreateTypeMirror(type);
+}
+
+
 static void ThrowMirroredCompilationError(const String& message) {
   Array& args = Array::Handle(Array::New(1));
   args.SetAt(0, message);
 
   Exceptions::ThrowByType(Exceptions::kMirroredCompilationError, args);
   UNREACHABLE();
 }
 
 
 static void ThrowInvokeError(const Error& error) {
@@ skipping 52 matching lines @@
   const Class& cls = Class::Handle(ref.GetClassReferent());
   const Function& func = Function::Handle(cls.signature_function());
   return CreateParameterMirrorList(func);
 }
 
 
 DEFINE_NATIVE_ENTRY(FunctionTypeMirror_return_type, 1) {
   GET_NON_NULL_NATIVE_ARGUMENT(MirrorReference, ref, arguments->NativeArgAt(0));
   const Class& cls = Class::Handle(ref.GetClassReferent());
   const Function& func = Function::Handle(cls.signature_function());
-  const AbstractType& return_type = AbstractType::Handle(func.result_type());
-  return CreateTypeMirror(return_type);
+  return func.result_type();
 }
 
 
 static bool FieldIsUninitialized(const Field& field) {
   ASSERT(!field.IsNull());
 
   // Return getter method for uninitialized fields, rather than the
   // field object, since the value in the field object will not be
   // initialized until the first time the getter is invoked.
   const Instance& value = Instance::Handle(field.value());
@@ skipping 142 matching lines @@
   DictionaryIterator entries(library);
 
   while (entries.HasNext()) {
     entry = entries.GetNext();
     if (entry.IsClass()) {
       const Class& klass = Class::Cast(entry);
       if (!klass.IsCanonicalSignatureClass()) {
         // The various implementations of public classes don't always have the
         // expected superinterfaces or other properties, so we filter them out.
         if (!RawObject::IsImplementationClassId(klass.id())) {
-          member_mirror = CreateClassMirror(klass, owner_mirror);
+          member_mirror = CreateClassMirror(klass,
+                                            AbstractType::Handle(),
+                                            owner_mirror);
           member_mirrors.Add(member_mirror);
         }
       }
     } else if (entry.IsField()) {
       const Field& field = Field::Cast(entry);
       member_mirror = CreateVariableMirror(field, owner_mirror);
       member_mirrors.Add(member_mirror);
     } else if (entry.IsFunction()) {
       const Function& func = Function::Cast(entry);
       if (func.kind() == RawFunction::kRegularFunction ||
           func.kind() == RawFunction::kGetterFunction ||
           func.kind() == RawFunction::kSetterFunction) {
         member_mirror = CreateMethodMirror(func, owner_mirror);
         member_mirrors.Add(member_mirror);
       }
     }
   }
 
   return member_mirrors.raw();
 }
 
 
 DEFINE_NATIVE_ENTRY(ClassMirror_type_variables, 1) {
   GET_NON_NULL_NATIVE_ARGUMENT(MirrorReference, ref, arguments->NativeArgAt(0));
   const Class& klass = Class::Handle(ref.GetClassReferent());
   return CreateTypeVariableList(klass);
 }
 
 
-DEFINE_NATIVE_ENTRY(LocalTypeVariableMirror_owner, 1) {
+DEFINE_NATIVE_ENTRY(ClassMirror_type_arguments, 1) {
+  GET_NON_NULL_NATIVE_ARGUMENT(AbstractType, type, arguments->NativeArgAt(0));
+
+  const AbstractTypeArguments& args =
+      AbstractTypeArguments::Handle(type.arguments());
+  if (args.IsNull()) {
+    return Object::empty_array().raw();
+  }
+
+  const Class& klass = Class::Handle(type.type_class());
+  const TypeArguments& params = TypeArguments::Handle(klass.type_parameters());
+  intptr_t num_params = 0;
+  if (!params.IsNull()) {
+    num_params = params.Length();
+  }

[regis, 2013/08/08 17:06:34]

You can replace the above 5 lines with
const intptr_t num_params = klass.NumTypeParameters();

[rmacnak, 2013/08/08 20:04:06]

Done.

+  const intptr_t num_inherited_args = args.Length() - num_params;
+
+  const Array& result = Array::Handle(Array::New(num_params));
+  AbstractType& arg_type = AbstractType::Handle();
+  Instance& type_mirror = Instance::Handle();
+  for (intptr_t i = 0; i < num_params; i++) {
+    arg_type ^= args.TypeAt(i + num_inherited_args);
+    type_mirror = CreateTypeMirror(arg_type);
+    result.SetAt(i, type_mirror);
+  }
+  return result.raw();
+}
+
+
+DEFINE_NATIVE_ENTRY(TypeVariableMirror_owner, 1) {
   GET_NON_NULL_NATIVE_ARGUMENT(TypeParameter, param, arguments->NativeArgAt(0));
   return CreateClassMirror(Class::Handle(param.parameterized_class()),
+                           AbstractType::Handle(),
                            Instance::null_instance());
 }
 
 
-DEFINE_NATIVE_ENTRY(LocalTypeVariableMirror_upper_bound, 1) {
+DEFINE_NATIVE_ENTRY(TypeVariableMirror_upper_bound, 1) {
   GET_NON_NULL_NATIVE_ARGUMENT(TypeParameter, param, arguments->NativeArgAt(0));
-  return CreateTypeMirror(AbstractType::Handle(param.bound()));
+  return param.bound();
 }
 
 
 // Invoke the function, or noSuchMethod if it is null. Propagate any unhandled
 // exceptions. Wrap and propagate any compilation errors.
 static RawObject* ReflectivelyInvokeDynamicFunction(const Instance& receiver,
                                                     const Function& function,
                                                     const String& target_name,
                                                     const Array& arguments) {
   // Note "arguments" is already the internal arguments with the receiver as
@@ skipping 204 matching lines @@
   Exceptions::ThrowByType(Exceptions::kNoSuchMethod, args);
   UNREACHABLE();
 }
 
 
 static void ThrowNoSuchMethod(const Class& klass,
                               const String& function_name,
                               const Function& function,
                               const InvocationMirror::Call call,
                               const InvocationMirror::Type type) {
-  AbstractTypeArguments& type_arguments = AbstractTypeArguments::Handle();
-  Type& pre_type = Type::Handle(
-      Type::New(klass, type_arguments, Scanner::kDummyTokenIndex));
-  pre_type.SetIsFinalized();
-  AbstractType& runtime_type = AbstractType::Handle(pre_type.Canonicalize());
+  AbstractType& runtime_type = AbstractType::Handle(RawTypeOfClass(klass));
 
   ThrowNoSuchMethod(runtime_type,
                     function_name,
                     function,
                     call,
                     type);
   UNREACHABLE();
 }
 
 
@@ skipping 363 matching lines @@
   GET_NON_NULL_NATIVE_ARGUMENT(MirrorReference, ref, arguments->NativeArgAt(0));
   const Function& func = Function::Handle(ref.GetFunctionReferent());
   if (func.IsNonImplicitClosureFunction()) {
     return CreateMethodMirror(Function::Handle(
         func.parent_function()), Object::null_instance());
   }
   const Class& owner = Class::Handle(func.Owner());
   if (owner.IsTopLevel()) {
     return CreateLibraryMirror(Library::Handle(owner.library()));
   }
-  return CreateClassMirror(owner, Object::null_instance());
+  return CreateClassMirror(owner,
+                           AbstractType::Handle(),
+                           Object::null_instance());
 }
 
 
 DEFINE_NATIVE_ENTRY(MethodMirror_parameters, 1) {
   GET_NON_NULL_NATIVE_ARGUMENT(MirrorReference, ref, arguments->NativeArgAt(0));
   const Function& func = Function::Handle(ref.GetFunctionReferent());
   return CreateParameterMirrorList(func);
 }
 
 
 DEFINE_NATIVE_ENTRY(MethodMirror_return_type, 1) {
   GET_NON_NULL_NATIVE_ARGUMENT(MirrorReference, ref, arguments->NativeArgAt(0));
   const Function& func = Function::Handle(ref.GetFunctionReferent());
   // We handle constructors in Dart code.
   ASSERT(!func.IsConstructor());
-  const AbstractType& return_type = AbstractType::Handle(func.result_type());
-  return CreateTypeMirror(return_type);
+  return func.result_type();
 }
 
 
 DEFINE_NATIVE_ENTRY(TypedefMirror_referent, 1) {
   GET_NON_NULL_NATIVE_ARGUMENT(MirrorReference, ref, arguments->NativeArgAt(0));
   const Class& cls = Class::Handle(ref.GetClassReferent());
   const Function& sig_func = Function::Handle(cls.signature_function());
   const Class& sig_cls = Class::Handle(sig_func.signature_class());
   return MirrorReference::New(sig_cls);
 }
 
 
 DEFINE_NATIVE_ENTRY(ParameterMirror_type, 2) {
   GET_NON_NULL_NATIVE_ARGUMENT(MirrorReference, ref, arguments->NativeArgAt(0));
   GET_NON_NULL_NATIVE_ARGUMENT(Smi, pos, arguments->NativeArgAt(1));
   const Function& func = Function::Handle(ref.GetFunctionReferent());
-  const AbstractType& param_type = AbstractType::Handle(func.ParameterTypeAt(
-      func.NumImplicitParameters() + pos.Value()));
-  return CreateTypeMirror(param_type);
+  return func.ParameterTypeAt(func.NumImplicitParameters() + pos.Value());
 }
 
 
 DEFINE_NATIVE_ENTRY(VariableMirror_type, 1) {
   GET_NON_NULL_NATIVE_ARGUMENT(MirrorReference, ref, arguments->NativeArgAt(0));
   const Field& field = Field::Handle(ref.GetFieldReferent());
-
-  const AbstractType& type = AbstractType::Handle(field.type());
-  return CreateTypeMirror(type);
+  return field.type();
 }
 
 }  // namespace dart