You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jena.apache.org by cl...@apache.org on 2014/01/01 16:29:27 UTC

svn commit: r1554630 - /jena/trunk/jena-core/src/main/java/com/hp/hpl/jena/graph/impl/CollectionGraph.java

Author: claude
Date: Wed Jan  1 15:29:26 2014
New Revision: 1554630

URL: http://svn.apache.org/r1554630
Log:
Added code for efficience when a Set is used as the collection

Modified:
    jena/trunk/jena-core/src/main/java/com/hp/hpl/jena/graph/impl/CollectionGraph.java

Modified: jena/trunk/jena-core/src/main/java/com/hp/hpl/jena/graph/impl/CollectionGraph.java
URL: http://svn.apache.org/viewvc/jena/trunk/jena-core/src/main/java/com/hp/hpl/jena/graph/impl/CollectionGraph.java?rev=1554630&r1=1554629&r2=1554630&view=diff
==============================================================================
--- jena/trunk/jena-core/src/main/java/com/hp/hpl/jena/graph/impl/CollectionGraph.java (original)
+++ jena/trunk/jena-core/src/main/java/com/hp/hpl/jena/graph/impl/CollectionGraph.java Wed Jan  1 15:29:26 2014
@@ -28,6 +28,7 @@ import com.hp.hpl.jena.util.iterator.Wra
 
 import java.util.Collection;
 import java.util.HashSet;
+import java.util.Set;
 
 /**
  * A simple graph implementation that wraps a collection of triples.
@@ -138,8 +139,9 @@ public class CollectionGraph extends Gra
 	}
 
 	// the collection
-	private Collection<Triple> triples;
-
+	private final Collection<Triple> triples;
+	private final boolean uniqueOnly;
+	
 	/**
 	 * Construct an empty graph using an empty HashSet.
 	 */
@@ -158,6 +160,7 @@ public class CollectionGraph extends Gra
 	{
 		super();
 		this.triples = triples;
+		this.uniqueOnly = triples instanceof Set;
 	}
 
 	@Override
@@ -170,7 +173,7 @@ public class CollectionGraph extends Gra
 	@Override
 	public void performAdd( final Triple t )
 	{
-		if (!triples.contains(t))
+		if (uniqueOnly || !triples.contains(t))
 		{
 			triples.add(t);
 		}