Hello Steven, Tuesday, January 9, 2007, 11:29:04 PM, you wrote:
AMDG
Andrey Semashev wrote:
Right, thanks for finding this. I'll try to fix it. I guess, I'll have to create a global mutex object for the assignment operations and lock it prior to other two locks and assignment. I think that this works: locking_state_machine& operator= (locking_state_machine const& that) { if(boost::addressof(that) != this) { const bool lock_this_first = boost::addressof(m_Mutex) < boost::addressof(that.m_Mutex); scoped_lock that_lock(lock_this_first? m_Mutex : that.m_Mutex); scoped_lock this_lock(lock_this_first? that.m_Mutex : m_Mutex); base_type::operator= (static_cast< base_type const& >(that)); } return *this; }
Fixed. I've attached the updated file. -- Best regards, Andrey mailto:andysem@mail.ru