You are viewing a plain text version of this content. The canonical link for it is here.
Posted to scm@excalibur.apache.org by sh...@apache.org on 2005/04/16 16:23:23 UTC

svn commit: r161570 - excalibur/trunk/components/thread/impl/src/java/org/apache/excalibur/thread/impl/DefaultThreadControl.java

Author: shash
Date: Sat Apr 16 07:23:23 2005
New Revision: 161570

URL: http://svn.apache.org/viewcvs?view=rev&rev=161570
Log:
Patch from: http://issues.apache.org/jira/browse/EXLBR-1
Fixes: There is a small timing window between the time the worker thread has finished executing and before a call to the finish() method of the ThreadControl is made. This can cause the threads waiting on a join(0) to never wake-up. 

Modified:
    excalibur/trunk/components/thread/impl/src/java/org/apache/excalibur/thread/impl/DefaultThreadControl.java

Modified: excalibur/trunk/components/thread/impl/src/java/org/apache/excalibur/thread/impl/DefaultThreadControl.java
URL: http://svn.apache.org/viewcvs/excalibur/trunk/components/thread/impl/src/java/org/apache/excalibur/thread/impl/DefaultThreadControl.java?view=diff&r1=161569&r2=161570
==============================================================================
--- excalibur/trunk/components/thread/impl/src/java/org/apache/excalibur/thread/impl/DefaultThreadControl.java (original)
+++ excalibur/trunk/components/thread/impl/src/java/org/apache/excalibur/thread/impl/DefaultThreadControl.java Sat Apr 16 07:23:23 2005
@@ -58,7 +58,7 @@
     {
         if (! isFinished() )
         {
-            m_thread.join(milliSeconds);
+            m_thread.wait(milliSeconds);
         }
     }
 



---------------------------------------------------------------------
To unsubscribe, e-mail: scm-unsubscribe@excalibur.apache.org
For additional commands, e-mail: scm-help@excalibur.apache.org