[
https://issues.apache.org/jira/browse/COMMONSSITE-183?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18063391#comment-18063391
]
Gary D. Gregory commented on COMMONSSITE-183:
---------------------------------------------
[~shanjiang]
Wrong project.
> `CPFloat`, `CPDouble`, `CPInt`, `CPLong`, `CPUTF8` — `compareTo` inconsistent
> with `equals`
> -------------------------------------------------------------------------------------------
>
> Key: COMMONSSITE-183
> URL: https://issues.apache.org/jira/browse/COMMONSSITE-183
> Project: Apache Commons All
> Issue Type: Bug
> Reporter: Shan Jiang
> Priority: Major
>
> ### Summary
> Five constant pool classes in `org.apache.commons.compress.harmony.pack200`
> implement
> `Comparable<T>` (via `CPConstant<T>`) with value-based `compareTo()`, but do
> not override
> `equals()`. Two constants with the same value will have `compareTo()` return
> 0 but `equals()`
> return false.
> ### Affected classes
> | Class | compareTo compares by | File |
> |-------|----------------------|------|
> | `CPFloat` | `Float.compare(value, obj.value)` | `.../pack200/CPFloat.java` |
> | `CPDouble` | `Double.compare(value, obj.value)` |
> `.../pack200/CPDouble.java` |
> | `CPInt` | `Integer.compare(value, obj.value)` | `.../pack200/CPInt.java` |
> | `CPLong` | `Long.compare(value, obj.value)` | `.../pack200/CPLong.java` |
> | `CPUTF8` | `utf8.compareTo(obj.utf8)` | `.../pack200/CPUTF8.java` |
> All extend `CPConstant<T>` which implements `Comparable<T>`.
> ### How this was found
> Detected by an automated JDK conformance oracle
> (`ComparableOracles.checkCompareToEqualsConsistency`).
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
