Changeset 21930 for trunk/src/winmm/os2timer.cpp

Timestamp:

Dec 30, 2011, 10:44:51 AM (14 years ago)

Author:

dmik

Message:

winmm: Disable forceful termination of timer threads.

This is absolutely unnecessary and may cause resource leaks
eventually leading to the application hang or crash.

The above symptoms are known to happen in Adobe Flash applications
which use a lot of timers which are started/stopped at a very high rate.
This fix gets rid of these symptoms.

File:

• trunk/src/winmm/os2timer.cpp (modified) (2 diffs)

trunk/src/winmm/os2timer.cpp

@@ -388 +388 @@
   StopTimer();
   
-  if(DosPostEventSem(TimerSem))
-  {  
-    //something went wrong, kill the thread
-    TerminateThread(hTimerThread, -1);
-  }
+  DosPostEventSem(TimerSem);
+
   TimerStatus = InActive;
 }
@@ -499 +496 @@
   //release object
   release();
-  
-  // mark this thread as terminated
-  ExitThread(0);
 }
 //******************************************************************************