DO NOT REPLY [Bug 24878] New: - DefaultThreadControl.java

[bu...@apache.org]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=24878>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=24878

DefaultThreadControl.java

           Summary: DefaultThreadControl.java
           Product: Avalon
           Version: unspecified
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: Major
          Priority: Other
         Component: Excalibur
        AssignedTo: dev@avalon.apache.org
        ReportedBy: geb@263.net


In DefaultThreadControl.java, these methods are synchronize calls but seem not 
neccesary.
        void join( final long milliSeconds ), void interrupt(),  void finish( 
final Throwable throwable )
And these methods will cause dead lock. If someone call join() to wait thread 
control finish before thread control has really finished, when thread control 
call finish() to notification, it will be blocked because it is locked by 
previous join() call.
So, my advice is to remove the synchronized call for these method.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@avalon.apache.org
For additional commands, e-mail: dev-help@avalon.apache.org