X-Git-Url: http://code.vuplus.com/gitweb/?a=blobdiff_plain;f=lib%2Fbase%2Fthread.cpp;h=fa923debe3ddf08aed24ec059b663a1b7486e071;hb=6a2f0acb3f92f433eb1ec4fd29a39532dadbaaca;hp=9878856eaa483007ccaa7d92f42043645c49a835;hpb=5655fee2e15bf1cfc0fe06a85e3c864dbfcd41ce;p=vuplus_dvbapp

diff --git a/lib/base/thread.cpp b/lib/base/thread.cpp
index 9878856..fa923de 100644
--- a/lib/base/thread.cpp
+++ b/lib/base/thread.cpp
@@ -2,7 +2,6 @@
 
 #include <stdio.h>
 #include <unistd.h>
-#include <assert.h>
 #include <lib/base/eerror.h>
 
 void eThread::thread_completed(void *ptr)
@@ -10,12 +9,9 @@ void eThread::thread_completed(void *ptr)
 	eThread *p = (eThread*) ptr;
 	p->m_alive = 0;
 
-		/* recover state */
-	if (!p->m_state.value())
-	{
-		p->m_state.up();
-		assert(p->m_state.value() == 1);
-	}
+		/* recover state in case thread was cancelled before calling hasStarted */
+	if (!p->m_started)
+		p->hasStarted();
 
 	p->thread_finished();
 }
@@ -43,11 +39,13 @@ int eThread::runAsync(int prio, int policy)
 		return -1;
 	
 	eDebug("after: %d", m_state.value());
-	assert(m_state.value() == 1); /* sync postconditions */
-	assert(!m_alive);
+	ASSERT(m_state.value() == 1); /* sync postconditions */
+	ASSERT(!m_alive);
 	m_state.down();
+	ASSERT(m_state.value() == 0);
 	
 	m_alive = 1;
+	m_started = 0;
 
 		/* start thread. */
 	pthread_attr_t attr;
@@ -89,9 +87,12 @@ eThread::~eThread()
 int eThread::sync(void)
 {
 	int res;
+	int debug_val_before = m_state.value();
 	m_state.down(); /* this might block */
 	res = m_alive;
-	assert(m_state.value() == 0);
+	if (m_state.value() != 0)
+		eFatal("eThread::sync: m_state.value() == %d - was %d before", m_state.value(), debug_val_before);
+	ASSERT(m_state.value() == 0);
 	m_state.up();
 	return res; /* 0: thread is guaranteed not to run. 1: state unknown. */
 }
@@ -121,6 +122,7 @@ void eThread::kill(bool sendcancel)
 
 void eThread::hasStarted()
 {
-	assert(!m_state.value());
+	ASSERT(!m_state.value());
+	m_started = 1;
 	m_state.up();
 }