<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="">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); 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><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="">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">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">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/">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></body></html>