RFR: 8320971: Use BufferedInputStream.buf directly when param of implTransferTo() is trusted [v16]

Brian Burkhalter bpb at openjdk.org
Thu Dec 21 17:31:55 UTC 2023


On Tue, 19 Dec 2023 10:18:11 GMT, Sergey Tsypanov <stsypanov at openjdk.org> wrote:

>> It looks like we can skip copying of `byte[]` in `BufferedInputStream.implTransferTo()` for `OutputStreams` residing in `java.io`.
>> 
>> See comment by @vlsi in https://github.com/openjdk/jdk/pull/10525/files#diff-e19c508d1bb6ee78697ecca66947c395adda0d9c49a85bf696e677ecbd977af1R612
>
> Sergey Tsypanov has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 20 additional commits since the last revision:
> 
>  - 8322292: Remove TransferToTrusted.checkedOutputStream()
>  - 8320971: Adjust JavaDoc
>  - Merge branch 'master' into 8320971
>  - 8320971: Revert irrelevant changes
>  - 8320971: Add more tests
>  - 8320971: Fix JavaDoc
>  - 8320971: Whitespaces
>  - 8320971: Fix build
>  - 8320971: Move IOStreams to com.sun.io
>  - Merge branch 'master' into 8320971
>  - ... and 10 more: https://git.openjdk.org/jdk/compare/54ca8c60...84686bc6

The test does not fail when run against the mainline (HEAD: Thu Dec 21 15:20:01 2023 +0000 UTC). As previously mentioned elsewhere, normally a deterministic test should fail before the proposed source patch is applied, and succeed thereafter.

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

PR Comment: https://git.openjdk.org/jdk/pull/16879#issuecomment-1866685521


More information about the core-libs-dev mailing list