Christian Wieninger wrote:
Thanks! So the following should work (please correct me if I'm wrong):
I add a cMutex object in the cList derived class and 2 functions:
class cThreadSafecList: public cList<...> {
...
private:
cMutex mutex;
public:
void Lock(void) { mutex.Lock(); }
void Unlock(void) { mutex.Unlock(); }
...
}
Its easier and more convenient if you simply do this:
class cThreadSafeList: public cList<...>, public cMutex {
This allows you to use the very comfortable cMutexLock:
void myfunction() {
cMutexLock MyLock(MyThreadsafeList);
// do critical stuff
// unlock will be called auto-magically here
}
The locking time
should be short to avoid a sluggish OSD, if one accesses
MyThreadSafecList in the main thread.
Don't lock too often, doing all the unlocking and re-locking may eat up
more time than simply holding the lock for a moment. Only unlock if you
don't need access for quite some time.
btw: A mutex lock is always assigned to a whole thread. Its safe to lock
the mutex several times from one thread. You can guard critical function
calls with a lock, and place another lock at the beginning of larger
functions to group all the small locks into one big.
btw2: Each cThread has a built-in mutex for free use. So if it matches
your situation, do this:
class cListWithThread: public cList<...>, public cThread {
Instead of cMutexLock, you can use cThreadLock here.
Cheers,
Udo
_______________________________________________
vdr mailing list
vdr@linuxtv.org
http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
