[
https://issues.apache.org/jira/browse/LANG-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Jonathan Baker updated LANG-1081:
---------------------------------
Attachment: DiffBuilder-equals_test_in_append_Objects.patch
DiffBuilderAppendsNonEqualObjectsFixed.java
DiffBuilderAppendsNonEqualObjects.java
Attached are examples of how to detect and correct the issue.
> DiffBuilder.append(String, Object left, Object right) does not do a
> left.equals(right) check.
> ---------------------------------------------------------------------------------------------
>
> Key: LANG-1081
> URL: https://issues.apache.org/jira/browse/LANG-1081
> Project: Commons Lang
> Issue Type: Bug
> Components: lang.builder.*
> Affects Versions: 3.3.2
> Reporter: Jonathan Baker
> Priority: Trivial
> Attachments: DiffBuilder-equals_test_in_append_Objects.patch,
> DiffBuilderAppendsNonEqualObjects.java,
> DiffBuilderAppendsNonEqualObjectsFixed.java
>
> Original Estimate: 1m
> Remaining Estimate: 1m
>
> Only testing == without testing equals() seems wrong.
> Should an equals() test be added after determining that the parameters are
> not arrays?
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
