<html><head><meta http-equiv="content-type" content="text/html; charset=utf-8"></head><body dir="auto"><div>Looks good to me.<br><br>/Magnus</div><div><br>8 dec. 2016 kl. 00:21 skrev Valerie Peng <<a href="mailto:valerie.peng@oracle.com">valerie.peng@oracle.com</a>>:<br><br></div><blockquote type="cite"><div>
<meta http-equiv="content-type" content="text/html; charset=utf-8">
<p>Anyone can help reviewing this?
<br>
<br>
The fix is straight forward, just renamed the DEBUG to J2UC_DEBUG
to address the E_MACRO_REDEFINED warning.
<br>
In addition, I also updated the nativeCrypto.h to remove the
workaround for a Solaris12-specific issue which has now been
fixed.<br>
</p>
<p><br>
Bug: <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8079898">https://bugs.openjdk.java.net/browse/JDK-8079898</a>
<br>
Webrev: <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/%7Evaleriep/8079898/webrev.00/">http://cr.openjdk.java.net/~valeriep/8079898/webrev.00/</a>
<br>
<br>
JPRT result looks fine.
<br>
<br>
Thanks,
<br>
<br>
Valerie
<br>
</p>
</div></blockquote></body></html>