I can argue that this is not p2 as related JDK-8201446, but it is definitely not p4.

On 02/07/2018 09:30, Phil Race wrote:
It is not necessary and I reject that it is a P3. Let's leave it for 12.

-phil.

On 07/02/2018 09:24 AM, Sergey Bylokhov wrote:
I think it is a good thing to drop this api completely in the same release as JDK-8201446. This is not a p4 and so it can be be fixed and integrated in jdk11 if the fix is ready.

On 02/07/2018 09:10, Phil Race wrote:
Sergey,

I think at this point, this is not a P3 for 11.
So I won't approve it for 11, but will consider it for 12.

-phil.

On 7/2/2018 6:36 AM, Sergey Bylokhov wrote:
Hello.
Please review the fix for jdk11.

Bug: https://bugs.openjdk.java.net/browse/JDK-8205537
Webrev: http://cr.openjdk.java.net/~serb/8205537/webrev.00/

sun.applet is an internal package contained some code related to implementation of applets and appletviewer. Some of its classes were dropped already: JDK-8204454, JDK-8203308. Now there are only 3 classes, all related to the applet security and we can drop them as well.

In the fix this package was removed, and the tests were updated to not use it. I am not sure how "ClassnameCharTest.java" is useful after applets removal, but since this test used subclass of the AppletClassLoader, I have copied some code from the removed class to the test.








--
Best regards, Sergey.

Reply via email to