X-Git-Url: http://code.vuplus.com/gitweb/?a=blobdiff_plain;f=lib%2Fbase%2Fthread.cpp;h=fa923debe3ddf08aed24ec059b663a1b7486e071;hb=30da484f79db1d8210a3dcb31b359484873dc240;hp=5353707dc9564f57f5a34476c6e8b558db360bcb;hpb=e40ddfb473102ade34ee1d0667d143ee204a2562;p=vuplus_dvbapp

diff --git a/lib/base/thread.cpp b/lib/base/thread.cpp
index 5353707..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,19 +87,23 @@ 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. */
 }
 
-void eThread::sendSignal(int sig)
+int eThread::sendSignal(int sig)
 {
 	if (m_alive)
-		pthread_kill(the_thread, sig);
+		return pthread_kill(the_thread, sig);
 	else
 		eDebug("send signal to non running thread");
+	return -1;
 }
 
 void eThread::kill(bool sendcancel)
@@ -120,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();
 }