> On Nov 15, 2019, at 4:30 AM, Sean Mullan <sean.mul...@oracle.com> wrote:
>
> On 11/4/19 8:14 PM, Weijun Wang wrote:
>>> Webrev:http://cr.openjdk.java.net/~weijun/8214024/webrev.00/
>> Will look at the code changes later.
>
> Looks good. Should you add the bugid to one of the tests such as
> sun/security/tools/keytool/KeyToolTest.java?
I think most of them (including KeyToolTest.java) are "adapted" to this change
rather than "verifying" it, therefore it seems unnecessary to add the bugid.
Only DeprecateKeyalg.java is updated to detect the removal and it's now renamed
to RemoveKeyAlgDefault.java.
>
> Also, please don't forget to add a Release Note.
OK.
Thanks,
Max
>
> --Sean
