On Tue, 16 Dec 2025 09:05:04 GMT, Lukasz Kostyra <[email protected]> wrote:

>> Jayathirth D V has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   create sampler states only once
>
> modules/javafx.graphics/src/main/native-prism-mtl/MetalContext.m line 239:
> 
>> 237: 
>> 238: - (void) create3DSamplerStates
>> 239: {
> 
> I think we could add something here to skip re-creating the sampler 
> descriptors. This can be called every time we switch to 3D (from 
> `nSetDeviceParametersFor3D`) and that way we could optimize this transition a 
> bit.

That's a good point.
Actually not just descriptors even the samplers states should be created only 
once. Updated the code.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/2005#discussion_r2622950228

Reply via email to