Add clarifying comment.

If data is of float type, then we assume that it is meant for
the alpha channel even if it is marked as R/G or B channels.

Change-Id: I8a95c3533def4a5bdb2b49e9489674bffb2a0cef

1 files changed, 9 insertions, 0 deletions

diff --git a/eclipse/plugins/com.android.ide.eclipse.gldebugger/src/com/android/ide/eclipse/gltrace/state/transforms/TexImageTransform.java b/eclipse/plugins/com.android.ide.eclipse.gldebugger/src/com/android/ide/eclipse/gltrace/state/transforms/TexImageTransform.java
index a88f21c..dde89ea 100644
--- a/eclipse/plugins/com.android.ide.eclipse.gldebugger/src/com/android/ide/eclipse/gltrace/state/transforms/TexImageTransform.java
+++ b/eclipse/plugins/com.android.ide.eclipse.gldebugger/src/com/android/ide/eclipse/gltrace/state/transforms/TexImageTransform.java
@@ -193,6 +193,15 @@ public class TexImageTransform implements IStateTransform {
             case GL_RED:
             case GL_GREEN:
             case GL_BLUE:
+                // GL_RED, GL_GREEN and GL_BLUE are all supposed to fill those respective
+                // channels, but we assume that the programmers intent was to use GL_ALPHA in order
+                // to overcome the issue that GL_ALPHA cannot be used with float data.
+                if (mType != GLEnum.GL_FLOAT) {
+                    throw new RuntimeException();
+                } else {
+                    // fall through - assume that it is GL_ALPHA
+                }
+                //$FALL-THROUGH$
             case GL_ALPHA:
                 return addRGBChannels(subImageData, width, height);
             case GL_LUMINANCE: