svn commit: r13229 - trunk/src_new/org/argouml/uml/reveng/ui/RESequenceDiagramDialog.java

Author: tfmorris
Date: 2007-08-03 13:05:46-0700
New Revision: 13229

Modified:
   trunk/src_new/org/argouml/uml/reveng/ui/RESequenceDiagramDialog.java

Log:
Add TODOs.  No functional change.

Modified: trunk/src_new/org/argouml/uml/reveng/ui/RESequenceDiagramDialog.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src_new/org/argouml/uml/reveng/ui/RESequenceDiagramDialog.java?view=diff&rev=13229&p1=trunk/src_new/org/argouml/uml/reveng/ui/RESequenceDiagramDialog.java&p2=trunk/src_new/org/argouml/uml/reveng/ui/RESequenceDiagramDialog.java&r1=13228&r2=13229
==============================================================================
--- trunk/src_new/org/argouml/uml/reveng/ui/RESequenceDiagramDialog.java	(original)
+++ trunk/src_new/org/argouml/uml/reveng/ui/RESequenceDiagramDialog.java	2007-08-03 13:05:46-0700
@@ -804,6 +804,7 @@
      */
     private Object permissionLookup(Object comp, String clsName) {
         Object theClassifier = null;
+        // TODO: This could use the new CoreHelper.getPackageImports()
         Collection cdeps = Model.getFacade().getClientDependencies(comp);
         Iterator iter1 = cdeps != null ? cdeps.iterator() : null;
         while (theClassifier == null && iter1 != null && iter1.hasNext()) {
@@ -816,6 +817,11 @@
                         && iter2 != null
                         && iter2.hasNext()) {
                     Object elem = iter2.next();
+                    // TODO: I'm not sure what this is trying to do, but it
+                    // probably isn't what it thinks it is.  The supplier to
+                    // an import is going to be a Package, which is not a
+                    // Classifier.  Perhaps this intends to process the 
+                    // ownedElements of the Package. - tfm - 20070803
                     if (Model.getFacade().isAClassifier(elem)
                          && clsName.equals(Model.getFacade().getName(elem))) {
                         theClassifier = elem;