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