[ 
https://issues.apache.org/jira/browse/TAMAYA-172?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Anatole Tresch reassigned TAMAYA-172:
-------------------------------------

    Assignee: Anatole Tresch

> Fix findbugs warning about wrong comparison via equals in 
> PropertySourceChangeBuilder
> -------------------------------------------------------------------------------------
>
>                 Key: TAMAYA-172
>                 URL: https://issues.apache.org/jira/browse/TAMAYA-172
>             Project: Tamaya
>          Issue Type: Bug
>          Components: Modules
>    Affects Versions: 0.2-incubating
>         Environment: 
> https://builds.apache.org/job/Tamaya-Master-branch/708/findbugsResult/HIGH/
>            Reporter: Philipp Ottlinger
>            Assignee: Anatole Tresch
>             Fix For: 0.3-incubating
>
>
> The findbugs warning indicates an error while comparing/merging property 
> values:
> {code}
> PropertySourceChangeBuilder.java:102, EC_UNRELATED_TYPES, Priority: High
> Call to org.apache.tamaya.spi.PropertyValue.equals(String) in 
> org.apache.tamaya.events.PropertySourceChangeBuilder.compare(PropertySource, 
> PropertySource)
> This method calls equals(Object) on two references of different class types 
> and analysis suggests they will be to objects of different classes at 
> runtime. Further, examination of the equals methods that would be invoked 
> suggest that either this call will always return false, or else the equals 
> method is not be symmetric (which is a property required by the contract for 
> equals in class Object). 
> {code}
> It's not really clear whether values or entry keys need to be compared here, 
> but its definitely wrong to compare String to PropertyValue at line 102 and 
> 110.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to