<AWT Dev> [13] Review Request: 8223766 Remains of ancient code in AWT initialization

Sergey Bylokhov Sergey.Bylokhov at oracle.com
Mon May 13 20:46:02 UTC 2019


Hi, Jay.
I deleted unused code inside the "initIds()", but "initIds()" methods itself(even empty) are still called from the java shared code.

On 13/05/2019 06:15, Jayathirth Rao wrote:
> Hi Sergey,
> 
> I see that you have removed content of JNI functions related ti initID's.
> Do we need the structure of these JNI functions, if yes can we add some kind of trace log to say there is no implementation.
> 
> Thanks,
> Jay
> 
>> On 13-May-2019, at 4:40 PM, Sergey Bylokhov <Sergey.Bylokhov at oracle.com> wrote:
>>
>> Hello.
>> Please review code cleanup for JDK 13.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8223766
>> Fix: http://cr.openjdk.java.net/~serb/8223766/webrev.00
>>
>> We have a few places in which the "initIDs()" methods initialize native data which are never used.
>> This code remains from the MToolkit and some old part of AWT on Windows. In the fix, this code was removed.
>>
>> Note that some of the initIds() methods became empty, in some future fix I'll try to delete them as well.
>>
>> -- 
>> Best regards, Sergey.
> 


-- 
Best regards, Sergey.


More information about the awt-dev mailing list