On Tue, 19 Mar 2024 22:23:46 GMT, Nir Lisker <nlis...@openjdk.org> wrote:
>> Andy Goryachev has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> for each
>
> tests/system/src/test/java/test/util/Util.java line 383:
>
>> 381: runAndWait(() -> {
>> 382: for (Window w : new ArrayList<>(Window.getWindows())) {
>> 383: w.hide();
>
> I think you can use `List.copyOf` instead of an `ArrayList` since it doesn't
> get modified (it's a one-time copy).
> The iteration can also be
>
> List.copyOf(Window.getWindows()).forEach(Window::hide);
>
> if you want.
The only issue with this style is - it's hard to set breakpoints.
In this case it's not that important.
Thank you!
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/1407#discussion_r1531200964
