On Tue, 15 Sep 2026 06:09:50 GMT, Alexander Zvegintsev <[email protected]> 
wrote:

>> Phil Race has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   8392349
>
> src/java.desktop/share/native/libawt/awt/image/BufImgSurfaceData.c line 362:
> 
>> 360:             initInverseGrayLut(pRgb, bisdo->lutsize, cData);
>> 361:         }
>> 362:         (*env)->ReleaseIntArrayElements(env, bisdo->lutarray, pRgb, 0);
> 
> Suggestion:
> 
>         (*env)->ReleaseIntArrayElements(env, bisdo->lutarray, pRgb, 
> JNI_ABORT);
> 
> 
> For consistency with the line 354, and it looks like that copying back is not 
> needed here.

yep, no need. Fixed.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/32862#discussion_r4019752832

Reply via email to