maurizio.cimadamore at oracle.com
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