But the "deprecation" argument is also for PolicyTool.
-Max On 03/29/2017 07:35 AM, joe darcy wrote:
Hi Max, Since entire classes were being covered by the annotation, I wanted to make clear that the removal warnings in those cases were for PolicyTool as opposed to being for something else. Thanks, -Joe On 3/28/2017 4:30 PM, Weijun Wang wrote: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
