sis-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From desruisse...@apache.org
Subject svn commit: r1776531 [1/4] - in /sis/branches/JDK8: application/sis-console/src/main/java/org/apache/sis/console/ core/sis-feature/src/main/java/org/apache/sis/feature/ core/sis-metadata/src/main/java/org/apache/sis/internal/metadata/ core/sis-metadata...
Date Fri, 30 Dec 2016 09:23:26 GMT
Author: desruisseaux
Date: Fri Dec 30 09:23:24 2016
New Revision: 1776531

URL: http://svn.apache.org/viewvc?rev=1776531&view=rev
Log:
Partial consolidation of Exception.getMessage() versus getLocalizedMessage() policy.
This commit contains opportunistic javadoc formatting in classes that have been reviewed as part of this policy consolidation.

Added:
    sis/branches/JDK8/core/sis-utility/src/main/java/org/apache/sis/util/LocalizedException.java
      - copied, changed from r1776530, sis/branches/JDK8/core/sis-utility/src/main/java/org/apache/sis/internal/util/LocalizedException.java
Removed:
    sis/branches/JDK8/core/sis-utility/src/main/java/org/apache/sis/internal/util/LocalizedException.java
Modified:
    sis/branches/JDK8/application/sis-console/src/main/java/org/apache/sis/console/Command.java
    sis/branches/JDK8/core/sis-feature/src/main/java/org/apache/sis/feature/AbstractAttribute.java
    sis/branches/JDK8/core/sis-feature/src/main/java/org/apache/sis/feature/DefaultAttributeType.java
    sis/branches/JDK8/core/sis-feature/src/main/java/org/apache/sis/feature/InvalidFeatureException.java
    sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/internal/metadata/ServicesForUtility.java
    sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/internal/metadata/sql/Initializer.java
    sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/io/wkt/AbstractParser.java
    sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/io/wkt/Element.java
    sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/io/wkt/Warnings.java
    sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/InvalidMetadataException.java
    sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/internal/referencing/ServicesForMetadata.java
    sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/parameter/Verifier.java
    sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/AuthorityFactories.java
    sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/factory/GeodeticObjectFactory.java
    sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/factory/IdentifiedObjectFinder.java
    sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/factory/IdentifiedObjectSet.java
    sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/factory/InvalidGeodeticParameterException.java
    sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/operation/DefaultCoordinateOperationFactory.java
    sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/operation/projection/ProjectionException.java
    sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/operation/transform/DefaultMathTransformFactory.java
    sis/branches/JDK8/core/sis-referencing/src/test/java/org/apache/sis/referencing/report/CoordinateReferenceSystems.java
    sis/branches/JDK8/core/sis-utility/src/main/java/org/apache/sis/internal/jaxb/gml/TM_Primitive.java
    sis/branches/JDK8/core/sis-utility/src/main/java/org/apache/sis/internal/util/LocalizedParseException.java
    sis/branches/JDK8/core/sis-utility/src/main/java/org/apache/sis/measure/Angle.java
    sis/branches/JDK8/core/sis-utility/src/main/java/org/apache/sis/setup/About.java
    sis/branches/JDK8/core/sis-utility/src/main/java/org/apache/sis/util/Exceptions.java
    sis/branches/JDK8/core/sis-utility/src/main/java/org/apache/sis/util/collection/TreeTableFormat.java
    sis/branches/JDK8/core/sis-utility/src/main/java/org/apache/sis/util/iso/AbstractInternationalString.java
    sis/branches/JDK8/core/sis-utility/src/main/java/org/apache/sis/util/resources/Errors.java
    sis/branches/JDK8/core/sis-utility/src/main/java/org/apache/sis/util/resources/Errors.properties
    sis/branches/JDK8/core/sis-utility/src/main/java/org/apache/sis/util/resources/Errors_fr.properties
    sis/branches/JDK8/core/sis-utility/src/main/java/org/apache/sis/util/resources/IndexedResourceBundle.java
    sis/branches/JDK8/core/sis-utility/src/main/java/org/apache/sis/util/resources/Loader.java
    sis/branches/JDK8/storage/sis-earth-observation/src/main/java/org/apache/sis/storage/earthobservation/LandsatStore.java
    sis/branches/JDK8/storage/sis-geotiff/src/main/java/org/apache/sis/storage/geotiff/GeoTiffStore.java
    sis/branches/JDK8/storage/sis-geotiff/src/main/java/org/apache/sis/storage/geotiff/Reader.java
    sis/branches/JDK8/storage/sis-netcdf/src/main/java/org/apache/sis/internal/netcdf/impl/ChannelDecoder.java
    sis/branches/JDK8/storage/sis-netcdf/src/main/java/org/apache/sis/storage/netcdf/NetcdfStore.java
    sis/branches/JDK8/storage/sis-storage/src/main/java/org/apache/sis/internal/storage/IOUtilities.java
    sis/branches/JDK8/storage/sis-storage/src/main/java/org/apache/sis/internal/storage/Resources.java
    sis/branches/JDK8/storage/sis-storage/src/main/java/org/apache/sis/internal/storage/Resources.properties
    sis/branches/JDK8/storage/sis-storage/src/main/java/org/apache/sis/internal/storage/Resources_fr.properties
    sis/branches/JDK8/storage/sis-storage/src/main/java/org/apache/sis/internal/storage/csv/Store.java
    sis/branches/JDK8/storage/sis-storage/src/main/java/org/apache/sis/internal/storage/wkt/Store.java
    sis/branches/JDK8/storage/sis-storage/src/main/java/org/apache/sis/storage/DataStore.java
    sis/branches/JDK8/storage/sis-storage/src/main/java/org/apache/sis/storage/DataStoreClosedException.java
    sis/branches/JDK8/storage/sis-storage/src/main/java/org/apache/sis/storage/DataStoreContentException.java
    sis/branches/JDK8/storage/sis-storage/src/main/java/org/apache/sis/storage/DataStoreException.java
    sis/branches/JDK8/storage/sis-storage/src/main/java/org/apache/sis/storage/DataStoreReferencingException.java
    sis/branches/JDK8/storage/sis-storage/src/main/java/org/apache/sis/storage/DataStoreRegistry.java
    sis/branches/JDK8/storage/sis-storage/src/main/java/org/apache/sis/storage/UnsupportedStorageException.java
    sis/branches/JDK8/storage/sis-xmlstore/src/main/java/org/apache/sis/internal/gpx/Store.java
    sis/branches/JDK8/storage/sis-xmlstore/src/main/java/org/apache/sis/internal/xml/StaxDataStore.java
    sis/branches/JDK8/storage/sis-xmlstore/src/main/java/org/apache/sis/internal/xml/StaxStreamReader.java

