sis-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From desruisse...@apache.org
Subject svn commit: r1779559 [15/21] - in /sis/trunk: ./ core/sis-feature/src/main/java/org/apache/sis/feature/ core/sis-metadata/src/main/java/org/apache/sis/internal/jaxb/code/ core/sis-metadata/src/main/java/org/apache/sis/internal/jaxb/geometry/ core/sis-m...
Date Fri, 20 Jan 2017 04:34:30 GMT
Modified: sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/system/OptionalDependency.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/system/OptionalDependency.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/system/OptionalDependency.java [UTF-8] (original)
+++ sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/system/OptionalDependency.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -42,8 +42,8 @@ public abstract class OptionalDependency
     /**
      * Creates a new optional dependency.
      *
-     * @param module A constant from the {@link Modules} class which identify the module that need the optional dependency.
-     * @param dependency The Maven artifact name (<strong>not</strong> a name from the {@link Modules} class)
+     * @param module  a constant from the {@link Modules} class which identify the module that need the optional dependency.
+     * @param dependency  the Maven artifact name (<strong>not</strong> a name from the {@link Modules} class)
      *        of the optional module on which the {@code module} depend.
      */
     protected OptionalDependency(final String module, final String dependency) {
@@ -75,12 +75,12 @@ public abstract class OptionalDependency
      * Returns the optional dependency, or {@code null} if not found.
      * This is a helper method for implementation of {@code getInstance()} static method in subclasses.
      *
-     * @param <T>            Compile-time type of the {@code type} argument.
-     * @param type           The subclass type.
-     * @param module         Same argument value than the one given to the {@linkplain #OptionalDependency constructor}.
-     * @param dependency     Same argument value than the one given to the {@linkplain #OptionalDependency constructor}.
-     * @param implementation The fully-qualified name of the class to instantiate by reflection.
-     * @return An instance of the {@code implementation} class, or {@code null} if not found.
+     * @param  <T>             compile-time type of the {@code type} argument.
+     * @param  type            the subclass type.
+     * @param  module          same argument value than the one given to the {@linkplain #OptionalDependency constructor}.
+     * @param  dependency      same argument value than the one given to the {@linkplain #OptionalDependency constructor}.
+     * @param  implementation  the fully-qualified name of the class to instantiate by reflection.
+     * @return an instance of the {@code implementation} class, or {@code null} if not found.
      */
     protected static <T extends OptionalDependency> T getInstance(final Class<T> type,
             final String module, final String dependency, final String implementation)
@@ -103,7 +103,7 @@ public abstract class OptionalDependency
      * Returns the exception to throw when a method requiring the optional dependency is invoked
      * but that module is not on the classpath.
      *
-     * @return The exception to throw.
+     * @return the exception to throw.
      */
     protected final UnsupportedOperationException moduleNotFound() {
         return new UnsupportedOperationException(Errors.format(Errors.Keys.MissingRequiredModule_1, dependency));

Modified: sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/system/ReferenceQueueConsumer.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/system/ReferenceQueueConsumer.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/system/ReferenceQueueConsumer.java [UTF-8] (original)
+++ sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/system/ReferenceQueueConsumer.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -65,8 +65,10 @@ public final class ReferenceQueueConsume
         synchronized (Threads.class) {
             final ReferenceQueueConsumer thread;
             Threads.lastCreatedDaemon = thread = new ReferenceQueueConsumer(Threads.lastCreatedDaemon);
-            // Call to Thread.start() must be outside the constructor
-            // (Reference: Goetz et al.: "Java Concurrency in Practice").
+            /*
+             * Call to Thread.start() must be outside the constructor
+             * (Reference: Goetz et al.: "Java Concurrency in Practice").
+             */
             thread.start();
         }
         if (Supervisor.ENABLED) {
@@ -120,8 +122,10 @@ public final class ReferenceQueueConsume
                     continue;
                 }
             } catch (InterruptedException exception) {
-                // Probably the 'killAll' method has been invoked.
-                // We need to test 'isKillRequested()' below.
+                /*
+                 * Probably the 'killAll' method has been invoked.
+                 * We need to test 'isKillRequested()' below.
+                 */
             } catch (Throwable exception) {
                 Logging.unexpectedException(Logging.getLogger(Loggers.SYSTEM), getClass(), "run", exception);
             }

Modified: sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/system/Semaphores.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/system/Semaphores.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/system/Semaphores.java [UTF-8] (original)
+++ sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/system/Semaphores.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -112,7 +112,7 @@ public final class Semaphores {
     /**
      * Clears the given flag.
      *
-     * @param flag  one of {@link #CONVERSION_AND_CRS}, {@link #ENCLOSED_IN_OPERATION} or other constants.
+     * @param  flag  one of {@link #CONVERSION_AND_CRS}, {@link #ENCLOSED_IN_OPERATION} or other constants.
      */
     public static void clear(final byte flag) {
         final Semaphores s = FLAGS.get();

Modified: sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/system/ServletListener.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/system/ServletListener.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/system/ServletListener.java [UTF-8] (original)
+++ sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/system/ServletListener.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -37,7 +37,7 @@ public final class ServletListener imple
     /**
      * Receives notification that the web application initialization process is starting.
      *
-     * @param event The context of the servlet being initialized.
+     * @param  event  the context of the servlet being initialized.
      */
     @Override
     public void contextInitialized(final ServletContextEvent event) {
@@ -48,7 +48,7 @@ public final class ServletListener imple
     /**
      * Receives notification that the application is about to be shutdown.
      *
-     * @param event The context of the servlet being shutdown.
+     * @param  event  the context of the servlet being shutdown.
      */
     @Override
     public void contextDestroyed(final ServletContextEvent event) {

Modified: sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/system/Shutdown.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/system/Shutdown.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/system/Shutdown.java [UTF-8] (original)
+++ sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/system/Shutdown.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -18,12 +18,10 @@ package org.apache.sis.internal.system;
 
 import java.util.List;
 import java.util.ArrayList;
+import java.util.Objects;
 import java.util.concurrent.Callable;
 import org.apache.sis.util.logging.Logging;
 
-// Branch-dependent imports
-import java.util.Objects;
-
 
 /**
  * A central place where to manage SIS shutdown process.
@@ -78,7 +76,7 @@ public final class Shutdown extends Thre
     /**
      * Returns the value set by the last call to {@link #setContainer(String)}.
      *
-     * @return Typically {@code "OSGi"}, {@code "Servlet"} or {@code null}.
+     * @return typically {@code "OSGi"}, {@code "Servlet"} or {@code null}.
      */
     public static String getContainer() {
         synchronized (resources) {
@@ -91,8 +89,8 @@ public final class Shutdown extends Thre
      * Example of such environments are OSG and servlet containers. In such case, the shutdown hook will not
      * be registered to the JVM {@link Runtime}.
      *
-     * @param env A description of the container. Should contain version information if possible.
-     *            Example: {@code "OSGi"} or {@code "JavaServer Web Dev Kit/1.0"}.
+     * @param  env  a description of the container. Should contain version information if possible.
+     *              Example: {@code "OSGi"} or {@code "JavaServer Web Dev Kit/1.0"}.
      */
     public static void setContainer(final String env) {
         Objects.requireNonNull(env);
@@ -108,7 +106,7 @@ public final class Shutdown extends Thre
      *
      * <p>The same resource shall not be added twice.</p>
      *
-     * @param resource The resource disposal to register for execution at shutdown time.
+     * @param  resource  the resource disposal to register for execution at shutdown time.
      */
     public static void register(final Callable<?> resource) {
         synchronized (resources) {
@@ -136,7 +134,7 @@ public final class Shutdown extends Thre
      * Unregisters a code from execution at JVM shutdown time.
      * This method uses identity comparison (it does not use {@link Object#equals(Object)}).
      *
-     * @param resource The resource disposal to cancel execution.
+     * @param  resource  the resource disposal to cancel execution.
      */
     public static void unregister(final Callable<?> resource) {
         synchronized (resources) {
@@ -152,7 +150,7 @@ public final class Shutdown extends Thre
     /**
      * Unregisters the supervisor MBean, executes the disposal tasks and shutdowns the {@code sis-utility} threads.
      *
-     * @param  caller The class invoking this method, to be used only for logging purpose, or {@code null}
+     * @param  caller  the class invoking this method, to be used only for logging purpose, or {@code null}
      *         if the logging system is not available anymore (i.e. the JVM itself is shutting down).
      * @throws Exception if an error occurred during unregistration of the supervisor MBean
      *         or during a resource disposal.

Modified: sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/system/Supervisor.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/system/Supervisor.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/system/Supervisor.java [UTF-8] (original)
+++ sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/system/Supervisor.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -111,7 +111,7 @@ public final class Supervisor extends St
      * has not been previously successfully {@linkplain #register() registered}, or if it has
      * already been unregistered.
      *
-     * @throws JMException If an error occurred during unregistration.
+     * @throws JMException if an error occurred during unregistration.
      */
     static synchronized void unregister() throws JMException {
         final ObjectName n = name;
@@ -126,7 +126,7 @@ public final class Supervisor extends St
     /**
      * Creates a new {@code Supervisor}.
      *
-     * @throws NotCompliantMBeanException Should never happen.
+     * @throws NotCompliantMBeanException should never happen.
      */
     public Supervisor() throws NotCompliantMBeanException {
         super(SupervisorMBean.class);
@@ -145,7 +145,7 @@ public final class Supervisor extends St
     /**
      * Returns the localized description for this MBean.
      *
-     * @return A localized description.
+     * @return a localized description.
      */
     @Override
     protected String getDescription(final MBeanInfo info) {
@@ -155,7 +155,7 @@ public final class Supervisor extends St
     /**
      * Returns the localized description for the given constructor, attribute or operation.
      *
-     * @return A localized description for the given attribute or operation.
+     * @return a localized description for the given attribute or operation.
      */
     @Override
     protected String getDescription(final MBeanFeatureInfo info) {
@@ -165,10 +165,10 @@ public final class Supervisor extends St
     /**
      * Returns the localized description for the given constructor parameter.
      *
-     * @param info     The constructor.
-     * @param param    The constructor parameter.
-     * @param sequence The parameter number (0 for the first parameter, 1 for the second, etc.)
-     * @return A localized description for the specified constructor parameter.
+     * @param  info      the constructor.
+     * @param  param     the constructor parameter.
+     * @param  sequence  the parameter number (0 for the first parameter, 1 for the second, etc.)
+     * @return a localized description for the specified constructor parameter.
      */
     @Override
     protected String getDescription(MBeanConstructorInfo info, MBeanParameterInfo param, int sequence) {
@@ -178,10 +178,10 @@ public final class Supervisor extends St
     /**
      * Returns the name of the given constructor parameter.
      *
-     * @param info     The constructor.
-     * @param param    The constructor parameter.
-     * @param sequence The parameter number (0 for the first parameter, 1 for the second, etc.)
-     * @return The name of the specified constructor parameter.
+     * @param  info      the constructor.
+     * @param  param     the constructor parameter.
+     * @param  sequence  the parameter number (0 for the first parameter, 1 for the second, etc.)
+     * @return the name of the specified constructor parameter.
      */
     @Override
     protected String getParameterName(MBeanConstructorInfo info, MBeanParameterInfo param, int sequence) {

Modified: sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/system/SupervisorMBean.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/system/SupervisorMBean.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/system/SupervisorMBean.java [UTF-8] (original)
+++ sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/system/SupervisorMBean.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -39,10 +39,10 @@ public interface SupervisorMBean {
      * Apache SIS version, Java version and operation system version; a "Classpath"
      * section containing bootstrap, extension and user classpath, <i>etc</i>.
      *
-     * @param  sections The section for which information are desired.
-     * @param  locale   The locale to use for formatting the texts in the tree, or {@code null} for the default.
-     * @param  timezone The timezone to use for formatting the dates, or {@code null} for the default.
-     * @return Configuration information, as a tree for grouping some configuration by sections.
+     * @param  sections  the section for which information are desired.
+     * @param  locale    the locale to use for formatting the texts in the tree, or {@code null} for the default.
+     * @param  timezone  the timezone to use for formatting the dates, or {@code null} for the default.
+     * @return configuration information, as a tree for grouping some configuration by sections.
      */
     TreeTable configuration(EnumSet<About> sections, Locale locale, TimeZone timezone);
 
@@ -50,8 +50,8 @@ public interface SupervisorMBean {
      * If there is something wrong with the current Apache SIS status,
      * returns descriptions of the problems. Otherwise returns {@code null}.
      *
-     * @param  locale The locale to use for reporting messages, or {@code null} for the default.
-     * @return A description of a problems in the library, or {@code null} if none.
+     * @param  locale  the locale to use for reporting messages, or {@code null} for the default.
+     * @return a description of a problems in the library, or {@code null} if none.
      */
     String[] warnings(Locale locale);
 }

Modified: sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/system/SystemListener.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/system/SystemListener.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/system/SystemListener.java [UTF-8] (original)
+++ sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/system/SystemListener.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -47,7 +47,7 @@ public abstract class SystemListener imp
     /**
      * Creates a new listener.
      *
-     * @param module The OSGi module symbolic name, as declared in the {@code Bundle-SymbolicName}
+     * @param module  the OSGi module symbolic name, as declared in the {@code Bundle-SymbolicName}
      *        entry of the {@code META-INF/MANIFEST.MF} file. Shall be one of {@link Modules} constants.
      */
     protected SystemListener(final String module) {
@@ -59,7 +59,7 @@ public abstract class SystemListener imp
      * This method doesn't check if the given listener is already present in the array,
      * unless assertions are enabled.
      *
-     * @param listener The listener to add. Can not be {@code null}.
+     * @param  listener  the listener to add. Can not be {@code null}.
      */
     public static synchronized void add(final SystemListener listener) {
         assert (listener != null) && !ArraysExt.contains(listeners, listener);
@@ -78,7 +78,7 @@ public abstract class SystemListener imp
      * Only one occurrence should exist, but this method check all of them as
      * a paranoiac check.`
      *
-     * @param listener The listener to remove.
+     * @param  listener  the listener to remove.
      */
     public static synchronized void remove(final SystemListener listener) {
         SystemListener[] list = listeners;
@@ -96,8 +96,8 @@ public abstract class SystemListener imp
      * Removes all listeners for the module of the given name.
      * This method is invoked by {@link OSGiActivator}.
      *
-     * @param name The name of the module for which to remove listeners, or {@code null} if the module
-     *        to uninstall does not have symbolic name (in which case it is probably not a SIS module).
+     * @param  name  the name of the module for which to remove listeners, or {@code null} if the module
+     *         to uninstall does not have symbolic name (in which case it is probably not a SIS module).
      */
     static synchronized void removeModule(final String name) {
         if (name != null) {

Modified: sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/AbstractMapEntry.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/AbstractMapEntry.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/AbstractMapEntry.java [UTF-8] (original)
+++ sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/AbstractMapEntry.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -17,20 +17,18 @@
 package org.apache.sis.internal.util;
 
 import java.util.Map;
+import java.util.Objects;
 import org.apache.sis.util.Debug;
 import org.apache.sis.util.CharSequences;
 import org.apache.sis.util.resources.Errors;
 
-// Branch-dependent imports
-import java.util.Objects;
-
 
 /**
  * Provides default implementations of {@link #equals(Object)}, {@link #hashCode()} and {@link #toString()}
  * for a map entry.
  *
- * @param <K> The type of keys maintained by the map.
- * @param <V> The type of mapped values.
+ * @param  <K>  the type of keys maintained by the map.
+ * @param  <V>  the type of mapped values.
  *
  * @author  Martin Desruisseaux (Geomatys)
  * @since   0.5
@@ -49,8 +47,8 @@ public abstract class AbstractMapEntry<K
      * The default implementation throws {@code UnsupportedOperationException}
      * for the convenience of unmodifiable map implementations.
      *
-     * @param  value The new value to be stored in this entry.
-     * @return The previous value (may be {@code null}).
+     * @param  value  the new value to be stored in this entry.
+     * @return the previous value (may be {@code null}).
      * @throws UnsupportedOperationException if this entry is unmodifiable.
      */
     @Override
@@ -62,7 +60,7 @@ public abstract class AbstractMapEntry<K
      * Compares the specified object with this entry for equality.
      * Criterion are specified by the {@link Map.Entry} contract.
      *
-     * @param object The other object to compare with this entry.
+     * @param  object  the other object to compare with this entry.
      */
     @Override
     public boolean equals(final Object object) {
@@ -97,8 +95,8 @@ public abstract class AbstractMapEntry<K
     /**
      * Returns only the first line of the string representation of the given value.
      *
-     * @param  value The value for which to get a string representation.
-     * @return The first line of the string representation of the given value.
+     * @param  value  the value for which to get a string representation.
+     * @return the first line of the string representation of the given value.
      */
     static String firstLine(final Object value) {
         String s = String.valueOf(value);

Modified: sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/CheckedArrayList.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/CheckedArrayList.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/CheckedArrayList.java [UTF-8] (original)
+++ sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/CheckedArrayList.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -46,7 +46,7 @@ import static org.apache.sis.util.Argume
  * The checks are performed only on a <cite>best effort</cite> basis. In current implementation,
  * holes are known to exist in use cases like {@code sublist(…).set(…)} or when using the list iterator.
  *
- * @param <E> The type of elements in the list.
+ * @param  <E>  the type of elements in the list.
  *
  * @author  Martin Desruisseaux (Geomatys)
  * @since   0.3

Modified: sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/CheckedHashSet.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/CheckedHashSet.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/CheckedHashSet.java [UTF-8] (original)
+++ sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/CheckedHashSet.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -37,7 +37,7 @@ import static org.apache.sis.util.Argume
  *   <li>Does not accept null elements.</li>
  * </ul>
  *
- * @param <E> The type of elements in the set.
+ * @param  <E>  the type of elements in the set.
  *
  * @author  Martin Desruisseaux (Geomatys)
  * @since   0.3
@@ -60,7 +60,7 @@ public final class CheckedHashSet<E> ext
     /**
      * Constructs a set of the specified type.
      *
-     * @param type The element type (can not be null).
+     * @param type  the element type (can not be null).
      */
     public CheckedHashSet(final Class<E> type) {
         super();
@@ -71,8 +71,8 @@ public final class CheckedHashSet<E> ext
     /**
      * Constructs a set of the specified type and initial capacity.
      *
-     * @param type The element type (should not be null).
-     * @param capacity The initial capacity.
+     * @param type      the element type (should not be null).
+     * @param capacity  the initial capacity.
      */
     public CheckedHashSet(final Class<E> type, final int capacity) {
         super(capacity);
@@ -91,7 +91,7 @@ public final class CheckedHashSet<E> ext
     /**
      * Adds the specified element to this set if it is not already present.
      *
-     * @param  element element to be added to this set.
+     * @param  element  element to be added to this set.
      * @return {@code true} if the set did not already contain the specified element.
      * @throws IllegalArgumentException if the specified element is not of the expected type.
      */

Modified: sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/CollectionsExt.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/CollectionsExt.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/CollectionsExt.java [UTF-8] (original)
+++ sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/CollectionsExt.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -22,6 +22,7 @@ import org.opengis.util.CodeList;
 import org.apache.sis.util.Static;
 import org.apache.sis.util.Numbers;
 import org.apache.sis.util.collection.CodeListSet;
+import org.apache.sis.util.collection.CheckedContainer;
 import org.apache.sis.util.resources.Errors;
 import org.opengis.parameter.InvalidParameterCardinalityException;
 
@@ -414,6 +415,65 @@ public final class CollectionsExt extend
     }
 
     /**
+     * Returns a unmodifiable version of the given collection.
+     * If the given collection is a {@link Set} or a {@link List}, then this method tries to
+     * return a collection of the same type. Other types are not guaranteed to be preserved.
+     *
+     * <p><em>The collection returned by this method may or may not be a view of the given collection</em>.
+     * Consequently this method shall be used <strong>only</strong> if the given collection will
+     * <strong>not</strong> be modified after this method call. In case of doubt, use the
+     * standard {@link Collections#unmodifiableCollection(Collection)} method instead.</p>
+     *
+     * @param  <E>         the type of elements in the collection.
+     * @param  collection  the collection to make unmodifiable, or {@code null}.
+     * @return a unmodifiable version of the given collection, or {@code null} if the given collection was null.
+     *
+     * @since 0.8
+     */
+    public static <E> Collection<E> unmodifiableOrCopy(Collection<E> collection) {
+        if (collection != null) {
+            if (collection instanceof Set<?>) {
+                return unmodifiableOrCopy((Set<E>) collection);
+            }
+            final int length = collection.size();
+            switch (length) {
+                case 0: {
+                    collection = Collections.emptyList();
+                    break;
+                }
+                case 1: {
+                    collection = Collections.singletonList(collection.iterator().next());
+                    break;
+                }
+                default: {
+                    if (collection instanceof UnmodifiableArrayList<?>) {
+                        break;                                              // List is already unmodifiable.
+                    }
+                    if (collection instanceof CheckedContainer<?>) {
+                        /*
+                         * We use UnmodifiableArrayList for avoiding one level of indirection. The fact that it
+                         * implements CheckedContainer is not a goal here, and is actually unsafe since we have
+                         * no guarantee (except Javadoc contract) that the <E> in CheckedContainer<E> is really
+                         * the same than in Collection<E>.  We tolerate this hole for now because we documented
+                         * the restriction in CheckedContainer javadoc, but future version may replace this block
+                         * by JDK9 collections.
+                         */
+                        @SuppressWarnings("unchecked")       // Okay if collection is compliant with CheckedContainer contract.
+                        final E[] array = (E[]) Array.newInstance(((CheckedContainer<E>) collection).getElementType(), length);
+                        collection = UnmodifiableArrayList.wrap(collection.toArray(array));
+                    } else if (collection instanceof List<?>) {
+                        collection = Collections.unmodifiableList((List<E>) collection);
+                    } else {
+                        collection = Collections.unmodifiableCollection(collection);
+                    }
+                    break;
+                }
+            }
+        }
+        return collection;
+    }
+
+    /**
      * Copies the content of the given collection to a new, unsynchronized, modifiable, in-memory
      * collection. The implementation class of the returned collection may be different than the
      * class of the collection given in argument. The following table gives the types mapping
@@ -519,7 +579,7 @@ public final class CollectionsExt extend
      * Returns a more compact representation of the given map. This method is similar to
      * {@link #unmodifiableOrCopy(Map)} except that it does not wrap the map in an unmodifiable
      * view. The intend is to avoid one level of indirection for performance and memory reasons.
-     * This is okay only if the map is kept in a private field and never escape outside this class.
+     * This is okay only if the map is kept in a private field and never escape outside that class.
      *
      * @param  <K>  the type of keys in the map.
      * @param  <V>  the type of values in the map.

Modified: sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/DefinitionURI.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/DefinitionURI.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/DefinitionURI.java [UTF-8] (original)
+++ sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/DefinitionURI.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -18,8 +18,8 @@ package org.apache.sis.internal.util;
 
 import java.util.Map;
 import java.util.Collections;
-
 import org.apache.sis.util.CharSequences;
+
 import static org.apache.sis.util.CharSequences.*;
 import static org.apache.sis.util.ArgumentChecks.ensureNonNull;
 import static org.apache.sis.internal.util.Utilities.appendUnicodeIdentifier;

Modified: sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/DoubleDouble.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/DoubleDouble.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/DoubleDouble.java [UTF-8] (original)
+++ sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/DoubleDouble.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -129,9 +129,11 @@ public final class DoubleDouble extends
      * @see #errorForWellKnownValue(double)
      */
     private static final double[] VALUES = {
-        // Some of the following constants have more fraction digits than necessary. We declare the extra
-        // digits for documentation purpose, and in order to have identical content than DoubleDoubleTest
-        // so that a plain copy-and-paste can be performed between those two classes.
+        /*
+         * Some of the following constants have more fraction digits than necessary. We declare the extra
+         * digits for documentation purpose, and in order to have identical content than DoubleDoubleTest
+         * so that a plain copy-and-paste can be performed between those two classes.
+         */
          0.000004848136811095359935899141023579480, // Arc-second to radians
          0.0002777777777777777777777777777777778,   // Second to degrees
          0.002777777777777777777777777777777778,    // 1/360°

Modified: sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/LazySet.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/LazySet.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/LazySet.java [UTF-8] (original)
+++ sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/LazySet.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -41,7 +41,7 @@ import org.apache.sis.internal.system.De
  *
  * <p>This class is not thread-safe. Synchronization, if desired, shall be done by the caller.</p>
  *
- * @param <E> the type of elements in the set.
+ * @param  <E>  the type of elements in the set.
  *
  * @author  Martin Desruisseaux (IRD, Geomatys)
  * @since   0.6
@@ -197,7 +197,7 @@ public class LazySet<E> extends SetOfUnk
      * Caches a new element. Subclasses can override this method is they want to substitute the given value
      * by another value.
      *
-     * @param  element  The element to add to the cache.
+     * @param  element  the element to add to the cache.
      */
     protected void cache(final E element) {
         if (numCached >= cachedElements.length) {

Modified: sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/LazySynchronizedIterator.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/LazySynchronizedIterator.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/LazySynchronizedIterator.java [UTF-8] (original)
+++ sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/LazySynchronizedIterator.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -28,7 +28,7 @@ import java.util.Iterator;
  * different threads can safely use their iterators concurrently even if the underlying {@code Iterable}s
  * were not thread-safe, because of the synchronization on {@code Iterable<E>} instances.</p>
  *
- * @param <E> The type of elements to be returned by the iterator.
+ * @param  <E>  the type of elements to be returned by the iterator.
  *
  * @author  Martin Desruisseaux (Geomatys)
  * @since   0.7

Modified: sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/MetadataServices.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/MetadataServices.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/MetadataServices.java [UTF-8] (original)
+++ sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/MetadataServices.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -62,7 +62,7 @@ public class MetadataServices extends Op
     /**
      * Returns the singleton instance.
      *
-     * @return The singleton instance.
+     * @return the singleton instance.
      */
     @SuppressWarnings("DoubleCheckedLocking")
     public static MetadataServices getInstance() {
@@ -94,8 +94,8 @@ public class MetadataServices extends Op
      * for which the two citations of interest are {@code "ISBN"} (International Standard Book Number) and
      * {@code "ISSN"} (International Standard Serial Number) citation.
      *
-     * @param  name The name of one of the citation constants defined in the {@code Citations} class.
-     * @return The requested citation, or {@code null} if the {@code sis-metadata} module is not available.
+     * @param  name  the name of one of the citation constants defined in the {@code Citations} class.
+     * @return the requested citation, or {@code null} if the {@code sis-metadata} module is not available.
      */
     public CitationConstant getCitationConstant(final String name) {
         return null;
@@ -106,8 +106,8 @@ public class MetadataServices extends Op
      * The metadata module will search in a database for information like a descriptive
      * title, abbreviations, identifiers, URL to a web site, <i>etc</i>.
      *
-     * @param  key The primary key of the desired citation.
-     * @return The requested citation, or {@code null} if the {@code sis-metadata} module is not available.
+     * @param  key  the primary key of the desired citation.
+     * @return the requested citation, or {@code null} if the {@code sis-metadata} module is not available.
      */
     public Citation createCitation(final String key) {
         return null;
@@ -123,9 +123,9 @@ public class MetadataServices extends Op
      *   <li>{@code "DataSource"}: URL to the data source, or error message.</li>
      * </ul>
      *
-     * @param  key A key identifying the information to return.
-     * @param  locale Language to use if possible.
-     * @return The information, or {@code null} if none.
+     * @param  key     a key identifying the information to return.
+     * @param  locale  language to use if possible.
+     * @return the information, or {@code null} if none.
      *
      * @see org.apache.sis.internal.metadata.ReferencingServices#getInformation(String)
      */

Modified: sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/SetOfUnknownSize.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/SetOfUnknownSize.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/SetOfUnknownSize.java [UTF-8] (original)
+++ sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/SetOfUnknownSize.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -28,7 +28,7 @@ import org.apache.sis.util.ArraysExt;
  * An alternative to {@code AbstractSet} for implementations having a costly {@link #size()} method.
  * This class overrides some methods in a way that avoid or reduce calls to {@link #size()}.
  *
- * @param <E> the type of elements in the set.
+ * @param  <E>  the type of elements in the set.
  *
  * @author  Martin Desruisseaux (Geomatys)
  * @since   0.7

Modified: sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/X364.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/X364.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/X364.java [UTF-8] (original)
+++ sis/trunk/core/sis-utility/src/main/java/org/apache/sis/internal/util/X364.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -119,8 +119,8 @@ public enum X364 {
     /**
      * Creates a new code.
      *
-     * @param code  The X.364 numerical code.
-     * @param color The color name, or {@code null} if none.
+     * @param code   the X.364 numerical code.
+     * @param color  the color name, or {@code null} if none.
      */
     @SuppressWarnings("ThisEscapedInObjectConstruction")
     private X364(final byte code, final String color) {
@@ -133,7 +133,7 @@ public enum X364 {
     /**
      * Creates a new background code.
      *
-     * @param foreground The X.364 code for a foreground color.
+     * @param foreground  the X.364 code for a foreground color.
      */
     @SuppressWarnings("ThisEscapedInObjectConstruction")
     private X364(final X364 foreground) {
@@ -145,7 +145,7 @@ public enum X364 {
     /**
      * Returns the enum for the foreground color.
      *
-     * @return The foreground color, or {@code this} if this enum is already a foreground color.
+     * @return the foreground color, or {@code this} if this enum is already a foreground color.
      */
     public X364 foreground() {
         return foreground;
@@ -154,7 +154,7 @@ public enum X364 {
     /**
      * Returns the enum for the background color.
      *
-     * @return The background color, or {@code this} if this enum is already a background color.
+     * @return the background color, or {@code this} if this enum is already a background color.
      */
     public X364 background() {
         return background;
@@ -163,15 +163,17 @@ public enum X364 {
     /**
      * Returns the ANSI escape sequence.
      *
-     * @return The ANSI escape sequence.
+     * @return the ANSI escape sequence.
      */
     public String sequence() {
         if (sequence == null) {
             sequence = (START + code + END).intern();
-            // We used the string.intern() method in order to avoid worrying about memory barrier
-            // (synchronization or volatile variable) since intern() does its own synchronization.
-            // The String will live for the whole library lifetime anyway, and if there is other
-            // X3.64 libraries on the JVM we may share the strings with them as a side effect.
+            /*
+             * We used the string.intern() method in order to avoid worrying about memory barrier
+             * (synchronization or volatile variable) since intern() does its own synchronization.
+             * The String will live for the whole library lifetime anyway, and if there is other
+             * X3.64 libraries on the JVM we may share the strings with them as a side effect.
+             */
         }
         return sequence;
     }
@@ -179,10 +181,10 @@ public enum X364 {
     /**
      * Removes all escape codes from the given string.
      *
-     * @param  text      The string which may contains escape codes.
-     * @param  fromIndex The index from which to start the process.
-     * @param  toIndex   The index after the last character to process.
-     * @return Text without the escape codes, or the given {@code text} reference if
+     * @param  text       the string which may contains escape codes.
+     * @param  fromIndex  the index from which to start the process.
+     * @param  toIndex    the index after the last character to process.
+     * @return text without the escape codes, or the given {@code text} reference if
      *         it didn't contained any escape codes.
      */
     public static CharSequence plain(final CharSequence text, int fromIndex, final int toIndex) {
@@ -221,10 +223,10 @@ search:     do {
      * CharSequences.codePointCount}({@linkplain #plain plain}(text))</code> without the
      * cost of creating a temporary string.
      *
-     * @param  text      The string which may contains escape codes.
-     * @param  fromIndex The index from which to start the computation.
-     * @param  toIndex   The index after the last character to take in account.
-     * @return The length of the given string without escape codes.
+     * @param  text       the string which may contains escape codes.
+     * @param  fromIndex  the index from which to start the computation.
+     * @param  toIndex    the index after the last character to take in account.
+     * @return the length of the given string without escape codes.
      */
     public static int lengthOfPlain(final CharSequence text, final int fromIndex, final int toIndex) {
         int i = CharSequences.indexOf(text, START, fromIndex, toIndex);
@@ -243,11 +245,11 @@ search: do {
             while (i < end) {
                 final char c = text.charAt(i++);
                 if (c < '0' || c > '9') {
-                    continue search; // Not an X.364 sequence.
+                    continue search;                                        // Not an X.364 sequence.
                 }
             }
             length += CharSequences.codePointCount(text, last, start);
-            last = ++i; // The ++ is for skipping the END character.
+            last = ++i;                                                     // The ++ is for skipping the END character.
         } while ((i = CharSequences.indexOf(text, START, i, toIndex)) >= 0);
         length += CharSequences.codePointCount(text, last, toIndex);
         assert CharSequences.codePointCount(plain(text, fromIndex, toIndex)) == length : text.subSequence(fromIndex, toIndex);
@@ -258,9 +260,9 @@ search: do {
      * Returns the enumeration value for the given color name.
      * The search is case-insensitive.
      *
-     * @param  color The color name.
-     * @return The code for the given color name.
-     * @throws IllegalArgumentException If no code has been found for the given color name.
+     * @param  color  the color name.
+     * @return the code for the given color name.
+     * @throws IllegalArgumentException if no code has been found for the given color name.
      */
     public static X364 forColorName(String color) throws IllegalArgumentException {
         color = CharSequences.trimWhitespaces(color);
@@ -296,7 +298,7 @@ search: do {
             }
             terminal = System.getenv("CLICOLOR");
         } catch (SecurityException e) {
-            return false; // Okay according javadoc.
+            return false;                               // Okay according javadoc.
         }
         if (terminal != null) try {
             return Integer.parseInt(terminal) != 0;

Modified: sis/trunk/core/sis-utility/src/main/java/org/apache/sis/io/AppendableWriter.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-utility/src/main/java/org/apache/sis/io/AppendableWriter.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-utility/src/main/java/org/apache/sis/io/AppendableWriter.java [UTF-8] (original)
+++ sis/trunk/core/sis-utility/src/main/java/org/apache/sis/io/AppendableWriter.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -39,7 +39,7 @@ final class AppendableWriter extends Wri
     /**
      * Creates a new filtered formatter which will send its output to the given stream or buffer.
      *
-     * @param out The underlying character output stream or buffer.
+     * @param out  the underlying character output stream or buffer.
      */
     AppendableWriter(final Appendable out) {
         super(getLock(out));
@@ -173,7 +173,7 @@ final class AppendableWriter extends Wri
      * Returns the content of the underlying {@link Appendable} as a string if possible,
      * or the localized <cite>"Unavailable content"</cite> string otherwise.
      *
-     * @return The content of the underlying {@code Appendable},
+     * @return the content of the underlying {@code Appendable},
      *         or a localized message for unavailable content.
      *
      * @see IO#content(Appendable)

Modified: sis/trunk/core/sis-utility/src/main/java/org/apache/sis/io/Appender.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-utility/src/main/java/org/apache/sis/io/Appender.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-utility/src/main/java/org/apache/sis/io/Appender.java [UTF-8] (original)
+++ sis/trunk/core/sis-utility/src/main/java/org/apache/sis/io/Appender.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -66,7 +66,7 @@ abstract class Appender implements Appen
     /**
      * Creates a new appender which will send its output to the given stream or buffer.
      *
-     * @param out The underlying character output stream or buffer.
+     * @param out  the underlying character output stream or buffer.
      */
     protected Appender(final Appendable out) {
         ArgumentChecks.ensureNonNull("out", out);
@@ -80,7 +80,7 @@ abstract class Appender implements Appen
      */
     final String lineSeparator(final CharSequence sequence, int start, final int end) {
         if (isHighSurrogate()) {
-            start++; // Skip invalid character.
+            start++;                                        // Skip invalid character.
         }
         while (start < end) {
             final int c = Character.codePointAt(sequence, start);
@@ -109,8 +109,11 @@ abstract class Appender implements Appen
             if (Character.isLowSurrogate(c)) {
                 return Character.toCodePoint(h, c);
             }
-            // Unpaired surrogate.  This is usually an error, but this not the fault of
-            // this class since we are processing data supplied by the user. Be lenient.
+            /*
+             * If we reach this point, we have unpaired surrogate.  This is usually an error,
+             * but this not the fault of this class since we are processing data supplied by
+             * the user. Be lenient.
+             */
         }
         if (Character.isHighSurrogate(c)) {
             highSurrogate = c;
@@ -132,10 +135,10 @@ abstract class Appender implements Appen
      * delegates to {@link #append(char)} and returns {@code start+1}. The intend is to avoid
      * processing a character sequence which starts by an invalid code point.
      *
-     * @param  sequence The character sequence to write.
-     * @param  start    Index of the first character to write by this method or by the caller.
-     * @param  end      Index after the last character to be written by the caller.
-     * @return Index of the first character which need to be written by the caller.
+     * @param  sequence  the character sequence to write.
+     * @param  start     index of the first character to write by this method or by the caller.
+     * @param  end       index after the last character to be written by the caller.
+     * @return index of the first character which need to be written by the caller.
      */
     final int appendSurrogate(final CharSequence sequence, int start, final int end) throws IOException {
         if (start != end && highSurrogate != 0) {
@@ -153,8 +156,8 @@ abstract class Appender implements Appen
     /**
      * Appends the given code point to the underlying {@link #out} stream or buffer.
      *
-     * @param  c The code point to append.
-     * @throws IOException If an error occurred while appending the code point.
+     * @param  c  the code point to append.
+     * @throws IOException if an error occurred while appending the code point.
      */
     final void appendCodePoint(final int c) throws IOException {
         if (Character.isBmpCodePoint(c)) {
@@ -171,9 +174,9 @@ abstract class Appender implements Appen
      * Appends the specified character sequence.
      * The default implementation delegates to {@link #append(CharSequence, int, int)}.
      *
-     * @param  sequence The character sequence to append, or {@code null}.
-     * @return A reference to this {@code Appendable}.
-     * @throws IOException If an I/O error occurred.
+     * @param  sequence  the character sequence to append, or {@code null}.
+     * @return a reference to this {@code Appendable}.
+     * @throws IOException if an I/O error occurred.
      */
     @Override
     public Appendable append(CharSequence sequence) throws IOException {
@@ -187,7 +190,7 @@ abstract class Appender implements Appen
      * Returns the content of this {@code Appendable} as a string if possible,
      * or the localized <cite>"Unavailable content"</cite> string otherwise.
      *
-     * @return The content of this {@code Appendable}, or a localized message for unavailable content.
+     * @return the content of this {@code Appendable}, or a localized message for unavailable content.
      *
      * @see IO#content(Appendable)
      */

Modified: sis/trunk/core/sis-utility/src/main/java/org/apache/sis/io/ClassFormat.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-utility/src/main/java/org/apache/sis/io/ClassFormat.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-utility/src/main/java/org/apache/sis/io/ClassFormat.java [UTF-8] (original)
+++ sis/trunk/core/sis-utility/src/main/java/org/apache/sis/io/ClassFormat.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -35,6 +35,7 @@ import org.apache.sis.util.Classes;
  * @version 0.3
  * @module
  */
+@SuppressWarnings("CloneableClassWithoutClone")   // Because this class does not contain field that need to be cloned.
 final class ClassFormat extends Format {
     /**
      * For cross-version compatibility.

Modified: sis/trunk/core/sis-utility/src/main/java/org/apache/sis/io/DefaultFormat.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-utility/src/main/java/org/apache/sis/io/DefaultFormat.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-utility/src/main/java/org/apache/sis/io/DefaultFormat.java [UTF-8] (original)
+++ sis/trunk/core/sis-utility/src/main/java/org/apache/sis/io/DefaultFormat.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -42,6 +42,7 @@ import org.apache.sis.internal.util.Loca
  * @version 0.3
  * @module
  */
+@SuppressWarnings("CloneableClassWithoutClone")   // Because this class does not contain field that need to be cloned.
 final class DefaultFormat extends Format {
     /**
      * For cross-version compatibility.

Modified: sis/trunk/core/sis-utility/src/main/java/org/apache/sis/io/IO.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-utility/src/main/java/org/apache/sis/io/IO.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-utility/src/main/java/org/apache/sis/io/IO.java [UTF-8] (original)
+++ sis/trunk/core/sis-utility/src/main/java/org/apache/sis/io/IO.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -49,7 +49,7 @@ public final class IO extends Static {
      * <p>Chains of wrappers are followed until a {@code Flushable} instance is found, if any.
      * The search stops at the first occurrence found.</p>
      *
-     * @param  out The stream or buffer to flush, or {@code null}.
+     * @param  out  the output stream, writer or buffer to flush, or {@code null}.
      * @throws IOException if an error occurred while flushing the given stream.
      */
     public static void flush(Appendable out) throws IOException {
@@ -72,7 +72,7 @@ public final class IO extends Static {
      * if any, is {@linkplain Flushable#flush() flushed}.
      * The search stops at the first {@code Closeable} occurrence found.</p>
      *
-     * @param  out The stream or buffer to close, or {@code null}.
+     * @param  out  the output stream, writer or buffer to close, or {@code null}.
      * @throws IOException if an error occurred while closing the given stream.
      */
     public static void close(Appendable out) throws IOException {
@@ -113,8 +113,8 @@ public final class IO extends Static {
      * to get proper content. In particular, this is necessary if the chain of {@code Appendable}s
      * contains {@link TableAppender} or {@link LineAppender} instances.</p>
      *
-     * @param  out The stream or buffer from which to get the content, or {@code null}.
-     * @return The content of the given stream of buffer, or {@code null} if unavailable.
+     * @param  out  the output stream, writer or buffer from which to get the content, or {@code null}.
+     * @return the content of the given stream of buffer, or {@code null} if unavailable.
      *
      * @see #flush(Appendable)
      */
@@ -153,9 +153,8 @@ public final class IO extends Static {
      * Any write operations performed on the returned writer will be forwarded
      * to the given {@code Appendable}.
      *
-     * @param  out The stream or buffer to view as a {@link Writer}, or {@code null}.
-     * @return A view of this {@code Appendable} as a writer, or {@code null} if the
-     *         given argument was null.
+     * @param  out  the output stream, writer or buffer to view as a {@link Writer}, or {@code null}.
+     * @return a view of this {@code Appendable} as a writer, or {@code null} if the given argument was null.
      */
     public static Writer asWriter(final Appendable out) {
         if (out == null || out instanceof Writer) {

Modified: sis/trunk/core/sis-utility/src/main/java/org/apache/sis/io/IdentifiedObjectFormat.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-utility/src/main/java/org/apache/sis/io/IdentifiedObjectFormat.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-utility/src/main/java/org/apache/sis/io/IdentifiedObjectFormat.java [UTF-8] (original)
+++ sis/trunk/core/sis-utility/src/main/java/org/apache/sis/io/IdentifiedObjectFormat.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -36,6 +36,7 @@ import org.apache.sis.util.resources.Voc
  * @version 0.4
  * @module
  */
+@SuppressWarnings("CloneableClassWithoutClone")   // Because this class does not contain field that need to be cloned.
 final class IdentifiedObjectFormat extends Format {
     /**
      * For cross-version compatibility.

Modified: sis/trunk/core/sis-utility/src/main/java/org/apache/sis/io/LineAppender.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-utility/src/main/java/org/apache/sis/io/LineAppender.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-utility/src/main/java/org/apache/sis/io/LineAppender.java [UTF-8] (original)
+++ sis/trunk/core/sis-utility/src/main/java/org/apache/sis/io/LineAppender.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -153,7 +153,7 @@ public class LineAppender extends Append
      *   <li>{@link #setLineSeparator(String)}</li>
      * </ul>
      *
-     * @param out The underlying stream or buffer to write to.
+     * @param out  the underlying stream or buffer to write to.
      */
     public LineAppender(final Appendable out) {
         super(out);
@@ -163,8 +163,8 @@ public class LineAppender extends Append
     /**
      * Constructs a formatter which will replaces line separators by the given string.
      *
-     * @param out                   The underlying stream or buffer to write to.
-     * @param lineSeparator         The line separator to send to {@code out}, or {@code null}
+     * @param out                   the underlying stream or buffer to write to.
+     * @param lineSeparator         the line separator to send to {@code out}, or {@code null}
      *                              for forwarding the EOL sequences unchanged.
      * @param isTabulationExpanded  {@code true} for expanding tabulations into spaces,
      *                              or {@code false} for sending {@code '\t'} characters as-is.
@@ -180,8 +180,8 @@ public class LineAppender extends Append
     /**
      * Constructs a formatter which will wrap the lines at a given maximal length.
      *
-     * @param out                   The underlying stream or buffer to write to.
-     * @param maximalLineLength     The maximal number of Unicode characters per line,
+     * @param out                   the underlying stream or buffer to write to.
+     * @param maximalLineLength     the maximal number of Unicode characters per line,
      *                              or {@link Integer#MAX_VALUE} if there is no limit.
      * @param isTabulationExpanded  {@code true} for expanding tabulations into spaces,
      *                              or {@code false} for forwarding {@code '\t'} characters as-is.
@@ -197,7 +197,7 @@ public class LineAppender extends Append
      * Returns the maximal line length, in unit of Unicode characters (code point count).
      * The default value is no limit.
      *
-     * @return The current maximal number of Unicode characters per line,
+     * @return the current maximal number of Unicode characters per line,
      *         or {@link Integer#MAX_VALUE} if there is no limit.
      */
     public int getMaximalLineLength() {
@@ -207,8 +207,8 @@ public class LineAppender extends Append
     /**
      * Sets the maximal line length, in units of Unicode characters (code point count).
      *
-     * @param length The new maximal number of Unicode characters per line,
-     *               or {@link Integer#MAX_VALUE} if there is no limit.
+     * @param  length  the new maximal number of Unicode characters per line,
+     *                 or {@link Integer#MAX_VALUE} if there is no limit.
      */
     public void setMaximalLineLength(final int length) {
         ArgumentChecks.ensureStrictlyPositive("length", length);
@@ -219,7 +219,7 @@ public class LineAppender extends Append
      * Returns the current tabulation width, in unit of Unicode characters (code point count).
      * The default value is 8.
      *
-     * @return The current tabulation width in number of Unicode characters.
+     * @return the current tabulation width in number of Unicode characters.
      */
     public int getTabulationWidth() {
         return tabulationWidth;
@@ -228,7 +228,7 @@ public class LineAppender extends Append
     /**
      * Sets the tabulation width, in unit of Unicode characters (code point count).
      *
-     * @param  width The new tabulation width. Must be greater than 0.
+     * @param  width  the new tabulation width. Must be greater than 0.
      * @throws IllegalArgumentException if {@code tabWidth} is not greater than 0
      *         or is unreasonably high.
      */
@@ -264,7 +264,7 @@ public class LineAppender extends Append
      * Returns the line separator to be sent to the underlying appendable,
      * or {@code null} if EOL sequences are forwarded unchanged.
      *
-     * @return The current line separator, or {@code null} if EOL are forwarded <i>as-is</i>.
+     * @return the current line separator, or {@code null} if EOL are forwarded <i>as-is</i>.
      */
     public String getLineSeparator() {
         return isEndOfLineReplaced ? lineSeparator : null;
@@ -277,7 +277,7 @@ public class LineAppender extends Append
      * If {@code null} (the default), then the line separators given to the {@code append}
      * methods are forwarded unchanged.
      *
-     * @param  lineSeparator The new line separator, or {@code null} for forwarding EOL <i>as-is</i>.
+     * @param  lineSeparator  the new line separator, or {@code null} for forwarding EOL <i>as-is</i>.
      *
      * @see Characters#isLineOrParagraphSeparator(int)
      */
@@ -308,12 +308,12 @@ public class LineAppender extends Append
      * responsibility.
      */
     private void endOfLine() throws IOException {
-        buffer.setLength(printableLength); // Reduce the amount of work for StringBuilder.deleteCharAt(int).
+        buffer.setLength(printableLength);      // Reduce the amount of work for StringBuilder.deleteCharAt(int).
         deleteSoftHyphen(printableLength - 1);
         transfer(printableLength);
         printableLength  = 0;
         codePointCount   = 0;
-        isEscapeSequence = false; // Handle line-breaks as "end of escape sequence".
+        isEscapeSequence = false;               // Handle line-breaks as "end of escape sequence".
         isNewLine        = true;
     }
 
@@ -321,7 +321,7 @@ public class LineAppender extends Append
      * Removes the soft hyphen characters from the given buffer. This is invoked
      * when the buffer is about to be written without being split on two lines.
      *
-     * @param i Index after the last character to check. This is either {@link printableLength}
+     * @param i index after the last character to check. This is either {@link printableLength}
      *          for checking all characters, or {@code printableLength-1} for preserving the last
      *          soft hyphen on the line (while removing all others).
      */
@@ -351,7 +351,7 @@ public class LineAppender extends Append
     /**
      * Writes the specified code point.
      *
-     * @throws IOException If an I/O error occurs.
+     * @throws IOException if an I/O error occurs.
      */
     @SuppressWarnings("fallthrough")
     private void write(final int c) throws IOException {
@@ -447,7 +447,7 @@ searchHyp:  for (int i=buffer.length();
                 switch (b) {
                     case '-': {
                         if (i>=n && !Character.isLetter(buffer.codePointBefore(i-n))) {
-                            break; // Continue searching previous characters.
+                            break;                      // Continue searching previous characters.
                         }
                         // fall through
                     }
@@ -482,9 +482,9 @@ searchHyp:  for (int i=buffer.length();
     /**
      * Writes a single character.
      *
-     * @param  c The character to append.
-     * @return A reference to this {@code Appendable}.
-     * @throws IOException If an I/O error occurs.
+     * @param  c  the character to append.
+     * @return a reference to this {@code Appendable}.
+     * @throws IOException if an I/O error occurs.
      */
     @Override
     public Appendable append(final char c) throws IOException {
@@ -498,11 +498,11 @@ searchHyp:  for (int i=buffer.length();
     /**
      * Writes a portion of a character sequence.
      *
-     * @param  sequence The character sequence to be written.
-     * @param  start    Index from which to start reading characters.
-     * @param  end      Index of the character following the last character to read.
-     * @return A reference to this {@code Appendable}.
-     * @throws IOException If an I/O error occurs.
+     * @param  sequence  the character sequence to be written.
+     * @param  start     index from which to start reading characters.
+     * @param  end       index of the character following the last character to read.
+     * @return a reference to this {@code Appendable}.
+     * @throws IOException if an I/O error occurs.
      */
     @Override
     public Appendable append(final CharSequence sequence, int start, final int end) throws IOException {
@@ -532,7 +532,7 @@ searchHyp:  for (int i=buffer.length();
      * are discarded, and the column position (for tabulation expansion calculation) is
      * reset to 0. This method does not write any line separator.
      *
-     * @throws IOException If an error occurred while sending the trailing non-white
+     * @throws IOException if an error occurred while sending the trailing non-white
      *         characters to the underlying stream.
      */
     public void clear() throws IOException {
@@ -551,7 +551,7 @@ searchHyp:  for (int i=buffer.length();
      * {@linkplain Characters#isLineOrParagraphSeparator(int) line or paragraph terminator},
      * or to invoke {@link #clear()}.</p>
      *
-     * @throws IOException If an I/O error occurs.
+     * @throws IOException if an I/O error occurs.
      */
     @Override
     public void flush() throws IOException {

Modified: sis/trunk/core/sis-utility/src/main/java/org/apache/sis/io/TableAppender.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-utility/src/main/java/org/apache/sis/io/TableAppender.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-utility/src/main/java/org/apache/sis/io/TableAppender.java [UTF-8] (original)
+++ sis/trunk/core/sis-utility/src/main/java/org/apache/sis/io/TableAppender.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -221,7 +221,7 @@ public class TableAppender extends Appen
     /**
      * Creates a new table formatter writing in an internal buffer with the specified column separator.
      *
-     * @param separator String to write between columns.
+     * @param separator  string to write between columns.
      */
     public TableAppender(final String separator) {
         this(new StringBuilder(256), separator);
@@ -233,7 +233,7 @@ public class TableAppender extends Appen
      * The default is a vertical double line for the left and right table borders, and a single
      * line between the columns.
      *
-     * @param out The underlying stream or buffer to write to.
+     * @param out  the underlying stream or buffer to write to.
      */
     public TableAppender(final Appendable out) {
         super(out);
@@ -245,8 +245,8 @@ public class TableAppender extends Appen
     /**
      * Creates a new table formatter writing in the given output with the specified column separator.
      *
-     * @param out The underlying stream or buffer to write to.
-     * @param separator String to write between columns.
+     * @param out        the underlying stream or buffer to write to.
+     * @param separator  string to write between columns.
      */
     public TableAppender(final Appendable out, final String separator) {
         super(out);
@@ -268,8 +268,8 @@ public class TableAppender extends Appen
      *
      * @param  horizontalBorder -1 for left border, +1 for right border,  0 for center.
      * @param  verticalBorder   -1 for top  border, +1 for bottom border, 0 for center.
-     * @param  horizontalChar   Character to use for horizontal line.
-     * @throws IOException      If the writing operation failed.
+     * @param  horizontalChar   character to use for horizontal line.
+     * @throws IOException      if the writing operation failed.
      */
     private void writeBorder(final int  horizontalBorder,
                              final int  verticalBorder,
@@ -346,8 +346,8 @@ public class TableAppender extends Appen
      *
      * The default value is {@code false}.
      *
-     * @param multiLines {@code true} true if EOL are used for line feeds inside
-     *        current cells, or {@code false} if EOL move to the next row.
+     * @param multiLines  {@code true} true if EOL are used for line feeds inside current cells,
+     *                    or {@code false} if EOL move to the next row.
      */
     public void setMultiLinesCells(final boolean multiLines) {
         multiLinesCells = multiLines;
@@ -357,7 +357,7 @@ public class TableAppender extends Appen
      * Returns the alignment of the text inside the current cell.
      * The default value is {@link #ALIGN_LEFT}.
      *
-     * @return Current cell alignment as one of the {@link #ALIGN_LEFT},
+     * @return current cell alignment as one of the {@link #ALIGN_LEFT},
      *         {@link #ALIGN_RIGHT} or {@link #ALIGN_CENTER} constants.
      */
     public byte getCellAlignment() {
@@ -372,8 +372,8 @@ public class TableAppender extends Appen
      *
      * <p>If this method is never invoked, then the default alignment is {@link #ALIGN_LEFT}.</p>
      *
-     * @param alignment The new cell alignment as one of the {@link #ALIGN_LEFT},
-     *        {@link #ALIGN_RIGHT} or {@link #ALIGN_CENTER} constants.
+     * @param  alignment  the new cell alignment as one of the {@link #ALIGN_LEFT},
+     *                    {@link #ALIGN_RIGHT} or {@link #ALIGN_CENTER} constants.
      */
     public void setCellAlignment(final byte alignment) {
         if (alignment < ALIGN_LEFT || alignment > ALIGN_RIGHT) {
@@ -386,12 +386,12 @@ public class TableAppender extends Appen
     /**
      * Returns the number of rows in this table. This count is reset to 0 by {@link #flush()}.
      *
-     * @return The number of rows in this table.
+     * @return the number of rows in this table.
      */
     public int getRowCount() {
         int count = currentRow;
         if (currentColumn != 0) {
-            count++; // Some writting has begun in the current row.
+            count++;                            // Some writting has begun in the current row.
         }
         return count;
     }
@@ -399,15 +399,15 @@ public class TableAppender extends Appen
     /**
      * Returns the number of columns in this table.
      *
-     * @return The number of columns in this table.
+     * @return the number of columns in this table.
      */
     public int getColumnCount() {
         return maximalColumnWidths.length;
     }
 
     /**
-     * Writes a single character. If {@link #isMultiLinesCells()} is {@code false}
-     * (which is the default), then:
+     * Writes a single character.
+     * If {@link #isMultiLinesCells()} is {@code false} (which is the default), then:
      *
      * <ul>
      *   <li>Tabulations ({@code '\t'}) are replaced by calls to {@link #nextColumn()}.</li>
@@ -415,7 +415,7 @@ public class TableAppender extends Appen
      *       line or paragraph separators} are replaced by calls to {@link #nextLine()}.</li>
      * </ul>
      *
-     * @param  c Character to write.
+     * @param  c  character to write.
      * @return {@code this}.
      */
     @Override
@@ -448,8 +448,8 @@ public class TableAppender extends Appen
     /**
      * Appends the specified character sequence.
      *
-     * @param  sequence The character sequence to append, or {@code null}.
-     * @return A reference to this {@code Appendable}.
+     * @param  sequence  the character sequence to append, or {@code null}.
+     * @return a reference to this {@code Appendable}.
      */
     @Override
     public TableAppender append(CharSequence sequence) {
@@ -463,9 +463,9 @@ public class TableAppender extends Appen
      * Writes a portion of a character sequence. Tabulations and line separators are
      * interpreted as by {@link #append(char)}.
      *
-     * @param  sequence The character sequence to be written.
-     * @param  start    Index from which to start reading characters.
-     * @param  end      Index of the character following the last character to read.
+     * @param  sequence  the character sequence to be written.
+     * @param  start     index from which to start reading characters.
+     * @param  end       index of the character following the last character to read.
      * @return {@code this}.
      */
     @Override
@@ -478,8 +478,10 @@ public class TableAppender extends Appen
         try {
             start = appendSurrogate(sequence, start, end);
         } catch (IOException e) {
-            // Should never happen, because appendSurrogate(…) delegates to append(char)
-            // which is overriden without 'throws IOException' clause in this class.
+            /*
+             * Should never happen, because appendSurrogate(…) delegates to append(char)
+             * which is overriden without 'throws IOException' clause in this class.
+             */
             throw new AssertionError(e);
         }
         if (start != end) {
@@ -545,7 +547,7 @@ public class TableAppender extends Appen
      * <p>Calling {@code nextColumn('*')} from the first character
      * in a cell is a convenient way to put a pad value in this cell.</p>
      *
-     * @param fill Character filling the cell (default to whitespace).
+     * @param  fill  character filling the cell (default to whitespace).
      */
     public void nextColumn(final char fill) {
         final String cellText = buffer.toString();
@@ -591,8 +593,8 @@ public class TableAppender extends Appen
      * <p>Calling {@code nextLine('-')} or {@code nextLine('═')} from the first column of a row
      * is a convenient way to fill this row with a line separator.</p>
      *
-     * @param fill Character filling the rest of the line (default to whitespace).
-     *             This character may be use as a row separator.
+     * @param  fill  character filling the rest of the line (default to whitespace).
+     *              This character may be use as a row separator.
      */
     public void nextLine(final char fill) {
         if (buffer.length() != 0) {
@@ -820,9 +822,9 @@ public class TableAppender extends Appen
      * Repeats a character. The {@code count} value may be negative,
      * which is handled as if it was zero.
      *
-     * @param out   The stream or buffer where to repeat the character.
-     * @param car   Character to write (usually ' ').
-     * @param count Number of repetition.
+     * @param  out    the stream or buffer where to repeat the character.
+     * @param  car    character to write (usually ' ').
+     * @param  count  number of repetition.
      */
     private static void repeat(final Appendable out, final char car, int count) throws IOException {
         while (--count >= 0) {

Modified: sis/trunk/core/sis-utility/src/main/java/org/apache/sis/math/ArrayVector.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-utility/src/main/java/org/apache/sis/math/ArrayVector.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-utility/src/main/java/org/apache/sis/math/ArrayVector.java [UTF-8] (original)
+++ sis/trunk/core/sis-utility/src/main/java/org/apache/sis/math/ArrayVector.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -613,9 +613,11 @@ abstract class ArrayVector<E extends Num
             return NumberRange.create(min, true, max, true);
         }
 
-        // Not worth to override 'increment(double)' because the array can not be long anyway
-        // (except if the increment is zero) and the implicit conversion of 'short' to 'int'
-        // performed by Java would make the implementation a little bit more tricky.
+        /*
+         * Not worth to override 'increment(double)' because the array can not be long anyway
+         * (except if the increment is zero) and the implicit conversion of 'short' to 'int'
+         * performed by Java would make the implementation a little bit more tricky.
+         */
     }
 
     /**
@@ -672,9 +674,11 @@ abstract class ArrayVector<E extends Num
             return NumberRange.create(min, true, max, true);
         }
 
-        // Not worth to override 'increment(double)' because the array can not be long anyway
-        // (except if the increment is zero) and the implicit conversion of 'byte' to 'int'
-        // performed by Java would make the implementation a little bit more tricky.
+        /*
+         * Not worth to override 'increment(double)' because the array can not be long anyway
+         * (except if the increment is zero) and the implicit conversion of 'byte' to 'int'
+         * performed by Java would make the implementation a little bit more tricky.
+         */
     }
 
     /**

Modified: sis/trunk/core/sis-utility/src/main/java/org/apache/sis/math/CompoundDirectPositions.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-utility/src/main/java/org/apache/sis/math/CompoundDirectPositions.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-utility/src/main/java/org/apache/sis/math/CompoundDirectPositions.java [UTF-8] (original)
+++ sis/trunk/core/sis-utility/src/main/java/org/apache/sis/math/CompoundDirectPositions.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -71,7 +71,7 @@ final class CompoundDirectPositions impl
     /**
      * Starts a new iteration.
      *
-     * @return Always {@code this}.
+     * @return always {@code this}.
      */
     @Override
     public Iterator<DirectPosition> iterator() {
@@ -90,7 +90,7 @@ final class CompoundDirectPositions impl
     /**
      * Sets this object to the next position and return it.
      *
-     * @return Always {@code this}.
+     * @return always {@code this}.
      */
     @Override
     public DirectPosition next() {
@@ -101,7 +101,7 @@ final class CompoundDirectPositions impl
     /**
      * Returns {@code this} since this object is already a direct position.
      *
-     * @return Always {@code this}.
+     * @return always {@code this}.
      */
     @Override
     public DirectPosition getDirectPosition() {
@@ -111,7 +111,7 @@ final class CompoundDirectPositions impl
     /**
      * Returns {@code null} since there is no CRS associated to this object.
      *
-     * @return Always {@code null}.
+     * @return always {@code null}.
      */
     @Override
     public CoordinateReferenceSystem getCoordinateReferenceSystem() {

Modified: sis/trunk/core/sis-utility/src/main/java/org/apache/sis/math/ConcatenatedVector.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-utility/src/main/java/org/apache/sis/math/ConcatenatedVector.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-utility/src/main/java/org/apache/sis/math/ConcatenatedVector.java [UTF-8] (original)
+++ sis/trunk/core/sis-utility/src/main/java/org/apache/sis/math/ConcatenatedVector.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -49,8 +49,8 @@ final class ConcatenatedVector extends V
     /**
      * Creates a concatenated vector.
      *
-     * @param first  the vector for the lower indices.
-     * @param second the vector for the higher indices.
+     * @param first   the vector for the lower indices.
+     * @param second  the vector for the higher indices.
      */
     public ConcatenatedVector(final Vector first, final Vector second) {
         this.first  = first;

Modified: sis/trunk/core/sis-utility/src/main/java/org/apache/sis/math/DecimalFunctions.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-utility/src/main/java/org/apache/sis/math/DecimalFunctions.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-utility/src/main/java/org/apache/sis/math/DecimalFunctions.java [UTF-8] (original)
+++ sis/trunk/core/sis-utility/src/main/java/org/apache/sis/math/DecimalFunctions.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -135,8 +135,8 @@ public final class DecimalFunctions exte
      *   return Double.parseDouble(Float.toString(value));
      * }
      *
-     * @param  value The {@code float} value to convert as a {@code double}.
-     * @return The given value as a {@code double} with the extra decimal fraction digits set to zero.
+     * @param  value  the {@code float} value to convert as a {@code double}.
+     * @return the given value as a {@code double} with the extra decimal fraction digits set to zero.
      */
     public static double floatToDouble(final float value) {
         /*

Modified: sis/trunk/core/sis-utility/src/main/java/org/apache/sis/math/FunctionProperty.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-utility/src/main/java/org/apache/sis/math/FunctionProperty.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-utility/src/main/java/org/apache/sis/math/FunctionProperty.java [UTF-8] (original)
+++ sis/trunk/core/sis-utility/src/main/java/org/apache/sis/math/FunctionProperty.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -159,7 +159,7 @@ public enum FunctionProperty {
      *   <li>{@link #SURJECTIVE}</li>
      * </ul>
      *
-     * @param  properties The properties of the function to test for bijectivity.
+     * @param  properties  the properties of the function to test for bijectivity.
      * @return {@code true} if a function having the given set of properties is bijective.
      */
     public static boolean isBijective(final Set<FunctionProperty> properties) {
@@ -176,7 +176,7 @@ public enum FunctionProperty {
      *   <li>{@link #ORDER_REVERSING}</li>
      * </ul>
      *
-     * @param  properties The properties of the function to test for monotonicity.
+     * @param  properties  the properties of the function to test for monotonicity.
      * @return {@code true} if a function having the given set of properties is monotonic.
      */
     public static boolean isMonotonic(final Set<FunctionProperty> properties) {



Mime
View raw message