Re: RFR 8145750: jjs fails to run simple scripts with security manager turned on

2015-12-20 Thread Alan Bateman
On 18/12/2015 12:55, Sundararajan Athijegannathan wrote: May be, not. But I tried giving only sun.reflect package access -- didn't work. There are a few doPrivileged blocks in dynalink code as well. This needs further analysis. But dynalink code was part of nashorn and so was getting AllPermis

Re: RFR 8145750: jjs fails to run simple scripts with security manager turned on

2015-12-20 Thread Sundararajan Athijegannathan
File a separate to track jdk.dynalink permissions: https://bugs.openjdk.java.net/browse/JDK-8145868 Adjusted test comments that Hannes mentioned and pushing changes in webrev @ http://cr.openjdk.java.net/~sundar/8145750/webrev.01/ Thanks -Sundar On 12/20/2015 9:53 PM, Alan Bateman wrote: On

Re: Review request for JDK-8143896: java.lang.Long is implicitly converted to double

2015-12-20 Thread Sundararajan Athijegannathan
Tests may be expanded to cover other Number subtypes like BigDecimal, BigInteger. -Sundar On 12/18/2015 8:24 PM, Hannes Wallnoefer wrote: Please review JDK-8143896: java.lang.Long is implicitly converted to double http://cr.openjdk.java.net/~hannesw/8143896/webrev/ Thanks, Hannes