From dace661ba6dc59e3ff1661dc99f344b7dce1ca41 Mon Sep 17 00:00:00 2001
From: Emil Ivov <emcho@jitsi.org>
Date: Thu, 30 Apr 2009 10:51:35 +0000
Subject: [PATCH] Fixes the fix of the previous fix ;)

---
 .../impl/protocol/icq/ContactGroupIcqImpl.java   | 16 +++++-----------
 1 file changed, 5 insertions(+), 11 deletions(-)

diff --git a/src/net/java/sip/communicator/impl/protocol/icq/ContactGroupIcqImpl.java b/src/net/java/sip/communicator/impl/protocol/icq/ContactGroupIcqImpl.java
index f54ae92ac..8006b67ab 100644
--- a/src/net/java/sip/communicator/impl/protocol/icq/ContactGroupIcqImpl.java
+++ b/src/net/java/sip/communicator/impl/protocol/icq/ContactGroupIcqImpl.java
@@ -153,22 +153,16 @@ void addContact(ContactIcqImpl contact)
      */
     void removeContact(ContactIcqImpl contact)
     {
-        for (Iterator<Map.Entry<String, Contact>> buddyIter
-                = buddies.entrySet().iterator();
-             buddyIter.hasNext();)
-        {
-            if (buddyIter.next().getValue().equals(contact))
-            {
-                buddyIter.remove();
-                break;
-            }
-        }
+        if (contact == null )
+            return;
+
+        buddies.remove(contact.getUIN().toLowerCase());
     }
 
     /**
      * Returns an Iterator over all contacts, member of this
      * <tt>ContactGroup</tt>.
-     * 
+     *
      * @return a java.util.Iterator over all contacts inside this
      *         <tt>ContactGroup</tt>. In case the group doesn't contain any
      *         members it will return an empty iterator.