commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jonathan Baker (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (LANG-1081) DiffBuilder.append(String, Object left, Object right) does not do a left.equals(right) check.
Date Wed, 14 Jan 2015 16:17:35 GMT

     [ 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)

Mime
View raw message