RFR: 8353845: com.sun.javafx.css.BitSet.equals(null) throws NPE [v2]

Michael Strauß mstrauss at openjdk.org
Wed Apr 9 09:07:40 UTC 2025


On Wed, 9 Apr 2025 08:59:37 GMT, Nir Lisker <nlisker at openjdk.org> wrote:

>> Michael Strauß has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   obj == null
>
> modules/javafx.graphics/src/test/java/test/com/sun/javafx/css/BitSetTest.java line 50:
> 
>> 48:     @Test
>> 49:     void equalsNullShouldBeFalse() {
>> 50:         assertFalse(set.equals(null));
> 
> Isn't `assertNotEquals` more suitable here?

Since this method specifically tests the `equals` method, I like to actually see the method in question.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1766#discussion_r2034847560


More information about the openjfx-dev mailing list