svn commit: r1140552 - /commons/sandbox/graph/trunk/src/main/java/org/apache/commons/graph/model/BaseMutableGraph.java

[simonetripodi]

Author: simonetripodi
Date: Tue Jun 28 12:15:06 2011
New Revision: 1140552

URL: http://svn.apache.org/viewvc?rev=1140552&view=rev
Log:
no needs to throw an exception for already present edges, automatically 
discarded

Modified:
    
commons/sandbox/graph/trunk/src/main/java/org/apache/commons/graph/model/BaseMutableGraph.java

Modified: 
commons/sandbox/graph/trunk/src/main/java/org/apache/commons/graph/model/BaseMutableGraph.java
URL: 
http://svn.apache.org/viewvc/commons/sandbox/graph/trunk/src/main/java/org/apache/commons/graph/model/BaseMutableGraph.java?rev=1140552&r1=1140551&r2=1140552&view=diff
==============================================================================
--- 
commons/sandbox/graph/trunk/src/main/java/org/apache/commons/graph/model/BaseMutableGraph.java
 (original)
+++ 
commons/sandbox/graph/trunk/src/main/java/org/apache/commons/graph/model/BaseMutableGraph.java
 Tue Jun 28 12:15:06 2011
@@ -112,10 +112,6 @@ public abstract class BaseMutableGraph<V
             throw new GraphException( "Null tail Vertex not admitted" );
         }
 
-        if ( !getAllEdges().add( e ) )
-        {
-            throw new GraphException( "Edge '%s' already present in the 
Graph", e );
-        }
         if ( !getAdjacencyList().containsKey( head ) )
         {
             throw new GraphException( "Head Vertex '%s' not present in the 
Graph", head );
@@ -125,6 +121,8 @@ public abstract class BaseMutableGraph<V
             throw new GraphException( "Tail Vertex '%s' not present in the 
Graph", tail );
         }
 
+        getAllEdges().add( e );
+
         internalAddEdge( head, e, tail );
 
         decorateAddEdge( head, e, tail );