You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jena.apache.org by an...@apache.org on 2013/08/02 13:25:25 UTC

svn commit: r1509634 - /jena/trunk/jena-arq/src/main/java/org/apache/jena/atlas/lib/Tuple.java

Author: andy
Date: Fri Aug  2 11:25:25 2013
New Revision: 1509634

URL: http://svn.apache.org/r1509634
Log:
Un-'final' class and expose the internal state to subclasses.

Modified:
    jena/trunk/jena-arq/src/main/java/org/apache/jena/atlas/lib/Tuple.java

Modified: jena/trunk/jena-arq/src/main/java/org/apache/jena/atlas/lib/Tuple.java
URL: http://svn.apache.org/viewvc/jena/trunk/jena-arq/src/main/java/org/apache/jena/atlas/lib/Tuple.java?rev=1509634&r1=1509633&r2=1509634&view=diff
==============================================================================
--- jena/trunk/jena-arq/src/main/java/org/apache/jena/atlas/lib/Tuple.java (original)
+++ jena/trunk/jena-arq/src/main/java/org/apache/jena/atlas/lib/Tuple.java Fri Aug  2 11:25:25 2013
@@ -28,13 +28,10 @@ import org.apache.jena.atlas.iterator.It
 import org.apache.jena.atlas.iterator.Transform ;
 
 /** Tuple class - tuples are immutable and must be created initialized */
-public final class Tuple<T> implements Iterable<T>
+public class Tuple<T> implements Iterable<T>
 {
-    // Interface this.
+    // Interface this?
     // Classes: TupleImpl, TupleSlice
-    
-    // Or use an ArrayList<T>
-    
 //    public static <X> Tuple<X> blankTuple(Class<X> clazz, int capacity)
 //    {
 //        @SuppressWarnings("unchecked")
@@ -73,9 +70,9 @@ public final class Tuple<T> implements I
     }
 
     // Alternative : Object[] (+ cast on access)
-    final T[] tuple ;
+    protected final T[] tuple ;
     
-    private Tuple(/*@SuppressWarnings("unchecked")*/ T...tuple)
+    protected Tuple(/*@SuppressWarnings("unchecked")*/ T...tuple)
     {
         this.tuple = ArrayUtils.copy(tuple) ;   // Take copy to stop 'tuple' being referenced after the call.
     }