On Fri, 25 Aug 2023 19:18:45 GMT, Harshitha Onkar <[email protected]> wrote:
>> In awt_MenuItem.cpp (712,22): ` mii.dwTypeData = (LPTSTR)(*sb)` produces >> invalid pointer cast warning when complied on clang and moreover this is a >> no-op. >> >> `mii.dwTypeData` is used only when **MIIM_STRING** flag is set in the fMask >> (as per >> [Docs](https://learn.microsoft.com/en-us/windows/win32/api/winuser/ns-winuser-menuiteminfoa)), >> which is not the case in JDK >> [Ln#705](https://github.com/openjdk/jdk/blob/e56d3bc2dab3d32453b6eda66e8434953c436084/src/java.desktop/windows/native/libawt/windows/awt_MenuItem.cpp#L706). >> Hence the assignment ` mii.dwTypeData = (LPTSTR)(*sb)` is not required and >> so is the label parameter. Additionally necessary cleanup is done at the >> following places - >> >> - WMenuItemPeer.java - to the native function call >> - awt_MenuItem.cpp - `WMenuItemPeer__1setLabel() ,_SetLabel(), SetLabel()` >> - awt_MenuItem.h >> >> Added a test which checks setLabel() functionality on Menu, MenuItem and >> PopupMenu. > > Harshitha Onkar has updated the pull request incrementally with one > additional commit since the last revision: > > removed robot.waitForIdle() calls > This proves _what's done in `_SetLabel` is **dead code**_: it's not used by > us, it's not used by the OS. Removing more than 100 lines which essentially > do nothing is a good thing in my opinion. OK. So it doesn't pull it because we don't tell it it is there to be read. Which does make it sound very much like JAWS is getting the data from somewhere else. ------------- PR Comment: https://git.openjdk.org/jdk/pull/15276#issuecomment-1701875314
