Odtnhj

I have an OpenGL object in a C++ class. Since I'm employing RAII, I want to have the destructor delete it. So my class looks something like:

class BufferObject

private:
 GLuint buff_;

public:
 BufferObject()
 
 glGenBuffers(1, &buff_);
 

 ~BufferObject()
 
 glDeleteBuffers(1, &buff_);
 

//Other members.
;

This seems like it works. But any time I do any of the following, I start getting various OpenGL errors when I use it:

vector<BufferObject> bufVec;

 BufferObject some_buffer;
 //Initialize some_buffer;
 bufVec.push_back(some_buffer);

bufVec.back(); //buffer doesn't work.

BufferObject InitBuffer()

 BufferObject buff;
 //Do stuff with `buff`
 return buff;


auto buff = InitBuffer(); //Returned buffer doesn't work.

What's going on?

Note: this is an attempt to build a canonical answer to these questions.

All of those operations copy the C++ object. Since your class did not define a copy constructor, you get the compiler-generated copy constructor. This simply copies all of the members of the object.

Consider the first example:

vector<BufferObject> bufVec;

 BufferObject some_buffer;
 //Initialize some_buffer;
 bufVec.push_back(some_buffer);

bufVec.back(); //buffer doesn't work.

When you call push_back, it copies some_buffer into a BufferObject in the vector. So, right before we exit that scope, there are two BufferObject objects.

But what OpenGL buffer object do they store? Well, they store the same one. After all, to C++, we just copied an integer. So both C++ objects store the same integer value.

When we exit that scope, some_buffer will be destroyed. Therefore, it will call glDeleteBuffers on this OpenGL object. But the object in the vector will still have its own copy of that OpenGL object name. Which has been destroyed.

So you can't use it anymore; hence the errors.

The same thing happens with your InitBuffer function. buff will be destroyed after it is copied into the return value, which makes the returned object worthless.

This is all due to a violation of the so-called "Rule of 3/5" in C++. You created a destructor without creating copy/move constructors/assignment operators. That's bad.

To solve this, your OpenGL object wrappers should be move-only types. You should delete the copy constructor and copy assignment operator, and provide move equivalents that set the moved-from object to object 0:

class BufferObject

private:
 GLuint buff_;

public:
 BufferObject()
 
 glGenBuffers(1, &buff_);
 

 BufferObject(const BufferObject &) = delete;
 BufferObject &operator=(const BufferObject &) = delete;

 BufferObject(BufferObject &&other) : buff_(other.buff_)
 
 other.buff_ = 0;
 

 BufferObject &operator=(BufferObject &&other)
 
 //ALWAYS check for self-assignment
 if(this != &other)
 
 Release();
 buff_ = other.buff_;
 other.buff_ = 0;
 

 return *this;
 

 ~BufferObject() Release();

 void Release();
 
 if(buff_)
 glDeleteBuffers(1, &buff_);
 

//Other members.
;

There are various other techniques for making move-only RAII wrappers for OpenGL objects.