svn commit: r17295 - trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/DnDJGraph.java

Michiel van der Wulp <[email protected]>
Newsgroups gmane.comp.lang.uml.argouml.cvs
Message-ID <[email protected]>
Author: mvw
Date: 2009-08-24 22:55:12-0700
New Revision: 17295

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

Log:
Added TODO comment.

Modified: trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/DnDJGraph.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/DnDJGraph.java?view=diff&pathrev=17295&r1=17294&r2=17295
==============================================================================
--- trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/DnDJGraph.java	(original)
+++ trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/DnDJGraph.java	2009-08-24 22:55:12-0700
@@ -1,5 +1,5 @@
 // $Id$
-// Copyright (c) 2005-2008 The Regents of the University of California. All
+// Copyright (c) 2005-2009 The Regents of the University of California. All
 // Rights Reserved. Permission to use, copy, modify, and distribute this
 // software and its documentation without fee, and without a written
 // agreement is hereby granted, provided that the above copyright notice
@@ -52,7 +52,7 @@
 /**
  * This is a JGraph with Drag and Drop capabilities.
  *
- * @author [email protected]
+ * @author mvw
  */
 class DnDJGraph
     extends JGraph
@@ -195,6 +195,8 @@
             
             Iterator i = modelElements.iterator();
             while (i.hasNext()) {
+                /* TODO: Why not call UMLDiagram.doesAccept() first, 
+                 * like in ClassDiagramRenderer?  */
                 FigNode figNode = ((UMLDiagram ) diagram).drop(i.next(),
                         dropTargetDropEvent.getLocation());

------------------------------------------------------
http://argouml.tigris.org/ds/viewMessage.do?dsForumId=5905&dsMessageId=2386914

To unsubscribe from this discussion, e-mail: [[email protected]].
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.