RFR: 8273349: Check uses of Stream::peek in controls and replace as needed [v3]
Andy Goryachev
angorya at openjdk.org
Thu Mar 28 15:51:36 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
a minor question: would it be much easier to understand if the code was written in conventional procedural style? a straightforward `for` loop?
-------------
PR Comment: https://git.openjdk.org/jfx/pull/1430#issuecomment-2025544489
More information about the openjfx-dev
mailing list