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

Reply via email to