<html><head><meta http-equiv="Content-Type" content="text/html; charset=UTF-8"></head><body><div><font face="sans-serif">Looks fine to me.</font></div><div><br></div><div><br></div><div><br></div><div id="composer_signature"><meta http-equiv="Content-Type" content="text/html; charset=UTF-8">--Jamil</div><div><br></div><div style="font-size:100%;color:#000000"><!-- originalMessage --><div>-------- Original message --------</div><div>From: Xuelei Fan <xuelei.fan@oracle.com> </div><div>Date: 12/2/19 7:36 AM (GMT-08:00) </div><div>To: security-dev@openjdk.java.net </div><div>Subject: RFR [14] JDK-8235183, Remove the "HACK CODE" in comment </div><div><br></div></div>Hi,<br><br>Could I get the following update reviewed?<br> http://cr.openjdk.java.net/~xuelei/8235183/webrev.00/<br><br>There are some comments in the SunJSSE implementation that come with <br>"HACK CODE". Which are not really bad code. They are just used to mark <br>unusual scenarios in case the block get removed in the future. However, <br>"HACK" may have negative impression for some code readers. It would be <br>nice to remove the "HACK CODE" and use some other words to mark the <br>unusual scenarios.<br><br>Thanks,<br>Xuelei<br></body></html>