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

[Jayathirth Rao]

Thanks for clarification changes are fine.

Regards,
Jay

> On 14-May-2019, at 2:16 AM, Sergey Bylokhov <Sergey.Bylokhov at oracle.com> wrote:
> 
> 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.