<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Hello, Mikhail.<div class=""><br class=""></div><div class="">The fix looks good to me.</div><div class=""><br class=""></div><div class="">With best regards. Petr.</div><div class=""><br class=""><div class=""><div><blockquote type="cite" class=""><div class="">On Oct 28, 2014, at 2:37 PM, mikhail cherkasov <<a href="mailto:mikhail.cherkasov@oracle.com" class="">mikhail.cherkasov@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class="">
  

    <meta http-equiv="content-type" content="text/html; charset=UTF-8" class="">
  
  <div text="#000000" bgcolor="#FFFFFF" class="">
    <div class="moz-text-plain" wrap="true" graphical-quote="true" style="font-family: -moz-fixed; font-size: 14px;" lang="x-western">
      <pre wrap="" class="">Hello, all.

Could you please review the following fix:
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~mcherkas/8051263/webrev.00/">http://cr.openjdk.java.net/~mcherkas/8051263/webrev.00/</a>
 
it's a backport of 
<a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8041572">https://bugs.openjdk.java.net/browse/JDK-8041572</a>
But due difference between 7 & 8 it isn't strict direct backport.
A part of original fix's changes are missed because jdk7 hasn't those code.

The backport is tested and it works fine.


Thanks,
Mikhail.
</pre>
    </div>
  </div>

</div></blockquote></div><br class=""></div></div></body></html>