On Thu, 2 Apr 2026 03:07:21 GMT, Sergey Bylokhov <[email protected]> wrote:
>> Phil Race has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> 8377568
>
> src/java.desktop/share/classes/java/awt/image/DataBufferFloat.java line 312:
>
>> 310: */
>> 311: public int getElem(int bank, int i) {
>> 312: checkIndex(i);
>
> this should be `checkIndex(bank, i)` instead
fixed
> test/jdk/java/awt/image/DataBuffer/DataBufferConstructorTest.java line 218:
>
>> 216: boolean signed, Class
>> expectedExceptionType) {
>> 217: try {
>> 218: new DataBufferShort(dataArray, size);
>
> Any reason why DataBufferUShort is not checked and signed param is ignored?
just missed adding it. fixed
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/29766#discussion_r3068960345
PR Review Comment: https://git.openjdk.org/jdk/pull/29766#discussion_r3068968054
