On Sep 9, 2014, at 18:45, Wang Weijun <weijun.w...@oracle.com> wrote:
>> <test/sun/security/krb5/auto/Context.java> >> 1) line 364: move it inside the if-block? Seems no value calling xstatus() >> when x is null. > > Correct. > >> 2) line 401: move it inside the if-block? Since if x is null, then there is >> no output following this println() call. > > No block needed if I move line 364 into if-block. Answered too quickly. The xstatus() method is for both ExtendedGSSContext and ExtendedGSSCredential (although no useless info for the latter now). I think I would keep my current code but pull the "if (cred != null)" block out of the "if (x != null)" one. Is that OK? Thanks Max
