svn commit: r13568 - trunk/src_new/org/argouml/kernel/MemberList.java

Author: bobtarling
Date: 2007-09-17 18:05:48-0700
New Revision: 13568

Modified:
   trunk/src_new/org/argouml/kernel/MemberList.java

Log:
Log if we try and remove a diagram that is already removed (I'm having thread problems during load that might be related to this)

Modified: trunk/src_new/org/argouml/kernel/MemberList.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src_new/org/argouml/kernel/MemberList.java?view=diff&rev=13568&p1=trunk/src_new/org/argouml/kernel/MemberList.java&p2=trunk/src_new/org/argouml/kernel/MemberList.java&r1=13567&r2=13568
==============================================================================
--- trunk/src_new/org/argouml/kernel/MemberList.java	(original)
+++ trunk/src_new/org/argouml/kernel/MemberList.java	2007-09-17 18:05:48-0700
@@ -101,7 +101,11 @@
             setTodoList(null);
             return true;
         } else {
-            return diagramMembers.remove(member);
+            final boolean removed = diagramMembers.remove(member);
+            if (!removed) {
+                LOG.warn("Failed to remove diagram member " + member);
+            }
+            return removed;
         }
     }
 
@@ -152,6 +156,7 @@
                 return true;
             }
         }
+        LOG.warn("Failed to remove diagram " + d);
         return false;
     }