Re: Code Review Request: JEP 140 Limited doPrivileged

2013-06-04 Thread Jeff Nisewanger
On 5/29/2013 5:44 AM, Florian Weimer wrote: On 05/29/2013 07:56 AM, Jeff Nisewanger wrote: The webrev is: http://cr.openjdk.java.net/~jdn/8014097/webrev.0/ The overloads make it cumbersome to use this feature with lambda expressions:

Re: Code Review Request: JEP 140 Limited doPrivileged

2013-06-04 Thread Jeff Nisewanger
Comments inline below: On 5/31/2013 11:53 AM, Mandy Chung wrote: Jeff, The limited doPrivilege is very useful and I'm hoping the libraries can begin using the limited doPrivileged once this is in jdk8. It's non-trivial stuff and I know that you're working on the unit tests and I can look at

Re: Code Review Request: JEP 140 Limited doPrivileged

2013-05-31 Thread Mandy Chung
Jeff, The limited doPrivilege is very useful and I'm hoping the libraries can begin using the limited doPrivileged once this is in jdk8. It's non-trivial stuff and I know that you're working on the unit tests and I can look at them when they are available. It looks fine to me and I don't see

Re: Code Review Request: JEP 140 Limited doPrivileged

2013-05-29 Thread Florian Weimer
On 05/29/2013 07:56 AM, Jeff Nisewanger wrote: The webrev is: http://cr.openjdk.java.net/~jdn/8014097/webrev.0/ The overloads make it cumbersome to use this feature with lambda expressions: http://mail.openjdk.java.net/pipermail/security-dev/2013-May/007434.html Could you rename the

Code Review Request: JEP 140 Limited doPrivileged

2013-05-28 Thread Jeff Nisewanger
This is the implementation of the JEP 140 feature for a limited privilege form of doPrivileged(). A test will be added in an updated webrev within the next day. The JEP is: http://openjdk.java.net/jeps/140 The webrev is: http://cr.openjdk.java.net/~jdn/8014097/webrev.0/