On Sun, 20 Nov 2022 17:41:28 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 refreshed the contents of this pull request, and previous > commits have been removed. The incremental views will show differences > compared to the previous content of the PR. The pull request contains one new > commit since the last revision: > > Jens' Proposal Thank you Markus for the changes. The latest version in `2e957354` looks fine to me. While we are at it, would you be willing to change the member variables `e` to `private final` and the `in` to `private`? From what I can see, I don't see any other class accessing these package private fields. ------------- PR: https://git.openjdk.org/jdk/pull/11249
