<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">Looks good to me.<div class=""><br class=""><div class="">
<div dir="auto" style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0); letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class=""><div>Thanks,</div><div>Rajan</div></div>
</div>
<div style=""><br class=""><blockquote type="cite" class=""><div class="">On Jun 8, 2020, at 4:34 AM, <a href="mailto:shivangi.g.gupta@oracle.com" class="">shivangi.g.gupta@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 class=""><p class="">Thanks Rajan. I changed the webrev and ran Mach5 too. Here is the
new webrev link: -</p><p class=""><a moz-do-not-send="true" href="http://cr.openjdk.java.net/~sshivang/reviews/8246330/webrev.01/" class=""><span style="caret-color: rgb(0, 0, 0); font-family: monospace; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: pre-wrap; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; float: none; display: inline !important;" class="">http://cr.openjdk.java.net/~sshivang/reviews/8246330/webrev.01/</span></a></p><p class=""><br class="">
</p><p class="">Thanks</p><p class="">Shivangi<br class="">
</p><p class=""><br class="">
</p>
<div class="moz-cite-prefix">On 06/06/20 7:37 am, Rajan Halade
wrote:<br class="">
</div>
<blockquote type="cite" cite="mid:7FDD0C33-ECE1-46E5-AC52-00849F2B8CE5@ORACLE.COM" class="">
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8" class="">
One minor comment -
<div class=""><br class="">
</div>
<div class="">
<div style="orphans: 2; widows: 2;" class="">Can you please put
try-catch around "<span style="orphans: 2; widows: 2;" class="">(new DisabledCurve()).run();” instead of over for
loop?</span></div>
<div style="orphans: 2; widows: 2;" class=""><span style="orphans: 2; widows: 2;" class=""><br class="">
</span></div>
<div style="orphans: 2; widows: 2;" class=""><span style="orphans: 2; widows: 2;" class="">Otherwise test looks
good.</span></div>
<div style="orphans: 2; widows: 2;" class=""><span style="orphans: 2; widows: 2;" class=""><br class="">
</span></div>
<div class="">
<div dir="auto" style="caret-color: rgb(0, 0, 0); letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">
<div class="">Thanks,</div>
<div class="">Rajan</div>
</div>
</div>
<div class=""><br class="">
<blockquote type="cite" class="">
<div class="">On Jun 4, 2020, at 8:41 PM, <a href="mailto:shivangi.g.gupta@oracle.com" class="" moz-do-not-send="true">shivangi.g.gupta@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 class=""><p class="">Corrected the links.<br class="">
</p>
<div class="moz-cite-prefix">On 04/06/20 10:59 pm, <a class="moz-txt-link-abbreviated" href="mailto:shivangi.g.gupta@oracle.com" moz-do-not-send="true">shivangi.g.gupta@oracle.com</a>
wrote:<br class="">
</div>
<blockquote type="cite" cite="mid:6aa9a1aa-cf2a-b806-1b57-b223752751d8@oracle.com" class="">
<meta http-equiv="content-type" content="text/html;
charset=UTF-8" class=""><p class="">Hi, <br class="">
<br class="">
May I please find a sponsor for this patch? <br class="">
<br class="">
Bug: <span style="caret-color: rgb(0, 0, 0); font-family: monospace; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: pre-wrap; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; float: none; display: inline !important;" class=""><a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8246330" moz-do-not-send="true">https://bugs.openjdk.java.net/browse/JDK-8246330</a></span>
<br class="">
<br class="">
Description:<span style="caret-color: rgb(23, 43,
77); color: rgb(23, 43, 77); font-family:
"DejaVu Sans", sans-serif; font-size:
14px; font-style: normal; font-variant-caps:
normal; font-weight: normal; letter-spacing:
normal; orphans: auto; text-align: start;
text-indent: 0px; text-transform: none;
white-space: normal; widows: auto; word-spacing:
0px; -webkit-text-size-adjust: auto;
-webkit-text-stroke-width: 0px; background-color:
rgb(255, 255, 255); text-decoration: none;
display: inline !important; float: none;" class="">Many
of EC named curves has been disabled with security
property "jdk.disabled.namedCurves". The purpose
of this Test is to verify the behavior of any EC
named curve from the disabled list.<span class="Apple-converted-space"> </span></span><br style="caret-color: rgb(23, 43, 77); color:
rgb(23, 43, 77); font-family: "DejaVu
Sans", sans-serif; font-size: 14px;
font-style: normal; font-variant-caps: normal;
font-weight: normal; letter-spacing: normal;
orphans: auto; text-align: start; text-indent:
0px; text-transform: none; white-space: normal;
widows: auto; word-spacing: 0px;
-webkit-text-size-adjust: auto;
-webkit-text-stroke-width: 0px; text-decoration:
none;" class="">
<span style="caret-color: rgb(23, 43, 77); color:
rgb(23, 43, 77); font-family: "DejaVu
Sans", sans-serif; font-size: 14px;
font-style: normal; font-variant-caps: normal;
font-weight: normal; letter-spacing: normal;
orphans: auto; text-align: start; text-indent:
0px; text-transform: none; white-space: normal;
widows: auto; word-spacing: 0px;
-webkit-text-size-adjust: auto;
-webkit-text-stroke-width: 0px; background-color:
rgb(255, 255, 255); text-decoration: none;
display: inline !important; float: none;" class="">This
Test will address a single name from the disabled
list as sect283r1.</span> <br class="">
<br class="">
<br class="">
Webrev: <span style="caret-color: rgb(0, 0, 0); font-family: monospace; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: pre-wrap; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; float: none; display: inline !important;" class=""><a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~sshivang/reviews/8246330/webrev.00/" moz-do-not-send="true">http://cr.openjdk.java.net/~sshivang/reviews/8246330/webrev.00/</a></span></p><p class=""> The patch has been tested on mach5, and
the individual test was passed. <br class="">
<br class="">
Thanks <br class="">
<br class="">
Shivangi </p>
</blockquote>
</div>
</div>
</blockquote>
</div>
<br class="">
</div>
</blockquote>
</div>
</div></blockquote></div><br class=""></div></body></html>