<html>
  <head>
    <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <br>
    <br>
    <div class="moz-cite-prefix">On 21/04/16 17:25, Liam Miller-Cushon
      wrote:<br>
    </div>
    <blockquote
cite="mid:CAL4Qsgt=L+HZ0uppYh1nbsBbFQfAURJp6TrE2yYkHCTfSz9PNg@mail.gmail.com"
      type="cite">
      <div dir="ltr">Oh, good question. There was a single usage in
        TransTypes that was recently deleted: <a moz-do-not-send="true"
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>
    </blockquote>
    I think so (yayy!)<br>
    <br>
    Maurizio<br>
    <blockquote
cite="mid:CAL4Qsgt=L+HZ0uppYh1nbsBbFQfAURJp6TrE2yYkHCTfSz9PNg@mail.gmail.com"
      type="cite">
      <div class="gmail_extra"><br>
        <div class="gmail_quote">On Thu, Apr 21, 2016 at 9:19 AM,
          Maurizio Cimadamore <span dir="ltr"><<a
              moz-do-not-send="true"
              href="mailto:maurizio.cimadamore@oracle.com"
              target="_blank"><a class="moz-txt-link-abbreviated" href="mailto:maurizio.cimadamore@oracle.com">maurizio.cimadamore@oracle.com</a></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 moz-do-not-send="true"
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>
    </blockquote>
    <br>
  </body>
</html>