On Mon, 4 Aug 2025 22:44:10 GMT, Nir Lisker <nlis...@openjdk.org> wrote:

>> Ambarish Rapte has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   kcr-andy: review comments
>
> modules/javafx.graphics/src/main/java/com/sun/prism/mtl/MTLResourceFactory.java
>  line 197:
> 
>> 195:         // contentX and contentY is set as 0 unlike D3D/ES2.
>> 196:         // The wrap mode are addressed, can be mapped to D3D/ES2 only 
>> if necessary.
>> 197:         return new MTLTexture(getContext(), resource, formatHint, 
>> wrapMode, allocw, alloch, 0, 0, allocw, alloch, useMipmap);
> 
> Same raw type.

Minor: This could just be `MTLTexture<>` instead of 
`MTLTexture<MTLTextureData>` because the type is inferred.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1824#discussion_r2258521498

Reply via email to