Re: JEP Review Request: Improve Security Manager Performance

2014-04-25 Thread David M. Lloyd
On 04/25/2014 02:35 PM, Sean Mullan wrote: On 04/25/2014 10:54 AM, David M. Lloyd wrote: On 04/25/2014 09:36 AM, Sean Mullan wrote: Please review a draft of a proposed research JEP to improve the performance of the Security Manager: http://cr.openjdk.java.net/~mullan/jeps/Improve-Security-Mana

Re: JEP Review Request: Improve Security Manager Performance

2014-04-25 Thread Sean Mullan
On 04/25/2014 10:54 AM, David M. Lloyd wrote: On 04/25/2014 09:36 AM, Sean Mullan wrote: Please review a draft of a proposed research JEP to improve the performance of the Security Manager: http://cr.openjdk.java.net/~mullan/jeps/Improve-Security-Manager-Performance.00 I am particularly inter

Re: Review request: 8040059 Change default policy for extensions to no permission

2014-04-25 Thread Mandy Chung
On 4/25/2014 12:55 AM, Erik Joelsson wrote: Hello Mandy, The logic looks fine. Just some style issues. I would like indentation for the conditionals to be 2 spaces as is currently the standard in the makefiles. I would also like to have POLICY_SRC_LIST to be declared empty with := instead of

Re: JEP Review Request: Improve Security Manager Performance

2014-04-25 Thread David M. Lloyd
On 04/25/2014 09:36 AM, Sean Mullan wrote: Please review a draft of a proposed research JEP to improve the performance of the Security Manager: http://cr.openjdk.java.net/~mullan/jeps/Improve-Security-Manager-Performance.00 I am particularly interested in any experience you have measuring or pr

JEP Review Request: Improve Security Manager Performance

2014-04-25 Thread Sean Mullan
Please review a draft of a proposed research JEP to improve the performance of the Security Manager: http://cr.openjdk.java.net/~mullan/jeps/Improve-Security-Manager-Performance.00 I am particularly interested in any experience you have measuring or profiling the performance of your code when

Re: Review request: 8040059 Change default policy for extensions to no permission

2014-04-25 Thread Erik Joelsson
Hello Mandy, The logic looks fine. Just some style issues. I would like indentation for the conditionals to be 2 spaces as is currently the standard in the makefiles. I would also like to have POLICY_SRC_LIST to be declared empty with := instead of just =. We only use = assignment when explic