Re: Inappropriate use of synchronized in PlainSocketImpl: Submit for Classpath

[Brian Jones]

"Wu, Gansha" <address@hidden> writes:

>     We have modified GNU Classpath so that ORP with Classpath can
>     run Volano. Here is the first of the patches needed for that.
>     We found that Classpath's implementation of PlainSocketImpl(in
>     java/net/PlainsocketImpl.java) overused synchronized keyword
>     before almost all the native methods such as read, write,
>     etc.. Generally speaking native socket implementation(e.g.,
>     Winsock) takes the responsibility for thread-safety, and if we
>     raise the responsibility up to Java language level, there'll be
>     problems here in multi-threaded senarios.

I submitted the two line change today, thanks.

Brian
-- 
Brian Jones <address@hidden>