<html>
  <head>
    <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <div class="moz-cite-prefix">On 16.09.2013 17:08, Anthony Petrov
      wrote:<br>
    </div>
    <blockquote cite="mid:523702D0.7050308@oracle.com" type="cite">Hi
      Sergey,
      <br>
      <br>
      1. There's some change in RepaintArea logic in LWRepaintArea.java.
      I'd suggest to push these changes as a separate fix.
      <br>
    </blockquote>
    There are 2 changed in the file:<br>
     - javadoc<br>
     - flushOnscreenGraphics() is a static method and now it is called
    via class name(LWComponentPeer) instead of instance of this class.<br>
    <blockquote cite="mid:523702D0.7050308@oracle.com" type="cite">
      <br>
      2. LWWindowPeer.java
      <br>
      <blockquote type="cite"> 46     public enum PeerType {
        <br>
      </blockquote>
      <br>
      Why should it not be static?
      <br>
    </blockquote>
    That's because "Nested enum types are implicitly <code
      class="literal">static</code>."<br>
    <a class="moz-txt-link-freetext" href="http://docs.oracle.com/javase/specs/jls/se7/html/jls-8.html#jls-8.9">http://docs.oracle.com/javase/specs/jls/se7/html/jls-8.html#jls-8.9</a><br>
    <blockquote cite="mid:523702D0.7050308@oracle.com" type="cite">
      <br>
      3.
      <br>
      <blockquote type="cite"> 132         LWWindowPeer ownerPeer =
        (owner != null) ? (LWWindowPeer) AWTAccessor
        <br>
         133                 .getComponentAccessor().getPeer(owner) :
        null;
        <br>
      </blockquote>
      <br>
      This could be better formatted as:
      <br>
      <br>
      ... owner == null ? null :&lt;CR&gt;
      <br>
      (LWWindowPeer) AWTAccessor...
      <br>
      <br>
      I.e. to avoid breaking the AWTAccessor call into multiple lines.
      The same at lines 396, 1166, and 1279.
      <br>
    </blockquote>
    Will fix it.<br>
    <blockquote cite="mid:523702D0.7050308@oracle.com" type="cite">
      <br>
      The rest of the fix looks fine to me.
      <br>
      <br>
      --
      <br>
      best regards,
      <br>
      Anthony
      <br>
      <br>
      On 09/12/13 18:53, Sergey Bylokhov wrote:
      <br>
      <blockquote type="cite">Hello,
        <br>
        Please review the fix for jdk 8.
        <br>
        - Documentation updated
        <br>
        - Component.getPeer() replaced with AWTAccessor.
        <br>
        - Generics code updated.
        <br>
        - For some methods access was changed to weaker
        <br>
        - SuppressWarnings were added when needed.
        <br>
        Plus some small general cleanup.
        <br>
        <br>
        Bug: <a class="moz-txt-link-freetext" href="http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8005255">http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8005255</a>
        <br>
        Webrev can be found at:
        <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~serb/8005255/webrev.00/">http://cr.openjdk.java.net/~serb/8005255/webrev.00/</a>
        <br>
        <br>
      </blockquote>
    </blockquote>
    <br>
    <br>
    <pre class="moz-signature" cols="72">-- 
Best regards, Sergey. </pre>
  </body>
</html>