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.
}