On Tue, 6 Sep 2022 16:56:03 GMT, Markus KARG <d...@openjdk.org> wrote:

>> src/java.base/share/classes/java/io/BufferedInputStream.java line 608:
>> 
>>> 606:     private long implTransferTo(OutputStream out) throws IOException {
>>> 607:         if (getClass() == BufferedInputStream.class
>>> 608:                 && ((count - pos) <= 0) && (markpos == -1)) {
>> 
>> Tiny thing: I think with 3 `&&`s we don't need parenthesis around `((count - 
>> pos) <= 0) && (markpos == -1))`. How about to leave leave only the one 
>> around `count - pos`?
>
> I was using the exact code proposed by @AlanBateman and I don't dare to touch 
> it. ;-)

@AlanBateman Please decide. :-)

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

PR: https://git.openjdk.org/jdk/pull/6935

Reply via email to