From: Fernando Nasser <fnasser@redhat.com>
Date: Fri, 19 Dec 2003 02:53:36 +0000 (+0000)
Subject: List.java (replaceItem): Prevent selection to move with replace and minimize flickering.
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=d416de057b728ba3bcde18362eb59be27305af7a;p=gcc.git

List.java (replaceItem): Prevent selection to move with replace and minimize flickering.

2003-12-18  Fernando Nasser  <fnasser@redhat.com>

        * java/awt/List.java (replaceItem): Prevent selection to move with
        replace and minimize flickering.

From-SVN: r74814
---

diff --git a/libjava/ChangeLog b/libjava/ChangeLog
index ecb767c01c6..231cf35a752 100644
--- a/libjava/ChangeLog
+++ b/libjava/ChangeLog
@@ -1,3 +1,8 @@
+2003-12-18  Fernando Nasser  <fnasser@redhat.com>
+
+	* java/awt/List.java (replaceItem): Prevent selection to move with
+	replace and minimize flickering.
+ 
 2003-12-18  Michael Koch  <konqueror@gmx.de>
 
 	* libltdl/ltdl.c: Define __private_extern__ if needed.
diff --git a/libjava/java/awt/List.java b/libjava/java/awt/List.java
index 23ca34fab0c..79b2faa6299 100644
--- a/libjava/java/awt/List.java
+++ b/libjava/java/awt/List.java
@@ -647,8 +647,21 @@ clear()
 public synchronized void
 replaceItem(String item, int index) throws IllegalArgumentException
 {
-  remove(index);
-  addItem(item, index);
+  if ((index < 0) || (index >= items.size()))
+    throw new IllegalArgumentException("Bad list index: " + index);
+
+  items.insertElementAt(item, index + 1);
+  items.removeElementAt (index);
+
+  if (peer != null)
+    {
+      ListPeer l = (ListPeer) peer;
+
+      /* We add first and then remove so that the selected
+	 item remains the same */
+      l.add (item, index + 1);
+      l.delItems (index, index);
+    }
 }
 
 /*************************************************************************/