RFR: 8273349: Check uses of Stream::peek in controls and replace as needed [v3]

Marius Hanl mhanl at openjdk.org
Thu Mar 28 17:37:37 UTC 2024


On Thu, 28 Mar 2024 08:49:44 GMT, drmarmac <duke at openjdk.org> wrote:

>> This PR removes potentially incorrect usages of Stream.peek().
>> The changed code should be covered by the tests that are already present.
>
> drmarmac has updated the pull request incrementally with two additional commits since the last revision:
> 
>  - Preallocate in SelectedIndicesList.set()
>  - Remove unused import

Looks good. Should be good regarding performance as well.

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

Marked as reviewed by mhanl (Committer).

PR Review: https://git.openjdk.org/jfx/pull/1430#pullrequestreview-1966854737


More information about the openjfx-dev mailing list