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

Weijun Wang weijun.wang at oracle.com
Tue Mar 28 23:41:24 UTC 2017


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:
>>
>> - 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