RFR: 8324941: POC for Headless platform for JavaFX [v4]

Johan Vos jvos at openjdk.org
Mon Jul 7 09:29:22 UTC 2025


On Fri, 27 Jun 2025 20:19:24 GMT, Andy Goryachev <angorya at 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


More information about the openjfx-dev mailing list