svn commit: r16209 - trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/FigEdgeModelElement.java

Author: mvw
Date: 2008-11-30 23:32:10-0800
New Revision: 16209

Modified:
   trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/FigEdgeModelElement.java

Log:
Added TODO.

Modified: trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/FigEdgeModelElement.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/FigEdgeModelElement.java?view=diff&rev=16209&p1=trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/FigEdgeModelElement.java&p2=trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/FigEdgeModelElement.java&r1=16208&r2=16209
==============================================================================
--- trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/FigEdgeModelElement.java	(original)
+++ trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/FigEdgeModelElement.java	2008-11-30 23:32:10-0800
@@ -951,7 +951,13 @@
      * @deprecated for 0.27.3 by tfmorris.  Separate initialization of 
      * notation providers from any later updates which are required. 
      * Initialization must be done in the constructor using methods which 
-     * can't be overriden.
+     * can't be overriden. <p>
+     * TODO: MVW: NotationProviders can not be updated - they
+     * are lightweight throw-away objects. Hence there is functionally
+     * only this method to create a (new) NotationProvider whenever
+     * any setting changes. E.g. when the notation language is 
+     * changed by the user, then the NPs are to be re-created.
+     * So, why split this in 2 methods?
      */
     protected void initNotationProviders(Object own) {
         if (notationProviderName != null) {