Checking the argument is not necessary and in the way it way it was done it

was not right.

2005-11-30  Robert Schuster  <robertschuster@fsfe.org>

        * java/beans/PropertyChangeSupport.java:
        (fireIndexedPropertyChange): Removed argument checking.

2 files changed, 6 insertions, 3 deletions

diff --git a/ChangeLog b/ChangeLog
index 652778498..718546e76 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2005-11-30  Robert Schuster  <robertschuster@fsfe.org>
+
+	* java/beans/PropertyChangeSupport.java:
+	(fireIndexedPropertyChange): Removed argument checking.
+
 2005-11-30  Jan Roehrich  <jan@roehrich.info>
 
 	* java/awt/datatransfer/SystemFlavorMap.java
diff --git a/java/beans/PropertyChangeSupport.java b/java/beans/PropertyChangeSupport.java
index da30f994d..1affb70eb 100644
--- a/java/beans/PropertyChangeSupport.java
+++ b/java/beans/PropertyChangeSupport.java
@@ -408,11 +408,9 @@ public class PropertyChangeSupport implements Serializable
   public void fireIndexedPropertyChange(String name, int index,
                                         Object oldValue, Object newValue)
   {
-    // FIXME: should we use equals() here?
-    if (oldValue == newValue && oldValue != null)
+      // Argument checking is done in firePropertyChange(PropertyChangeEvent) .
       firePropertyChange(new IndexedPropertyChangeEvent(source, name,
                                                         oldValue, newValue,
-                                                        index));
   }
 
   /**