On 11/08/11 17:27, Alan Bateman wrote:
Chris Hegarty wrote:
Here is a first stab at fixing this issue.

  http://cr.openjdk.java.net/~chegar/7014860/webrev.00/webrev/

-Chris.
The spec clarification seems reasonable to me. Small typo "the streams available method" to "the stream's available method".

Good to see the NIO socket adapter included in the test. One comment on the test is that the Thread.dumpStack is a bit odd. It might be better to pass in a test name into the test method and have that printed by the failure message.

-Alan.

Just wondering what the motivation for changing this is? I can see the logic in the change all right, but has anyone actually encountered this problem (of an inconsistency between InputStream.available() and the number of bytes that can be returned)?

- Michael

Reply via email to