RFR 8191516: OutputStream.write(byte[], int, int) could have fewer parameter bounds checks
Paul Sandoz
paul.sandoz at oracle.com
Tue Nov 21 02:32:29 UTC 2017
> On 20 Nov 2017, at 16:53, Brian Burkhalter <brian.burkhalter at oracle.com> wrote:
>
> https://bugs.openjdk.java.net/browse/JDK-8191516
> http://cr.openjdk.java.net/~bpb/8191516/webrev.00/
>
> Change OutputStream.write(byte[],int,int) to use the same three parameter bounds checks used by InputStream.read(byte[],int,int) instead of the five checks currently used. This change is covered by the existing test jdk/java/io/OutputStream/WriteParams.java.
>
See also Objects.checkFromIndexSize if you wanna use that instead.
Also the if len == 0 check is probably redundant, i doubt it makes any difference given the condition needs to be checked before entering the loop.
Paul.
More information about the core-libs-dev
mailing list