Still a problem with eraseNonReal, put this mutex another way

cosurgi · May 1, 2015 · 28d9acf · 28d9acf
1 parent 0a0e677
commit 28d9acf
Showing 1 changed file with 10 additions and 2 deletions.
diff --git a/core/InteractionContainer.cpp b/core/InteractionContainer.cpp
@@ -130,8 +130,16 @@ void InteractionContainer::requestErase(Interaction* I){
 }
 
 void InteractionContainer::eraseNonReal(){
-	boost::mutex::scoped_lock lock(drawloopmutex);
-	FOREACH(const shared_ptr<Interaction>& i, *this) if(!i->isReal()) this->erase(i->getId1(),i->getId2(),i->linIx);
+	FOREACH(const shared_ptr<Interaction>& i, *this)
+	{
+		bool isNotReal(false);
+		{
+			boost::mutex::scoped_lock lock(drawloopmutex);
+			isNotReal = not i->isReal();
+		}
+		if(isNotReal)
+			this->erase(i->getId1(),i->getId2(),i->linIx);
+	}
 }
 
 // compare interaction based on their first id