RFR: 8273349: Check uses of Stream::peek in controls and replace as needed [v3]
Andy Goryachev
angorya at openjdk.org
Thu Mar 28 22:34:35 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
Thank you for clarifications!
Does not look like this is called from a loop, so whatever inefficiency we have is likely to be ok.
-------------
Marked as reviewed by angorya (Reviewer).
PR Review: https://git.openjdk.org/jfx/pull/1430#pullrequestreview-1967473815
More information about the openjfx-dev
mailing list