sis-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From desruisse...@apache.org
Subject svn commit: r1593151 - in /sis/branches/JDK8/core: sis-feature/src/main/java/org/apache/sis/feature/PropertyList.java sis-utility/src/main/java/org/apache/sis/internal/util/CheckedArrayList.java
Date Wed, 07 May 2014 22:37:26 GMT
Author: desruisseaux
Date: Wed May  7 22:37:25 2014
New Revision: 1593151

URL: http://svn.apache.org/r1593151
Log:
Removal of the PropertyList class, since the approach allowing a [0 ... n] cardinality for
Attribute instance has been abandoned
(see comment in commit 1593149 for more details). The case of PropertySingleton still needs
to be re-evaluated.

Removed:
    sis/branches/JDK8/core/sis-feature/src/main/java/org/apache/sis/feature/PropertyList.java
Modified:
    sis/branches/JDK8/core/sis-utility/src/main/java/org/apache/sis/internal/util/CheckedArrayList.java

Modified: sis/branches/JDK8/core/sis-utility/src/main/java/org/apache/sis/internal/util/CheckedArrayList.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-utility/src/main/java/org/apache/sis/internal/util/CheckedArrayList.java?rev=1593151&r1=1593150&r2=1593151&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-utility/src/main/java/org/apache/sis/internal/util/CheckedArrayList.java
[UTF-8] (original)
+++ sis/branches/JDK8/core/sis-utility/src/main/java/org/apache/sis/internal/util/CheckedArrayList.java
[UTF-8] Wed May  7 22:37:25 2014
@@ -194,17 +194,17 @@ public final class CheckedArrayList<E> e
     }
 
     /**
-     * A wrapper around the given array for use by {@link CheckedArrayList#addAll(Collection)}
only, or by other
-     * implementations using the same strategy. This wrapper violates some {@link List} method
contracts,  so it
-     * shall really be used only as a temporary object for passing array to {@code ArrayList.addAll(…)}
methods.
+     * A wrapper around the given array for use by {@link CheckedArrayList#addAll(Collection)}
only.
+     * This wrapper violates some {@link List} method contracts, so it shall really be used
only as
+     * a temporary object for passing array to {@code ArrayList.addAll(…)} methods.
      * In particular {@link #toArray()} returns directly the internal array, because this
is the method to be
      * invoked by {@code ArrayList.addAll(…)} (this is actually the only important method
in this wrapper).
      *
      * @param <E> The type or list elements.
      */
-    public static final class Mediator<E> extends AbstractList<E> {
+    private static final class Mediator<E> extends AbstractList<E> {
         private final E[] array;
-        public Mediator(final E[] array)    {this.array = array;}
+        Mediator(final E[] array)           {this.array = array;}
         @Override public int size()         {return array.length;}
         @Override public E   get(int index) {return array[index];}
         @Override public E[] toArray()      {return array;} // See class javadoc.



Mime
View raw message