glGetError() INVALID_OPERATION (1282)

glGetError returns the current error state of the OpenGL state machine. After calling glGetError, the error state reverts to GL_NO_ERROR and it will remain so until an OpenGL function is called which sets an error state.

If you find an unexpected error state you need to find the function(s) that set it. At some point glGetError is GL_NO_ERROR right before a GL call but not afterwards. This function is not used correctly.

It's probably not a bad idea to have (as a bare minimum solution) something like assert(glGetError() == GL_NO_ERROR); after each GL call. That will immediately complain about error conditions in debug builds but compile down to nothing in release builds.

Disable texturing before drawing non-textured things? And enable it.

glDisable(TEXTURE_2D);

But I guess we need more info.

most likely you forget to bind some necessary data to the current shader, pls post your code

render for debugging with glPolygonmode in wireframe, helps alot

(and dont use black as background color ;D )

I've got to admit that this was the first time I've actually used glGetError() to track stuff down, but once I realized it holds the last error, I kept running it through my initialization code until I found the method that does it. Calling glGetError() after each gl* call is expensive on OES implementations, so if I do that, I'll have to setup a bunch of #ifdef preprocessors so that I can run my builds w/o being bogged down.

As far as glDisable(GL_TEXTURE_2D), I've been checking that, but this where I think the issue could lie because I actually don't call commonly used glEnable/glDisable states directly. Instead, I have my own method that holds a synchronization variable to determine whether the state is already enabled before calling glEnable(), and vice-versa. This meant to cull unnecessary calls for OES builds. I think something could be wrong with glBindTexture because I modified one of my wrapper methods recently for that particular call. I'll also have to check glActiveTexture's wrapper function...

My models also don't use just textures directly, they use a Material object that is configured to work with the corresponding config of my uber shader built for that model to render. My Material class has some logic when uploading texture types that don't exist across each mesh in my model (uber shaders work at the per-model level for efficiency). So, if some parts of my model uses normal mapping, but others don't, instead of loading two shaders, I just load one with normal mapping, and for materials that don't have normal maps associated with it, it'll send a default 2x2 normal map. If I'm using diffuse maps, then they default meshes without ambient maps to a white texture.

These textured vertices aren't lit, however, and I know it's going through the lighting shader too, so I think it's some sort of GL state issues like you've all pointed out.

glGetError returns the current error state of the OpenGL state machine. After calling glGetError, the error state reverts to GL_NO_ERROR and it will remain so until an OpenGL function is called which sets an error state.

That is not entirely correct.

If you check http://www.opengl.org/sdk/docs/man/xhtml/glGetError.xml, you'll find that more than one error may be recorded by OpenGL, and that glGetError() should be called in a loop until GL_NO_ERROR is returned.

I've only had 1 OpenGL error at once, but I was thinking it acts more like a stack that'll show the most current out of a queue up to some defined max