Wherever SecurityException is thrown, javadoc could have @throws and one line explanation. Even though it is a RuntimeException, JDK javadoc elsewhere documents SecurityException explicitly.

Other than that +1

PS. This webrev shows differences due to other patches/fixes as well ...

-Sundar

On 10/9/2015 10:39 PM, Attila Szegedi wrote:
Please review JDK-8139304 "Remove some obsolete Dynalink classes" at 
<http://cr.openjdk.java.net/~attila/8139304/webrev.jdk9-01> for 
<https://bugs.openjdk.java.net/browse/JDK-8139304>

Thanks,
   Attila.

Reply via email to