void WINAPI ThreadFunc(Thread* thread){
while(1) {
thread->Run();
}
}
This means that your thread objects will loop forever. This is not the standard way of doing things...usually you just call thread->Run() and if the thread needs to loop do it inside teh Run() method (i.e. the implementation). the way you have it, there is no way to free up the thread context since it runs forever. Furthermore, you will crash your system when you try to delete the thread (suddenly your thread pointer is invalid).
You need some way of determining what happens if someone tries to delete a thread that''s still running (this will cause stack corruptions up the wazzoo!). This means you need to provide a getStatus() function in Thread and in the destructor if block until the status is "complete" or whatever.
Bzzz...back to the drawing board with that class.
Regards,
Jeff
[ CodeDread ]
