Sure, fine with me.
Thanks,
Valerie

On 9/9/2014 3:53 AM, Wang Weijun wrote:
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

Reply via email to