RFR [9] 8138978: Examine usages of sun.misc.IOUtils

[Paul Sandoz]

> On 8 Oct 2015, at 10:46, Chris Hegarty <chris.hegarty at oracle.com> wrote:
>> It reads at most “seqlen" bytes, so the array may be larger than necessary, which might be ok depending on whether one can trust "seqlen”.
> 
> We do no trust ‘seqlen’. :-(
> 

Good!


>> The following pattern occurs a few times:
>> 
>> byte[] b = is.readAllBytes();
>> if (len !- -1 && b.length != len) throw new EOFException(…)
>> 
>> A further useful addition to consider would be an IS.readFulyl(int expectedLength).
> 
> Yes, I think a variant of readNBytes(long expectedLength),
> would  be a useful addition. I can file a bug and add it to
> the I/O grab bag, 6852033 [1]. Then a further round of
> cleanup could possibly remove IOUtils completely.
> 

Ok.

Paul.