On Sun, 6 Sep 2026 20:17:34 GMT, Marius Hanl <[email protected]> wrote:

>> Michael Strauß has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   review comments
>
> modules/javafx.base/src/test/java/test/util/property/PropertyMetadataVerifier.java
>  line 300:
> 
>> 298:         } while (propertyClass != null);
>> 299: 
>> 300:         throw new AssertionError();
> 
> Minor: Should we add a message here?

I wouldn't overthink this, since this code will never be reached. It's not 
something you can do wrong when you use this testing tool.

-------------

PR Review Comment: https://git.openjdk.org/jfx/pull/2015#discussion_r3945256313

Reply via email to