I'm trying to get VBO's to work in OpenGL. I'm following a ton of tutorials, and vertex arrays are working fine, but I get a runtime crash when using vertex buffer objects. Here's the code:
// global vars:
GLfloat *vertices;
GLfloat *texcoords;
GLuint vbo;
When I change the following lines, it works, but that's using vertex arrays, rather than vbo's.
// These lines cause a crash
glVertexPointer(3, GL_FLOAT, 0, 0);
glTexCoordPointer(2, GL_FLOAT, 0, (GLvoid*)(sizeof(vertices)))
// These lines work, but use vertex arrays rather than vbos
glVertexPointer(3, GL_FLOAT, 0, vertices);
glTexCoordPointer(2, GL_FLOAT, 0, texcoords);
The actual crash is an "unhandled exception" on the line that contains glDrawArrays.
Here are any relevant system specs:
Language: C++
IDE: Visual Studio 2010 Professional
OS: Windows 7 Ultimate
GPU: nVidia GeForce 9650m GT (supports up to GL 3.3)
Hi, sizeof(vertices)
isn't doing what you are hoping it does, I think. sizeof(vertices) will (probably) be 4, since it is just a pointer. You probably want something more like: sizeof(float) * 3 * NUM_VERTICES
which will give you the size in bytes of your vertex array. Some explanation:
(1) using float, because that's the data type of your vertex coordinates
(2) multiplying by 3, because there are three floats per vertices (x, y, z)
Also, keep in mind that the last parameter in glVertexPointer works differently when you have a buffer bound.
You probably want something more like: sizeof(float) * 3 * NUM_VERTICES
You're right, that code was wrong. I'm actually using the following: glTexCoordPointer(2, GL_FLOAT, 0, (GLvoid*)(9 * sizeof(GLfloat)));
Still doesn't work though. If I take the texture coordinates out altogether, and just try to render the vertex coordinates, it still crashes at the same place.
I'm guessing that second snippet isn't right either. glTexCoordPointer's 4th parameter, when you have a buffer bound, is the offset into the buffer where the texture coordinates begin. Unless your vertex array is made up of only three points, (3 vertices with 3 floats each), 9*sizeof(float) is going to be wrong. <br><br>Also, in pretty much every place (in your original post) where you are using sizeof, it is being used incorrectly, so make sure to double check them all. If you want to post an updated sample that might help.
<div><br></div><div>edit: I guess you are just drawing three points. It might help to see where you are creating/populating the arrays as well.</div>
Why would it matter what creating my arrays look like? If I use the vertex array method of passing the pointer to the array into the gpu every frame, then it works, correct textures and vertices, so clearly the arrays are fine.
I don't think it's relevant, but here ya go:
vertices = new GLfloat[9];
texcoords = new GLfloat[6];
vertices[0] = 0.0f;
...
vertices[8] = 1.0f;
texcoords[0] = 0.0f;
...
texcoords[5] = 1.0f;
I'm also using GLee. My gl.h header is version 1.2 (standard on every windows machine) and I'm getting all of my buffer functions declared in GLee.h. Not sure if that makes a difference.
Just trying to eliminate possible problems. If you can post your latest vbo setup/drawing code that would be helpful too.
I can post the entire source file in about 5 hours. I think I might take the vbo specific code from the program and see if it doesn't work in a program all by itself. I have a few other things going on including keyboard/mouse input, displaying the framerate, moving my character, collision detection. I've commented almost all of that out though, so I could focus on the switch from vertex arrays to vbos.
Still, regardless of all that other stuff, why would it crash on glDrawArrays... I wonder if my framerate code is calling glDrawArrays... Can you call that function twice per render loop?
Solved. I am an idiot. My issue was on the following line:
GLuint vbuffer = 0
I was declaring a local variable instead of using the global vbo, so that by the time I made it to my render function, the global vbo was still NULL. FML. Textures still don't work with the fixed code though, but I'm sure I can figure that one out. Thanks for taking a look, Scott.
