<div dir="ltr">Oh, good question. There was a single usage in TransTypes that was recently deleted: <a href="http://hg.openjdk.java.net/jdk9/dev/langtools/rev/2a83d2947323">http://hg.openjdk.java.net/jdk9/dev/langtools/rev/2a83d2947323</a><div><br></div><div>So can the method be removed?</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Apr 21, 2016 at 9:19 AM, Maurizio Cimadamore <span dir="ltr"><<a href="mailto:maurizio.cimadamore@oracle.com" target="_blank">maurizio.cimadamore@oracle.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Is the method even used? My IDE says it's only used recursively within the same method...<span class="HOEnZb"><font color="#888888"><br>
<br>
Maurizio</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
On 21/04/16 16:57, Liam Miller-Cushon wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi, I have a question about MethodSymbol.implementedIn:<br>
<a href="http://hg.openjdk.java.net/jdk9/dev/langtools/file/9b37d0f28609/src/jdk.compiler/share/classes/com/sun/tools/javac/code/Symbol.java#l1620" rel="noreferrer" target="_blank">http://hg.openjdk.java.net/jdk9/dev/langtools/file/9b37d0f28609/src/jdk.compiler/share/classes/com/sun/tools/javac/code/Symbol.java#l1620</a><br>
<br>
I think the FIXME about using subst() is incorrect, Types.memberType takes care of that.<br>
<br>
But testing that the return types are the same still looks wrong, because it doesn't handle implementations with covariant return types.<br>
<br>
I'm not sure why implementedIn is testing return types at all: it calls MethodSymbol.overrides, which checks return types.<br>
<br>
So perhaps the call to isSameType in implementedIn can just be removed? Deleting it doesn't break any tests.<br>
</blockquote>
<br>
</div></div></blockquote></div><br></div>