
<html><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><br class=""><blockquote type="cite" class="">On Nov 19, 2014, at 22:10, Sean Mullan <<a href="mailto:sean.mullan@oracle.com" class="">sean.mullan@oracle.com</a>> wrote:<br class=""><br class="">On 11/18/2014 02:47 AM, Wang Weijun wrote:<br class=""><blockquote type="cite" class="">Re-request for code review at<br class=""><br class=""><a href="http://cr.openjdk.java.net/~weijun/8061253/webrev.01/" class="">http://cr.openjdk.java.net/~weijun/8061253/webrev.01/</a><br class=""></blockquote><br class="">* Principal:<br class=""><br class="">This should also be included in the @implSpec, is it? You might need another implSpec tag before this, or remove the <p> as it isn't really necessary.<br class=""><br class=""><p>Subclasses may override this with a different implementation, if<br class="">necessary.<br class=""></blockquote><div class=""><br class=""></div>Looks like one is enough.<div class=""><br class=""></div><div class=""><img height="315" width="764" apple-width="yes" apple-height="yes" apple-inline="yes" id="B0A2C393-54A4-4964-B4B9-058D34520E8B" src="cid:816786A6-BA25-4F99-BE2F-9B84F02C33E3" class=""><br class=""><blockquote type="cite" class=""><br class="">* General<br class=""><br class="">Sometimes you say "the Foo" and other times you say "this Foo". I think you should try to be consistent. I prefer "this Foo".<br class=""></blockquote><div class=""><br class=""></div>OK, I'll go through it.</div><div class=""><br class=""></div><div class="">Thanks</div><div class="">Max</div><div class=""><br class=""><blockquote type="cite" class=""><br class="">--Sean<br class=""><br class=""><blockquote type="cite" class=""><br class="">Most are still spec changes, with only one code change in<br class="">KerberosTicket.java where KerberosPrincipal.toString() is changed to<br class="">KerberosPrincipal.getName(). This is also because the spec for<br class="">getName() is more suitable in this call.<br class=""><br class="">*Jamil*: I usually don't touch copyright years and let RE handle it.<br class="">You might say this is not a good habit.<br class=""><br class="">Thanks Max<br class=""><br class=""><blockquote type="cite" class="">On Nov 15, 2014, at 08:05, Wang Weijun <<a href="mailto:weijun.wang@oracle.com" class="">weijun.wang@oracle.com</a>><br class="">wrote:<br class=""><br class="">Webrev withdrawn. I'd like to make more changes.<br class=""><br class="">Thanks Max<br class=""><br class=""><blockquote type="cite" class="">On Nov 14, 2014, at 10:38, Wang Weijun <<a href="mailto:weijun.wang@oracle.com" class="">weijun.wang@oracle.com</a>><br class="">wrote:<br class=""><br class="">Hi All<br class=""><br class="">This is a doc change only bug. Fix at<br class=""><br class=""><a href="http://cr.openjdk.java.net/~weijun/8061253/webrev.00/" class="">http://cr.openjdk.java.net/~weijun/8061253/webrev.00/</a><br class=""><br class="">Please take a review.<br class=""><br class="">Thanks Max<br class=""><br class=""></blockquote><br class=""></blockquote><br class=""></blockquote></blockquote><br class=""></div></body></html>