RFR: 8271024: Implement macOS Metal Rendering Pipeline [v3]
Martin Fox
mfox at openjdk.org
Mon Jun 23 18:29:36 UTC 2025
On Mon, 23 Jun 2025 12:55:32 GMT, Jayathirth D V <jdv at openjdk.org> wrote:
>> Ambarish Rapte has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Glass changes after Metal PR inputs
>
> Glass changes are updated based on review inputs.
>
> Also @arapte noticed commented out GLASS_POOL_PUSH/POP logic in GlassView.m which is updated to match mainline code. Since this enables common autoreleasepool, explicit autorelease calls for Vertex & Index buffer in MetalMesh.m are removed. Latest code is tested again and headful CI runs are green with both OpenGL and Metal.
@jayathirthrao Thanks for renaming the classes. Much appreciated.
> 4. Regarding setOpaque() in GlassWindow.m :
That setOpaque: call still isn't being executed. The code is looking at the GlassLayer, not the GlassLayerCGL, so the isKindOfClass: is failing and setOpaque: is never called.
But there's no point in changing that code until we can reproduce the original PickTest3D bug and I haven't been able to do that. Have you? The bug (JDK-8095004) was entered in 2013 but the version of PickTest3D in apps/toys was added in early 2015. I don't know how closely it matches the code that caused the original bug report.
The original bug report doesn't add up. The comments state that the problem stems from using a Phong material that has a diffuse color with an alpha of 0.5. That should cause the individual objects to become translucent which is what happens when I tweak the PickTest3D code. But the screen shot in the bug report shows solid objects; the entire scene is washed out but the individual objects are opaque.
I can't test this on my Windows box probably because I'm running in a VM. Prism reports that the D3D renderer is running without error but I still get blank screens except for the ColorCube toy.
-------------
PR Comment: https://git.openjdk.org/jfx/pull/1824#issuecomment-2997510033
More information about the openjfx-dev
mailing list