RFR of 8177683: Suppress lint removal warnings in jdk.security and jdk.policytool

Weijun Wang weijun.wang at oracle.com
Tue Mar 28 23:30:52 UTC 2017


Hi Joe

This looks fine to me.

One question:

- at SuppressWarnings("deprecation")
+ at 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
>
>



More information about the security-dev mailing list