svn commit: r14817 - trunk/src/argouml-core-model/src/org/argouml/model/ModelEventPump.java

Author: tfmorris
Date: 2008-05-28 12:30:07-0700
New Revision: 14817

Modified:
   trunk/src/argouml-core-model/src/org/argouml/model/ModelEventPump.java

Log:
Issue 5110: Make recursive use of flush illegal.  Synchronize event counter on a separate mutex.

Modified: trunk/src/argouml-core-model/src/org/argouml/model/ModelEventPump.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-core-model/src/org/argouml/model/ModelEventPump.java?view=diff&rev=14817&p1=trunk/src/argouml-core-model/src/org/argouml/model/ModelEventPump.java&p2=trunk/src/argouml-core-model/src/org/argouml/model/ModelEventPump.java&r1=14816&r2=14817
==============================================================================
--- trunk/src/argouml-core-model/src/org/argouml/model/ModelEventPump.java	(original)
+++ trunk/src/argouml-core-model/src/org/argouml/model/ModelEventPump.java	2008-05-28 12:30:07-0700
@@ -224,8 +224,12 @@
     void stopPumpingEvents();
 
     /**
-     * Flush events from the ModelEventPump.
-     * Blocks until all events have been delivered.
+     * Flush events from the ModelEventPump. Blocks until all events have been
+     * delivered.
+     * <p>
+     * <em>NOTE: </em> This may not be called from a 
+     * {@link PropertyChangeListener#propertyChange(java.beans.PropertyChangeEvent)}
+     * method which has been invoked by the ModelEventPump.
      */
     void flushModelEvents();