On Fri, 30 Sep 2022 14:46:35 GMT, Alexander Zvegintsev <azveg...@openjdk.org>
wrote:
>> Alisen Chung has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> changed loop end condition to ind-1
>
> src/java.desktop/unix/native/libawt_xawt/awt/awt_GraphicsEnv.c line 575:
>
>> 573: cleanup:
>> 574: if (success != JNI_TRUE) {
>> 575: for (int i = 0; i < ind-1; i++) {
>
> New `int i` declaration shadows previously declared `i`, I think it can be
> reused.
I tried building without the new int i and it does seem to be fine. Probably
worth it to follow the rest of the class's format and reuse the i var for
getAllConfigs.
-------------
PR: https://git.openjdk.org/jdk/pull/10378
