On Mon, Jun 9, 2008 at 6:14 PM, Steven Watanabe <watanabesj@gmail.com> wrote:

AMDG
[...]

// assignment operator
CircularBuffer& operator= (const CircularBuffer& other)
{

CircularBuffer temp(other);

why do you need this temp, if it is not used further?

WriteLock w_lock(rw_mutex);

using std::swap;
swap(m_capacity, other.m_capacity);
swap(m_buffer, other.m_buffer);

return *this;
}

In Christ,
Steven Watanabe

With Kind Regards,
Ovanes