Hi Joe
This looks fine to me.
One question:
-@SuppressWarnings("deprecation")
+@SuppressWarnings({"deprecation",
+ "removal"}) // PolicyTool
Why did you split the line to 2?
Thanks
Max
On 03/28/2017 09:09 AM, joe darcy wrote:
Hello,
The jdk.security and jdk.policytool modules use various APIs that are
deprecated for removal. Until the types in question are actually
removed, the lint removal warnings should be suppressed in support of
having a warnings-free build. Please review the webrev:
http://cr.openjdk.java.net/~darcy/8177683.0/
(This issue is a subtask of JDK-8177553: Address removal lint warnings
in the JDK build. I'd prefer to get JDK-8177553 addressed in JDK 9;
however, if that is not approved as part of rampdown 2, I'll push the
associated changes to JDK 10.)
Thanks,
-Joe
