Yup, it finally happened.
I am making a voxel terrain generation project to learn OpenGL. It was my abstraction of vertex arrays. Initially, when I created this class, it generated an ID in the constructor, but then when I introduced multithreading, I needed to stop doing that in the constructor (bad design, I know—need to study design patterns). So I introduced a boolean to initialize it when the first call to
Bind() is made. But I didn't set it to false at that time. I saw chunks rendering, but there were gaps between them. So I started debugging, and honestly, the VertexArray class wasn't even on my mind. I just printed the VAO values in the output along with some other data. Although the values were somewhat random, I ignored it because OpenGL only guarantees unique unused values, not how they're generated. But then in the middle, I saw some were small and continuous like 1, 2, ..., 10. Then I put a print statement in the Generate() function of VertexArray and realized it wasn't even being called.Yup, that's my rant. And here's the ugly code I wrote:
cpp
class VertexArray {
public:
explicit VertexArray(bool lazy = false);
~VertexArray();
// Returns the vertex array ID
GLuint id() const { return arrayid; }
void Generate();
// Binds this vertex array
void Bind();
void UnBind();
// Adds and enables the attribute
void AddAttribute(Attribute attribute);
private:
GLuint arrayid{};
};
https://redd.it/1prafjr
@r_cpp
