<html>
  <head>
    <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    Thanks for the reply Anton and thanks for the fix you are working
    on.  It will be good to get at least one other set of eyes on the
    changes as I am not very familiar with that code (but am starting to
    because I'm looking into fixing some accessibility issues raised by
    the Andriod Studio team).  -Pete<br>
    <br>
    <div class="moz-cite-prefix">On 12/22/15 3:45 PM, Anton Tarasov
      wrote:<br>
    </div>
    <blockquote
      cite="mid:B93F00A2-7815-481E-9EA4-5EC38A7A8F84@jetbrains.com"
      type="cite">
      <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
      Hi Pete,
      <div class=""><br class="">
      </div>
      <div class="">Thanks for the review!</div>
      <div class=""><br class="">
      </div>
      <div class="">
        <div>
          <blockquote type="cite" class="">
            <div class="">On 22 Dec 2015, at 23:07, Pete Brunet <<a
                moz-do-not-send="true"
                href="mailto:peter.brunet@oracle.com" class=""><a class="moz-txt-link-abbreviated" href="mailto:peter.brunet@oracle.com">peter.brunet@oracle.com</a></a>>
              wrote:</div>
            <br class="Apple-interchange-newline">
            <div class="">
              <meta content="text/html; charset=utf-8"
                http-equiv="Content-Type" class="">
              <div bgcolor="#FFFFFF" text="#000000" class=""> Hi Anton,
                Some comments/questions:<br class="">
                - Some copyright dates need updating<br class="">
              </div>
            </div>
          </blockquote>
          <div><br class="">
          </div>
          <div>Indeed, I’ll update them.</div>
          <br class="">
          <blockquote type="cite" class="">
            <div class="">
              <div bgcolor="#FFFFFF" text="#000000" class=""> - Line
                1112 of JavaComponentAccessibility: does the release of
                jaccessible cause a release of jparent?<br class="">
              </div>
            </div>
          </blockquote>
          <div><br class="">
          </div>
          <div>As I can see, jparent there is only a ref to jComponent,
            which in its turn is a JNIGlobalRef (or anyway is a class
            field). So, I don’t see the need to delete it… (or did I
            miss something?)</div>
          <br class="">
          <blockquote type="cite" class="">
            <div class="">
              <div bgcolor="#FFFFFF" text="#000000" class=""> - Line
                7155 of Component.java: is that the only place where
                this means is needed?<br class="">
              </div>
            </div>
          </blockquote>
          <div><br class="">
          </div>
          If you mean to call AC.dispose() than - yes, I think so. We
          rather don’t want to dispose the context until the Component
          goes out of the UI hierarchy, which is when
          Component.removeNotify() is _always_ getting called (for hw
          & lw components).</div>
        <div><br class="">
        </div>
        <div>Anton.</div>
        <div><br class="">
          <blockquote type="cite" class="">
            <div class="">
              <div bgcolor="#FFFFFF" text="#000000" class=""> <br
                  class="">
                Pete<br class="">
                <br class="">
                <div class="moz-cite-prefix">On 12/22/15 8:10 AM, Anton
                  Tarasov wrote:<br class="">
                </div>
                <blockquote
                  cite="mid:086C9CD4-8AF3-4A58-98EB-75A457E68295@jetbrains.com"
                  type="cite" class="">
                  <meta http-equiv="Content-Type" content="text/html;
                    charset=utf-8" class="">
                  Hi guys!
                  <div class=""><br class="">
                  </div>
                  <div class="">Could you please review the problem I’ve
                    filed and the suggested fix?</div>
                  <div class=""><br class="">
                  </div>
                  <div class="">bug: <a moz-do-not-send="true"
                      href="https://bugs.openjdk.java.net/browse/JDK-8145984"
                      class="">JDK-8145984</a> sun.lwawt.macosx.CAccessible
                    leaks</div>
                  <div class="">webrev: <a moz-do-not-send="true"
                      href="http://cr.openjdk.java.net/%7Eant/JDK-8145984/jdk9/webrev.0"
                      class="">http://cr.openjdk.java.net/~ant/JDK-8145984/jdk9/webrev.0</a></div>
                  <div class=""><br class="">
                  </div>
                  <div class="">(This is to be addressed in 8u/9. The
                    webrev for 8u is in JIRA, it’s identical except the
                    paths.)</div>
                  <div class=""><br class="">
                  </div>
                  <div class="">Please, find the details in JIRA.</div>
                  <div class=""><br class="">
                  </div>
                  <div class="">Thanks!</div>
                  <div class="">Anton.</div>
                </blockquote>
                <br class="">
              </div>
            </div>
          </blockquote>
        </div>
        <br class="">
      </div>
    </blockquote>
    <br>
  </body>
</html>