Re: [patch] NetComponents - TelnetInputStream.java threading issue

["Daniel F. Savarese" <df...@savarese.org>]

I wrote:
>In the patch, making __hasReachedEOF and __isClosed volatile doesn't do
>anything.  Those are not static variables.  Changing the notify() to

Er, what I meant to say is that assignments to those variables are already
protected by critical sections.  Your patch removed one of those protections,
so the volatile would have been appropriate.  I left in the protection.

daniel






--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>