This is the mail archive of the java-patches@sources.redhat.com mailing list for the Java project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: java.io PipedInputStream PipedOutputStream PipedReader PipedWriter


>>>>> "Bryce" == Bryce McKinlay <bryce@albatross.co.nz> writes:

Bryce> The Piped I/O classes we inherited from classpath were somewhat
Bryce> broken (especially PipedReader/PipedWriter). These new
Bryce> implementations are much simpler and hopefully less buggy -
Bryce> they work for my code and pass all the test cases I found (JCL,
Bryce> Kaffe, Classpath).

There are some Mauve tests too.

BTW I noticed in the Reader classes you replaced synchronized(lock)
with synchronization on the pipe itself.  This seems wrong.  I thought
Readers were always supposed to synchronize on their lock object.
It might only matter if the user tries to subclass PipedReader.

Tom

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]