On 02/05/2015 09:27, Chris Hegarty wrote:
:
Thanks, this was an editing issue. Removed.
I think the javadoc looks quite good now, except may be the first
statement "Reads some bytes ...". It might be clearer to start with
"Reads a given number of bytes ...". The subsequent text makes the short
read case and the return value clear.
As Alan has commented, another readAllBytes() returning a byte[] maybe useful
too ( but a different use case ). Let’s park this momentarily, while I sketch
up the readAllBytes variant, so we can ensure that the typical use cases have
been addressed. Doing so may feedback into the spec of this method. I’ll push
this latest draft into the sandbox so it is not lost.
Yes, a separate use-case but once that I would expect to be common.
-Alan.
