On Sun, 5 Jul 2026 02:39:04 GMT, Michael Strauß <[email protected]> wrote:
>> modules/javafx.graphics/src/test/java/test/javafx/scene/layout/HeaderBarTest.java
>> line 167:
>>
>>> 165: @Test
>>> 166: void attachedPropertyPublishesExplicitValue() {
>>> 167: var colorScheme = new ColorScheme[1];
>>
>> Minor: `AtomicReference` could be used. I personally like it a bit more
>
> I don't like that one: `AtomicReference` is an atomic reference, not a
> general-purpose mutable reference wrapper.
I also think AtomicReference will be a better choice. An array is probably ok
here because it's a single-threaded test.
-------------
PR Review Comment: https://git.openjdk.org/jfx/pull/2188#discussion_r3531642392
