RFR: 8246739: InputStream.skipNBytes could be implemented more efficiently [v2]

Brian Burkhalter bpb at openjdk.java.net
Fri Nov 20 16:52:09 UTC 2020


On Fri, 20 Nov 2020 11:43:11 GMT, sergus13 <github.com+74766043+sergus13 at openjdk.org> wrote:

>> Brian Burkhalter has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - 8246739: InputStream.skipNBytes could be implemented more efficiently
>>  - 8246739: InputStream.skipNBytes could be implemented more efficiently
>
> src/java.base/share/classes/java/io/InputStream.java line 605:
> 
>> 603:             } else if (ns == 0) { // no bytes skipped
>> 604:                 // read one byte to check for EOS
>> 605:                 if (read() < 0) {
> 
> Wouldn't it be better to compare return value with -1 (as it was before)?
> Since the documentation of read method says:
> _Returns:
>     the next byte of data, or **-1 if the end of the stream is reached**._

Indeed that would match the letter of the specification of `read()` but I don't think it would make any practical difference.

-------------

PR: https://git.openjdk.java.net/jdk/pull/1329


More information about the core-libs-dev mailing list