Modified: sis/branches/JDK8/application/sis-console/src/main/java/org/apache/sis/console/Command.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/application/sis-console/src/main/java/org/apache/sis/console/Command.java?rev=1776531&r1=1776530&r2=1776531&view=diff
==============================================================================
--- sis/branches/JDK8/application/sis-console/src/main/java/org/apache/sis/console/Command.java [UTF-8] (original)
+++ sis/branches/JDK8/application/sis-console/src/main/java/org/apache/sis/console/Command.java [UTF-8] Fri Dec 30 09:23:24 2016
@@ -128,9 +128,9 @@ public final class Command {
      * Creates a new command for the given arguments. The first value in the given array which is
      * not an option is taken as the command name. All other values are options or filenames.
      *
-     * @param  args The command-line arguments.
-     * @throws InvalidCommandException If an invalid command has been given.
-     * @throws InvalidOptionException If the given arguments contain an invalid option.
+     * @param  args  the command-line arguments.
+     * @throws InvalidCommandException if an invalid command has been given.
+     * @throws InvalidOptionException if the given arguments contain an invalid option.
      */
     protected Command(final String[] args) throws InvalidCommandException, InvalidOptionException {
         int commandIndex = -1;
@@ -176,7 +176,7 @@ public final class Command {
      * by the {@link #exitCodeFor(Throwable)} method.
      *
      * @return 0 on success, or an exit code if the command failed for a reason other than a Java exception.
-     * @throws Exception If an error occurred during the command execution. This is typically, but not limited, to
+     * @throws Exception if an error occurred during the command execution. This is typically, but not limited, to
      *         {@link IOException}, {@link SQLException}, {@link DataStoreException} or {@link TransformException}.
      */
     public int run() throws Exception {
@@ -199,8 +199,8 @@ public final class Command {
      * {@linkplain Throwable#getCause() causes} until an exception matching a {@code *_EXIT_CODE}
      * constant is found.
      *
-     * @param  cause The exception for which to get the exit code.
-     * @return The exit code as one of the {@code *_EXIT_CODE} constant, or {@link #OTHER_ERROR_EXIT_CODE} if unknown.
+     * @param  cause  the exception for which to get the exit code.
+     * @return the exit code as one of the {@code *_EXIT_CODE} constant, or {@link #OTHER_ERROR_EXIT_CODE} if unknown.
      */
     public static int exitCodeFor(Throwable cause) {
         while (cause != null) {
@@ -217,7 +217,7 @@ public final class Command {
      * Prints the message of the given exception. This method is invoked only when the error occurred before
      * the {@link CommandRunner} has been built, otherwise the {@link CommandRunner#err} printer shall be used.
      *
-     * @param args The command line arguments, used only for detecting if the {@code --debug} option was present.
+     * @param  args  the command line arguments, used only for detecting if the {@code --debug} option was present.
      */
     private static void error(final String[] args, final Exception e) {
         final boolean debug = ArraysExt.containsIgnoreCase(args, Option.PREFIX + "debug");
@@ -245,7 +245,7 @@ public final class Command {
     /**
      * Prints the information to the standard output stream.
      *
-     * @param args Command-line options.
+     * @param  args  command-line options.
      */
     public static void main(final String[] args) {
         /*

Modified: sis/branches/JDK8/core/sis-feature/src/main/java/org/apache/sis/feature/AbstractAttribute.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-feature/src/main/java/org/apache/sis/feature/AbstractAttribute.java?rev=1776531&r1=1776530&r2=1776531&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-feature/src/main/java/org/apache/sis/feature/AbstractAttribute.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-feature/src/main/java/org/apache/sis/feature/AbstractAttribute.java [UTF-8] Fri Dec 30 09:23:24 2016
@@ -181,8 +181,9 @@ public abstract class AbstractAttribute<
                 characteristics = newCharacteristicsMap();
                 characteristics.values().addAll(Arrays.asList(characterizedBy));
             }
-        } catch (RuntimeException e) { // At least ClassCastException, NullPointerException, IllegalArgumentException and IllegalStateException.
-            throw (IOException) new InvalidObjectException(e.getMessage()).initCause(e);
+        } catch (RuntimeException e) {
+            // At least ClassCastException, NullPointerException, IllegalArgumentException and IllegalStateException.
+            throw (IOException) new InvalidObjectException(e.getLocalizedMessage()).initCause(e);
         }
     }
 

Modified: sis/branches/JDK8/core/sis-feature/src/main/java/org/apache/sis/feature/DefaultAttributeType.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-feature/src/main/java/org/apache/sis/feature/DefaultAttributeType.java?rev=1776531&r1=1776530&r2=1776531&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-feature/src/main/java/org/apache/sis/feature/DefaultAttributeType.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-feature/src/main/java/org/apache/sis/feature/DefaultAttributeType.java [UTF-8] Fri Dec 30 09:23:24 2016
@@ -224,8 +224,9 @@ public class DefaultAttributeType<V> ext
             if (characterizedBy != null) {
                 characteristics = CharacteristicTypeMap.create(this, characterizedBy);
             }
-        } catch (RuntimeException e) { // At least ClassCastException, NullPointerException and IllegalArgumentException.
-            throw (IOException) new InvalidObjectException(e.getMessage()).initCause(e);
+        } catch (RuntimeException e) {
+            // At least ClassCastException, NullPointerException and IllegalArgumentException.
+            throw (IOException) new InvalidObjectException(e.getLocalizedMessage()).initCause(e);
         }
     }
 

Modified: sis/branches/JDK8/core/sis-feature/src/main/java/org/apache/sis/feature/InvalidFeatureException.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-feature/src/main/java/org/apache/sis/feature/InvalidFeatureException.java?rev=1776531&r1=1776530&r2=1776531&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-feature/src/main/java/org/apache/sis/feature/InvalidFeatureException.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-feature/src/main/java/org/apache/sis/feature/InvalidFeatureException.java [UTF-8] Fri Dec 30 09:23:24 2016
@@ -16,9 +16,8 @@
  */
 package org.apache.sis.feature;
 
-import java.util.Locale;
 import org.opengis.util.InternationalString;
-import org.apache.sis.internal.util.LocalizedException;
+import org.apache.sis.util.LocalizedException;
 
 // Branch-dependent imports
 import org.opengis.feature.Feature;
@@ -34,7 +33,7 @@ import org.opengis.feature.InvalidProper
  *
  * @author  Martin Desruisseaux (Geomatys)
  * @since   0.7
- * @version 0.7
+ * @version 0.8
  * @module
  *
  * @see Features#validate(Feature)
@@ -61,14 +60,12 @@ final class InvalidFeatureException exte
     }
 
     /**
-     * Returns the message localized in the given language, or in a default language if the requested
-     * localization is not available.
+     * Return the message in various locales.
      *
-     * @param  locale  the desired language.
-     * @return the message in the given locale, or in a default locale if the requested localization is not available.
+     * @return the exception message.
      */
     @Override
-    public String getLocalizedMessage(final Locale locale) {
-        return message.toString(locale);
+    public InternationalString getInternationalMessage() {
+        return message;
     }
 }

Modified: sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/internal/metadata/ServicesForUtility.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/internal/metadata/ServicesForUtility.java?rev=1776531&r1=1776530&r2=1776531&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/internal/metadata/ServicesForUtility.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/internal/metadata/ServicesForUtility.java [UTF-8] Fri Dec 30 09:23:24 2016
@@ -61,8 +61,8 @@ public final class ServicesForUtility ex
     /**
      * Returns the constant defined in the {@link Citations} class for the given name.
      *
-     * @param  name The name of one of the citation constants defined in the {@code Citations} class.
-     * @return The requested citation, or {@code null} if there is no constant for the given name.
+     * @param  name  the name of one of the citation constants defined in the {@code Citations} class.
+     * @return the requested citation, or {@code null} if there is no constant for the given name.
      */
     @Override
     public CitationConstant getCitationConstant(final String name) {
@@ -73,8 +73,8 @@ public final class ServicesForUtility ex
     /**
      * Returns the build-in citation for the given primary key, or {@code null}.
      *
-     * @param  key The primary key of the desired citation.
-     * @return The requested citation, or {@code null} if unknown.
+     * @param  key  the primary key of the desired citation.
+     * @return the requested citation, or {@code null} if unknown.
      *
      * @todo The content is hard-coded for now. But the plan for a future version is to fetch richer information
      *       from a database, including for example the responsible party and the URL. However that method would
@@ -219,9 +219,9 @@ public final class ServicesForUtility ex
      * Returns information about the Apache SIS configuration.
      * See super-class for a list of keys.
      *
-     * @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.
      */
     @Override
     public String getInformation(final String key, final Locale locale) {

Modified: sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/internal/metadata/sql/Initializer.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/internal/metadata/sql/Initializer.java?rev=1776531&r1=1776530&r2=1776531&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/internal/metadata/sql/Initializer.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/internal/metadata/sql/Initializer.java [UTF-8] Fri Dec 30 09:23:24 2016
@@ -432,7 +432,7 @@ public abstract class Initializer {
             try {
                 ds.getConnection().close();     // Does the actual shutdown.
             } catch (SQLException e) {          // This is the expected exception.
-                final LogRecord record = new LogRecord(Level.FINE, e.getLocalizedMessage());
+                final LogRecord record = new LogRecord(Level.FINE, e.getMessage());
                 if (!isSuccessfulShutdown(e)) {
                     record.setLevel(Level.WARNING);
                     record.setThrown(e);

Modified: sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/io/wkt/AbstractParser.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/io/wkt/AbstractParser.java?rev=1776531&r1=1776530&r2=1776531&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/io/wkt/AbstractParser.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/io/wkt/AbstractParser.java [UTF-8] Fri Dec 30 09:23:24 2016
@@ -218,7 +218,7 @@ abstract class AbstractParser implements
             if (cause instanceof FactoryException) {
                 throw (FactoryException) cause;
             }
-            throw new FactoryException(exception.getMessage(), exception);
+            throw new FactoryException(exception.getLocalizedMessage(), exception);
         }
         final Warnings warnings = getAndClearWarnings(value);
         if (warnings != null) {

Modified: sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/io/wkt/Element.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/io/wkt/Element.java?rev=1776531&r1=1776530&r2=1776531&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/io/wkt/Element.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/io/wkt/Element.java [UTF-8] Fri Dec 30 09:23:24 2016
@@ -113,8 +113,8 @@ final class Element implements Serializa
     /**
      * Constructs a root element.
      *
-     * @param name An arbitrary name for the root element.
-     * @param singleton The only children for this root.
+     * @param name       an arbitrary name for the root element.
+     * @param singleton  the only children for this root.
      */
     Element(final String name, final Element singleton) {
         keyword = name;
@@ -158,10 +158,10 @@ final class Element implements Serializa
      *     be used for sharing unique instance of each value if possible.</li>
      * </ul>
      *
-     * @param text         The text to parse.
-     * @param position     On input, the position where to start parsing from.
-     *                     On output, the first character after the separator.
-     * @param sharedValues If parsing a fragment, a map with the values found in other elements. Otherwise {@code null}.
+     * @param text          the text to parse.
+     * @param position      on input, the position where to start parsing from.
+     *                      On output, the first character after the separator.
+     * @param sharedValues  if parsing a fragment, a map with the values found in other elements. Otherwise {@code null}.
      */
     Element(final AbstractParser parser, final String text, final ParsePosition position,
             final Map<Object,Object> sharedValues) throws ParseException
@@ -377,8 +377,8 @@ final class Element implements Serializa
      * <code>"Error in &lt;{@link #keyword}&gt;"</code> will be prepend to the message.
      * The error index will be the starting index of this {@code Element}.
      *
-     * @param  cause The cause of the failure, or {@code null} if none.
-     * @return The exception to be thrown.
+     * @param  cause  the cause of the failure, or {@code null} if none.
+     * @return the exception to be thrown.
      */
     final ParseException parseFailed(final Exception cause) {
         return (ParseException) new LocalizedParseException(locale, Errors.Keys.ErrorIn_2,
@@ -391,9 +391,9 @@ final class Element implements Serializa
      * Properties {@link ParsePosition#getIndex()} and {@link ParsePosition#getErrorIndex()}
      * must be accurate before this method is invoked.
      *
-     * @param  text The unparsable string.
-     * @param  position The position in the string.
-     * @return An exception with a formatted error message.
+     * @param  text      the unparsable string.
+     * @param  position  the position in the string.
+     * @return an exception with a formatted error message.
      */
     private ParseException unparsableString(final String text, final ParsePosition position) {
         final short errorKey;
@@ -414,9 +414,9 @@ final class Element implements Serializa
     /**
      * Returns an exception saying that a character is missing.
      *
-     * @param c          The missing character.
-     * @param errorIndex The error position.
-     * @param position   The position to update with the error index.
+     * @param c           the missing character.
+     * @param errorIndex  the error position.
+     * @param position    the position to update with the error index.
      */
     private ParseException missingCharacter(final int c, final int errorIndex, final ParsePosition position) {
         position.setIndex(offset);
@@ -429,7 +429,7 @@ final class Element implements Serializa
     /**
      * Returns an exception saying that a sub-element is missing.
      *
-     * @param key The name of the missing sub-element.
+     * @param key  the name of the missing sub-element.
      */
     final ParseException missingComponent(final String key) {
         int error = offset;
@@ -443,8 +443,8 @@ final class Element implements Serializa
     /**
      * Returns a {@link ParseException} for a child keyword which is unknown.
      *
-     * @param  expected Keyword of a typical element. Used only if this element contains no child element.
-     * @return The exception to be thrown.
+     * @param  expected  keyword of a typical element. Used only if this element contains no child element.
+     * @return the exception to be thrown.
      */
     final ParseException missingOrUnknownComponent(final String expected) {
         String name = null;
@@ -476,8 +476,8 @@ final class Element implements Serializa
      * But it would be a {@code CSFactory} contract violation, so the user would get a {@link NullPointerException}
      * later. For making easier to trace the cause, we throw here an exception with a similar error message.</p>
      *
-     * @param  cs The illegal coordinate system.
-     * @return The exception to be thrown.
+     * @param  cs  the illegal coordinate system.
+     * @return the exception to be thrown.
      */
     final ParseException illegalCS(final CoordinateSystem cs) {
         final short key;
@@ -504,7 +504,7 @@ final class Element implements Serializa
     /**
      * Returns the next value (not a child element) without removing it.
      *
-     * @return The next value, or {@code null} if none.
+     * @return the next value, or {@code null} if none.
      */
     public Object peekValue() {
         final Iterator<Object> iterator = list.iterator();
@@ -520,8 +520,8 @@ final class Element implements Serializa
     /**
      * Removes the next {@link Date} from the list and returns it.
      *
-     * @param  key The parameter name. Used for formatting an error message if no date is found.
-     * @return The next {@link Date} on the list.
+     * @param  key  the parameter name. Used for formatting an error message if no date is found.
+     * @return the next {@link Date} on the list.
      * @throws ParseException if no more date is available.
      */
     public Date pullDate(final String key) throws ParseException {
@@ -539,8 +539,8 @@ final class Element implements Serializa
     /**
      * Removes the next {@link Number} from the list and returns it.
      *
-     * @param  key The parameter name. Used for formatting an error message if no number is found.
-     * @return The next {@link Number} on the list as a {@code double}.
+     * @param  key  the parameter name. Used for formatting an error message if no number is found.
+     * @return the next {@link Number} on the list as a {@code double}.
      * @throws ParseException if no more number is available.
      */
     public double pullDouble(final String key) throws ParseException {
@@ -558,8 +558,8 @@ final class Element implements Serializa
     /**
      * Removes the next {@link Number} from the list and returns it as an integer.
      *
-     * @param  key The parameter name. Used for formatting an error message if no number is found.
-     * @return The next {@link Number} on the list as an {@code int}.
+     * @param  key  the parameter name. Used for formatting an error message if no number is found.
+     * @return the next {@link Number} on the list as an {@code int}.
      * @throws ParseException if no more number is available, or the number is not an integer.
      */
     public int pullInteger(final String key) throws ParseException {
@@ -582,8 +582,8 @@ final class Element implements Serializa
     /**
      * Removes the next {@link Boolean} from the list and returns it.
      *
-     * @param  key The parameter name. Used for formatting an error message if no boolean is found.
-     * @return The next {@link Boolean} on the list as a {@code boolean}.
+     * @param  key  the parameter name. Used for formatting an error message if no boolean is found.
+     * @return the next {@link Boolean} on the list as a {@code boolean}.
      * @throws ParseException if no more boolean is available.
      */
     public boolean pullBoolean(final String key) throws ParseException {
@@ -601,8 +601,8 @@ final class Element implements Serializa
     /**
      * Removes the next {@link String} from the list and returns it.
      *
-     * @param  key The parameter name. Used for formatting an error message if no number is found.
-     * @return The next {@link String} on the list.
+     * @param  key  the parameter name. Used for formatting an error message if no number is found.
+     * @return the next {@link String} on the list.
      * @throws ParseException if no more string is available.
      */
     public String pullString(final String key) throws ParseException {
@@ -620,8 +620,8 @@ final class Element implements Serializa
     /**
      * Removes the next {@link Object} from the list and returns it.
      *
-     * @param  key The parameter name. Used for formatting an error message if no number is found.
-     * @return The next {@link Object} on the list (never {@code null}).
+     * @param  key  the parameter name. Used for formatting an error message if no number is found.
+     * @return the next {@link Object} on the list (never {@code null}).
      * @throws ParseException if no more object is available.
      */
     public Object pullObject(final String key) throws ParseException {
@@ -649,9 +649,9 @@ final class Element implements Serializa
      *       does not match.</li>
      * </ul>
      *
-     * @param  mode {@link AbstractParser#FIRST}, {@link AbstractParser#OPTIONAL} or {@link AbstractParser#MANDATORY}.
-     * @param  keys The element names (e.g. {@code "PrimeMeridian"}).
-     * @return The next {@link Element} of the given names found on the list, or {@code null} if none.
+     * @param  mode  {@link AbstractParser#FIRST}, {@link AbstractParser#OPTIONAL} or {@link AbstractParser#MANDATORY}.
+     * @param  keys  the element names (e.g. {@code "PrimeMeridian"}).
+     * @return the next {@link Element} of the given names found on the list, or {@code null} if none.
      * @throws ParseException if {@code mode} is {@code MANDATORY} and no element of the given names was found.
      */
     public Element pullElement(final int mode, final String... keys) throws ParseException {
@@ -684,8 +684,8 @@ final class Element implements Serializa
      * Removes and returns the next {@link Element} with no bracket.
      * The key is used only for only for formatting an error message.
      *
-     * @param  key The parameter name. Used only for formatting an error message.
-     * @return The next {@link Element} in the list, with no bracket.
+     * @param  key  the parameter name. Used only for formatting an error message.
+     * @return the next {@link Element} in the list, with no bracket.
      * @throws ParseException if no more void element is available.
      */
     public Element pullVoidElement(final String key) throws ParseException {
@@ -706,8 +706,8 @@ final class Element implements Serializa
     /**
      * Removes the next object of the given type from the list and returns it, if presents.
      *
-     * @param  type The object type.
-     * @return The next object on the list, or {@code null} if none.
+     * @param  type  the object type.
+     * @return the next object on the list, or {@code null} if none.
      */
     @SuppressWarnings("unchecked")
     public <T> T pullOptional(final Class<T> type) {
@@ -749,8 +749,8 @@ final class Element implements Serializa
      *       This list is used for helping the users to locate the ignored elements.</li>
      * </ul>
      *
-     * @param  ignoredElements The collection where to declare ignored elements.
-     * @throws ParseException If the list still contains some unprocessed values.
+     * @param  ignoredElements  the collection where to declare ignored elements.
+     * @throws ParseException if the list still contains some unprocessed values.
      */
     final void close(final Map<String, List<String>> ignoredElements) throws ParseException {
         if (list != null) {
@@ -780,8 +780,8 @@ final class Element implements Serializa
     /**
      * Implementation of {@link #toString()} to be invoked recursively.
      *
-     * @param buffer Where to format.
-     * @param margin Number of space to put in the left margin.
+     * @param  buffer  where to format.
+     * @param  margin  number of spaces to put in the left margin.
      */
     @Debug
     private void format(final StringBuilder buffer, int margin, final String lineSeparator) {

Modified: sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/io/wkt/Warnings.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/io/wkt/Warnings.java?rev=1776531&r1=1776530&r2=1776531&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/io/wkt/Warnings.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/io/wkt/Warnings.java [UTF-8] Fri Dec 30 09:23:24 2016
@@ -147,9 +147,9 @@ public final class Warnings implements L
     /**
      * Creates a new object for declaring warnings.
      *
-     * @param locale The locale for reporting warning messages.
-     * @param isParsing {@code false} if formatting, or {@code true} if parsing.
-     * @param ignoredElements The {@link Parser#ignoredElements} map, or an empty map (can not be null).
+     * @param locale           the locale for reporting warning messages.
+     * @param isParsing        {@code false} if formatting, or {@code true} if parsing.
+     * @param ignoredElements  the {@link Parser#ignoredElements} map, or an empty map (can not be null).
      */
     Warnings(final Locale locale, final boolean isParsing, final Map<String, List<String>> ignoredElements) {
         this.errorLocale     = locale;
@@ -176,11 +176,11 @@ public final class Warnings implements L
     /**
      * Adds a warning. At least one of {@code message} or {@code cause} shall be non-null.
      *
-     * @param message The message, or {@code null}.
-     * @param cause   The exception that caused the warning, or {@code null}
-     * @param source  The location of the exception, or {@code null}. If non-null, then {@code source[0]} shall be
-     *                the keyword of the WKT element where the exception occurred, and {@code source[1]} the keyword
-     *                of the parent of {@code source[0]}.
+     * @param message  the message, or {@code null}.
+     * @param cause    the exception that caused the warning, or {@code null}
+     * @param source   the location of the exception, or {@code null}. If non-null, then {@code source[0]} shall be
+     *                 the keyword of the WKT element where the exception occurred, and {@code source[1]} the keyword
+     *                 of the parent of {@code source[0]}.
      */
     final void add(final InternationalString message, final Exception cause, final String[] source) {
         assert (message != null) || (cause != null);
@@ -213,7 +213,7 @@ public final class Warnings implements L
      * This is not necessarily the same locale than the one used for parsing and formatting dates and numbers
      * in the WKT.
      *
-     * @return The locale or warning messages are reported.
+     * @return the locale or warning messages are reported.
      */
     @Override
     public Locale getLocale() {
@@ -226,7 +226,7 @@ public final class Warnings implements L
      * then this method returns the value of {@code IdentifiedObject.getName().getCode()}.
      * Otherwise this method returns a simple class name.
      *
-     * @return The name of the root element, or {@code null} if unknown.
+     * @return the name of the root element, or {@code null} if unknown.
      */
     public String getRootElement() {
         return root;
@@ -235,7 +235,7 @@ public final class Warnings implements L
     /**
      * Returns the number of warning messages.
      *
-     * @return The number of warning messages.
+     * @return the number of warning messages.
      */
     public final int getNumMessages() {
         return (messages != null) ? messages.size() / 2 : 0;
@@ -245,7 +245,7 @@ public final class Warnings implements L
      * Returns a warning message.
      *
      * @param  index 0 for the first warning, 1 for the second warning, <i>etc.</i> until {@link #getNumMessages()} - 1.
-     * @return The <var>i</var>-th warning message.
+     * @return the <var>i</var>-th warning message.
      */
     public String getMessage(int index) {
         ArgumentChecks.ensureValidIndex(getNumMessages(), index);
@@ -255,7 +255,7 @@ public final class Warnings implements L
             return i18n.toString(errorLocale);
         } else {
             final Exception cause = (Exception) messages.get(index + 1);
-            final String[] sources = exceptionSources.get(cause);   // See comment in 'toString(Locale)'.
+            final String[] sources = exceptionSources.get(cause);           // See comment in 'toString(Locale)'.
             if (sources != null) {
                 return Errors.getResources(errorLocale).getString(Errors.Keys.UnparsableStringInElement_2, sources);
             } else {
@@ -267,8 +267,8 @@ public final class Warnings implements L
     /**
      * Returns the exception which was the cause of the message at the given index, or {@code null} if none.
      *
-     * @param  index The value given to {@link #getMessage(int)}.
-     * @return The exception which was the cause of the warning message, or {@code null} if none.
+     * @param  index  the value given to {@link #getMessage(int)}.
+     * @return the exception which was the cause of the warning message, or {@code null} if none.
      */
     public Exception getException(final int index) {
         ArgumentChecks.ensureValidIndex(getNumMessages(), index);
@@ -279,7 +279,7 @@ public final class Warnings implements L
      * Returns the non-fatal exceptions that occurred during the parsing or formatting.
      * If no exception occurred, returns an empty set.
      *
-     * @return The non-fatal exceptions that occurred.
+     * @return the non-fatal exceptions that occurred.
      */
     public Set<Exception> getExceptions() {
         return (exceptionSources != null) ? exceptionSources.keySet() : Collections.emptySet();
@@ -295,8 +295,8 @@ public final class Warnings implements L
      * but with path elements stored in reverse order.
      * </div>
      *
-     * @param  ex The exception for which to get the source.
-     * @return The keywords of the WKT element where the given exception occurred, or {@code null} if unknown.
+     * @param  ex  the exception for which to get the source.
+     * @return the keywords of the WKT element where the given exception occurred, or {@code null} if unknown.
      */
     public String[] getExceptionSource(final Exception ex) {
         return (exceptionSources != null) ? exceptionSources.get(ex) : null;
@@ -305,7 +305,7 @@ public final class Warnings implements L
     /**
      * Returns the keywords of all unknown elements found during the WKT parsing.
      *
-     * @return The keywords of unknown WKT elements, or an empty set if none.
+     * @return the keywords of unknown WKT elements, or an empty set if none.
      */
     public Set<String> getUnknownElements() {
         return ignoredElements.keySet();
@@ -318,8 +318,8 @@ public final class Warnings implements L
      *
      * <p>The returned collection elements are in no particular order.</p>
      *
-     * @param element The keyword of the unknown element.
-     * @return The keywords of elements where the given unknown element was found.
+     * @param  element  the keyword of the unknown element.
+     * @return the keywords of elements where the given unknown element was found.
      */
     public Collection<String> getUnknownElementLocations(final String element) {
         return ignoredElements.get(element);
@@ -330,7 +330,7 @@ public final class Warnings implements L
      * The locale used by this method is given by {@link #getLocale()}.
      * This is usually the locale given to the {@link WKTFormat} constructor.
      *
-     * @return A string representation of the warning messages.
+     * @return a string representation of the warning messages.
      */
     @Override
     public String toString() {
@@ -341,8 +341,8 @@ public final class Warnings implements L
      * Returns a string representation of the warning messages in the given locale.
      * This method formats the warnings in a bullet list.
      *
-     * @param  locale The locale to use for formatting warning messages.
-     * @return A string representation of the warning messages.
+     * @param  locale  the locale to use for formatting warning messages.
+     * @return a string representation of the warning messages.
      */
     public String toString(final Locale locale) {
         final StringBuilder buffer = new StringBuilder(250);

Modified: sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/InvalidMetadataException.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/InvalidMetadataException.java?rev=1776531&r1=1776530&r2=1776531&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/InvalidMetadataException.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/InvalidMetadataException.java [UTF-8] Fri Dec 30 09:23:24 2016
@@ -49,7 +49,7 @@ public class InvalidMetadataException ex
      * @since 0.8
      */
     public InvalidMetadataException(final Throwable cause) {
-        super(cause.getLocalizedMessage(), cause);
+        super(cause);
     }
 
     /**

Modified: sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/internal/referencing/ServicesForMetadata.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/internal/referencing/ServicesForMetadata.java?rev=1776531&r1=1776530&r2=1776531&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/internal/referencing/ServicesForMetadata.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/internal/referencing/ServicesForMetadata.java [UTF-8] Fri Dec 30 09:23:24 2016
@@ -139,11 +139,11 @@ public final class ServicesForMetadata e
      * the horizontal extent. If the {@code crs} argument is null, then it is caller's responsibility
      * to ensure that the given envelope is two-dimensional.
      *
-     * @param  envelope The source envelope.
-     * @param  target The target bounding box.
-     * @param  crs The envelope CRS, or {@code null} if unknown.
-     * @param  normalizedCRS The horizontal component of the given CRS, or null if the {@code crs} argument is null.
-     * @throws TransformException If the given envelope can not be transformed.
+     * @param  envelope       the source envelope.
+     * @param  target         the target bounding box.
+     * @param  crs            the envelope CRS, or {@code null} if unknown.
+     * @param  normalizedCRS  the horizontal component of the given CRS, or null if the {@code crs} argument is null.
+     * @throws TransformException if the given envelope can not be transformed.
      */
     private void setGeographicExtent(Envelope envelope, final DefaultGeographicBoundingBox target,
             final CoordinateReferenceSystem crs, final GeographicCRS normalizedCRS) throws TransformException
@@ -187,10 +187,10 @@ public final class ServicesForMetadata e
      * If the {@code crs} argument is null, then it is caller's responsibility to ensure
      * that the given envelope is one-dimensional.
      *
-     * @param  envelope    The source envelope.
-     * @param  target      The target vertical extent.
-     * @param  crs         The envelope CRS, or {@code null} if unknown.
-     * @param  verticalCRS The vertical component of the given CRS, or null if the {@code crs} argument is null.
+     * @param  envelope     the source envelope.
+     * @param  target       the target vertical extent.
+     * @param  crs          the envelope CRS, or {@code null} if unknown.
+     * @param  verticalCRS  the vertical component of the given CRS, or null if the {@code crs} argument is null.
      */
     private static void setVerticalExtent(final Envelope envelope, final DefaultVerticalExtent target,
             final CoordinateReferenceSystem crs, final VerticalCRS verticalCRS)
@@ -210,11 +210,11 @@ public final class ServicesForMetadata e
     /**
      * Implementation of the public {@code setBounds} methods for the temporal extent.
      *
-     * @param  envelope    The source envelope.
-     * @param  target      The target temporal extent.
-     * @param  crs         The envelope CRS (mandatory, can not be {@code null}).
-     * @param  verticalCRS The temporal component of the given CRS (mandatory).
-     * @throws UnsupportedOperationException If no implementation of {@code TemporalFactory} has been found
+     * @param  envelope     the source envelope.
+     * @param  target       the target temporal extent.
+     * @param  crs          the envelope CRS (mandatory, can not be {@code null}).
+     * @param  verticalCRS  the temporal component of the given CRS (mandatory).
+     * @throws UnsupportedOperationException if no implementation of {@code TemporalFactory} has been found
      *         on the classpath.
      */
     private static void setTemporalExtent(final Envelope envelope, final DefaultTemporalExtent target,
@@ -233,9 +233,9 @@ public final class ServicesForMetadata e
      * to a geographic CRS (without datum shift if possible). Otherwise, the envelope is assumed already
      * in a geographic CRS using (<var>longitude</var>, <var>latitude</var>) axis order.
      *
-     * @param  envelope The source envelope.
-     * @param  target The target bounding box where to store envelope information.
-     * @throws TransformException If the given envelope can not be transformed.
+     * @param  envelope  the source envelope.
+     * @param  target    the target bounding box where to store envelope information.
+     * @throws TransformException if the given envelope can not be transformed.
      */
     @Override
     public void setBounds(Envelope envelope, final DefaultGeographicBoundingBox target) throws TransformException {
@@ -255,9 +255,9 @@ public final class ServicesForMetadata e
      * Sets a vertical extent with the value inferred from the given envelope.
      * Only the vertical ordinates are extracted; all other ordinates are ignored.
      *
-     * @param  envelope The source envelope.
-     * @param  target The target vertical extent where to store envelope information.
-     * @throws TransformException If no vertical component can be extracted from the given envelope.
+     * @param  envelope  the source envelope.
+     * @param  target    the target vertical extent where to store envelope information.
+     * @throws TransformException if no vertical component can be extracted from the given envelope.
      */
     @Override
     public void setBounds(final Envelope envelope, final DefaultVerticalExtent target) throws TransformException {
@@ -273,9 +273,9 @@ public final class ServicesForMetadata e
      * Sets a temporal extent with the value inferred from the given envelope.
      * Only the vertical ordinates are extracted; all other ordinates are ignored.
      *
-     * @param  envelope The source envelope.
-     * @param  target The target temporal extent where to store envelope information.
-     * @throws TransformException If no temporal component can be extracted from the given envelope.
+     * @param  envelope  the source envelope.
+     * @param  target    the target temporal extent where to store envelope information.
+     * @throws TransformException if no temporal component can be extracted from the given envelope.
      */
     @Override
     public void setBounds(final Envelope envelope, final DefaultTemporalExtent target) throws TransformException {
@@ -292,9 +292,9 @@ public final class ServicesForMetadata e
      * If the given {@code target} has more geographic or vertical extents than needed (0 or 1), then the
      * extraneous extents are removed.
      *
-     * @param  envelope The source envelope.
-     * @param  target The target spatio-temporal extent where to store envelope information.
-     * @throws TransformException If no temporal component can be extracted from the given envelope.
+     * @param  envelope  the source envelope.
+     * @param  target    the target spatiotemporal extent where to store envelope information.
+     * @throws TransformException if no temporal component can be extracted from the given envelope.
      */
     @Override
     public void setBounds(final Envelope envelope, final DefaultSpatialTemporalExtent target) throws TransformException {
@@ -359,9 +359,9 @@ public final class ServicesForMetadata e
     /**
      * Initializes a horizontal, vertical and temporal extent with the values inferred from the given envelope.
      *
-     * @param  envelope The source envelope.
-     * @param  target The target extent where to store envelope information.
-     * @throws TransformException If a coordinate transformation was required and failed.
+     * @param  envelope  the source envelope.
+     * @param  target    the target extent where to store envelope information.
+     * @throws TransformException if a coordinate transformation was required and failed.
      */
     @Override
     public void addElements(final Envelope envelope, final DefaultExtent target) throws TransformException {
@@ -402,8 +402,8 @@ public final class ServicesForMetadata e
     /**
      * Returns a fully implemented parameter descriptor.
      *
-     * @param  parameter A partially implemented parameter descriptor, or {@code null}.
-     * @return A fully implemented parameter descriptor, or {@code null} if the given argument was null.
+     * @param  parameter  a partially implemented parameter descriptor, or {@code null}.
+     * @return a fully implemented parameter descriptor, or {@code null} if the given argument was null.
      */
     @Override
     public ParameterDescriptor<?> toImplementation(final ParameterDescriptor<?> parameter) {
@@ -413,8 +413,8 @@ public final class ServicesForMetadata e
     /**
      * Converts the given object in a {@code FormattableObject} instance.
      *
-     * @param  object The object to wrap.
-     * @return The given object converted to a {@code FormattableObject} instance.
+     * @param  object  the object to wrap.
+     * @return the given object converted to a {@code FormattableObject} instance.
      */
     @Override
     public FormattableObject toFormattableObject(final IdentifiedObject object) {
@@ -426,9 +426,9 @@ public final class ServicesForMetadata e
      * object is not already an instance of {@code FormattableObject} before to invoke this method. This method
      * returns {@code null} if it can not convert the object.
      *
-     * @param  object The object to wrap.
-     * @param  internal {@code true} if the formatting convention is {@code Convention.INTERNAL}.
-     * @return The given object converted to a {@code FormattableObject} instance, or {@code null}.
+     * @param  object    the object to wrap.
+     * @param  internal  {@code true} if the formatting convention is {@code Convention.INTERNAL}.
+     * @return the given object converted to a {@code FormattableObject} instance, or {@code null}.
      *
      * @since 0.6
      */
@@ -468,7 +468,7 @@ public final class ServicesForMetadata e
     /**
      * Returns a coordinate reference system for heights above the mean seal level.
      *
-     * @return The "Mean Seal Level (MSL) height" coordinate reference system.
+     * @return the "Mean Seal Level (MSL) height" coordinate reference system.
      *
      * @since 0.6
      */
@@ -480,7 +480,7 @@ public final class ServicesForMetadata e
     /**
      * Returns the Greenwich prime meridian.
      *
-     * @return The Greenwich prime meridian.
+     * @return the Greenwich prime meridian.
      *
      * @since 0.6
      */
@@ -492,8 +492,8 @@ public final class ServicesForMetadata e
     /**
      * Returns the coordinate system of a geocentric CRS using axes in the given unit of measurement.
      *
-     * @param  linearUnit The unit of measurement for the geocentric CRS axes.
-     * @return The coordinate system for a geocentric CRS with axes using the given unit of measurement.
+     * @param  linearUnit  the unit of measurement for the geocentric CRS axes.
+     * @return the coordinate system for a geocentric CRS with axes using the given unit of measurement.
      *
      * @since 0.6
      */
@@ -506,8 +506,8 @@ public final class ServicesForMetadata e
      * Converts a geocentric coordinate system from the legacy WKT 1 to the current ISO 19111 standard.
      * This method replaces the (Other, East, North) directions by (Geocentric X, Geocentric Y, Geocentric Z).
      *
-     * @param  cs The geocentric coordinate system to upgrade.
-     * @return The upgraded coordinate system, or {@code cs} if there is no change to apply.
+     * @param  cs  the geocentric coordinate system to upgrade.
+     * @return the upgraded coordinate system, or {@code cs} if there is no change to apply.
      *
      * @since 0.6
      */
@@ -521,9 +521,9 @@ public final class ServicesForMetadata e
      * since that legacy format did not specified any information about the coordinate system in use.
      * This method should not need to be invoked for parsing WKT version 2.
      *
-     * @param  properties The coordinate system name, and optionally other properties.
-     * @param  axes The axes of the unknown coordinate system.
-     * @return An "abstract" coordinate system using the given axes.
+     * @param  properties  the coordinate system name, and optionally other properties.
+     * @param  axes        the axes of the unknown coordinate system.
+     * @return an "abstract" coordinate system using the given axes.
      *
      * @since 0.6
      */
@@ -538,12 +538,12 @@ public final class ServicesForMetadata e
      * Note that this method is needed only for WKT 1 parsing, since WKT provides enough information for using
      * the standard factories.
      *
-     * @param  properties    The properties to be given to the {@code DerivedCRS} and {@code Conversion} objects.
-     * @param  baseCRS       Coordinate reference system to base the derived CRS on.
-     * @param  method        The coordinate operation method (mandatory in all cases).
-     * @param  baseToDerived Transform from positions in the base CRS to positions in this target CRS.
-     * @param  derivedCS     The coordinate system for the derived CRS.
-     * @return The newly created derived CRS, potentially implementing an additional CRS interface.
+     * @param  properties     the properties to be given to the {@code DerivedCRS} and {@code Conversion} objects.
+     * @param  baseCRS        coordinate reference system to base the derived CRS on.
+     * @param  method         the coordinate operation method (mandatory in all cases).
+     * @param  baseToDerived  transform from positions in the base CRS to positions in this target CRS.
+     * @param  derivedCS      the coordinate system for the derived CRS.
+     * @return the newly created derived CRS, potentially implementing an additional CRS interface.
      *
      * @since 0.6
      */
@@ -561,10 +561,10 @@ public final class ServicesForMetadata e
      * Returns an axis direction from a pole along a meridian.
      * The given meridian is usually, but not necessarily, relative to the Greenwich meridian.
      *
-     * @param  baseDirection The base direction, which must be {@link AxisDirection#NORTH} or {@link AxisDirection#SOUTH}.
-     * @param  meridian The meridian in degrees, relative to a unspecified (usually Greenwich) prime meridian.
+     * @param  baseDirection  the base direction, which must be {@link AxisDirection#NORTH} or {@link AxisDirection#SOUTH}.
+     * @param  meridian       the meridian in degrees, relative to a unspecified (usually Greenwich) prime meridian.
      *         Meridians in the East hemisphere are positive and meridians in the West hemisphere are negative.
-     * @return The axis direction along the given meridian.
+     * @return the axis direction along the given meridian.
      *
      * @since 0.6
      */
@@ -576,8 +576,8 @@ public final class ServicesForMetadata e
     /**
      * Creates the {@code TOWGS84} element during parsing of a WKT version 1.
      *
-     * @param  values The 7 Bursa-Wolf parameter values.
-     * @return The {@link BursaWolfParameters}.
+     * @param  values  the 7 Bursa-Wolf parameter values.
+     * @return the {@link BursaWolfParameters}.
      *
      * @since 0.6
      */
@@ -592,13 +592,13 @@ public final class ServicesForMetadata e
      * Creates a single operation from the given properties.
      * This method is provided here because not yet available in GeoAPI interfaces.
      *
-     * @param  properties The properties to be given to the identified object.
-     * @param  sourceCRS  The source CRS.
-     * @param  targetCRS  The target CRS.
-     * @param  interpolationCRS The CRS of additional coordinates needed for the operation, or {@code null} if none.
-     * @param  method     The coordinate operation method (mandatory in all cases).
-     * @param  factory    The factory to use.
-     * @return The coordinate operation created from the given arguments.
+     * @param  properties        the properties to be given to the identified object.
+     * @param  sourceCRS         the source CRS.
+     * @param  targetCRS         the target CRS.
+     * @param  interpolationCRS  the CRS of additional coordinates needed for the operation, or {@code null} if none.
+     * @param  method            the coordinate operation method (mandatory in all cases).
+     * @param  factory           the factory to use.
+     * @return the coordinate operation created from the given arguments.
      * @throws FactoryException if the object creation failed.
      *
      * @since 0.6
@@ -626,11 +626,11 @@ public final class ServicesForMetadata e
      * If the given properties are empty and the {@code mtFactory} is the system default, then this method
      * returns the system default {@code CoordinateOperationFactory} instead of creating a new one.
      *
-     * @param  properties The default properties.
-     * @param  mtFactory  The math transform factory to use.
-     * @param  crsFactory The factory to use if the operation factory needs to create CRS for intermediate steps.
-     * @param  csFactory  The factory to use if the operation factory needs to create CS for intermediate steps.
-     * @return The coordinate operation factory to use.
+     * @param  properties  the default properties.
+     * @param  mtFactory   the math transform factory to use.
+     * @param  crsFactory  the factory to use if the operation factory needs to create CRS for intermediate steps.
+     * @param  csFactory   the factory to use if the operation factory needs to create CS for intermediate steps.
+     * @return the coordinate operation factory to use.
      *
      * @since 0.7
      */
@@ -657,8 +657,8 @@ public final class ServicesForMetadata e
     /**
      * Returns the properties of the given object.
      *
-     * @param  object The object from which to get the properties.
-     * @return The properties of the given object.
+     * @param  object  the object from which to get the properties.
+     * @return the properties of the given object.
      *
      * @since 0.6
      */
@@ -671,8 +671,8 @@ public final class ServicesForMetadata e
      * Returns {@code true} if the {@linkplain AbstractIdentifiedObject#getName() primary name} or an aliases
      * of the given object matches the given name.
      *
-     * @param  object The object for which to check the name or alias.
-     * @param  name The name to compare with the object name or aliases.
+     * @param  object  the object for which to check the name or alias.
+     * @param  name    the name to compare with the object name or aliases.
      * @return {@code true} if the primary name of at least one alias matches the specified {@code name}.
      *
      * @since 0.6
@@ -686,9 +686,9 @@ public final class ServicesForMetadata e
      * Returns information about the Apache SIS configuration.
      * See super-class for a list of keys.
      *
-     * @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.
      */
     @Override
     public String getInformation(final String key, final Locale locale) {

Modified: sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/parameter/Verifier.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/parameter/Verifier.java?rev=1776531&r1=1776530&r2=1776531&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/parameter/Verifier.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/parameter/Verifier.java [UTF-8] Fri Dec 30 09:23:24 2016
@@ -98,9 +98,8 @@ final class Verifier {
      * {@linkplain ParameterDescriptor#getValidValues() set of valid values}.
      * If the value fails any of those tests, then an exception is thrown.
      *
-     * @param  <T> The type of parameter value. The given {@code value} should typically be an instance of this class.
-     *             This is not required by this method signature but is checked by this method implementation.
-     *
+     * @param  <T>         the type of parameter value. The given {@code value} should typically be an instance of this class.
+     *                     This is not required by this method signature but is checked by this method implementation.
      * @param  descriptor  the parameter descriptor to check against.
      * @param  value       the value to check, or {@code null}.
      * @param  unit        the unit of the value to check, or {@code null}.

Modified: sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/AuthorityFactories.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/AuthorityFactories.java?rev=1776531&r1=1776530&r2=1776531&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/AuthorityFactories.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/AuthorityFactories.java [UTF-8] Fri Dec 30 09:23:24 2016
@@ -186,7 +186,7 @@ final class AuthorityFactories<T extends
      * {@link CRS#getAuthorityFactory(String)}, which is the public facade for {@link #EPSG()}.
      */
     private static void log(final Exception e, final boolean isWarning) {
-        String message = e.getLocalizedMessage();
+        String message = e.getMessage();        // Prefer the locale of system administrator.
         if (message == null) {
             message = e.toString();
         }

Modified: sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/factory/GeodeticObjectFactory.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/factory/GeodeticObjectFactory.java?rev=1776531&r1=1776530&r2=1776531&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/factory/GeodeticObjectFactory.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/factory/GeodeticObjectFactory.java [UTF-8] Fri Dec 30 09:23:24 2016
@@ -254,7 +254,7 @@ public class GeodeticObjectFactory exten
         if (properties == null || properties.isEmpty()) {
             properties = Collections.emptyMap();
         } else {
-            properties = CollectionsExt.compact(new HashMap<String,Object>(properties));
+            properties = CollectionsExt.compact(new HashMap<>(properties));
         }
         defaultProperties = properties;
         pool = new WeakHashSet<>(AbstractIdentifiedObject.class);
@@ -1643,7 +1643,7 @@ public class GeodeticObjectFactory exten
             Throwable cause = e.getCause();
             while (cause != null) {
                 if (cause instanceof ParameterNotFoundException) {
-                    throw new InvalidGeodeticParameterException(e.getMessage(), cause);     // More accurate exception.
+                    throw new InvalidGeodeticParameterException(e.getLocalizedMessage(), cause);
                 }
                 cause = cause.getCause();
             }

Modified: sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/factory/IdentifiedObjectFinder.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/factory/IdentifiedObjectFinder.java?rev=1776531&r1=1776530&r2=1776531&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/factory/IdentifiedObjectFinder.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/factory/IdentifiedObjectFinder.java [UTF-8] Fri Dec 30 09:23:24 2016
@@ -562,7 +562,11 @@ public class IdentifiedObjectFinder {
      * Invoked when an exception occurred during the creation of a candidate from a code.
      */
     private static void exceptionOccurred(final FactoryException exception) {
-        final LogRecord record = new LogRecord(Level.FINER, exception.getLocalizedMessage());
+        /*
+         * use 'getMessage()' instead of 'getLocalizedMessage()' for
+         * giving preference to the locale of system administrator.
+         */
+        final LogRecord record = new LogRecord(Level.FINER, exception.getMessage());
         record.setLoggerName(Loggers.CRS_FACTORY);
         Logging.log(IdentifiedObjectFinder.class, "find", record);
     }

Modified: sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/factory/IdentifiedObjectSet.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/factory/IdentifiedObjectSet.java?rev=1776531&r1=1776530&r2=1776531&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/factory/IdentifiedObjectSet.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/factory/IdentifiedObjectSet.java [UTF-8] Fri Dec 30 09:23:24 2016
@@ -79,7 +79,7 @@ import java.util.Objects;
  * However, implementors are encouraged to wrap in {@linkplain java.util.Collections#unmodifiableSet unmodifiable set}
  * if they intend to cache {@code IdentifiedObjectSet} instances.
  *
- * @param <T> The type of objects to be included in this set.
+ * @param  <T>  the type of objects to be included in this set.
  *
  * @author  Martin Desruisseaux (IRD, Geomatys)
  * @since   0.7
@@ -127,8 +127,8 @@ public class IdentifiedObjectSet<T exten
      * to {@link #addAuthorityCode(String)} for deferred {@code IdentifiedObject} creation,
      * or to {@link #add(IdentifiedObject)} for already instantiated objects.
      *
-     * @param factory The factory to use for deferred {@code IdentifiedObject} instances creation.
-     * @param type The type of objects included in this set.
+     * @param  factory  the factory to use for deferred {@code IdentifiedObject} instances creation.
+     * @param  type     the type of objects included in this set.
      */
     public IdentifiedObjectSet(final AuthorityFactory factory, final Class<T> type) {
         ArgumentChecks.ensureNonNull("factory", factory);
@@ -142,7 +142,7 @@ public class IdentifiedObjectSet<T exten
      * Returns the locale to use for error messages and warnings.
      * The default implementation inherits the {@link #factory} locale, if any.
      *
-     * @return The locale, or {@code null} if not explicitly defined.
+     * @return the locale, or {@code null} if not explicitly defined.
      */
     @Override
     public Locale getLocale() {
@@ -152,7 +152,7 @@ public class IdentifiedObjectSet<T exten
     /**
      * Returns the type of {@code IdentifiedObject} included in this set.
      *
-     * @return The type of {@code IdentifiedObject} included in this set.
+     * @return the type of {@code IdentifiedObject} included in this set.
      */
     @Override
     public Class<T> getElementType() {
@@ -174,7 +174,7 @@ public class IdentifiedObjectSet<T exten
      * Returns the number of objects available in this set. Note that this number may decrease
      * during the iteration process if the creation of some {@code IdentifiedObject}s failed.
      *
-     * @return The number of objects available in this set.
+     * @return the number of objects available in this set.
      */
     @Override
     public int size() {
@@ -187,7 +187,7 @@ public class IdentifiedObjectSet<T exten
      * Returns the authority codes of all {@code IdentifiedObject}s contained in this collection, in insertion order.
      * This method does not trig the {@linkplain #createObject(String) creation} of any object.
      *
-     * @return The authority codes in iteration order.
+     * @return the authority codes in iteration order.
      */
     public String[] getAuthorityCodes() {
         return codes().clone();
@@ -219,7 +219,7 @@ public class IdentifiedObjectSet<T exten
      * the ones in the array returned by {@link #getAuthorityCodes()} but in a different order, then this method
      * just sets the new ordering.</div>
      *
-     * @param codes The authority codes of identified objects to store in this set.
+     * @param  codes the authority codes of identified objects to store in this set.
      *
      * @see #addAuthorityCode(String)
      */
@@ -240,7 +240,7 @@ public class IdentifiedObjectSet<T exten
      * will instantiate an {@code IdentifiedObject} for the given code only when first needed.
      * Otherwise this collection is unchanged.
      *
-     * @param  code The code authority code of the {@code IdentifiedObject} to include in this set.
+     * @param  code  the code authority code of the {@code IdentifiedObject} to include in this set.
      */
     public void addAuthorityCode(final String code) {
         synchronized (objects) {
@@ -256,7 +256,7 @@ public class IdentifiedObjectSet<T exten
      * If this collection already contains an object using the same authority code than the given object,
      * then the old object is replaced by the new one regardless of whether the objects themselves are equal or not.
      *
-     * @param object The object to add to the set.
+     * @param  object  the object to add to the set.
      * @return {@code true} if this set changed as a result of this call.
      *
      * @see #getAuthorityCode(IdentifiedObject)
@@ -335,7 +335,7 @@ public class IdentifiedObjectSet<T exten
     /**
      * Returns {@code true} if this collection contains the specified {@code IdentifiedObject}.
      *
-     * @param  object The {@code IdentifiedObject} to test for presence in this set.
+     * @param  object  the {@code IdentifiedObject} to test for presence in this set.
      * @return {@code true} if the given object is presents in this set.
      */
     @Override
@@ -346,7 +346,7 @@ public class IdentifiedObjectSet<T exten
     /**
      * Removes the object for the given code.
      *
-     * @param code The code of the object to remove.
+     * @param  code  the code of the object to remove.
      */
     final void removeAuthorityCode(final String code) {
         synchronized (objects) {
@@ -358,7 +358,7 @@ public class IdentifiedObjectSet<T exten
     /**
      * Removes the specified {@code IdentifiedObject} from this collection, if it is present.
      *
-     * @param  object The {@code IdentifiedObject} to remove from this set.
+     * @param  object  the {@code IdentifiedObject} to remove from this set.
      * @return {@code true} if this set changed as a result of this call.
      */
     @Override
@@ -380,7 +380,7 @@ public class IdentifiedObjectSet<T exten
     /**
      * Removes from this collection all of its elements that are contained in the specified collection.
      *
-     * @param  collection The {@code IdentifiedObject}s to remove from this set.
+     * @param  collection  the {@code IdentifiedObject}s to remove from this set.
      * @return {@code true} if this set changed as a result of this call.
      */
     @Override
@@ -400,7 +400,7 @@ public class IdentifiedObjectSet<T exten
      * <p>This iterator is <strong>not</strong> thread safe – iteration should be done in a single thread.
      * However the iterator is robust to concurrent changes in {@code IdentifiedObjectSet} during iteration.</p>
      *
-     * @return An iterator over all {@code IdentifiedObject} instances in this set, in insertion order.
+     * @return an iterator over all {@code IdentifiedObject} instances in this set, in insertion order.
      * @throws BackingStoreException if an error occurred while creating the iterator.
      */
     @Override
@@ -479,8 +479,8 @@ public class IdentifiedObjectSet<T exten
      * {@link FactoryException} (except the ones accepted as {@linkplain #isRecoverableFailure recoverable failures})
      * are thrown as if they were never wrapped into {@link BackingStoreException}.
      *
-     * @param n The number of object to resolve. If this number is equals or greater than {@link #size()}, then
-     *          this method ensures that all {@code IdentifiedObject} instances in this collection are created.
+     * @param  n  the number of object to resolve. If this number is equals or greater than {@link #size()}, then
+     *            this method ensures that all {@code IdentifiedObject} instances in this collection are created.
      * @throws FactoryException if an {@linkplain #createObject(String) object creation} failed.
      */
     public void resolve(int n) throws FactoryException {
@@ -510,8 +510,8 @@ public class IdentifiedObjectSet<T exten
      *
      * Subclasses may override this method if they want to use a different identifiers.
      *
-     * @param  object The object for which to get the authority code.
-     * @return The authority code of the given identified object.
+     * @param  object  the object for which to get the authority code.
+     * @return the authority code of the given identified object.
      */
     protected String getAuthorityCode(final T object) {
         final Identifier id = IdentifiedObjects.getIdentifier(object, factory.getAuthority());
@@ -522,9 +522,9 @@ public class IdentifiedObjectSet<T exten
      * Creates an object for the specified authority code.
      * This method is invoked during the iteration process if an object was not already created.
      *
-     * @param  code The code for which to create the identified object.
-     * @return The identified object created from the given code.
-     * @throws FactoryException If the object creation failed.
+     * @param  code  the code for which to create the identified object.
+     * @return the identified object created from the given code.
+     * @throws FactoryException if the object creation failed.
      */
     protected T createObject(final String code) throws FactoryException {
         return type.cast(proxy.createFromAPI(factory, code));
@@ -549,7 +549,7 @@ public class IdentifiedObjectSet<T exten
      *   <li>Otherwise returns {@code false}.</li>
      * </ul>
      *
-     * @param  exception The exception that occurred while creating an object.
+     * @param  exception  the exception that occurred while creating an object.
      * @return {@code true} if the given exception should be considered recoverable,
      *         or {@code false} if it should be considered fatal.
      */
@@ -568,7 +568,7 @@ public class IdentifiedObjectSet<T exten
         final StringBuilder trace = new StringBuilder(180);
         while (cause != null) {
             trace.append(lineSeparator).append("  • ").append(Classes.getShortClassName(cause));
-            final String message = cause.getLocalizedMessage();
+            final String message = cause.getMessage();      // Prefer the local of system administrator.
             if (message != null) {
                 trace.append(": ").append(message);
             }

Modified: sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/factory/InvalidGeodeticParameterException.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/factory/InvalidGeodeticParameterException.java?rev=1776531&r1=1776530&r2=1776531&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/factory/InvalidGeodeticParameterException.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/factory/InvalidGeodeticParameterException.java [UTF-8] Fri Dec 30 09:23:24 2016
@@ -49,7 +49,7 @@ public class InvalidGeodeticParameterExc
     /**
      * Construct an exception with the specified detail message.
      *
-     * @param  message The detail message. The detail message is saved
+     * @param  message  the detail message. The detail message is saved
      *         for later retrieval by the {@link #getMessage()} method.
      */
     public InvalidGeodeticParameterException(String message) {
@@ -62,7 +62,7 @@ public class InvalidGeodeticParameterExc
      * <p>This constructor is not public because its behavior is slightly different than the default JDK behavior:
      * the message is set to the throwable message instead than to {@code throwable.toString()}.</p>
      *
-     * @param  cause The cause for this exception. The cause is saved
+     * @param  cause  the cause for this exception. The cause is saved
      *         for later retrieval by the {@link #getCause()} method.
      */
     InvalidGeodeticParameterException(Throwable cause) {
@@ -72,9 +72,9 @@ public class InvalidGeodeticParameterExc
     /**
      * Construct an exception with the specified detail message and cause.
      *
-     * @param  message The detail message. The detail message is saved
+     * @param  message  the detail message. The detail message is saved
      *         for later retrieval by the {@link #getMessage()} method.
-     * @param  cause The cause for this exception. The cause is saved
+     * @param  cause  the cause for this exception. The cause is saved
      *         for later retrieval by the {@link #getCause()} method.
      */
     public InvalidGeodeticParameterException(String message, Throwable cause) {

Modified: sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/operation/DefaultCoordinateOperationFactory.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/operation/DefaultCoordinateOperationFactory.java?rev=1776531&r1=1776530&r2=1776531&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/operation/DefaultCoordinateOperationFactory.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/operation/DefaultCoordinateOperationFactory.java [UTF-8] Fri Dec 30 09:23:24 2016
@@ -149,8 +149,8 @@ public class DefaultCoordinateOperationF
      * {@code DefaultCoordinateOperationFactory} will fallback on the map given to this constructor
      * for any property not present in the map provided to a {@code createFoo(Map<String,?>, …)} method.
      *
-     * @param properties the default properties, or {@code null} if none.
-     * @param factory the factory to use for creating
+     * @param properties  the default properties, or {@code null} if none.
+     * @param factory     the factory to use for creating
      *        {@linkplain org.apache.sis.referencing.operation.transform.AbstractMathTransform math transforms},
      *        or {@code null} for the default factory.
      */
@@ -196,8 +196,8 @@ public class DefaultCoordinateOperationF
      * CoordinateReferenceSystem, CoordinateReferenceSystem, CoordinateReferenceSystem, MathTransform)
      * coordinate operation constructor}.</p>
      *
-     * @param  properties The user-supplied properties.
-     * @return The union of the given properties with the default properties.
+     * @param  properties  the user-supplied properties.
+     * @return the union of the given properties with the default properties.
      */
     protected Map<String,?> complete(final Map<String,?> properties) {
         ArgumentChecks.ensureNonNull("properties", properties);
@@ -230,7 +230,7 @@ public class DefaultCoordinateOperationF
      * Returns the underlying math transform factory. This factory is used for constructing {@link MathTransform}
      * dependencies for all {@linkplain AbstractCoordinateOperation coordinate operations} instances.
      *
-     * @return The underlying math transform factory.
+     * @return the underlying math transform factory.
      */
     final MathTransformFactory getMathTransformFactory() {
         MathTransformFactory factory = mtFactory;
@@ -263,8 +263,8 @@ public class DefaultCoordinateOperationF
      * non-{@linkplain org.apache.sis.util.Deprecable#isDeprecated() deprecated} matching method is returned.
      * If all matching methods are deprecated, the first one is returned.</p>
      *
-     * @param  name The name of the operation method to fetch.
-     * @return The operation method of the given name.
+     * @param  name  the name of the operation method to fetch.
+     * @return the operation method of the given name.
      * @throws FactoryException if the requested operation method can not be fetched.
      *
      * @see DefaultMathTransformFactory#getOperationMethod(String)
@@ -324,11 +324,11 @@ public class DefaultCoordinateOperationF
      *   </tr>
      * </table>
      *
-     * @param  properties       Set of properties. Shall contain at least {@code "name"}.
-     * @param  sourceDimensions Number of dimensions in the source CRS of this operation method, or {@code null}.
-     * @param  targetDimensions Number of dimensions in the target CRS of this operation method, or {@code null}.
-     * @param  parameters       Description of parameters expected by this operation.
-     * @return The operation method created from the given arguments.
+     * @param  properties        set of properties. Shall contain at least {@code "name"}.
+     * @param  sourceDimensions  number of dimensions in the source CRS of this operation method, or {@code null}.
+     * @param  targetDimensions  number of dimensions in the target CRS of this operation method, or {@code null}.
+     * @param  parameters        description of parameters expected by this operation.
+     * @return the operation method created from the given arguments.
      * @throws FactoryException if the object creation failed.
      *
      * @see DefaultOperationMethod#DefaultOperationMethod(Map, Integer, Integer, ParameterDescriptorGroup)
@@ -384,10 +384,10 @@ public class DefaultCoordinateOperationF
      *   </tr>
      * </table>
      *
-     * @param  properties The properties to be given to the identified object.
-     * @param  method     The operation method.
-     * @param  parameters The parameter values.
-     * @return The defining conversion created from the given arguments.
+     * @param  properties  the properties to be given to the identified object.
+     * @param  method      the operation method.
+     * @param  parameters  the parameter values.
+     * @return the defining conversion created from the given arguments.
      * @throws FactoryException if the object creation failed.
      *
      * @see DefaultConversion#DefaultConversion(Map, OperationMethod, MathTransform, ParameterValueGroup)
@@ -414,8 +414,8 @@ public class DefaultCoordinateOperationF
      * target CRS exists in the source CRS, but not necessarily in the same order.
      * The target CRS may have less datum than the source CRS.
      *
-     * @param sourceCRS The target CRS.
-     * @param targetCRS The source CRS.
+     * @param  sourceCRS  the target CRS.
+     * @param  targetCRS  the source CRS.
      * @return {@code true} if all datum in the {@code targetCRS} exists in the {@code sourceCRS}.
      */
     private static boolean isConversion(final CoordinateReferenceSystem sourceCRS,
@@ -477,13 +477,13 @@ next:   for (int i=components.size(); --
      *   </tr>
      * </table>
      *
-     * @param  properties The properties to be given to the identified object.
-     * @param  sourceCRS  The source CRS.
-     * @param  targetCRS  The target CRS.
-     * @param  interpolationCRS The CRS of additional coordinates needed for the operation, or {@code null} if none.
-     * @param  method     The coordinate operation method (mandatory in all cases).
-     * @param  transform  Transform from positions in the source CRS to positions in the target CRS.
-     * @return The coordinate operation created from the given arguments.
+     * @param  properties        the properties to be given to the identified object.
+     * @param  sourceCRS         the source CRS.
+     * @param  targetCRS         the target CRS.
+     * @param  interpolationCRS  the CRS of additional coordinates needed for the operation, or {@code null} if none.
+     * @param  method            the coordinate operation method (mandatory in all cases).
+     * @param  transform         transform from positions in the source CRS to positions in the target CRS.
+     * @return the coordinate operation created from the given arguments.
      * @throws FactoryException if the object creation failed.
      *
      * @see DefaultOperationMethod#getOperationType()
@@ -637,9 +637,9 @@ next:   for (int i=components.size(); --
      *   </tr>
      * </table>
      *
-     * @param  properties The properties to be given to the identified object.
-     * @param  operations The sequence of operations. Shall contains at least two operations.
-     * @return The concatenated operation created from the given arguments.
+     * @param  properties  the properties to be given to the identified object.
+     * @param  operations  the sequence of operations. Shall contains at least two operations.
+     * @return the concatenated operation created from the given arguments.
      * @throws FactoryException if the object creation failed.
      */
     @Override

Modified: sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/operation/projection/ProjectionException.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/operation/projection/ProjectionException.java?rev=1776531&r1=1776530&r2=1776531&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/operation/projection/ProjectionException.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/operation/projection/ProjectionException.java [UTF-8] Fri Dec 30 09:23:24 2016
@@ -72,7 +72,8 @@ public class ProjectionException extends
      * @param  cause  the cause, or {@code null} if none.
      */
     public ProjectionException(final Throwable cause) {
-        super(cause.getLocalizedMessage(), cause);
+        // Reproduce the behavior of standard Throwable(Throwable) constructor.
+        super((cause != null) ? cause.toString() : null, cause);
     }
 
     /**



Mime
View raw message