RFR: JDK-8297299 SequenceInputStream should not use Vector [v6]
Jaikiran Pai
jpai at openjdk.org
Tue Nov 22 01:45:29 UTC 2022
On Mon, 21 Nov 2022 17:43:23 GMT, Markus KARG <duke at openjdk.org> wrote:
>> There is no need to use a temporary Vector within the constructor of SynchronizedInputStream, as more efficient (non-synchronized) alternative code (like List.of) will do the same in possibly less time. While the optimization is not dramatic, it still makes sense to replace Vector unless synchronization is really needed.
>
> Markus KARG has updated the pull request incrementally with one additional commit since the last revision:
>
> private member variables
Thank you Markus for these changes. The latest state in `8d298318` looks good to me. I'll run some tests today before sponsoring this.
-------------
Marked as reviewed by jpai (Reviewer).
PR: https://git.openjdk.org/jdk/pull/11249
More information about the core-libs-dev
mailing list