Bug in Cylinder (solved)

Arrgh, just noticed that this issue has already been solved. I only didn’t update from CVS the last few days - my fault. smack :smiley:

This topic can be closed/deleted/whatever…





Yeah, me again…



I believe there’s a bug in com.jme.scene.shape.Cylinder. When using a Cylinder with a CullState that culls back faces, the bottom plane of the cylinder is visible only on the inside. And it seems that issue has not been posted yet.



Some pics to illustrate my point:



A backface-culled Cylinder’s top plane





The same Cylinder’s bottom plane:





Obviously, the orientation of the triangles of the Cylinder’s bottom is wrong, therefore the wrong side gets culled. Of course, this display error can be avoided by not setting a CullState. Anyway, I’d be happy if this problem could be fixed in the future, as the X3D loader I wrote for jME recently also supports backface culling of cylinders specified in an X3D scene. As long as the attribute solid of an X3D Cylinder element is not set to “false” (default “true”), this bug in jME’s Cylinder causes a display error…