Re: RFR 8074835/8074836: Resolve disabled warnings for libj2gss/libosxkrb5

2015-03-16 Thread Wang Weijun
> On Mar 16, 2015, at 22:04, Magnus Ihse Bursie > wrote: > > On 2015-03-16 04:12, Wang Weijun wrote: >> Hi All >> >> Please review the change at >> >>http://cr.openjdk.java.net/~weijun/8074836/webrev.00/ >> >> Thanks >> Max >> > > Looks good to me. Thanks for fixing this. > > I'm just

Re: RFR 8074835/8074836: Resolve disabled warnings for libj2gss/libosxkrb5

2015-03-16 Thread Magnus Ihse Bursie
On 2015-03-16 04:12, Wang Weijun wrote: Hi All Please review the change at http://cr.openjdk.java.net/~weijun/8074836/webrev.00/ Thanks Max Looks good to me. Thanks for fixing this. I'm just curious about the new deprecated-declarations warning. I didn't see it when I excluded the war

[9] request for review: 8048147: Privilege tests with JAAS Subject.doAs

2015-03-16 Thread Artem Smotrakov
Hello, Please review a couple of new tests for privilege checks on Subject.doAs(). https://bugs.openjdk.java.net/browse/JDK-8048147 http://cr.openjdk.java.net/~asmotrak/8048147/webrev.00/ Artem

Re: RFR 8074835/8074836: Resolve disabled warnings for libj2gss/libosxkrb5

2015-03-16 Thread Erik Joelsson
Looks good to me. Thanks for fixing warnings! /Erik On 2015-03-16 04:12, Wang Weijun wrote: Hi All Please review the change at http://cr.openjdk.java.net/~weijun/8074836/webrev.00/ Thanks Max