On Fri, 27 Jun 2025 20:19:24 GMT, Andy Goryachev <ango...@openjdk.org> wrote:
>> Johan Vos has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Process reviewer comments
>
> modules/javafx.graphics/src/main/java/com/sun/glass/ui/headless/HeadlessWindow.java
> line 329:
>
>> 327: for (int j = 0; j < width; j++) {
>> 328: int idx = i * width + j;
>> 329: int fidx = (y + i) * 1000 + x + j;
>
> where does 1000 come from?
it's the stride used in the ByteBuffer holding the frame. I've moved it to a
final int and removed all hard occurrences of 1000 .
> modules/javafx.graphics/src/main/java/com/sun/glass/ui/headless/HeadlessWindow.java
> line 341:
>
>> 339: int offsetX = this.getX();
>> 340: int offsetY = this.getY();
>> 341: int stride = 1000;
>
> same magic 1000 here
it's the stride used in the ByteBuffer holding the frame. I've moved it to a
final int and removed all hard occurrences of 1000 .
> modules/javafx.graphics/src/main/java/com/sun/glass/ui/headless/HeadlessWindow.java
> line 356:
>
>> 354:
>> 355: void clearRect(int x0, int w0, int y0, int h0) {
>> 356: int stride = 1000;
>
> and here...
it's the stride used in the ByteBuffer holding the frame. I've moved it to a
final int and removed all hard occurrences of 1000 .
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1836#discussion_r2189477073
PR Review Comment: https://git.openjdk.org/jfx/pull/1836#discussion_r2189477376
PR Review Comment: https://git.openjdk.org/jfx/pull/1836#discussion_r2189477514
