> On Jan 24, 2019, at 6:20 PM, Mandy Chung <mandy.ch...@oracle.com> wrote:
> 
> 
> 
> On 1/24/19 1:55 PM, Lance Andersen wrote:
>> 
>>> Alternatively you can simply do:
>>>             PrivilegedExceptionAction<Path> pea = () -> 
>>> tempPath.toRealPath();
>>>             return AccessController.doPrivileged(pea); 
>>> 
>>> In this case, limited doPrivileged does not make whole lot of difference 
>>> jdk.zip explicitly grants for limited permissions and this doPrivileged 
>>> block is one single operation, toRealPath.
>> 
>> Do you have a preference here either way?  I had that originally but changed 
>> it  to make it clearer why it was done.
> 
> For this simple case, I would avoid using limited doPrivileged.

Okie Dokie, I made the change and also ran optimize imports on the test:  
http://cr.openjdk.java.net/~lancea/8210469/webrev.01/index.html

Best
Lance
> 
> Mandy

 <http://oracle.com/us/design/oracle-email-sig-198324.gif>
 <http://oracle.com/us/design/oracle-email-sig-198324.gif> 
<http://oracle.com/us/design/oracle-email-sig-198324.gif>
 <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| 
Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering 
1 Network Drive 
Burlington, MA 01803
lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>



Reply via email to