svn commit: r13692 - trunk/src_new/org/argouml/notation/providers/java/NotationUtilityJava.java

Author: tfmorris
Date: 2007-10-25 10:47:44-0700
New Revision: 13692

Modified:
   trunk/src_new/org/argouml/notation/providers/java/NotationUtilityJava.java

Log:
Add TODO

Modified: trunk/src_new/org/argouml/notation/providers/java/NotationUtilityJava.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src_new/org/argouml/notation/providers/java/NotationUtilityJava.java?view=diff&rev=13692&p1=trunk/src_new/org/argouml/notation/providers/java/NotationUtilityJava.java&p2=trunk/src_new/org/argouml/notation/providers/java/NotationUtilityJava.java&r1=13691&r2=13692
==============================================================================
--- trunk/src_new/org/argouml/notation/providers/java/NotationUtilityJava.java	(original)
+++ trunk/src_new/org/argouml/notation/providers/java/NotationUtilityJava.java	2007-10-25 10:47:44-0700
@@ -40,7 +40,8 @@
     /**
      * The constructor - nothing to construct.
      */
-    NotationUtilityJava() { }
+    NotationUtilityJava() {
+    }
 
     /**
      * Returns a visibility String either for a VisibilityKind, but also 
@@ -53,6 +54,8 @@
      */
     static String generateVisibility(Object o) {
         if (Model.getFacade().isAFeature(o)) {
+            // TODO: The src_visibility tag doesn't appear to be created
+            // anywhere by ArgoUML currently
             Object tv = Model.getFacade().getTaggedValue(o, "src_visibility");
             if (tv != null) {
                 Object tvValue = Model.getFacade().getValue(tv);
@@ -201,7 +204,7 @@
                 ns = Model.getFacade().getNamespace(ns);
             }
             while (!stack.isEmpty()) {
-                s.append(stack.pop() + ".");
+                s.append(stack.pop()).append(".");
             }
 
             if (s.length() > 0 && !(s.lastIndexOf(".") == s.length() - 1)) {