On Thu, 28 Mar 2024 08:49:44 GMT, drmarmac <d...@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