svn commit: r14328 - trunk/src/app/src/org/argouml/uml/diagram/sequence/ui/FigMessagePort.java

Author: bobtarling
Date: 2008-04-12 03:29:52-0700
New Revision: 14328

Modified:
   trunk/src/app/src/org/argouml/uml/diagram/sequence/ui/FigMessagePort.java

Log:
Throw an exception if an error occurs during load.

Modified: trunk/src/app/src/org/argouml/uml/diagram/sequence/ui/FigMessagePort.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/app/src/org/argouml/uml/diagram/sequence/ui/FigMessagePort.java?view=diff&rev=14328&p1=trunk/src/app/src/org/argouml/uml/diagram/sequence/ui/FigMessagePort.java&p2=trunk/src/app/src/org/argouml/uml/diagram/sequence/ui/FigMessagePort.java&r1=14327&r2=14328
==============================================================================
--- trunk/src/app/src/org/argouml/uml/diagram/sequence/ui/FigMessagePort.java	(original)
+++ trunk/src/app/src/org/argouml/uml/diagram/sequence/ui/FigMessagePort.java	2008-04-12 03:29:52-0700
@@ -67,13 +67,14 @@
      * @see org.tigris.gef.presentation.FigGroup#addFig(org.tigris.gef.presentation.Fig)
      */
     public void addFig(Fig toAdd) {
-        if (toAdd instanceof FigLine) {
-            if (getFigs().size() == 0) {
-                toAdd.setVisible(false);
-                super.addFig(toAdd);
-            }
+        if (!(toAdd instanceof FigLine)) {
+            throw new IllegalArgumentException("Unexpect Fig " + toAdd);
+        }
+        if (getFigs().size() == 0) {
+            toAdd.setVisible(false);
+            super.addFig(toAdd);
         } else {
-            LOG.error("Unexpect Fig " + toAdd);
+            // is this an error condition also?
         }
     }