[Committed] Inconsistencies in JOGL- and LWJGLTextureState

hey there,



While trying to resolve issue http://www.jmonkeyengine.com/forum/index.php?topic=10563.0 I think I found a few bugs in the TextureStates. They look like classic Copy&Paste Bugs. One block with T dimension done, copied and all (but one) occurances of T changed to R. This seemed to happen a few times.



Check it out. Maybe I’m wrong though  :|



Index: src/com/jme/scene/state/jogl/JOGLTextureState.java
===================================================================
--- src/com/jme/scene/state/jogl/JOGLTextureState.java   (revision 4561)
+++ src/com/jme/scene/state/jogl/JOGLTextureState.java   (working copy)
@@ -1472,7 +1472,7 @@
                     checkAndSetUnit(unit, record);
                     gl.glTexGeni(GL.GL_R, GL.GL_TEXTURE_GEN_MODE,
                             GL.GL_NORMAL_MAP);
-                    unitRecord.textureGenTMode = GL.GL_NORMAL_MAP;
+                    unitRecord.textureGenRMode = GL.GL_NORMAL_MAP;
                 }
 
                 if (!unitRecord.isValid() || unitRecord.textureGenQ) {
@@ -1519,7 +1519,7 @@
                     checkAndSetUnit(unit, record);
                     gl.glTexGeni(GL.GL_R, GL.GL_TEXTURE_GEN_MODE,
                             GL.GL_REFLECTION_MAP);
-                    unitRecord.textureGenTMode = GL.GL_REFLECTION_MAP;
+                    unitRecord.textureGenRMode = GL.GL_REFLECTION_MAP;
                 }
 
                 if (!unitRecord.isValid() || unitRecord.textureGenQ) {
@@ -1611,14 +1611,14 @@
                         || unitRecord.textureGenQMode != GL.GL_OBJECT_LINEAR) {
                     gl.glTexGeni(GL.GL_Q, GL.GL_TEXTURE_GEN_MODE,
                             GL.GL_OBJECT_LINEAR);
-                    unitRecord.textureGenSMode = GL.GL_OBJECT_LINEAR;
+                    unitRecord.textureGenQMode = GL.GL_OBJECT_LINEAR;
                 }
 
                 if (!unitRecord.isValid()
                         || unitRecord.textureGenRMode != GL.GL_OBJECT_LINEAR) {
                     gl.glTexGeni(GL.GL_R, GL.GL_TEXTURE_GEN_MODE,
                             GL.GL_OBJECT_LINEAR);
-                    unitRecord.textureGenTMode = GL.GL_OBJECT_LINEAR;
+                    unitRecord.textureGenRMode = GL.GL_OBJECT_LINEAR;
                 }
 
                 if (!unitRecord.isValid()
Index: src/com/jme/scene/state/lwjgl/LWJGLTextureState.java
===================================================================
--- src/com/jme/scene/state/lwjgl/LWJGLTextureState.java   (revision 4561)
+++ src/com/jme/scene/state/lwjgl/LWJGLTextureState.java   (working copy)
@@ -1509,7 +1509,7 @@
                     checkAndSetUnit(unit, record);
                     GL11.glTexGeni(GL11.GL_R, GL11.GL_TEXTURE_GEN_MODE,
                             ARBTextureCubeMap.GL_NORMAL_MAP_ARB);
-                    unitRecord.textureGenTMode = ARBTextureCubeMap.GL_NORMAL_MAP_ARB;
+                    unitRecord.textureGenRMode = ARBTextureCubeMap.GL_NORMAL_MAP_ARB;
                 }
 
                 if (!unitRecord.isValid() || unitRecord.textureGenQ) {
@@ -1550,13 +1550,13 @@
                             ARBTextureCubeMap.GL_REFLECTION_MAP_ARB);
                     unitRecord.textureGenTMode = ARBTextureCubeMap.GL_REFLECTION_MAP_ARB;
                 }
-
+                
                 if (!unitRecord.isValid()
                         || unitRecord.textureGenRMode != ARBTextureCubeMap.GL_REFLECTION_MAP_ARB) {
                     checkAndSetUnit(unit, record);
                     GL11.glTexGeni(GL11.GL_R, GL11.GL_TEXTURE_GEN_MODE,
                             ARBTextureCubeMap.GL_REFLECTION_MAP_ARB);
-                    unitRecord.textureGenTMode = ARBTextureCubeMap.GL_REFLECTION_MAP_ARB;
+                    unitRecord.textureGenRMode = ARBTextureCubeMap.GL_REFLECTION_MAP_ARB;
                 }
 
                 if (!unitRecord.isValid() || unitRecord.textureGenQ) {
@@ -1648,14 +1648,14 @@
                         || unitRecord.textureGenQMode != GL11.GL_OBJECT_LINEAR) {
                     GL11.glTexGeni(GL11.GL_Q, GL11.GL_TEXTURE_GEN_MODE,
                             GL11.GL_OBJECT_LINEAR);
-                    unitRecord.textureGenSMode = GL11.GL_OBJECT_LINEAR;
+                    unitRecord.textureGenQMode = GL11.GL_OBJECT_LINEAR;
                 }
 
                 if (!unitRecord.isValid()
                         || unitRecord.textureGenRMode != GL11.GL_OBJECT_LINEAR) {
                     GL11.glTexGeni(GL11.GL_R, GL11.GL_TEXTURE_GEN_MODE,
                             GL11.GL_OBJECT_LINEAR);
-                    unitRecord.textureGenTMode = GL11.GL_OBJECT_LINEAR;
+                    unitRecord.textureGenRMode = GL11.GL_OBJECT_LINEAR;
                 }
 
                 if (!unitRecord.isValid()

can someone confirm this?