On 5/8/2018 1:50 PM, Maurizio Cimadamore wrote:
The [Type, Method, Field]Descriptor API looks neat - I have two comments on this:

* I believe the name descriptor is going to be confusing with the rest of the API, as there are a lot of classes whose name ends with 'Desc'

Then perhaps these types belong somewhere else besides jlic?

* we discussed internally to sprinkle parameters on java/lang/reflect/Member (e.g. have a type parameter T <: TypeDescriptor<T>, and have a method type() which returns a T). That way we could have Member to be a full live representation of a reflective member, including access to owner (a Class<?>), name (String) and, finally, a type (some TypeDescriptor T).

Yes, this patch doesn't yet do this but we can certainly go there once we have agreement on the approach.


Reply via email to