I think this should be an enhancement, and not a bug. Is this mainly for
a performance improvement?
Also, this change puts more of a dependency on sun.security.action which
I think we want to avoid, as it would be nice to eventually stop
exporting sun.security.action so as to minimize the exports from
java.base to the java.security.jgss module. You can double-check with
Mandy or Alan on whether my line of thinking is correct on that.
Thanks,
Sean
On 8/13/18 6:24 AM, Weijun Wang wrote:
Please take a review at
http://cr.openjdk.java.net/~weijun/8209416/webrev.00/
This is all about refactoring "AccessController.doPrivileged(new GetPropertyAction(x))"
into "GetPropertyAction.privilegedGetProperty(x)".
I've already introduced a new GetBooleanAction::privilegedGetProperty method.
Noreg-cleanup.
Thanks
Max
