RFR: 8273349: Check uses of Stream::peek in controls and replace as needed [v3]
Andy Goryachev
angorya at openjdk.org
Thu Mar 28 16:04:38 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
modules/javafx.controls/src/main/java/javafx/scene/control/ControlUtils.java line 40:
> 38: import java.util.List;
> 39: import java.util.function.IntPredicate;
> 40: import java.util.stream.Collectors;
and thank you for removing the unused import ... JDK-8289845
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1430#discussion_r1543225967
More information about the openjfx-dev
mailing list