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

Reply via email to