Thanks. Will add some comments.

-Max

在 Oct 22, 2012,7:30 PM,Xuelei Fan <[email protected]> 写道:

> Looks fine to me.  I would suggest you add a few lines about the prefix
> of KRB5CCNAME in the method comment.
> 
> Xuelei
> 
> On 10/22/2012 2:38 PM, Weijun Wang wrote:
>> Please take a look at
>> 
>>   http://cr.openjdk.java.net/~weijun/8001208/webrev.00/
>> 
>> An old test is enhanced to check for the fix.
>> 
>> Honestly, it might be best to check for KRB5CCNAME outside
>> FileCredentialsCache since the ccache type can be something other than
>> FILE:. It will touch more files and I decide to delay this enhancement
>> until we start supporting more real types.
>> 
>> Also, I want to backport this fix to 7u, so it had better be simple.
>> 
>> Thanks
>> Max
> 

Reply via email to