<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <div class="moz-cite-prefix">Hi, Paul.<br>
      adding 2d-dev@<br>
      <br>
      media: sound/awt/swing part looks fine.<br>
      Note that this part of the fix should be pushed to client forest.<br>
      <br>
      On 5/12/14 2:03 PM, Paul Sandoz wrote:<br>
    </div>
    <blockquote
      cite="mid:0CB20945-D7D7-4151-A546-80634F014A46@oracle.com"
      type="cite">
      <meta http-equiv="Content-Type" content="text/html;
        charset=ISO-8859-1">
      <div>Hi,</div>
      <div><br>
      </div>
      <div>This is a request for review of Otavio's patch
        replacing StringBuffer with StringBuilder within OpenJDK. (I
        also need to review it.)</div>
      <div><br>
      </div>
      <div>It covers many areas and i have grouped the patches into such
        areas to aid reviewing. When commenting please including
        core-libs.</div>
      <div><br>
      </div>
      <div>Jtreg tests showed no regressions, but when reviewing we need
        to be mindful of the context e.g. if the buffer escapes and can
        cross thread boundaries. </div>
      <div><br>
      </div>
      <div>
        <div>This is also an experiment to see if we can review the
          whole thing under one bug :-) if things prove problematic and
          slow we can split it out. Many files are touched but there are
          not many changes to each file and changes are very formulaic.</div>
      </div>
      <div><br>
      </div>
      <div>I have also included ASM related changes, but i suspect we
        may have to leave those alone since such changes will make it
        more difficult to pull in ASM from upstream.</div>
      <div> </div>
      <div>-</div>
      <div><br>
      </div>
      <div>Otavio, for the record can you reply to this thread posting
        your single ("uber") patch as textual attachment? (IIUC such
        attachments should now be supported by the email server).</div>
      <div><br>
      </div>
      <div>Paul.</div>
      <div><br>
      </div>
      <div>- core</div>
      <div><a moz-do-not-send="true"
href="http://cr.openjdk.java.net/%7Epsandoz/jdk9/sb/JDK-8041679-buffer-to-builder-core/webrev/">http://cr.openjdk.java.net/~psandoz/jdk9/sb/JDK-8041679-buffer-to-builder-core/webrev/</a></div>
      <div><br>
      </div>
      <div>- io</div>
      <div><a moz-do-not-send="true"
href="http://cr.openjdk.java.net/%7Epsandoz/jdk9/sb/JDK-8041679-buffer-to-builder-io/webrev/">http://cr.openjdk.java.net/~psandoz/jdk9/sb/JDK-8041679-buffer-to-builder-io/webrev/</a></div>
      <div><br>
      </div>
      <div>- management</div>
      <div><a moz-do-not-send="true"
href="http://cr.openjdk.java.net/%7Epsandoz/jdk9/sb/JDK-8041679-buffer-to-builder-management/webrev/">http://cr.openjdk.java.net/~psandoz/jdk9/sb/JDK-8041679-buffer-to-builder-management/webrev/</a></div>
      <div><br>
      </div>
      <div>- rmi</div>
      <div><a moz-do-not-send="true"
href="http://cr.openjdk.java.net/%7Epsandoz/jdk9/sb/JDK-8041679-buffer-to-builder-rmi/webrev/">http://cr.openjdk.java.net/~psandoz/jdk9/sb/JDK-8041679-buffer-to-builder-rmi/webrev/</a></div>
      <div><br>
      </div>
      <div>- security</div>
      <div><a moz-do-not-send="true"
href="http://cr.openjdk.java.net/%7Epsandoz/jdk9/sb/JDK-8041679-buffer-to-builder-security/webrev/">http://cr.openjdk.java.net/~psandoz/jdk9/sb/JDK-8041679-buffer-to-builder-security/webrev/</a></div>
      <div><br>
      </div>
      <div><br>
      </div>
      <div>- tools</div>
      <div><a moz-do-not-send="true"
href="http://cr.openjdk.java.net/%7Epsandoz/jdk9/sb/JDK-8041679-buffer-to-builder-tools/webrev/">http://cr.openjdk.java.net/~psandoz/jdk9/sb/JDK-8041679-buffer-to-builder-tools/webrev/</a></div>
      <div><br>
      </div>
      <div><br>
      </div>
      <div>- graphics/media</div>
      <div><a moz-do-not-send="true"
href="http://cr.openjdk.java.net/%7Epsandoz/jdk9/sb/JDK-8041679-buffer-to-builder-media/webrev/">http://cr.openjdk.java.net/~psandoz/jdk9/sb/JDK-8041679-buffer-to-builder-media/webrev/</a></div>
      <div><br>
      </div>
      <div><br>
      </div>
      <div>- asm</div>
      <div><a moz-do-not-send="true"
href="http://cr.openjdk.java.net/%7Epsandoz/jdk9/sb/JDK-8041679-buffer-to-builder-asm/webrev/">http://cr.openjdk.java.net/~psandoz/jdk9/sb/JDK-8041679-buffer-to-builder-asm/webrev/</a></div>
    </blockquote>
    <br>
    <br>
    <pre class="moz-signature" cols="72">-- 
Best regards, Sergey. </pre>
  </body>
</html>