Uploaded image for project: 'Tamaya'
  1. Tamaya
  2. TAMAYA-172

Fix findbugs warning about wrong comparison via equals in PropertySourceChangeBuilder

    XMLWordPrintableJSON

Details

    Description

      The findbugs warning indicates an error while comparing/merging property values:

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

      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.

      Attachments

        Activity

          People

            anatole Anatole Tresch
            pottlinger Philipp Ottlinger
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: