On Mon, 4 Apr 2022 18:13:28 GMT, Phil Race <p...@openjdk.org> wrote: >> Artem Semenov has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Is the check for the AccessibleRole.POPUP_MENU is needed here? How it will >> work if the Menu is added to some container other than POPUP_MENU? > > src/java.desktop/macosx/native/libawt_lwawt/awt/a11y/CommonComponentAccessibility.m > line 129: > >> 127: * Here we should keep all the mapping between the accessibility >> roles and implementing classes >> 128: */ >> 129: rolesMap = [[NSMutableDictionary alloc] initWithCapacity:51]; > > I suppose 51 just seemed like a strange number ?
I removed one position from this dictionary, and therefore reduced the capacity. ------------- PR: https://git.openjdk.java.net/jdk/pull/8043
