<html><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;">Please take a review at<br class=""><br class=""> <a href="http://cr.openjdk.java.net/~weijun/8199154/webrev.00/" class="">http://cr.openjdk.java.net/~weijun/8199154/webrev.00/</a><br class=""><br class="">In LdapLoginModule.java, there seems to be an erroneous DL inside DL. I removed the outer DL and its incomplete DD. There is no visual difference after the change.<br class=""><br class="">In Krb5LoginModule.java, there were only DDs and no DT. I've simply removed all D? tags. The output is little more compact. See the attached screenshot.<div class=""><br class=""><img apple-inline="yes" id="1B5C1636-E14C-4829-ADC4-7F03B82A244E" width="294" height="240" src="cid:18EAF65D-8A8F-4BF8-B342-029DBFB7B9E2" class=""><br class=""><br class=""></div><div class="">Thanks<br class="">Max</div></body></html>