On Mon, 21 Nov 2022 17:43:23 GMT, Markus KARG <d...@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