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

2017-03-28 Thread Weijun Wang
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. T

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

2017-03-28 Thread joe darcy
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

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

2017-03-28 Thread Weijun Wang
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 mod

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

2017-03-27 Thread joe darcy
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.ja