Maurizio Cimadamore maurizio.cimadamore at
Thu Apr 21 16:19:47 UTC 2016

Is the method even used? My IDE says it's only used recursively within 
the same method...


On 21/04/16 16:57, Liam Miller-Cushon wrote:
> Hi, I have a question about MethodSymbol.implementedIn:
> I think the FIXME about using subst() is incorrect, Types.memberType 
> takes care of that.
> But testing that the return types are the same still looks wrong, 
> because it doesn't handle implementations with covariant return types.
> I'm not sure why implementedIn is testing return types at all: it 
> calls MethodSymbol.overrides, which checks return types.
> So perhaps the call to isSameType in implementedIn can just be 
> removed? Deleting it doesn't break any tests.

More information about the compiler-dev mailing list