On Fri, 23 Jun 2023 14:28:51 GMT, Alexey Ivanov <aiva...@openjdk.org> wrote:
>> The declaration and implementation have to match. > > To minimise the number of changes, we can go for using `jint` in > `AwtMenu::GetItem`. > > What do you thing, @djelinski and @TheShermanTanker? Hmm, I lean towards jint as I feel it conveys the fact that it is a Java parameter clearer, intuitively to me it makes sense that a Java integer type would still work in a C++ for loop in native code ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/14125#discussion_r1239893754