sis-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From desruisse...@apache.org
Subject svn commit: r1779506 [5/9] - in /sis/branches/JDK8/core: sis-feature/src/main/java/org/apache/sis/feature/ sis-metadata/src/main/java/org/apache/sis/io/wkt/ sis-metadata/src/main/java/org/apache/sis/metadata/ sis-metadata/src/main/java/org/apache/sis/m...
Date Thu, 19 Jan 2017 18:46:46 GMT
Modified: sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/content/DefaultSampleDimension.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/content/DefaultSampleDimension.java?rev=1779506&r1=1779505&r2=1779506&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/content/DefaultSampleDimension.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/content/DefaultSampleDimension.java [UTF-8] Thu Jan 19 18:46:44 2017
@@ -50,6 +50,7 @@ import static org.apache.sis.internal.me
  * @since   0.5
  * @module
  */
+@SuppressWarnings("CloneableClassWithoutClone")                 // ModifiableMetadata needs shallow clones.
 @XmlType(name = "MD_SampleDimension_Type", propOrder = {
     "maxValue",
     "minValue",
@@ -157,7 +158,7 @@ public class DefaultSampleDimension exte
      * metadata instances can also be obtained by unmarshalling an invalid XML document.
      * </div>
      *
-     * @param object The metadata to copy values from, or {@code null} if none.
+     * @param  object  the metadata to copy values from, or {@code null} if none.
      *
      * @see #castOrCopy(SampleDimension)
      */
@@ -196,8 +197,8 @@ public class DefaultSampleDimension exte
      *       metadata contained in the given object are not recursively copied.</li>
      * </ul>
      *
-     * @param  object The object to get as a SIS implementation, or {@code null} if none.
-     * @return A SIS implementation containing the values of the given object (may be the
+     * @param  object  the object to get as a SIS implementation, or {@code null} if none.
+     * @return a SIS implementation containing the values of the given object (may be the
      *         given object itself), or {@code null} if the argument was null.
      */
     public static DefaultSampleDimension castOrCopy(final SampleDimension object) {
@@ -214,7 +215,7 @@ public class DefaultSampleDimension exte
     /**
      * Returns the minimum value of data values in each dimension included in the resource.
      *
-     * @return Minimum value of data values in each dimension included in the resource, or {@code null} if unspecified.
+     * @return minimum value of data values in each dimension included in the resource, or {@code null} if unspecified.
      */
     @Override
     @XmlElement(name = "minValue")
@@ -225,7 +226,7 @@ public class DefaultSampleDimension exte
     /**
      * Sets the minimum value of data values in each dimension included in the resource.
      *
-     * @param newValue The new new minimum value.
+     * @param  newValue  the new new minimum value.
      */
     public void setMinValue(final Double newValue) {
         checkWritePermission();
@@ -235,7 +236,7 @@ public class DefaultSampleDimension exte
     /**
      * Returns the maximum value of data values in each dimension included in the resource.
      *
-     * @return Maximum value of data values in each dimension included in the resource, or {@code null} if unspecified.
+     * @return maximum value of data values in each dimension included in the resource, or {@code null} if unspecified.
      */
     @Override
     @XmlElement(name = "maxValue")
@@ -246,7 +247,7 @@ public class DefaultSampleDimension exte
     /**
      * Sets the maximum value of data values in each dimension included in the resource.
      *
-     * @param newValue The new new maximum value.
+     * @param  newValue  the new new maximum value.
      */
     public void setMaxValue(final Double newValue) {
         checkWritePermission();
@@ -256,7 +257,7 @@ public class DefaultSampleDimension exte
     /**
      * Returns the mean value of data values in each dimension included in the resource.
      *
-     * @return The mean value of data values in each dimension included in the resource, or {@code null} if none.
+     * @return the mean value of data values in each dimension included in the resource, or {@code null} if none.
      */
     @Override
 /// @XmlElement(name = "meanValue")
@@ -267,7 +268,7 @@ public class DefaultSampleDimension exte
     /**
      * Sets the mean value of data values in each dimension included in the resource.
      *
-     * @param newValue The new mean value of data values in each dimension included in the resource.
+     * @param  newValue  the new mean value of data values in each dimension included in the resource.
      */
     public void setMeanValue(final Double newValue) {
         checkWritePermission();
@@ -277,7 +278,7 @@ public class DefaultSampleDimension exte
     /**
      * Returns the number of values used in a thematic classification resource.
      *
-     * @return The number of values used in a thematic classification resource, or {@code null} if none.
+     * @return the number of values used in a thematic classification resource, or {@code null} if none.
      */
     @Override
     @ValueRange(minimum = 0)
@@ -289,7 +290,7 @@ public class DefaultSampleDimension exte
     /**
      * Sets the number of values used in a thematic classification resource.
      *
-     * @param newValue The new number of values used in a thematic classification resource.
+     * @param  newValue  the new number of values used in a thematic classification resource.
      * @throws IllegalArgumentException if the given value is negative.
      */
     public void setNumberOfValues(final Integer newValue) {
@@ -302,7 +303,7 @@ public class DefaultSampleDimension exte
     /**
      * Returns the standard deviation of data values in each dimension included in the resource.
      *
-     * @return Standard deviation of data values in each dimension included in the resource, or {@code null} if none.
+     * @return standard deviation of data values in each dimension included in the resource, or {@code null} if none.
      */
     @Override
 /// @XmlElement(name = "standardDeviation")
@@ -313,7 +314,7 @@ public class DefaultSampleDimension exte
     /**
      * Sets the standard deviation of data values in each dimension included in the resource.
      *
-     * @param newValue The new standard deviation of data values in each dimension included in the resource.
+     * @param  newValue  the new standard deviation of data values in each dimension included in the resource.
      */
     public void setStandardDeviation(final Double newValue) {
         checkWritePermission();
@@ -323,7 +324,7 @@ public class DefaultSampleDimension exte
     /**
      * Returns the units of data in the dimension.
      *
-     * @return The units of data in the dimension, or {@code null} if unspecified.
+     * @return the units of data in the dimension, or {@code null} if unspecified.
      */
     @Override
     @XmlElement(name = "units")
@@ -334,7 +335,7 @@ public class DefaultSampleDimension exte
     /**
      * Sets the units of data in the dimension.
      *
-     * @param newValue The new units of data in the dimension.
+     * @param  newValue  the new units of data in the dimension.
      */
     public void setUnits(final Unit<?> newValue) {
         checkWritePermission();
@@ -344,7 +345,7 @@ public class DefaultSampleDimension exte
     /**
      * Returns the scale factor which has been applied to the cell value.
      *
-     * @return Scale factor which has been applied to the cell value, or {@code null} if none.
+     * @return scale factor which has been applied to the cell value, or {@code null} if none.
      */
     @Override
 /// @XmlElement(name = "scaleFactor")
@@ -355,7 +356,7 @@ public class DefaultSampleDimension exte
     /**
      * Sets the scale factor which has been applied to the cell value.
      *
-     * @param newValue The new scale factor which has been applied to the cell value.
+     * @param  newValue  the new scale factor which has been applied to the cell value.
      */
     public void setScaleFactor(final Double newValue) {
         checkWritePermission();
@@ -365,7 +366,7 @@ public class DefaultSampleDimension exte
     /**
      * Returns the physical value corresponding to a cell value of zero.
      *
-     * @return The physical value corresponding to a cell value of zero, or {@code null} if none.
+     * @return the physical value corresponding to a cell value of zero, or {@code null} if none.
      */
     @Override
 /// @XmlElement(name = "offset")
@@ -376,7 +377,7 @@ public class DefaultSampleDimension exte
     /**
      * Sets the physical value corresponding to a cell value of zero.
      *
-     * @param newValue The new physical value corresponding to a cell value of zero, or {@code null} if none..
+     * @param  newValue  the new physical value corresponding to a cell value of zero, or {@code null} if none..
      */
     public void setOffset(final Double newValue) {
         checkWritePermission();
@@ -386,7 +387,7 @@ public class DefaultSampleDimension exte
     /**
      * Returns type of transfer function to be used when scaling a physical value for a given element.
      *
-     * @return Type of transfer function, or {@code null}.
+     * @return type of transfer function, or {@code null}.
      */
     @Override
     public TransferFunctionType getTransferFunctionType() {
@@ -396,7 +397,7 @@ public class DefaultSampleDimension exte
     /**
      * Sets the type of transfer function to be used when scaling a physical value for a given element.
      *
-     * @param newValue The new transfer function value.
+     * @param  newValue  the new transfer function value.
      */
     public void setTransferFunctionType(final TransferFunctionType newValue) {
         checkWritePermission();
@@ -407,7 +408,7 @@ public class DefaultSampleDimension exte
      * Returns the maximum number of significant bits in the uncompressed representation
      * for the value in each band of each pixel.
      *
-     * @return Maximum number of significant bits in the uncompressed representation
+     * @return maximum number of significant bits in the uncompressed representation
      *         for the value in each band of each pixel, or {@code null} if none.
      */
     @Override
@@ -421,7 +422,7 @@ public class DefaultSampleDimension exte
      * Sets the maximum number of significant bits in the uncompressed representation
      * for the value in each band of each pixel.
      *
-     * @param newValue The new maximum number of significant bits.
+     * @param  newValue  the new maximum number of significant bits.
      * @throws IllegalArgumentException if the given value is zero or negative.
      */
     public void setBitsPerValue(final Integer newValue) {
@@ -435,7 +436,7 @@ public class DefaultSampleDimension exte
      * Returns the smallest distance between which separate points can be distinguished,
      * as specified in instrument design.
      *
-     * @return Smallest distance between which separate points can be distinguished, or {@code null}.
+     * @return smallest distance between which separate points can be distinguished, or {@code null}.
      */
     @Override
     @ValueRange(minimum = 0, isMinIncluded = false)
@@ -447,7 +448,7 @@ public class DefaultSampleDimension exte
      * Sets the smallest distance between which separate points can be distinguished,
      * as specified in instrument design.
      *
-     * @param newValue The new nominal spatial resolution.
+     * @param  newValue  the new nominal spatial resolution.
      * @throws IllegalArgumentException if the given value is negative.
      */
     public void setNominalSpatialResolution(final Double newValue) {
@@ -460,7 +461,7 @@ public class DefaultSampleDimension exte
     /**
      * Returns type of other attribute description.
      *
-     * @return Type of other attribute description, or {@code null} if none.
+     * @return type of other attribute description, or {@code null} if none.
      */
     @Override
 /// @XmlElement(name = "otherPropertyType")
@@ -471,7 +472,7 @@ public class DefaultSampleDimension exte
     /**
      * Sets a new type of other attribute description.
      *
-     * @param newValue The new type of other attribute description.
+     * @param  newValue  the new type of other attribute description.
      */
     public void setOtherPropertyType(final RecordType newValue) {
         checkWritePermission();
@@ -494,7 +495,7 @@ public class DefaultSampleDimension exte
      * Sets a new instance of other/attributeType that defines attributes not explicitly
      * included in {@link CoverageContentType}.
      *
-     * @param newValue The new instance of other/attributeType.
+     * @param  newValue  the new instance of other/attributeType.
      */
     public void setOtherProperty(final Record newValue) {
         checkWritePermission();

Modified: sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/distribution/DefaultDataFile.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/distribution/DefaultDataFile.java?rev=1779506&r1=1779505&r2=1779506&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/distribution/DefaultDataFile.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/distribution/DefaultDataFile.java [UTF-8] Thu Jan 19 18:46:44 2017
@@ -45,6 +45,7 @@ import org.apache.sis.metadata.iso.ISOMe
  * @version 0.3
  * @module
  */
+@SuppressWarnings("CloneableClassWithoutClone")                 // ModifiableMetadata needs shallow clones.
 @XmlType(name = "MX_DataFile_Type", propOrder = {
     "featureTypes",
     "fileFormat"
@@ -80,7 +81,7 @@ public class DefaultDataFile extends ISO
      * This is a <cite>shallow</cite> copy constructor, since the other metadata contained in the
      * given object are not recursively copied.
      *
-     * @param object the metadata to copy values from, or {@code null} if none.
+     * @param object  the metadata to copy values from, or {@code null} if none.
      *
      * @see #castOrCopy(DataFile)
      */
@@ -106,7 +107,7 @@ public class DefaultDataFile extends ISO
      *       metadata contained in the given object are not recursively copied.</li>
      * </ul>
      *
-     * @param  object the object to get as a SIS implementation, or {@code null} if none.
+     * @param  object  the object to get as a SIS implementation, or {@code null} if none.
      * @return a SIS implementation containing the values of the given object (may be the
      *         given object itself), or {@code null} if the argument was null.
      */
@@ -134,7 +135,7 @@ public class DefaultDataFile extends ISO
     /**
      * Sets the list of feature types concerned by the transfer data file.
      *
-     * @param newValues the new feature type values.
+     * @param newValues  the new feature type values.
      */
     public void setFeatureTypes(final Collection<? extends LocalName> newValues) {
         featureTypes = writeCollection(newValues, featureTypes, LocalName.class);
@@ -154,7 +155,7 @@ public class DefaultDataFile extends ISO
     /**
      * Sets the format of the transfer data file.
      *
-     * @param newValue the new file format value.
+     * @param newValue  the new file format value.
      */
     public void setFileFormat(final Format newValue) {
         checkWritePermission();

Modified: sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/distribution/DefaultDigitalTransferOptions.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/distribution/DefaultDigitalTransferOptions.java?rev=1779506&r1=1779505&r2=1779506&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/distribution/DefaultDigitalTransferOptions.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/distribution/DefaultDigitalTransferOptions.java [UTF-8] Thu Jan 19 18:46:44 2017
@@ -52,6 +52,7 @@ import static org.apache.sis.internal.me
  * @version 0.5
  * @module
  */
+@SuppressWarnings("CloneableClassWithoutClone")                 // ModifiableMetadata needs shallow clones.
 @XmlType(name = "MD_DigitalTransferOptions_Type", propOrder = {
     "unitsOfDistribution",
     "transferSize",
@@ -114,7 +115,7 @@ public class DefaultDigitalTransferOptio
      * metadata instances can also be obtained by unmarshalling an invalid XML document.
      * </div>
      *
-     * @param object The metadata to copy values from, or {@code null} if none.
+     * @param  object  the metadata to copy values from, or {@code null} if none.
      *
      * @see #castOrCopy(DigitalTransferOptions)
      */
@@ -144,8 +145,8 @@ public class DefaultDigitalTransferOptio
      *       metadata contained in the given object are not recursively copied.</li>
      * </ul>
      *
-     * @param  object The object to get as a SIS implementation, or {@code null} if none.
-     * @return A SIS implementation containing the values of the given object (may be the
+     * @param  object  the object to get as a SIS implementation, or {@code null} if none.
+     * @return a SIS implementation containing the values of the given object (may be the
      *         given object itself), or {@code null} if the argument was null.
      */
     public static DefaultDigitalTransferOptions castOrCopy(final DigitalTransferOptions object) {
@@ -158,7 +159,7 @@ public class DefaultDigitalTransferOptio
     /**
      * Returns tiles, layers, geographic areas, <i>etc.</i>, in which data is available.
      *
-     * @return Tiles, layers, geographic areas, <cite>etc.</cite> in which data is available, or {@code null}.
+     * @return tiles, layers, geographic areas, <cite>etc.</cite> in which data is available, or {@code null}.
      */
     @Override
     @XmlElement(name = "unitsOfDistribution")
@@ -169,7 +170,7 @@ public class DefaultDigitalTransferOptio
     /**
      * Sets tiles, layers, geographic areas, <i>etc.</i>, in which data is available.
      *
-     * @param newValue The new units of distribution.
+     * @param  newValue  the new units of distribution.
      */
     public void setUnitsOfDistribution(final InternationalString newValue) {
         checkWritePermission();
@@ -180,7 +181,7 @@ public class DefaultDigitalTransferOptio
      * Returns an estimated size of a unit in the specified transfer format, expressed in megabytes.
      * The transfer size is greater than zero.
      *
-     * @return Estimated size of a unit in the specified transfer format in megabytes, or {@code null}.
+     * @return estimated size of a unit in the specified transfer format in megabytes, or {@code null}.
      */
     @Override
     @XmlElement(name = "transferSize")
@@ -193,7 +194,7 @@ public class DefaultDigitalTransferOptio
      * Sets an estimated size of a unit in the specified transfer format, expressed in megabytes.
      * The transfer shall be greater than zero.
      *
-     * @param newValue The new transfer size, or {@code null}.
+     * @param  newValue  the new transfer size, or {@code null}.
      * @throws IllegalArgumentException if the given value is NaN or negative.
      */
     public void setTransferSize(final Double newValue) {
@@ -206,7 +207,7 @@ public class DefaultDigitalTransferOptio
     /**
      * Returns information about online sources from which the resource can be obtained.
      *
-     * @return Online sources from which the resource can be obtained.
+     * @return online sources from which the resource can be obtained.
      */
     @Override
     @XmlElement(name = "onLine")
@@ -217,7 +218,7 @@ public class DefaultDigitalTransferOptio
     /**
      * Sets information about online sources from which the resource can be obtained.
      *
-     * @param newValues The new online sources.
+     * @param  newValues  the new online sources.
      */
     public void setOnLines(final Collection<? extends OnlineResource> newValues) {
         onLines = writeCollection(newValues, onLines, OnlineResource.class);
@@ -226,7 +227,7 @@ public class DefaultDigitalTransferOptio
     /**
      * Returns information about offline media on which the resource can be obtained.
      *
-     * @return Offline media on which the resource can be obtained.
+     * @return offline media on which the resource can be obtained.
      *
      * @since 0.5
      */
@@ -238,7 +239,7 @@ public class DefaultDigitalTransferOptio
     /**
      * Sets information about offline media on which the resource can be obtained.
      *
-     * @param newValues The new offline media.
+     * @param  newValues  the new offline media.
      *
      * @since 0.5
      */
@@ -249,7 +250,7 @@ public class DefaultDigitalTransferOptio
     /**
      * Returns information about offline media on which the resource can be obtained.
      *
-     * @return Offline media on which the resource can be obtained, or {@code null}.
+     * @return offline media on which the resource can be obtained, or {@code null}.
      *
      * @deprecated As of ISO 19115:2014, replaced by {@link #getOffLines()}.
      */
@@ -263,7 +264,7 @@ public class DefaultDigitalTransferOptio
     /**
      * Sets information about offline media on which the resource can be obtained.
      *
-     * @param newValue The new offline media.
+     * @param  newValue  the new offline media.
      *
      * @deprecated As of ISO 19115:2014, replaced by {@link #setOffLines(Collection)}.
      */
@@ -275,7 +276,7 @@ public class DefaultDigitalTransferOptio
     /**
      * Returns the rate of occurrence of distribution.
      *
-     * @return Rate of occurrence of distribution, or {@code null} if none.
+     * @return rate of occurrence of distribution, or {@code null} if none.
      *
      * @since 0.5
      */
@@ -287,7 +288,7 @@ public class DefaultDigitalTransferOptio
     /**
      * Sets the rate of occurrence of distribution.
      *
-     * @param newValue The new rate of occurrence of distribution.
+     * @param  newValue  the new rate of occurrence of distribution.
      *
      * @since 0.5
      */
@@ -299,7 +300,7 @@ public class DefaultDigitalTransferOptio
     /**
      * Returns the formats of distribution.
      *
-     * @return Formats of distribution.
+     * @return formats of distribution.
      *
      * @since 0.5
      */
@@ -311,7 +312,7 @@ public class DefaultDigitalTransferOptio
     /**
      * Sets the formats of distribution.
      *
-     * @param newValues The new formats of distribution.
+     * @param  newValues  the new formats of distribution.
      *
      * @since 0.5
      */

Modified: sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/distribution/DefaultDistribution.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/distribution/DefaultDistribution.java?rev=1779506&r1=1779505&r2=1779506&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/distribution/DefaultDistribution.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/distribution/DefaultDistribution.java [UTF-8] Thu Jan 19 18:46:44 2017
@@ -47,6 +47,7 @@ import org.apache.sis.metadata.iso.ISOMe
  * @version 0.5
  * @module
  */
+@SuppressWarnings("CloneableClassWithoutClone")                 // ModifiableMetadata needs shallow clones.
 @XmlType(name = "MD_Distribution_Type", propOrder = {
     "distributionFormats",
     "distributors",
@@ -91,7 +92,7 @@ public class DefaultDistribution extends
      * This is a <cite>shallow</cite> copy constructor, since the other metadata contained in the
      * given object are not recursively copied.
      *
-     * @param object The metadata to copy values from, or {@code null} if none.
+     * @param  object  the metadata to copy values from, or {@code null} if none.
      *
      * @see #castOrCopy(Distribution)
      */
@@ -119,8 +120,8 @@ public class DefaultDistribution extends
      *       metadata contained in the given object are not recursively copied.</li>
      * </ul>
      *
-     * @param  object The object to get as a SIS implementation, or {@code null} if none.
-     * @return A SIS implementation containing the values of the given object (may be the
+     * @param  object  the object to get as a SIS implementation, or {@code null} if none.
+     * @return a SIS implementation containing the values of the given object (may be the
      *         given object itself), or {@code null} if the argument was null.
      */
     public static DefaultDistribution castOrCopy(final Distribution object) {
@@ -133,7 +134,7 @@ public class DefaultDistribution extends
     /**
      * Returns a brief description of a set of distribution options.
      *
-     * @return Brief description of a set of distribution options.
+     * @return brief description of a set of distribution options.
      *
      * @since 0.5
      */
@@ -145,7 +146,7 @@ public class DefaultDistribution extends
     /**
      * Sets a brief description of a set of distribution options.
      *
-     * @param newValue The new description.
+     * @param  newValue  the new description.
      *
      * @since 0.5
      */
@@ -157,7 +158,7 @@ public class DefaultDistribution extends
     /**
      * Provides a description of the format of the data to be distributed.
      *
-     * @return Description of the format of the data to be distributed.
+     * @return description of the format of the data to be distributed.
      */
     @Override
     @XmlElement(name = "distributionFormat")
@@ -168,7 +169,7 @@ public class DefaultDistribution extends
     /**
      * Sets a description of the format of the data to be distributed.
      *
-     * @param newValues The new distribution formats.
+     * @param  newValues  the new distribution formats.
      */
     public void setDistributionFormats(final Collection<? extends Format> newValues) {
         distributionFormats = writeCollection(newValues, distributionFormats, Format.class);
@@ -177,7 +178,7 @@ public class DefaultDistribution extends
     /**
      * Provides information about the distributor.
      *
-     * @return Information about the distributor.
+     * @return information about the distributor.
      */
     @Override
     @XmlElement(name = "distributor")
@@ -188,7 +189,7 @@ public class DefaultDistribution extends
     /**
      * Sets information about the distributor.
      *
-     * @param newValues The new distributors.
+     * @param  newValues  the new distributors.
      */
     public void setDistributors(final Collection<? extends Distributor> newValues) {
         distributors = writeCollection(newValues, distributors, Distributor.class);
@@ -197,7 +198,7 @@ public class DefaultDistribution extends
     /**
      * Provides information about technical means and media by which a resource is obtained from the distributor.
      *
-     * @return Technical means and media by which a resource is obtained from the distributor.
+     * @return technical means and media by which a resource is obtained from the distributor.
      */
     @Override
     @XmlElement(name = "transferOptions")
@@ -209,7 +210,7 @@ public class DefaultDistribution extends
      * Sets information about technical means and media by which a resource is obtained
      * from the distributor.
      *
-     * @param newValues The new transfer options.
+     * @param  newValues  the new transfer options.
      */
     public void setTransferOptions(final Collection<? extends DigitalTransferOptions> newValues) {
         transferOptions = writeCollection(newValues, transferOptions, DigitalTransferOptions.class);

Modified: sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/distribution/DefaultDistributor.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/distribution/DefaultDistributor.java?rev=1779506&r1=1779505&r2=1779506&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/distribution/DefaultDistributor.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/distribution/DefaultDistributor.java [UTF-8] Thu Jan 19 18:46:44 2017
@@ -47,6 +47,7 @@ import org.apache.sis.metadata.iso.ISOMe
  * @version 0.5
  * @module
  */
+@SuppressWarnings("CloneableClassWithoutClone")                 // ModifiableMetadata needs shallow clones.
 @XmlType(name = "MD_Distributor_Type", propOrder = {
     "distributorContact",
     "distributionOrderProcesses",
@@ -90,7 +91,7 @@ public class DefaultDistributor extends
     /**
      * Creates a distributor with the specified contact.
      *
-     * @param distributorContact Party from whom the resource may be obtained, or {@code null}.
+     * @param distributorContact  party from whom the resource may be obtained, or {@code null}.
      */
     public DefaultDistributor(final Responsibility distributorContact) {
         this.distributorContact = distributorContact;
@@ -101,7 +102,7 @@ public class DefaultDistributor extends
      * This is a <cite>shallow</cite> copy constructor, since the other metadata contained in the
      * given object are not recursively copied.
      *
-     * @param object The metadata to copy values from, or {@code null} if none.
+     * @param  object  the metadata to copy values from, or {@code null} if none.
      *
      * @see #castOrCopy(Distributor)
      */
@@ -129,8 +130,8 @@ public class DefaultDistributor extends
      *       metadata contained in the given object are not recursively copied.</li>
      * </ul>
      *
-     * @param  object The object to get as a SIS implementation, or {@code null} if none.
-     * @return A SIS implementation containing the values of the given object (may be the
+     * @param  object  the object to get as a SIS implementation, or {@code null} if none.
+     * @return a SIS implementation containing the values of the given object (may be the
      *         given object itself), or {@code null} if the argument was null.
      */
     public static DefaultDistributor castOrCopy(final Distributor object) {
@@ -143,7 +144,7 @@ public class DefaultDistributor extends
     /**
      * Party from whom the resource may be obtained.
      *
-     * @return Party from whom the resource may be obtained, or {@code null}.
+     * @return party from whom the resource may be obtained, or {@code null}.
      */
     @Override
     @XmlElement(name = "distributorContact", required = true)
@@ -154,7 +155,7 @@ public class DefaultDistributor extends
     /**
      * Sets the party from whom the resource may be obtained.
      *
-     * @param newValue The new distributor contact.
+     * @param  newValue  the new distributor contact.
      */
     public void setDistributorContact(final Responsibility newValue) {
         checkWritePermission();
@@ -165,7 +166,7 @@ public class DefaultDistributor extends
      * Provides information about how the resource may be obtained,
      * and related instructions and fee information.
      *
-     * @return Information about how the resource may be obtained.
+     * @return information about how the resource may be obtained.
      */
     @Override
     @XmlElement(name = "distributionOrderProcess")
@@ -177,7 +178,7 @@ public class DefaultDistributor extends
      * Sets information about how the resource may be obtained,
      * and related instructions and fee information.
      *
-     * @param newValues The new distribution order processes.
+     * @param  newValues  the new distribution order processes.
      */
     public void setDistributionOrderProcesses(final Collection<? extends StandardOrderProcess> newValues) {
         distributionOrderProcesses = writeCollection(newValues, distributionOrderProcesses, StandardOrderProcess.class);
@@ -186,7 +187,7 @@ public class DefaultDistributor extends
     /**
      * Provides information about the format used by the distributor.
      *
-     * @return Information about the format used by the distributor.
+     * @return information about the format used by the distributor.
      */
     @Override
     @XmlElement(name = "distributorFormat")
@@ -197,7 +198,7 @@ public class DefaultDistributor extends
     /**
      * Sets information about the format used by the distributor.
      *
-     * @param newValues The new distributor formats.
+     * @param  newValues  the new distributor formats.
      */
     public void setDistributorFormats(final Collection<? extends Format> newValues) {
         distributorFormats = writeCollection(newValues, distributorFormats, Format.class);
@@ -206,7 +207,7 @@ public class DefaultDistributor extends
     /**
      * Provides information about the technical means and media used by the distributor.
      *
-     * @return Information about the technical means and media used by the distributor.
+     * @return information about the technical means and media used by the distributor.
      */
     @Override
     @XmlElement(name = "distributorTransferOptions")
@@ -217,7 +218,7 @@ public class DefaultDistributor extends
     /**
      * Provides information about the technical means and media used by the distributor.
      *
-     * @param newValues The new distributor transfer options.
+     * @param  newValues  the new distributor transfer options.
      */
     public void setDistributorTransferOptions(final Collection<? extends DigitalTransferOptions> newValues) {
         distributorTransferOptions = writeCollection(newValues, distributorTransferOptions, DigitalTransferOptions.class);

Modified: sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/distribution/DefaultFormat.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/distribution/DefaultFormat.java?rev=1779506&r1=1779505&r2=1779506&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/distribution/DefaultFormat.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/distribution/DefaultFormat.java [UTF-8] Thu Jan 19 18:46:44 2017
@@ -75,6 +75,7 @@ import java.util.function.BiConsumer;
  * @version 0.8
  * @module
  */
+@SuppressWarnings("CloneableClassWithoutClone")                 // ModifiableMetadata needs shallow clones.
 @XmlType(name = "MD_Format_Type", propOrder = {
     "name",
     "version",

Modified: sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/distribution/DefaultMedium.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/distribution/DefaultMedium.java?rev=1779506&r1=1779505&r2=1779506&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/distribution/DefaultMedium.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/distribution/DefaultMedium.java [UTF-8] Thu Jan 19 18:46:44 2017
@@ -55,6 +55,7 @@ import static org.apache.sis.internal.me
  * @version 0.5
  * @module
  */
+@SuppressWarnings("CloneableClassWithoutClone")                 // ModifiableMetadata needs shallow clones.
 @XmlType(name = "MD_Medium_Type", propOrder = {
     "name",
     "densities",
@@ -119,7 +120,7 @@ public class DefaultMedium extends ISOMe
      * metadata instances can also be obtained by unmarshalling an invalid XML document.
      * </div>
      *
-     * @param object The metadata to copy values from, or {@code null} if none.
+     * @param  object  the metadata to copy values from, or {@code null} if none.
      *
      * @see #castOrCopy(Medium)
      */
@@ -150,8 +151,8 @@ public class DefaultMedium extends ISOMe
      *       metadata contained in the given object are not recursively copied.</li>
      * </ul>
      *
-     * @param  object The object to get as a SIS implementation, or {@code null} if none.
-     * @return A SIS implementation containing the values of the given object (may be the
+     * @param  object  the object to get as a SIS implementation, or {@code null} if none.
+     * @return a SIS implementation containing the values of the given object (may be the
      *         given object itself), or {@code null} if the argument was null.
      */
     public static DefaultMedium castOrCopy(final Medium object) {
@@ -164,7 +165,7 @@ public class DefaultMedium extends ISOMe
     /**
      * Returns the name of the medium on which the resource can be received.
      *
-     * @return Name of the medium, or {@code null}.
+     * @return name of the medium, or {@code null}.
      */
     @Override
     @XmlElement(name = "name")
@@ -175,7 +176,7 @@ public class DefaultMedium extends ISOMe
     /**
      * Sets the name of the medium on which the resource can be received.
      *
-     * @param newValue The new name.
+     * @param  newValue  the new name.
      */
     public void setName(final MediumName newValue) {
         checkWritePermission();
@@ -186,7 +187,7 @@ public class DefaultMedium extends ISOMe
      * Returns the density at which the data is recorded.
      * The number shall be greater than zero.
      *
-     * @return Density at which the data is recorded, or {@code null}.
+     * @return density at which the data is recorded, or {@code null}.
      *
      * @since 0.5
      */
@@ -200,7 +201,7 @@ public class DefaultMedium extends ISOMe
      * Sets density at which the data is recorded.
      * The number shall be greater than zero.
      *
-     * @param newValue The new density.
+     * @param  newValue  the new density.
      * @throws IllegalArgumentException if the given value is NaN, zero or negative.
      *
      * @since 0.5
@@ -215,7 +216,7 @@ public class DefaultMedium extends ISOMe
     /**
      * @deprecated As of ISO 19115:2014, replaced by {@link #getDensity()}.
      *
-     * @return Density at which the data is recorded, or {@code null}.
+     * @return density at which the data is recorded, or {@code null}.
      */
     @Override
     @Deprecated
@@ -248,7 +249,7 @@ public class DefaultMedium extends ISOMe
     /**
      * @deprecated As of ISO 19115:2014, replaced by {@link #setDensity(Double)}.
      *
-     * @param newValues The new densities.
+     * @param  newValues  the new densities.
      */
     @Deprecated
     public void setDensities(final Collection<? extends Double> newValues) {
@@ -258,7 +259,7 @@ public class DefaultMedium extends ISOMe
     /**
      * Returns the units of measure for the recording density.
      *
-     * @return Units of measure for the recording density, or {@code null}.
+     * @return units of measure for the recording density, or {@code null}.
      */
     @Override
     @XmlElement(name = "densityUnits")
@@ -269,7 +270,7 @@ public class DefaultMedium extends ISOMe
     /**
      * Sets the units of measure for the recording density.
      *
-     * @param newValue The new density units.
+     * @param  newValue  the new density units.
      */
     public void setDensityUnits(final Unit<?> newValue) {
         checkWritePermission();
@@ -279,7 +280,7 @@ public class DefaultMedium extends ISOMe
     /**
      * Returns the number of items in the media identified.
      *
-     * @return Number of items in the media identified, or {@code null}.
+     * @return number of items in the media identified, or {@code null}.
      */
     @Override
     @ValueRange(minimum = 0)
@@ -291,7 +292,7 @@ public class DefaultMedium extends ISOMe
     /**
      * Sets the number of items in the media identified.
      *
-     * @param newValue The new volumes, or {@code null}.
+     * @param  newValue  the new volumes, or {@code null}.
      * @throws IllegalArgumentException if the given value is negative.
      */
     public void setVolumes(final Integer newValue) {
@@ -304,7 +305,7 @@ public class DefaultMedium extends ISOMe
     /**
      * Returns the method used to write to the medium.
      *
-     * @return Method used to write to the medium, or {@code null}.
+     * @return method used to write to the medium, or {@code null}.
      */
     @Override
     @XmlElement(name = "mediumFormat")
@@ -315,7 +316,7 @@ public class DefaultMedium extends ISOMe
     /**
      * Sets the method used to write to the medium.
      *
-     * @param newValues The new medium formats.
+     * @param  newValues  the new medium formats.
      */
     public void setMediumFormats(final Collection<? extends MediumFormat> newValues) {
         mediumFormats = writeCollection(newValues, mediumFormats, MediumFormat.class);
@@ -324,7 +325,7 @@ public class DefaultMedium extends ISOMe
     /**
      * Returns a description of other limitations or requirements for using the medium.
      *
-     * @return Description of other limitations for using the medium, or {@code null}.
+     * @return description of other limitations for using the medium, or {@code null}.
      */
     @Override
     @XmlElement(name = "mediumNote")
@@ -335,7 +336,7 @@ public class DefaultMedium extends ISOMe
     /**
      * Sets a description of other limitations or requirements for using the medium.
      *
-     * @param newValue The new medium note.
+     * @param  newValue  the new medium note.
      */
     public void setMediumNote(final InternationalString newValue) {
         checkWritePermission();
@@ -345,7 +346,7 @@ public class DefaultMedium extends ISOMe
     /**
      * Returns a unique identifier for an instance of the medium.
      *
-     * @return Unique identifier, or {@code null} if none.
+     * @return unique identifier, or {@code null} if none.
      *
      * @since 0.5
      */
@@ -358,7 +359,7 @@ public class DefaultMedium extends ISOMe
     /**
      * Sets a unique identifier for an instance of the medium.
      *
-     * @param newValue The new identifier.
+     * @param  newValue  the new identifier.
      *
      * @since 0.5
      */

Modified: sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/distribution/DefaultStandardOrderProcess.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/distribution/DefaultStandardOrderProcess.java?rev=1779506&r1=1779505&r2=1779506&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/distribution/DefaultStandardOrderProcess.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/distribution/DefaultStandardOrderProcess.java [UTF-8] Thu Jan 19 18:46:44 2017
@@ -51,6 +51,7 @@ import static org.apache.sis.internal.me
  * @version 0.5
  * @module
  */
+@SuppressWarnings("CloneableClassWithoutClone")                 // ModifiableMetadata needs shallow clones.
 @XmlType(name = "MD_StandardOrderProcess_Type", propOrder = {
     "fees",
     "plannedAvailableDateTime",
@@ -112,7 +113,7 @@ public class DefaultStandardOrderProcess
      * This is a <cite>shallow</cite> copy constructor, since the other metadata contained in the
      * given object are not recursively copied.
      *
-     * @param object The metadata to copy values from, or {@code null} if none.
+     * @param  object  the metadata to copy values from, or {@code null} if none.
      *
      * @see #castOrCopy(StandardOrderProcess)
      */
@@ -142,8 +143,8 @@ public class DefaultStandardOrderProcess
      *       metadata contained in the given object are not recursively copied.</li>
      * </ul>
      *
-     * @param  object The object to get as a SIS implementation, or {@code null} if none.
-     * @return A SIS implementation containing the values of the given object (may be the
+     * @param  object  the object to get as a SIS implementation, or {@code null} if none.
+     * @return a SIS implementation containing the values of the given object (may be the
      *         given object itself), or {@code null} if the argument was null.
      */
     public static DefaultStandardOrderProcess castOrCopy(final StandardOrderProcess object) {
@@ -158,7 +159,7 @@ public class DefaultStandardOrderProcess
      * Include monetary units (as specified in ISO 4217).
      * The monetary units may also be available with {@link #getCurrency()}.
      *
-     * @return Fees and terms for retrieving the resource, or {@code null}.
+     * @return fees and terms for retrieving the resource, or {@code null}.
      *
      * @see #getCurrency()
      */
@@ -172,7 +173,7 @@ public class DefaultStandardOrderProcess
      * Sets fees and terms for retrieving the resource.
      * Include monetary units (as specified in ISO 4217).
      *
-     * @param newValue The new fees.
+     * @param  newValue  the new fees.
      *
      * @see #setCurrency(Currency)
      */
@@ -190,7 +191,7 @@ public class DefaultStandardOrderProcess
      * However if this method returns a non-null value, then that value is required to be consistent
      * with the fees text.</p>
      *
-     * @return The fees monetary units, or {@code null} if none or unknown.
+     * @return the fees monetary units, or {@code null} if none or unknown.
      *
      * @since 0.5
      *
@@ -206,7 +207,7 @@ public class DefaultStandardOrderProcess
      * Callers should ensure that the given currency is consistent with the currency
      * in the {@linkplain #getFees() fees} text.
      *
-     * @param newValue The new currency.
+     * @param  newValue  the new currency.
      *
      * @since 0.5
      *
@@ -220,7 +221,7 @@ public class DefaultStandardOrderProcess
     /**
      * Returns the date and time when the dataset will be available.
      *
-     * @return Date and time when the dataset will be available, or {@code null}.
+     * @return date and time when the dataset will be available, or {@code null}.
      */
     @Override
     @XmlElement(name = "plannedAvailableDateTime")
@@ -231,7 +232,7 @@ public class DefaultStandardOrderProcess
     /**
      * Sets the date and time when the dataset will be available.
      *
-     * @param newValue The new planned available time.
+     * @param  newValue  the new planned available time.
      */
     public void setPlannedAvailableDateTime(final Date newValue) {
         checkWritePermission();
@@ -241,7 +242,7 @@ public class DefaultStandardOrderProcess
     /**
      * Returns general instructions, terms and services provided by the distributor.
      *
-     * @return General instructions, terms and services provided by the distributor, or {@code null}.
+     * @return general instructions, terms and services provided by the distributor, or {@code null}.
      */
     @Override
     @XmlElement(name = "orderingInstructions")
@@ -252,7 +253,7 @@ public class DefaultStandardOrderProcess
     /**
      * Sets general instructions, terms and services provided by the distributor.
      *
-     * @param newValue The new ordering instructions.
+     * @param  newValue  the new ordering instructions.
      */
     public void setOrderingInstructions(final InternationalString newValue) {
         checkWritePermission();
@@ -262,7 +263,7 @@ public class DefaultStandardOrderProcess
     /**
      * Returns typical turnaround time for the filling of an order.
      *
-     * @return Typical turnaround time for the filling of an order, or {@code null}.
+     * @return typical turnaround time for the filling of an order, or {@code null}.
      */
     @Override
     @XmlElement(name = "turnaround")
@@ -273,7 +274,7 @@ public class DefaultStandardOrderProcess
     /**
      * Sets typical turnaround time for the filling of an order.
      *
-     * @param newValue The new turnaround.
+     * @param  newValue  the new turnaround.
      */
     public void setTurnaround(final InternationalString newValue) {
         checkWritePermission();
@@ -283,7 +284,7 @@ public class DefaultStandardOrderProcess
     /**
      * Returns the description of the {@linkplain #getOrderOptions() order options} record.
      *
-     * @return Description of the order options record, or {@code null} if none.
+     * @return description of the order options record, or {@code null} if none.
      *
      * @since 0.5
      *
@@ -310,7 +311,7 @@ public class DefaultStandardOrderProcess
     /**
      * Returns the request/purchase choices.
      *
-     * @return Request/purchase choices.
+     * @return request/purchase choices.
      *
      * @since 0.5
      *

Modified: sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/extent/AbstractGeographicExtent.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/extent/AbstractGeographicExtent.java?rev=1779506&r1=1779505&r2=1779506&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/extent/AbstractGeographicExtent.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/extent/AbstractGeographicExtent.java [UTF-8] Thu Jan 19 18:46:44 2017
@@ -46,6 +46,7 @@ import org.apache.sis.metadata.iso.ISOMe
  * @version 0.4
  * @module
  */
+@SuppressWarnings("CloneableClassWithoutClone")                 // ModifiableMetadata needs shallow clones.
 @XmlType(name = "AbstractEX_GeographicExtent_Type")
 @XmlRootElement(name = "EX_GeographicExtent")
 @XmlSeeAlso({
@@ -78,7 +79,7 @@ public class AbstractGeographicExtent ex
     /**
      * Constructs a geographic extent initialized with the specified inclusion value.
      *
-     * @param inclusion Whether the bounding polygon encompasses an area covered by the data.
+     * @param inclusion  whether the bounding polygon encompasses an area covered by the data.
      */
     public AbstractGeographicExtent(final boolean inclusion) {
         this.inclusion = inclusion;
@@ -89,7 +90,7 @@ public class AbstractGeographicExtent ex
      * This is a <cite>shallow</cite> copy constructor, since the other metadata contained in the
      * given object are not recursively copied.
      *
-     * @param object The metadata to copy values from, or {@code null} if none.
+     * @param object  the metadata to copy values from, or {@code null} if none.
      *
      * @see #castOrCopy(GeographicExtent)
      */
@@ -119,8 +120,8 @@ public class AbstractGeographicExtent ex
      *       metadata contained in the given object are not recursively copied.</li>
      * </ul>
      *
-     * @param  object The object to get as a SIS implementation, or {@code null} if none.
-     * @return A SIS implementation containing the values of the given object (may be the
+     * @param  object  the object to get as a SIS implementation, or {@code null} if none.
+     * @return a SIS implementation containing the values of the given object (may be the
      *         given object itself), or {@code null} if the argument was null.
      */
     public static AbstractGeographicExtent castOrCopy(final GeographicExtent object) {

Modified: sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/extent/DefaultBoundingPolygon.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/extent/DefaultBoundingPolygon.java?rev=1779506&r1=1779505&r2=1779506&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/extent/DefaultBoundingPolygon.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/extent/DefaultBoundingPolygon.java [UTF-8] Thu Jan 19 18:46:44 2017
@@ -46,6 +46,7 @@ import org.opengis.metadata.extent.Bound
  * @version 0.3
  * @module
  */
+@SuppressWarnings("CloneableClassWithoutClone")                 // ModifiableMetadata needs shallow clones.
 @XmlType(name = "EX_BoundingPolygon_Type")
 @XmlRootElement(name = "EX_BoundingPolygon")
 public class DefaultBoundingPolygon extends AbstractGeographicExtent implements BoundingPolygon {
@@ -68,7 +69,7 @@ public class DefaultBoundingPolygon exte
     /**
      * Creates a bounding polygon initialized to the specified polygon.
      *
-     * @param polygon The sets of points defining the bounding polygon.
+     * @param polygon  the sets of points defining the bounding polygon.
      */
     public DefaultBoundingPolygon(final Geometry polygon) {
         polygons = singleton(polygon, Geometry.class);
@@ -79,7 +80,7 @@ public class DefaultBoundingPolygon exte
      * This is a <cite>shallow</cite> copy constructor, since the other metadata contained in the
      * given object are not recursively copied.
      *
-     * @param object The metadata to copy values from, or {@code null} if none.
+     * @param object  the metadata to copy values from, or {@code null} if none.
      *
      * @see #castOrCopy(BoundingPolygon)
      */
@@ -104,8 +105,8 @@ public class DefaultBoundingPolygon exte
      *       metadata contained in the given object are not recursively copied.</li>
      * </ul>
      *
-     * @param  object The object to get as a SIS implementation, or {@code null} if none.
-     * @return A SIS implementation containing the values of the given object (may be the
+     * @param  object  the object to get as a SIS implementation, or {@code null} if none.
+     * @return a SIS implementation containing the values of the given object (may be the
      *         given object itself), or {@code null} if the argument was null.
      */
     public static DefaultBoundingPolygon castOrCopy(final BoundingPolygon object) {
@@ -118,7 +119,7 @@ public class DefaultBoundingPolygon exte
     /**
      * Returns the sets of points defining the bounding polygon or other geometry.
      *
-     * @return The sets of points defining the resource boundary.
+     * @return the sets of points defining the resource boundary.
      */
     @Override
     @XmlElement(name = "polygon", required = true)
@@ -129,7 +130,7 @@ public class DefaultBoundingPolygon exte
     /**
      * Sets the sets of points defining the resource boundary.
      *
-     * @param newValues The new boundaries.
+     * @param  newValues  the new boundaries.
      */
     public void setPolygons(final Collection<? extends Geometry> newValues) {
         polygons = writeCollection(newValues, polygons, Geometry.class);

Modified: sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/extent/DefaultExtent.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/extent/DefaultExtent.java?rev=1779506&r1=1779505&r2=1779506&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/extent/DefaultExtent.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/extent/DefaultExtent.java [UTF-8] Thu Jan 19 18:46:44 2017
@@ -68,6 +68,7 @@ import org.apache.sis.internal.metadata.
  * @see org.apache.sis.referencing.AbstractReferenceSystem#getDomainOfValidity()
  * @see org.apache.sis.referencing.datum.AbstractDatum#getDomainOfValidity()
  */
+@SuppressWarnings("CloneableClassWithoutClone")                 // ModifiableMetadata needs shallow clones.
 @XmlType(name = "EX_Extent_Type", propOrder = {
     "description",
     "geographicElements",

Modified: sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/extent/DefaultGeographicBoundingBox.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/extent/DefaultGeographicBoundingBox.java?rev=1779506&r1=1779505&r2=1779506&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/extent/DefaultGeographicBoundingBox.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/extent/DefaultGeographicBoundingBox.java [UTF-8] Thu Jan 19 18:46:44 2017
@@ -105,6 +105,7 @@ import static java.lang.Double.doubleToL
  *
  * @see org.apache.sis.geometry.GeneralEnvelope
  */
+@SuppressWarnings("CloneableClassWithoutClone")                 // ModifiableMetadata needs shallow clones.
 @XmlType(name = "EX_GeographicBoundingBox_Type", propOrder = {
     "westBoundLongitude",
     "eastBoundLongitude",
@@ -252,8 +253,10 @@ public class DefaultGeographicBoundingBo
             return true;
         }
         final boolean p = value;
-        // (value == Boolean.FALSE) is an optimization for a common case avoiding PrimitiveTypeProperties check.
-        // DO NOT REPLACE BY 'equals' OR 'booleanValue()' - the exact reference value matter.
+        /*
+         * (value == Boolean.FALSE) is an optimization for a common case avoiding PrimitiveTypeProperties check.
+         * DO NOT REPLACE BY 'equals' OR 'booleanValue()' - the exact reference value matter.
+         */
         if (p || (value == Boolean.FALSE) || !(PrimitiveTypeProperties.property(value) instanceof NilReason)) {
             return p;
         }

Modified: sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/extent/DefaultGeographicDescription.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/extent/DefaultGeographicDescription.java?rev=1779506&r1=1779505&r2=1779506&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/extent/DefaultGeographicDescription.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/extent/DefaultGeographicDescription.java [UTF-8] Thu Jan 19 18:46:44 2017
@@ -56,6 +56,7 @@ import org.apache.sis.util.iso.Types;
  * @version 0.6
  * @module
  */
+@SuppressWarnings("CloneableClassWithoutClone")                 // ModifiableMetadata needs shallow clones.
 @XmlType(name = "EX_GeographicDescription_Type")
 @XmlRootElement(name = "EX_GeographicDescription")
 public class DefaultGeographicDescription extends AbstractGeographicExtent implements GeographicDescription {
@@ -87,8 +88,8 @@ public class DefaultGeographicDescriptio
      * <code>new DefaultGeographicDescription({@link org.apache.sis.metadata.iso.citation.Citations#EPSG}, "1731")</code>
      * can stand for <cite>“France – mainland north of 48.15°N”</cite>.</div>
      *
-     * @param authority The authority of the identifier code, or {@code null} if none.
-     * @param code The identifier code used to represent a geographic area, or {@code null} if none.
+     * @param authority  the authority of the identifier code, or {@code null} if none.
+     * @param code       the identifier code used to represent a geographic area, or {@code null} if none.
      */
     public DefaultGeographicDescription(final Citation authority, final String code) {
         super(true);
@@ -109,7 +110,7 @@ public class DefaultGeographicDescriptio
      *       of the given {@code description}.</li>
      * </ul>
      *
-     * @param description The natural language description of the meaning of the code value, or {@code null} if none.
+     * @param description  the natural language description of the meaning of the code value, or {@code null} if none.
      *
      * @since 0.6
      */
@@ -135,7 +136,7 @@ public class DefaultGeographicDescriptio
      * This is a <cite>shallow</cite> copy constructor, since the other metadata contained in the
      * given object are not recursively copied.
      *
-     * @param object The metadata to copy values from, or {@code null} if none.
+     * @param object  the metadata to copy values from, or {@code null} if none.
      *
      * @see #castOrCopy(GeographicDescription)
      */
@@ -160,8 +161,8 @@ public class DefaultGeographicDescriptio
      *       metadata contained in the given object are not recursively copied.</li>
      * </ul>
      *
-     * @param  object The object to get as a SIS implementation, or {@code null} if none.
-     * @return A SIS implementation containing the values of the given object (may be the
+     * @param  object  the object to get as a SIS implementation, or {@code null} if none.
+     * @return a SIS implementation containing the values of the given object (may be the
      *         given object itself), or {@code null} if the argument was null.
      */
     public static DefaultGeographicDescription castOrCopy(final GeographicDescription object) {
@@ -182,7 +183,7 @@ public class DefaultGeographicDescriptio
      *   <li>the <cite>“France – mainland north of 48.15°N”</cite> description.</li>
      * </ul></div>
      *
-     * @return The identifier used to represent a geographic area, or {@code null}.
+     * @return the identifier used to represent a geographic area, or {@code null}.
      */
     @Override
     @XmlElement(name = "geographicIdentifier", required = true)
@@ -193,7 +194,7 @@ public class DefaultGeographicDescriptio
     /**
      * Sets the identifier used to represent a geographic area.
      *
-     * @param newValue The new geographic identifier.
+     * @param  newValue  the new geographic identifier.
      */
     public void setGeographicIdentifier(final Identifier newValue) {
         checkWritePermission();

Modified: sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/extent/DefaultSpatialTemporalExtent.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/extent/DefaultSpatialTemporalExtent.java?rev=1779506&r1=1779505&r2=1779506&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/extent/DefaultSpatialTemporalExtent.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/extent/DefaultSpatialTemporalExtent.java [UTF-8] Thu Jan 19 18:46:44 2017
@@ -50,6 +50,7 @@ import org.apache.sis.internal.metadata.
  * @version 0.5
  * @module
  */
+@SuppressWarnings("CloneableClassWithoutClone")                 // ModifiableMetadata needs shallow clones.
 @XmlType(name = "EX_SpatialTemporalExtent_Type")
 @XmlRootElement(name = "EX_SpatialTemporalExtent")
 public class DefaultSpatialTemporalExtent extends DefaultTemporalExtent implements SpatialTemporalExtent {
@@ -78,9 +79,9 @@ public class DefaultSpatialTemporalExten
     /**
      * Constructs a new spatial-temporal extent initialized to the specified values.
      *
-     * @param spatialExtent  The spatial extent component of composite spatial and temporal extent.
-     * @param verticalExtent The vertical extent component, or {@code null} if none.
-     * @param extent         The date and time for the content of the dataset, or {@code null} if unspecified.
+     * @param spatialExtent   the spatial extent component of composite spatial and temporal extent.
+     * @param verticalExtent  the vertical extent component, or {@code null} if none.
+     * @param extent          the date and time for the content of the dataset, or {@code null} if unspecified.
      *
      * @since 0.5
      */
@@ -98,7 +99,7 @@ public class DefaultSpatialTemporalExten
      * This is a <cite>shallow</cite> copy constructor, since the other metadata contained in the
      * given object are not recursively copied.
      *
-     * @param object The metadata to copy values from, or {@code null} if none.
+     * @param  object  the metadata to copy values from, or {@code null} if none.
      *
      * @see #castOrCopy(SpatialTemporalExtent)
      */
@@ -124,8 +125,8 @@ public class DefaultSpatialTemporalExten
      *       metadata contained in the given object are not recursively copied.</li>
      * </ul>
      *
-     * @param  object The object to get as a SIS implementation, or {@code null} if none.
-     * @return A SIS implementation containing the values of the given object (may be the
+     * @param  object  the object to get as a SIS implementation, or {@code null} if none.
+     * @return a SIS implementation containing the values of the given object (may be the
      *         given object itself), or {@code null} if the argument was null.
      */
     public static DefaultSpatialTemporalExtent castOrCopy(final SpatialTemporalExtent object) {
@@ -138,7 +139,7 @@ public class DefaultSpatialTemporalExten
     /**
      * Returns the spatial extent component of composite spatial and temporal extent.
      *
-     * @return The list of geographic extents (never {@code null}).
+     * @return the list of geographic extents (never {@code null}).
      */
     @Override
     @XmlElement(name = "spatialExtent", required = true)
@@ -149,7 +150,7 @@ public class DefaultSpatialTemporalExten
     /**
      * Sets the spatial extent component of composite spatial and temporal extent.
      *
-     * @param newValues The new spatial extent.
+     * @param  newValues  the new spatial extent.
      */
     public void setSpatialExtent(final Collection<? extends GeographicExtent> newValues) {
         spatialExtent = writeCollection(newValues, spatialExtent, GeographicExtent.class);
@@ -158,7 +159,7 @@ public class DefaultSpatialTemporalExten
     /**
      * Returns the vertical extent component.
      *
-     * @return Vertical extent component, or {@code null} if none.
+     * @return vertical extent component, or {@code null} if none.
      *
      * @since 0.5
      */
@@ -171,7 +172,7 @@ public class DefaultSpatialTemporalExten
     /**
      * Sets the vertical extent component.
      *
-     * @param newValue The new vertical extent component.
+     * @param  newValue  the new vertical extent component.
      *
      * @since 0.5
      */
@@ -210,7 +211,7 @@ public class DefaultSpatialTemporalExten
      * <b>Note:</b> This method is available only if the {@code sis-referencing} module is
      * available on the classpath.
      *
-     * @param  envelope The envelope to use for setting this spatio-temporal extent.
+     * @param  envelope  the envelope to use for setting this spatio-temporal extent.
      * @throws UnsupportedOperationException if the referencing module is not on the classpath.
      * @throws TransformException if the envelope can not be transformed to a temporal extent.
      */

Modified: sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/extent/DefaultTemporalExtent.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/extent/DefaultTemporalExtent.java?rev=1779506&r1=1779505&r2=1779506&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/extent/DefaultTemporalExtent.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/extent/DefaultTemporalExtent.java [UTF-8] Thu Jan 19 18:46:44 2017
@@ -60,6 +60,7 @@ import org.apache.sis.internal.metadata.
  * @version 0.3
  * @module
  */
+@SuppressWarnings("CloneableClassWithoutClone")                 // ModifiableMetadata needs shallow clones.
 @XmlType(name = "EX_TemporalExtent_Type")
 @XmlRootElement(name = "EX_TemporalExtent")
 @XmlSeeAlso(DefaultSpatialTemporalExtent.class)

Modified: sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/extent/DefaultVerticalExtent.java
URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/extent/DefaultVerticalExtent.java?rev=1779506&r1=1779505&r2=1779506&view=diff
==============================================================================
--- sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/extent/DefaultVerticalExtent.java [UTF-8] (original)
+++ sis/branches/JDK8/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/extent/DefaultVerticalExtent.java [UTF-8] Thu Jan 19 18:46:44 2017
@@ -53,6 +53,7 @@ import org.apache.sis.internal.metadata.
  * @version 0.3
  * @module
  */
+@SuppressWarnings("CloneableClassWithoutClone")                 // ModifiableMetadata needs shallow clones.
 @XmlType(name = "EX_VerticalExtent_Type", propOrder = {
     "minimumValue",
     "maximumValue",
@@ -91,9 +92,9 @@ public class DefaultVerticalExtent exten
     /**
      * Creates a vertical extent initialized to the specified values.
      *
-     * @param minimumValue The lowest vertical extent contained in the dataset, or {@link Double#NaN} if none.
-     * @param maximumValue The highest vertical extent contained in the dataset, or {@link Double#NaN} if none.
-     * @param verticalCRS  The information about the vertical coordinate reference system, or {@code null}.
+     * @param minimumValue  the lowest vertical extent contained in the dataset, or {@link Double#NaN} if none.
+     * @param maximumValue  the highest vertical extent contained in the dataset, or {@link Double#NaN} if none.
+     * @param verticalCRS   the information about the vertical coordinate reference system, or {@code null}.
      */
     public DefaultVerticalExtent(final double minimumValue,
                                  final double maximumValue,
@@ -109,7 +110,7 @@ public class DefaultVerticalExtent exten
      * This is a <cite>shallow</cite> copy constructor, since the other metadata contained in the
      * given object are not recursively copied.
      *
-     * @param object The metadata to copy values from, or {@code null} if none.
+     * @param  object  the metadata to copy values from, or {@code null} if none.
      *
      * @see #castOrCopy(VerticalExtent)
      */
@@ -136,8 +137,8 @@ public class DefaultVerticalExtent exten
      *       metadata contained in the given object are not recursively copied.</li>
      * </ul>
      *
-     * @param  object The object to get as a SIS implementation, or {@code null} if none.
-     * @return A SIS implementation containing the values of the given object (may be the
+     * @param  object  the object to get as a SIS implementation, or {@code null} if none.
+     * @return a SIS implementation containing the values of the given object (may be the
      *         given object itself), or {@code null} if the argument was null.
      */
     public static DefaultVerticalExtent castOrCopy(final VerticalExtent object) {
@@ -150,7 +151,7 @@ public class DefaultVerticalExtent exten
     /**
      * Returns the lowest vertical extent contained in the dataset.
      *
-     * @return The lowest vertical extent, or {@code null}.
+     * @return the lowest vertical extent, or {@code null}.
      */
     @Override
     @XmlElement(name = "minimumValue", required = true)
@@ -162,7 +163,7 @@ public class DefaultVerticalExtent exten
     /**
      * Sets the lowest vertical extent contained in the dataset.
      *
-     * @param newValue The new minimum value.
+     * @param  newValue  the new minimum value.
      */
     public void setMinimumValue(final Double newValue) {
         checkWritePermission();
@@ -172,7 +173,7 @@ public class DefaultVerticalExtent exten
     /**
      * Returns the highest vertical extent contained in the dataset.
      *
-     * @return The highest vertical extent, or {@code null}.
+     * @return the highest vertical extent, or {@code null}.
      */
     @Override
     @XmlElement(name = "maximumValue", required = true)
@@ -184,7 +185,7 @@ public class DefaultVerticalExtent exten
     /**
      * Sets the highest vertical extent contained in the dataset.
      *
-     * @param newValue The new maximum value.
+     * @param  newValue  the new maximum value.
      */
     public void setMaximumValue(final Double newValue) {
         checkWritePermission();
@@ -196,7 +197,7 @@ public class DefaultVerticalExtent exten
      * which the maximum and minimum elevation values are measured.
      * The CRS identification includes unit of measure.
      *
-     * @return The vertical CRS, or {@code null}.
+     * @return the vertical CRS, or {@code null}.
      */
     @Override
     @XmlElement(name = "verticalCRS", required = true)
@@ -208,7 +209,7 @@ public class DefaultVerticalExtent exten
      * Sets the information about the vertical coordinate reference system to
      * which the maximum and minimum elevation values are measured.
      *
-     * @param newValue The new vertical CRS.
+     * @param  newValue  the new vertical CRS.
      */
     public void setVerticalCRS(final VerticalCRS newValue) {
         checkWritePermission();
@@ -222,7 +223,7 @@ public class DefaultVerticalExtent exten
      *
      * <p><b>Note:</b> this method is available only if the referencing module is on the classpath.</p>
      *
-     * @param  envelope The envelope to use for setting this vertical extent.
+     * @param  envelope  the envelope to use for setting this vertical extent.
      * @throws UnsupportedOperationException if the referencing module is not on the classpath.
      * @throws TransformException if the envelope can not be transformed to a vertical extent.
      *



Mime
View raw message