On Wed, 6 Aug 2025 23:17:35 GMT, Nir Lisker <[email protected]> wrote:

>> MTLMesh is the geometry i.e. vertex buffer, index buffer required to render 
>> a 3D Shape. So, if we create say 10 Spheres as `Sphere(50)`, then it creates 
>> 10 MTLMeshView and only 1 MTLMesh shared among MTLMeshView.
>> 
>> The mesh is used on native side, Please check MetalMesh.render() method in 
>> the PR. 
>> This aligns very much with D3DMesh, ES2Mesh.
>
> Ah, it's used in the native side. Please add a comment saying so because it's 
> flagged as an unused field in Java.

Added a short comment and another line that is present in D3DMeshView and 
ES2MeshView.

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

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

Reply via email to