RFR: 7083187: Class CSS.CssValue is missing implementations of equals() and hashCode() [v7]

Alexey Ivanov aivanov at openjdk.org
Mon May 22 10:56:55 UTC 2023


On Mon, 22 May 2023 09:51:12 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:

> I was not sure how to get the other value which is told in the spec where it says percentage needs another value to be passed in. Also checking for percentage equality resulted in regression in one closed test bug4690812.java so percentage equality was not handled.
> 
> https://github.com/openjdk/jdk/blob/8aa50288a1a6a853bf4d8d97b1849d5744ed7a32/src/java.desktop/share/classes/javax/swing/text/html/CSS.java#L2607-L2609

I don't think you need to. In this case, AttributeSets are compared. All the relative units will result in different computed value depending on the element they're applied to.

I'll take a look at the test. I can't see how implementation of `equals` affects that test.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/13405#discussion_r1200345789



More information about the client-libs-dev mailing list