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:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to