[security-dev 01329]: Code review request (was Re: CCC request approved: 6853328: Support OK-AS-DELEGATE flag)

View: New views
2 Messages — Rating Filter:   Alert me  

Parent Message unknown [security-dev 01329]: Code review request (was Re: CCC request approved: 6853328: Support OK-AS-DELEGATE flag)

by weijun.wang :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Hi Valerie

The CCC is finally approved. Please take a review at the code changes:

    http://cr.openjdk.java.net/~weijun/6853328/webrev.00/

Or, have you reviewed this before? It's been a long time since the  
webrev is created but I cannot find your reply in my old mails.

Thanks
Max

On Oct 20, 2009, at 11:27 PM:

> Your CCC request
>
>    6853328: Support OK-AS-DELEGATE flag
>
> has been approved.
>


[security-dev 01354]: Re: Code review request (was Re: CCC request approved: 6853328: Support OK-AS-DELEGATE flag)

by Valerie Peng :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Hi, Max,

The changes look fine.
src/share/classes/sun/security/jgss/spnego/SpNegoContext.java: line 708 has a typo, i.e. intiiator should be initiator.

Thanks,
Valerie

On 10/20/09 16:54, Max (Weijun) Wang wrote:
Hi Valerie

The CCC is finally approved. Please take a review at the code changes:

   http://cr.openjdk.java.net/~weijun/6853328/webrev.00/

Or, have you reviewed this before? It's been a long time since the webrev is created but I cannot find your reply in my old mails.

Thanks
Max

On Oct 20, 2009, at 11:27 PM:

Your CCC request

   6853328: Support OK-AS-DELEGATE flag

has been approved.