Thread: Operator overloading problem

You should change it to be a free function:

Code:

CBuffer operator+(const CBuffer& lhs, const CBuffer& rhs);

If necessary, make it a friend function, though I recommend implementing it in terms of an operator+= such as:

Code:

CBuffer& CBuffer::operator+=(const CBuffer& rhs)

Incidentally, your copy assignment operator should take a const CBuffer&, not a CBuffer&, as its argument.

I tried to define outside the class like you have, but get the same error.
Anyways i made it like this:

Code:

class CBuffer
{
  static CBuffer retBuffer;
  ..
  ..
};

and in all my sub functions where i need to return a new CBuffer i do this:

Code:

CBuffer& CBuffer::operator+(CBuffer& pOther)
{
  retBuffer.clear();
  ...
  return retBuffer;
}

Btw how does putting const affect it?

Im using mingw btw

I tried to define outside the class like you have, but get the same error.

I am not entirely sure what is the problem, so I am guessing that it is const-correctness.

Btw how does putting const affect it?

You are not going to (and should not) change the arguments passed. In any case, your copy assignment operator is wrong, and needs to be const-correct.

I think you should remove your static CBuffer member variable (is that even legal?), and are you sure you need an operator+? Do not overload operators unless it is necessary and intuitive.