gpu: fixed vbo attribute binding when one of the attribute on the buffer does not exist in the shader

src/gpu/shader.cpp

@@ -342,7 +342,6 @@ ShaderAttrib Shader::GetAttribLocation(char const *attr,
         if (l < 0)
         {
             Log::Warn("tried to query invalid attribute: %s\n", attr);
-            l = 0;
         }
         else
         {

src/gpu/vertexbuffer.cpp

@@ -356,7 +356,8 @@ void VertexDeclaration::SetStream(VertexBuffer *vb, ShaderAttrib attr1,
         uint32_t index = l[n].m_flags & 0xffff;
 
 #   if !defined __CELLOS_LV2__
-        glEnableVertexAttribArray((GLint)reg);
+		if (reg != 0xffffffff)
+	        glEnableVertexAttribArray((GLint)reg);
 #   else
         switch (usage)
         {
@@ -438,32 +439,35 @@ void VertexDeclaration::SetStream(VertexBuffer *vb, ShaderAttrib attr1,
 
 
 #   if !defined __CELLOS_LV2__
-        if (tlut[type_index].type == GL_FLOAT
+		if (reg != 0xffffffff)
-             || tlut[type_index].type == GL_DOUBLE
+		{
-             || tlut[type_index].type == GL_BYTE
+			if (tlut[type_index].type == GL_FLOAT
-             || tlut[type_index].type == GL_UNSIGNED_BYTE
+				 || tlut[type_index].type == GL_DOUBLE
+				 || tlut[type_index].type == GL_BYTE
+				 || tlut[type_index].type == GL_UNSIGNED_BYTE
 #       if defined USE_GLEW && !defined __APPLE__
-             /* If this is not available, don't use it */
+				 /* If this is not available, don't use it */
-             || !glVertexAttribIPointer
+				 || !glVertexAttribIPointer
 #       endif
-             || false)
+				 || false)
-        {
+			{
-            /* Normalize unsigned bytes by default, because it's usually
+				/* Normalize unsigned bytes by default, because it's usually
-             * some color information. */
+				 * some color information. */
-            GLboolean normalize = (tlut[type_index].type == GL_UNSIGNED_BYTE)
+				GLboolean normalize = (tlut[type_index].type == GL_UNSIGNED_BYTE)
-                               || (tlut[type_index].type == GL_BYTE);
+								   || (tlut[type_index].type == GL_BYTE);
-            glVertexAttribPointer((GLint)reg, tlut[type_index].size,
+				glVertexAttribPointer((GLint)reg, tlut[type_index].size,
-                                  tlut[type_index].type, normalize,
+									  tlut[type_index].type, normalize,
-                                  stride, (GLvoid const *)(uintptr_t)offset);
+									  stride, (GLvoid const *)(uintptr_t)offset);
-        }
+			}
 #       if defined GL_VERSION_3_0
-        else
+			else
-        {
+			{
-            glVertexAttribIPointer((GLint)reg, tlut[type_index].size,
+				glVertexAttribIPointer((GLint)reg, tlut[type_index].size,
-                                   tlut[type_index].type,
+									   tlut[type_index].type,
-                                   stride, (GLvoid const *)(uintptr_t)offset);
+									   stride, (GLvoid const *)(uintptr_t)offset);
-        }
+			}
 #       endif
+		}
 #   else
         switch (usage)
         {