sis-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From desruisse...@apache.org
Subject svn commit: r1779559 [11/21] - in /sis/trunk: ./ core/sis-feature/src/main/java/org/apache/sis/feature/ core/sis-metadata/src/main/java/org/apache/sis/internal/jaxb/code/ core/sis-metadata/src/main/java/org/apache/sis/internal/jaxb/geometry/ core/sis-m...
Date Fri, 20 Jan 2017 04:34:30 GMT
Modified: sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/identification/DefaultUsage.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/identification/DefaultUsage.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/identification/DefaultUsage.java [UTF-8] (original)
+++ sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/identification/DefaultUsage.java [UTF-8] Fri Jan 20 04:34:26 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_Usage_Type", propOrder = {
     "specificUsage",
     "usageDate",
@@ -87,8 +88,7 @@ public class DefaultUsage extends ISOMet
     private InternationalString userDeterminedLimitations;
 
     /**
-     * Identification of and means of communicating with person(s) and organization(s)
-     * using the resource(s).
+     * Identification of and means of communicating with person(s) and organization(s) using the resource(s).
      */
     private Collection<ResponsibleParty> userContactInfo;
 
@@ -117,8 +117,8 @@ public class DefaultUsage extends ISOMet
     /**
      * Creates an usage initialized to the specified values.
      *
-     * @param specificUsage   Brief description of the resource and/or resource series usage, or {@code null} if none.
-     * @param userContactInfo Means of communicating with person(s) and organization(s), or {@code null} if none.
+     * @param specificUsage    brief description of the resource and/or resource series usage, or {@code null} if none.
+     * @param userContactInfo  means of communicating with person(s) and organization(s), or {@code null} if none.
      */
     public DefaultUsage(final CharSequence specificUsage,
                         final ResponsibleParty userContactInfo)
@@ -132,7 +132,7 @@ public class DefaultUsage extends ISOMet
      * 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(Usage)
      */
@@ -166,8 +166,8 @@ public class DefaultUsage extends ISOMet
      *       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 DefaultUsage castOrCopy(final Usage object) {
@@ -180,7 +180,7 @@ public class DefaultUsage extends ISOMet
     /**
      * Returns a brief description of the resource and/or resource series usage.
      *
-     * @return Description of the resource usage, or {@code null}.
+     * @return description of the resource usage, or {@code null}.
      */
     @Override
     @XmlElement(name = "specificUsage", required = true)
@@ -191,7 +191,7 @@ public class DefaultUsage extends ISOMet
     /**
      * Sets a brief description of the resource and/or resource series usage.
      *
-     * @param newValue The new specific usage.
+     * @param  newValue  the new specific usage.
      */
     public void setSpecificUsage(final InternationalString newValue) {
         checkWritePermission();
@@ -201,7 +201,7 @@ public class DefaultUsage extends ISOMet
     /**
      * Returns the date and time of the first use or range of uses of the resource and/or resource series.
      *
-     * @return Date of the first use of the resource, or {@code null}.
+     * @return date of the first use of the resource, or {@code null}.
      */
     @Override
     @XmlElement(name = "usageDateTime")
@@ -212,7 +212,7 @@ public class DefaultUsage extends ISOMet
     /**
      * Sets the date and time of the first use.
      *
-     * @param newValue The new usage date.
+     * @param  newValue  the new usage date.
      */
     public void setUsageDate(final Date newValue)  {
         checkWritePermission();
@@ -222,7 +222,7 @@ public class DefaultUsage extends ISOMet
     /**
      * Returns applications, determined by the user for which the resource and/or resource series is not suitable.
      *
-     * @return Applications for which the resource and/or resource series is not suitable, or {@code null}.
+     * @return applications for which the resource and/or resource series is not suitable, or {@code null}.
      */
     @Override
     @XmlElement(name = "userDeterminedLimitations")
@@ -233,7 +233,7 @@ public class DefaultUsage extends ISOMet
     /**
      * Sets applications, determined by the user for which the resource and/or resource series is not suitable.
      *
-     * @param newValue The new user determined limitations.
+     * @param  newValue  the new user determined limitations.
      */
     public void setUserDeterminedLimitations(final InternationalString newValue) {
         checkWritePermission();
@@ -248,7 +248,7 @@ public class DefaultUsage extends ISOMet
      * This change may be applied in GeoAPI 4.0.
      * </div>
      *
-     * @return Means of communicating with person(s) and organization(s) using the resource(s).
+     * @return means of communicating with person(s) and organization(s) using the resource(s).
      */
     @Override
     @XmlElement(name = "userContactInfo", required = true)
@@ -264,7 +264,7 @@ public class DefaultUsage extends ISOMet
      * This change may be applied in GeoAPI 4.0.
      * </div>
      *
-     * @param newValues The new user contact info.
+     * @param  newValues  the new user contact info.
      */
     public void setUserContactInfo(final Collection<? extends ResponsibleParty> newValues) {
         userContactInfo = writeCollection(newValues, userContactInfo, ResponsibleParty.class);
@@ -273,7 +273,7 @@ public class DefaultUsage extends ISOMet
     /**
      * Responses to the user-determined limitations.
      *
-     * @return Response to the user-determined limitations.
+     * @return response to the user-determined limitations.
      *
      * @since 0.5
      */
@@ -286,7 +286,7 @@ public class DefaultUsage extends ISOMet
     /**
      * Sets a new response to the user-determined limitations.
      *
-     * @param newValues The new response to the user-determined limitations.
+     * @param  newValues  the new response to the user-determined limitations.
      *
      * @since 0.5
      */
@@ -297,7 +297,7 @@ public class DefaultUsage extends ISOMet
     /**
      * Publications that describe usage of data.
      *
-     * @return Publications that describe usage of data.
+     * @return publications that describe usage of data.
      *
      * @since 0.5
      */
@@ -310,7 +310,7 @@ public class DefaultUsage extends ISOMet
     /**
      * Sets the publications that describe usage of data.
      *
-     * @param newValues The new publications.
+     * @param  newValues  the new publications.
      *
      * @since 0.5
      */
@@ -322,7 +322,7 @@ public class DefaultUsage extends ISOMet
      * Citation of a description of known issues associated with the resource
      * along with proposed solutions if available.
      *
-     * @return Citation of a description of known issues associated with the resource.
+     * @return citation of a description of known issues associated with the resource.
      *
      * @since 0.5
      */
@@ -336,7 +336,7 @@ public class DefaultUsage extends ISOMet
      * Sets a new citation of a description of known issues associated with the resource
      * along with proposed solutions if available.
      *
-     * @param newValues The new citation of a description.
+     * @param  newValues  the new citation of a description.
      *
      * @since 0.5
      */

Modified: sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/lineage/DefaultAlgorithm.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/lineage/DefaultAlgorithm.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/lineage/DefaultAlgorithm.java [UTF-8] (original)
+++ sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/lineage/DefaultAlgorithm.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -45,6 +45,7 @@ import org.apache.sis.xml.Namespaces;
  * @version 0.3
  * @module
  */
+@SuppressWarnings("CloneableClassWithoutClone")                 // ModifiableMetadata needs shallow clones.
 @XmlType(name = "LE_Algorithm_Type", propOrder = {
     "citation",
     "description"
@@ -77,7 +78,7 @@ public class DefaultAlgorithm extends IS
      * 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(Algorithm)
      */
@@ -103,8 +104,8 @@ public class DefaultAlgorithm extends IS
      *       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 DefaultAlgorithm castOrCopy(final Algorithm object) {
@@ -117,7 +118,7 @@ public class DefaultAlgorithm extends IS
     /**
      * Returns the information identifying the algorithm and version or date.
      *
-     * @return Algorithm and version or date, or {@code null}.
+     * @return algorithm and version or date, or {@code null}.
      */
     @Override
     @XmlElement(name = "citation", namespace = Namespaces.GMI, required = true)
@@ -128,7 +129,7 @@ public class DefaultAlgorithm extends IS
     /**
      * Sets the information identifying the algorithm and version or date.
      *
-     * @param newValue The new citation value.
+     * @param  newValue  the new citation value.
      */
     public void setCitation(final Citation newValue) {
         checkWritePermission();
@@ -138,7 +139,7 @@ public class DefaultAlgorithm extends IS
     /**
      * Returns the information describing the algorithm used to generate the data.
      *
-     * @return Algorithm used to generate the data, or {@code null}.
+     * @return algorithm used to generate the data, or {@code null}.
      */
     @Override
     @XmlElement(name = "description", namespace = Namespaces.GMI, required = true)
@@ -149,7 +150,7 @@ public class DefaultAlgorithm extends IS
     /**
      * Sets the information describing the algorithm used to generate the data.
      *
-     * @param newValue The new description value.
+     * @param  newValue  the new description value.
      */
     public void setDescription(final InternationalString newValue) {
         checkWritePermission();

Modified: sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/lineage/DefaultLineage.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/lineage/DefaultLineage.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/lineage/DefaultLineage.java [UTF-8] (original)
+++ sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/lineage/DefaultLineage.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -60,6 +60,7 @@ import static org.opengis.annotation.Spe
  * @version 0.5
  * @module
  */
+@SuppressWarnings("CloneableClassWithoutClone")                 // ModifiableMetadata needs shallow clones.
 @XmlType(name = "LI_Lineage_Type", propOrder = {
     "statement",
 /// "scope",
@@ -113,7 +114,7 @@ public class DefaultLineage extends ISOM
      * 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(Lineage)
      */
@@ -144,8 +145,8 @@ public class DefaultLineage extends ISOM
      *       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 DefaultLineage castOrCopy(final Lineage object) {
@@ -160,7 +161,7 @@ public class DefaultLineage extends ISOM
      * Can be provided only if {@linkplain DefaultScope#getLevel scope level}
      * is {@link ScopeCode#DATASET DATASET} or {@link ScopeCode#SERIES SERIES}.
      *
-     * @return Explanation of the data producer's knowledge about the lineage, or {@code null}.
+     * @return explanation of the data producer's knowledge about the lineage, or {@code null}.
      */
     @Override
     @XmlElement(name = "statement")
@@ -171,7 +172,7 @@ public class DefaultLineage extends ISOM
     /**
      * Sets the general explanation of the data producer’s knowledge about the lineage of a dataset.
      *
-     * @param newValue The new statement.
+     * @param  newValue  the new statement.
      */
     public void setStatement(final InternationalString newValue) {
         checkWritePermission();
@@ -181,7 +182,7 @@ public class DefaultLineage extends ISOM
     /**
      * Returns the type of resource and / or extents to which the lineage information applies.
      *
-     * @return Type of resource and / or extents to which the lineage information applies.
+     * @return type of resource and / or extents to which the lineage information applies.
      *
      * @since 0.5
      */
@@ -194,7 +195,7 @@ public class DefaultLineage extends ISOM
     /**
      * Sets the type of resource and / or extents to which the lineage information applies.
      *
-     * @param newValue The new type of resource.
+     * @param  newValue  the new type of resource.
      *
      * @since 0.5
      */
@@ -206,7 +207,7 @@ public class DefaultLineage extends ISOM
     /**
      * Returns additional documentation.
      *
-     * @return Additional documentation.
+     * @return additional documentation.
      *
      * @since 0.5
      */
@@ -230,7 +231,7 @@ public class DefaultLineage extends ISOM
     /**
      * Returns the information about about events in the life of a resource specified by the scope.
      *
-     * @return Information about events in the life of a resource.
+     * @return information about events in the life of a resource.
      */
     @Override
     @XmlElement(name = "processStep")
@@ -241,7 +242,7 @@ public class DefaultLineage extends ISOM
     /**
      * Sets information about events in the life of a resource specified by the scope.
      *
-     * @param newValues The new process steps.
+     * @param  newValues  the new process steps.
      */
     public void setProcessSteps(final Collection<? extends ProcessStep> newValues)  {
         processSteps = writeCollection(newValues, processSteps, ProcessStep.class);
@@ -250,7 +251,7 @@ public class DefaultLineage extends ISOM
     /**
      * Returns information about the source data used in creating the data specified by the scope.
      *
-     * @return Information about the source data.
+     * @return information about the source data.
      */
     @Override
     @XmlElement(name = "source")
@@ -261,7 +262,7 @@ public class DefaultLineage extends ISOM
     /**
      * Sets information about the source data used in creating the data specified by the scope.
      *
-     * @param newValues The new sources.
+     * @param  newValues  the new sources.
      */
     public void setSources(final Collection<? extends Source> newValues) {
         sources = writeCollection(newValues, sources, Source.class);

Modified: sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/lineage/DefaultNominalResolution.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/lineage/DefaultNominalResolution.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/lineage/DefaultNominalResolution.java [UTF-8] (original)
+++ sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/lineage/DefaultNominalResolution.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -45,6 +45,7 @@ import static org.apache.sis.internal.me
  * @version 0.5
  * @module
  */
+@SuppressWarnings("CloneableClassWithoutClone")                 // ModifiableMetadata needs shallow clones.
 @XmlType(name = "LE_NominalResolution_Type", propOrder = {
     "scanningResolution",
     "groundResolution"
@@ -86,7 +87,7 @@ public class DefaultNominalResolution ex
      * 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(NominalResolution)
      */
@@ -112,8 +113,8 @@ public class DefaultNominalResolution 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 DefaultNominalResolution castOrCopy(final NominalResolution object) {
@@ -127,7 +128,7 @@ public class DefaultNominalResolution ex
      * Returns the distance between consistent parts of (centre, left side, right side)
      * adjacent pixels in the scan plane.
      *
-     * @return Distance between consistent parts of adjacent pixels in the scan plane, or {@code null}.
+     * @return distance between consistent parts of adjacent pixels in the scan plane, or {@code null}.
      */
     @Override
     @ValueRange(minimum=0, isMinIncluded=false)
@@ -140,7 +141,7 @@ public class DefaultNominalResolution ex
      * Sets the distance between consistent parts of (centre, left side, right side) adjacent
      * pixels in the scan plane.
      *
-     * @param newValue The new scanning resolution value.
+     * @param  newValue  the new scanning resolution value.
      * @throws IllegalArgumentException if the given value is NaN, zero or negative.
      */
     public void setScanningResolution(final Double newValue) {
@@ -154,7 +155,7 @@ public class DefaultNominalResolution ex
      * Returns the distance between consistent parts of (centre, left side, right side) adjacent
      * pixels in the object space.
      *
-     * @return Distance between consistent parts of adjacent pixels in the object space, or {@code null}.
+     * @return distance between consistent parts of adjacent pixels in the object space, or {@code null}.
      */
     @Override
     @ValueRange(minimum=0, isMinIncluded=false)
@@ -167,7 +168,7 @@ public class DefaultNominalResolution ex
      * Sets the distance between consistent parts of (centre, left side, right side) adjacent pixels
      * in the object space.
      *
-     * @param newValue The new ground resolution value.
+     * @param  newValue  the new ground resolution value.
      * @throws IllegalArgumentException if the given value is NaN, zero or negative.
      */
     public void setGroundResolution(final Double newValue) {

Modified: sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/lineage/DefaultProcessStep.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/lineage/DefaultProcessStep.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/lineage/DefaultProcessStep.java [UTF-8] (original)
+++ sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/lineage/DefaultProcessStep.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -62,6 +62,7 @@ import static org.apache.sis.internal.me
  * @version 0.5
  * @module
  */
+@SuppressWarnings("CloneableClassWithoutClone")                 // ModifiableMetadata needs shallow clones.
 @XmlType(name = "LI_ProcessStep_Type", propOrder = {
     "description",
     "rationale",
@@ -144,7 +145,7 @@ public class DefaultProcessStep extends
     /**
      * Creates a process step initialized to the given description.
      *
-     * @param description Description of the event, including related parameters or tolerances.
+     * @param description  description of the event, including related parameters or tolerances.
      */
     public DefaultProcessStep(final CharSequence description) {
         this.description = Types.toInternationalString(description);
@@ -155,7 +156,7 @@ public class DefaultProcessStep 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(ProcessStep)
      */
@@ -191,8 +192,8 @@ public class DefaultProcessStep 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 DefaultProcessStep castOrCopy(final ProcessStep object) {
@@ -205,7 +206,7 @@ public class DefaultProcessStep extends
     /**
      * Returns the description of the event, including related parameters or tolerances.
      *
-     * @return Description of the event, or {@code null}.
+     * @return description of the event, or {@code null}.
      */
     @Override
     @XmlElement(name = "description", required = true)
@@ -216,7 +217,7 @@ public class DefaultProcessStep extends
     /**
      * Sets the description of the event, including related parameters or tolerances.
      *
-     * @param newValue The new description.
+     * @param  newValue  the new description.
      */
     public void setDescription(final InternationalString newValue) {
         checkWritePermission();
@@ -226,7 +227,7 @@ public class DefaultProcessStep extends
     /**
      * Returns the requirement or purpose for the process step.
      *
-     * @return Requirement or purpose for the process step, or {@code null}.
+     * @return requirement or purpose for the process step, or {@code null}.
      */
     @Override
     @XmlElement(name = "rationale")
@@ -237,7 +238,7 @@ public class DefaultProcessStep extends
     /**
      * Sets the requirement or purpose for the process step.
      *
-     * @param newValue The new rationale.
+     * @param  newValue  the new rationale.
      */
     public void setRationale(final InternationalString newValue) {
         checkWritePermission();
@@ -247,7 +248,7 @@ public class DefaultProcessStep extends
     /**
      * Returns the date and time or range of date and time on or over which the process step occurred.
      *
-     * @return Date on or over which the process step occurred, or {@code null}.
+     * @return date on or over which the process step occurred, or {@code null}.
      */
     @Override
     @XmlElement(name = "dateTime")
@@ -258,7 +259,7 @@ public class DefaultProcessStep extends
     /**
      * Sets the date and time or range of date and time on or over which the process step occurred.
      *
-     * @param newValue The new date.
+     * @param  newValue  the new date.
      */
     public void setDate(final Date newValue) {
         checkWritePermission();
@@ -274,7 +275,7 @@ public class DefaultProcessStep extends
      * This change may be applied in GeoAPI 4.0.
      * </div>
      *
-     * @return Means of communication with person(s) and organization(s) associated with the process step.
+     * @return means of communication with person(s) and organization(s) associated with the process step.
      */
     @Override
     @XmlElement(name = "processor")
@@ -291,7 +292,7 @@ public class DefaultProcessStep extends
      * This change may be applied in GeoAPI 4.0.
      * </div>
      *
-     * @param newValues The new processors.
+     * @param  newValues  the new processors.
      */
     public void setProcessors(final Collection<? extends ResponsibleParty> newValues) {
         processors = writeCollection(newValues, processors, ResponsibleParty.class);
@@ -300,7 +301,7 @@ public class DefaultProcessStep extends
     /**
      * Returns the process step documentation.
      *
-     * @return Process step documentation.
+     * @return process step documentation.
      *
      * @since 0.5
      */
@@ -313,7 +314,7 @@ public class DefaultProcessStep extends
     /**
      * Sets the process step documentation.
      *
-     * @param newValues The new documentation.
+     * @param  newValues  the new documentation.
      *
      * @since 0.5
      */
@@ -324,7 +325,7 @@ public class DefaultProcessStep extends
     /**
      * Returns the type of resource and / or extent to which the process step applies.
      *
-     * @return Type of resource, or {@code null} if none.
+     * @return type of resource, or {@code null} if none.
      *
      * @since 0.5
      */
@@ -337,7 +338,7 @@ public class DefaultProcessStep extends
     /**
      * Sets the type of resource and / or extent to which the process step applies.
      *
-     * @param newValue The new type of resource.
+     * @param  newValue  the new type of resource.
      *
      * @since 0.5
      */
@@ -349,7 +350,7 @@ public class DefaultProcessStep extends
     /**
      * Returns the information about the source data used in creating the data specified by the scope.
      *
-     * @return Information about the source data used in creating the data.
+     * @return information about the source data used in creating the data.
      */
     @Override
     @XmlElement(name = "source")
@@ -360,7 +361,7 @@ public class DefaultProcessStep extends
     /**
      * Information about the source data used in creating the data specified by the scope.
      *
-     * @param newValues The new sources.
+     * @param  newValues  the new sources.
      */
     public void setSources(final Collection<? extends Source> newValues) {
         sources = writeCollection(newValues, sources, Source.class);
@@ -369,7 +370,7 @@ public class DefaultProcessStep extends
     /**
      * Returns the description of the product generated as a result of the process step.
      *
-     * @return Product generated as a result of the process step.
+     * @return product generated as a result of the process step.
      */
     @Override
     @XmlElement(name = "output", namespace = Namespaces.GMI)
@@ -380,7 +381,7 @@ public class DefaultProcessStep extends
     /**
      * Sets the description of the product generated as a result of the process step.
      *
-     * @param newValues The new output values.
+     * @param  newValues  the new output values.
      */
     public void setOutputs(final Collection<? extends Source> newValues) {
         outputs = writeCollection(newValues, outputs, Source.class);
@@ -391,7 +392,7 @@ public class DefaultProcessStep extends
      * was applied to derive geographic data from the raw instrument measurements, such
      * as datasets, software used, and the processing environment.
      *
-     * @return Procedure by which the algorithm was applied to derive geographic data, or {@code null}.
+     * @return procedure by which the algorithm was applied to derive geographic data, or {@code null}.
      */
     @Override
     @XmlElement(name = "processingInformation", namespace = Namespaces.GMI)
@@ -404,7 +405,7 @@ public class DefaultProcessStep extends
      * applied to derive geographic data from the raw instrument measurements, such as
      * datasets, software used, and the processing environment.
      *
-     * @param newValue The new processing information value.
+     * @param  newValue  the new processing information value.
      */
     public void setProcessingInformation(final Processing newValue) {
         checkWritePermission();
@@ -414,7 +415,7 @@ public class DefaultProcessStep extends
     /**
      * Returns the report generated by the process step.
      *
-     * @return Report generated by the process step.
+     * @return report generated by the process step.
      */
     @Override
     @XmlElement(name = "report", namespace = Namespaces.GMI)
@@ -425,7 +426,7 @@ public class DefaultProcessStep extends
     /**
      * Sets the report generated by the process step.
      *
-     * @param newValues The new process step report values.
+     * @param  newValues  the new process step report values.
      */
     public void setReports(final Collection<? extends ProcessStepReport> newValues) {
         reports = writeCollection(newValues, reports, ProcessStepReport.class);

Modified: sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/lineage/DefaultProcessStepReport.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/lineage/DefaultProcessStepReport.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/lineage/DefaultProcessStepReport.java [UTF-8] (original)
+++ sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/lineage/DefaultProcessStepReport.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -44,6 +44,7 @@ import org.apache.sis.xml.Namespaces;
  * @version 0.3
  * @module
  */
+@SuppressWarnings("CloneableClassWithoutClone")                 // ModifiableMetadata needs shallow clones.
 @XmlType(name = "LE_ProcessStepReport_Type", propOrder = {
     "name",
     "description",
@@ -82,7 +83,7 @@ public class DefaultProcessStepReport 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(ProcessStepReport)
      */
@@ -109,8 +110,8 @@ public class DefaultProcessStepReport 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 DefaultProcessStepReport castOrCopy(final ProcessStepReport object) {
@@ -123,7 +124,7 @@ public class DefaultProcessStepReport ex
     /**
      * Returns the name of the processing report.
      *
-     * @return Name of the processing report, or {@code null}.
+     * @return name of the processing report, or {@code null}.
      */
     @Override
     @XmlElement(name = "name", namespace = Namespaces.GMI, required = true)
@@ -134,7 +135,7 @@ public class DefaultProcessStepReport ex
     /**
      * Sets the name of the processing report.
      *
-     * @param newValue The new name value.
+     * @param  newValue  the new name value.
      */
     public void setName(final InternationalString newValue) {
         checkWritePermission();
@@ -144,7 +145,7 @@ public class DefaultProcessStepReport ex
     /**
      * Returns the textual description of what occurred during the process step.
      *
-     * @return What occurred during the process step, or {@code null}.
+     * @return what occurred during the process step, or {@code null}.
      */
     @Override
     @XmlElement(name = "description", namespace = Namespaces.GMI)
@@ -155,7 +156,7 @@ public class DefaultProcessStepReport ex
     /**
      * Sets the textual description of what occurred during the process step.
      *
-     * @param newValue The new description value.
+     * @param  newValue  the new description value.
      */
     public void setDescription(final InternationalString newValue) {
         checkWritePermission();
@@ -165,7 +166,7 @@ public class DefaultProcessStepReport ex
     /**
      * Returns the type of file that contains the processing report.
      *
-     * @return Type of file that contains the processing report, or {@code null}.
+     * @return type of file that contains the processing report, or {@code null}.
      */
     @Override
     @XmlElement(name = "fileType", namespace = Namespaces.GMI)
@@ -176,7 +177,7 @@ public class DefaultProcessStepReport ex
     /**
      * Sets the type of file that contains the processing report.
      *
-     * @param newValue The new file type value.
+     * @param  newValue  the new file type value.
      */
     public void setFileType(final InternationalString newValue) {
         checkWritePermission();

Modified: sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/lineage/DefaultProcessing.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/lineage/DefaultProcessing.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/lineage/DefaultProcessing.java [UTF-8] (original)
+++ sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/lineage/DefaultProcessing.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -49,6 +49,7 @@ import org.apache.sis.internal.jaxb.NonM
  * @version 0.3
  * @module
  */
+@SuppressWarnings("CloneableClassWithoutClone")                 // ModifiableMetadata needs shallow clones.
 @XmlType(name = "LE_Processing_Type", propOrder = {
     "identifier",
     "softwareReferences",
@@ -101,7 +102,7 @@ public class DefaultProcessing extends I
      * 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(Processing)
      */
@@ -131,8 +132,8 @@ public class DefaultProcessing extends I
      *       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 DefaultProcessing castOrCopy(final Processing object) {
@@ -145,7 +146,7 @@ public class DefaultProcessing extends I
     /**
      * Returns the information to identify the processing package that produced the data.
      *
-     * @return Identifier of the processing package that produced the data, or {@code null}.
+     * @return identifier of the processing package that produced the data, or {@code null}.
      */
     @Override
     @XmlElement(name = "identifier", namespace = Namespaces.GMI, required = true)
@@ -156,7 +157,7 @@ public class DefaultProcessing extends I
     /**
      * Sets the information to identify the processing package that produced the data.
      *
-     * @param newValue The new identifier value.
+     * @param  newValue  the new identifier value.
      */
     public void setIdentifier(final Identifier newValue) {
         checkWritePermission();
@@ -167,7 +168,7 @@ public class DefaultProcessing extends I
     /**
      * Returns the reference to document describing processing software.
      *
-     * @return Document describing processing software.
+     * @return document describing processing software.
      */
     @Override
     @XmlElement(name = "softwareReference", namespace = Namespaces.GMI)
@@ -178,7 +179,7 @@ public class DefaultProcessing extends I
     /**
      * Sets the reference to document describing processing software.
      *
-     * @param newValues The new software references values.
+     * @param  newValues  the new software references values.
      */
     public void setSoftwareReferences(final Collection<? extends Citation> newValues) {
         softwareReferences = writeCollection(newValues, softwareReferences, Citation.class);
@@ -187,7 +188,7 @@ public class DefaultProcessing extends I
     /**
      * Returns the additional details about the processing procedures. {@code null} if unspecified.
      *
-     * @return Processing procedures, or {@code null}.
+     * @return processing procedures, or {@code null}.
      */
     @Override
     @XmlElement(name = "procedureDescription", namespace = Namespaces.GMI)
@@ -198,7 +199,7 @@ public class DefaultProcessing extends I
     /**
      * Sets the additional details about the processing procedures.
      *
-     * @param newValue The new procedure description value.
+     * @param  newValue  the new procedure description value.
      */
     public void setProcedureDescription(final InternationalString newValue) {
         checkWritePermission();
@@ -208,7 +209,7 @@ public class DefaultProcessing extends I
     /**
      * Returns the reference to documentation describing the processing.
      *
-     * @return Documentation describing the processing.
+     * @return documentation describing the processing.
      */
     @Override
     @XmlElement(name = "documentation", namespace = Namespaces.GMI)
@@ -219,7 +220,7 @@ public class DefaultProcessing extends I
     /**
      * Sets the reference to documentation describing the processing.
      *
-     * @param newValues The new documentations values.
+     * @param  newValues  the new documentations values.
      */
     public void setDocumentations(final Collection<? extends Citation> newValues) {
         documentations = writeCollection(newValues, documentations, Citation.class);
@@ -228,7 +229,7 @@ public class DefaultProcessing extends I
     /**
      * Returns the parameters to control the processing operations, entered at run time.
      *
-     * @return Parameters to control the processing operations, or {@code null}.
+     * @return parameters to control the processing operations, or {@code null}.
      */
     @Override
     @XmlElement(name = "runTimeParameters", namespace = Namespaces.GMI)
@@ -239,7 +240,7 @@ public class DefaultProcessing extends I
     /**
      * Sets the parameters to control the processing operations, entered at run time.
      *
-     * @param newValue The new runtime parameter value.
+     * @param  newValue  the new runtime parameter value.
      */
     public void setRunTimeParameters(final InternationalString newValue) {
         checkWritePermission();
@@ -250,7 +251,7 @@ public class DefaultProcessing extends I
      * Returns the details of the methodology by which geographic information was derived from the
      * instrument readings.
      *
-     * @return Methodology by which geographic information was derived from the instrument readings.
+     * @return methodology by which geographic information was derived from the instrument readings.
      */
     @Override
     @XmlElement(name = "algorithm", namespace = Namespaces.GMI)
@@ -262,7 +263,7 @@ public class DefaultProcessing extends I
      * Sets the details of the methodology by which geographic information was derived from the
      * instrument readings.
      *
-     * @param newValues The new algorithms values.
+     * @param  newValues  the new algorithms values.
      */
     public void setAlgorithms(final Collection<? extends Algorithm> newValues) {
         algorithms = writeCollection(newValues, algorithms, Algorithm.class);

Modified: sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/lineage/DefaultSource.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/lineage/DefaultSource.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/lineage/DefaultSource.java [UTF-8] (original)
+++ sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/lineage/DefaultSource.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -69,6 +69,7 @@ import static org.opengis.annotation.Spe
  * @version 0.5
  * @module
  */
+@SuppressWarnings("CloneableClassWithoutClone")                 // ModifiableMetadata needs shallow clones.
 @XmlType(name = "LI_Source_Type", propOrder = {
     "description",
     "scaleDenominator",
@@ -140,7 +141,7 @@ public class DefaultSource extends ISOMe
     /**
      * Creates a source initialized with the given description.
      *
-     * @param description A detailed description of the level of the source data, or {@code null}.
+     * @param description  a detailed description of the level of the source data, or {@code null}.
      */
     public DefaultSource(final CharSequence description) {
         this.description = Types.toInternationalString(description);
@@ -151,7 +152,7 @@ public class DefaultSource extends ISOMe
      * 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(Source)
      */
@@ -189,8 +190,8 @@ public class DefaultSource 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 DefaultSource castOrCopy(final Source object) {
@@ -203,7 +204,7 @@ public class DefaultSource extends ISOMe
     /**
      * Returns a detailed description of the level of the source data.
      *
-     * @return Description of the level of the source data, or {@code null}.
+     * @return description of the level of the source data, or {@code null}.
      */
     @Override
     @XmlElement(name = "description")
@@ -214,7 +215,7 @@ public class DefaultSource extends ISOMe
     /**
      * Sets a detailed description of the level of the source data.
      *
-     * @param newValue The new description.
+     * @param  newValue  the new description.
      */
     public void setDescription(final InternationalString newValue) {
         checkWritePermission();
@@ -224,7 +225,7 @@ public class DefaultSource extends ISOMe
     /**
      * Returns the spatial resolution expressed as a scale factor, an angle or a level of detail.
      *
-     * @return Spatial resolution expressed as a scale factor, an angle or a level of detail, or {@code null} if none.
+     * @return spatial resolution expressed as a scale factor, an angle or a level of detail, or {@code null} if none.
      *
      * @since 0.5
      */
@@ -237,7 +238,7 @@ public class DefaultSource extends ISOMe
     /**
      * Sets the spatial resolution expressed as a scale factor, an angle or a level of detail.
      *
-     * @param newValue The new spatial resolution.
+     * @param  newValue  the new spatial resolution.
      *
      * @since 0.5
      */
@@ -251,7 +252,7 @@ public class DefaultSource extends ISOMe
      * This method fetches the value from the
      * {@linkplain #getSourceSpatialResolution() source spatial resolution}.
      *
-     * @return Representative fraction on a source map, or {@code null}.
+     * @return representative fraction on a source map, or {@code null}.
      *
      * @deprecated As of ISO 19115:2014, moved to {@link DefaultResolution#getEquivalentScale()}.
      */
@@ -268,7 +269,7 @@ public class DefaultSource extends ISOMe
      * This method stores the value in the
      * {@linkplain #setSourceSpatialResolution(Resolution) source spatial resolution}.
      *
-     * @param newValue The new scale denominator.
+     * @param  newValue  the new scale denominator.
      *
      * @deprecated As of ISO 19115:2014, moved to {@link DefaultResolution#setEquivalentScale(RepresentativeFraction)}.
      */
@@ -284,8 +285,10 @@ public class DefaultSource extends ISOMe
                 resolution = new DefaultResolution(newValue);
             }
         }
-        // Invoke the non-deprecated setter method only if the reference changed,
-        // for consistency with other deprecated setter methods in metadata module.
+        /*
+         * Invoke the non-deprecated setter method only if the reference changed,
+         * for consistency with other deprecated setter methods in metadata module.
+         */
         if (resolution != sourceSpatialResolution) {
             setSourceSpatialResolution(resolution);
         }
@@ -294,7 +297,7 @@ public class DefaultSource extends ISOMe
     /**
      * Returns the spatial reference system used by the source data.
      *
-     * @return Spatial reference system used by the source data, or {@code null}.
+     * @return spatial reference system used by the source data, or {@code null}.
      *
      * @todo We need to annotate the referencing module before we can annotate this method.
      */
@@ -307,7 +310,7 @@ public class DefaultSource extends ISOMe
     /**
      * Sets the spatial reference system used by the source data.
      *
-     * @param newValue The new reference system.
+     * @param  newValue  the new reference system.
      */
     public void setSourceReferenceSystem(final ReferenceSystem newValue) {
         checkWritePermission();
@@ -317,7 +320,7 @@ public class DefaultSource extends ISOMe
     /**
      * Returns the recommended reference to be used for the source data.
      *
-     * @return Recommended reference to be used for the source data, or {@code null}.
+     * @return recommended reference to be used for the source data, or {@code null}.
      */
     @Override
     @XmlElement(name = "sourceCitation")
@@ -328,7 +331,7 @@ public class DefaultSource extends ISOMe
     /**
      * Sets the recommended reference to be used for the source data.
      *
-     * @param newValue The new source citation.
+     * @param  newValue  the new source citation.
      */
     public void setSourceCitation(final Citation newValue) {
         checkWritePermission();
@@ -338,7 +341,7 @@ public class DefaultSource extends ISOMe
     /**
      * Returns the references to metadata for the source.
      *
-     * @return References to metadata for the source.
+     * @return references to metadata for the source.
      *
      * @since 0.5
      */
@@ -351,7 +354,7 @@ public class DefaultSource extends ISOMe
     /**
      * Sets the references to metadata for the source.
      *
-     * @param newValues The new references.
+     * @param  newValues  the new references.
      *
      * @since 0.5
      */
@@ -363,7 +366,7 @@ public class DefaultSource extends ISOMe
      * Return the type and / or extent of the source.
      * This information should be provided if the {@linkplain #getDescription() description} is not provided.
      *
-     * @return Type and / or extent of the source, or {@code null} if none.
+     * @return type and / or extent of the source, or {@code null} if none.
      *
      * @since 0.5
      */
@@ -376,7 +379,7 @@ public class DefaultSource extends ISOMe
     /**
      * Sets the type and / or extent of the source.
      *
-     * @param newValue The new type and / or extent of the source.
+     * @param  newValue  the new type and / or extent of the source.
      *
      * @since 0.5
      */
@@ -389,7 +392,7 @@ public class DefaultSource extends ISOMe
      * Returns the information about the spatial, vertical and temporal extent of the source data.
      * This method fetches the values from the {@linkplain #getScope() scope}.
      *
-     * @return Information about the extent of the source data.
+     * @return information about the extent of the source data.
      *
      * @deprecated As of ISO 19115:2014, moved to {@link DefaultScope#getExtents()}.
      */
@@ -413,7 +416,7 @@ public class DefaultSource extends ISOMe
      * Information about the spatial, vertical and temporal extent of the source data.
      * This method stores the values in the {@linkplain #setScope(Scope) scope}.
      *
-     * @param newValues The new source extents.
+     * @param  newValues  the new source extents.
      *
      * @deprecated As of ISO 19115:2014, moved to {@link DefaultScope#setExtents(Collection)}.
      */
@@ -431,7 +434,7 @@ public class DefaultSource extends ISOMe
     /**
      * Returns information about process steps in which this source was used.
      *
-     * @return Information about process steps in which this source was used.
+     * @return information about process steps in which this source was used.
      */
     @Override
     @XmlElement(name = "sourceStep")
@@ -442,7 +445,7 @@ public class DefaultSource extends ISOMe
     /**
      * Sets information about process steps in which this source was used.
      *
-     * @param newValues The new process steps.
+     * @param  newValues  the new process steps.
      */
     public void setSourceSteps(final Collection<? extends ProcessStep> newValues) {
         sourceSteps = writeCollection(newValues, sourceSteps, ProcessStep.class);
@@ -451,7 +454,7 @@ public class DefaultSource extends ISOMe
     /**
      * Returns the processing level of the source data. {@code null} if unspecified.
      *
-     * @return Processing level of the source data, or {@code null}.
+     * @return processing level of the source data, or {@code null}.
      */
     @Override
     @XmlElement(name = "processedLevel", namespace = Namespaces.GMI)
@@ -462,7 +465,7 @@ public class DefaultSource extends ISOMe
     /**
      * Sets the processing level of the source data.
      *
-     * @param newValue The new processed level value.
+     * @param  newValue  the new processed level value.
      */
     public void setProcessedLevel(final Identifier newValue) {
         checkWritePermission();
@@ -472,7 +475,7 @@ public class DefaultSource extends ISOMe
     /**
      * Returns the distance between consistent parts (centre, left side, right side) of two adjacent pixels.
      *
-     * @return Distance between consistent parts of two adjacent pixels, or {@code null}.
+     * @return distance between consistent parts of two adjacent pixels, or {@code null}.
      */
     @Override
     @XmlElement(name = "resolution", namespace = Namespaces.GMI)
@@ -483,7 +486,7 @@ public class DefaultSource extends ISOMe
     /**
      * Sets the distance between consistent parts (centre, left side, right side) of two adjacent pixels.
      *
-     * @param newValue The new nominal resolution value.
+     * @param  newValue  the new nominal resolution value.
      */
     public void setResolution(final NominalResolution newValue) {
         checkWritePermission();

Modified: sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/maintenance/DefaultMaintenanceInformation.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/maintenance/DefaultMaintenanceInformation.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/maintenance/DefaultMaintenanceInformation.java [UTF-8] (original)
+++ sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/maintenance/DefaultMaintenanceInformation.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -63,6 +63,7 @@ import static org.opengis.annotation.Spe
  * @version 0.5
  * @module
  */
+@SuppressWarnings("CloneableClassWithoutClone")                 // ModifiableMetadata needs shallow clones.
 @XmlType(name = "MD_MaintenanceInformation_Type", propOrder = {
     "maintenanceAndUpdateFrequency",
     "dateOfNextUpdate",
@@ -125,7 +126,7 @@ public class DefaultMaintenanceInformati
     /**
      * Creates a maintenance information.
      *
-     * @param maintenanceAndUpdateFrequency The frequency with which changes and additions are
+     * @param maintenanceAndUpdateFrequency  the frequency with which changes and additions are
      *        made to the resource after the initial resource is completed, or {@code null} if none.
      */
     public DefaultMaintenanceInformation(final MaintenanceFrequency maintenanceAndUpdateFrequency) {
@@ -137,7 +138,7 @@ public class DefaultMaintenanceInformati
      * 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(MaintenanceInformation)
      */
@@ -174,8 +175,8 @@ public class DefaultMaintenanceInformati
      *       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 DefaultMaintenanceInformation castOrCopy(final MaintenanceInformation object) {
@@ -189,7 +190,7 @@ public class DefaultMaintenanceInformati
      * Returns the frequency with which changes and additions are made to the resource
      * after the initial resource is completed.
      *
-     * @return Frequency with which changes and additions are made to the resource, or {@code null}.
+     * @return frequency with which changes and additions are made to the resource, or {@code null}.
      */
     @Override
     @XmlElement(name = "maintenanceAndUpdateFrequency", required = true)
@@ -201,7 +202,7 @@ public class DefaultMaintenanceInformati
      * Sets the frequency with which changes and additions are made to the resource
      * after the initial resource is completed.
      *
-     * @param newValue The new maintenance frequency.
+     * @param  newValue  the new maintenance frequency.
      */
     public void setMaintenanceAndUpdateFrequency(final MaintenanceFrequency newValue) {
         checkWritePermission();
@@ -211,7 +212,7 @@ public class DefaultMaintenanceInformati
     /**
      * Return the date information associated with maintenance of resource.
      *
-     * @return Date information associated with maintenance of resource.
+     * @return date information associated with maintenance of resource.
      *
      * @since 0.5
      */
@@ -224,7 +225,7 @@ public class DefaultMaintenanceInformati
     /**
      * Sets the date information associated with maintenance of resource.
      *
-     * @param newValues The new date information associated with maintenance of resource.
+     * @param  newValues  the new date information associated with maintenance of resource.
      *
      * @since 0.5
      */
@@ -236,7 +237,7 @@ public class DefaultMaintenanceInformati
      * Returns the scheduled revision date for resource.
      * This method fetches the value from the {@linkplain #getMaintenanceDates() maintenance dates}.
      *
-     * @return Scheduled revision date, or {@code null}.
+     * @return scheduled revision date, or {@code null}.
      *
      * @deprecated As of ISO 19115:2014, replaced by {@link #getMaintenanceDates()} in order to enable inclusion
      *             of a {@link DateType} to describe the type of the date. The associated date type is
@@ -261,7 +262,7 @@ public class DefaultMaintenanceInformati
      * Sets the scheduled revision date for resource.
      * This method stores the value in the {@linkplain #getMaintenanceDates() maintenance dates}.
      *
-     * @param newValue The new date of next update.
+     * @param  newValue  the new date of next update.
      */
     @Deprecated
     public void setDateOfNextUpdate(final Date newValue) {
@@ -296,7 +297,7 @@ public class DefaultMaintenanceInformati
     /**
      * Returns the maintenance period other than those defined.
      *
-     * @return The maintenance period, or {@code null}.
+     * @return the maintenance period, or {@code null}.
      */
     @Override
     @XmlElement(name = "userDefinedMaintenanceFrequency")
@@ -307,7 +308,7 @@ public class DefaultMaintenanceInformati
     /**
      * Sets the maintenance period other than those defined.
      *
-     * @param newValue The new user defined maintenance frequency.
+     * @param  newValue  the new user defined maintenance frequency.
      */
     public void setUserDefinedMaintenanceFrequency(final PeriodDuration newValue) {
         checkWritePermission();
@@ -330,7 +331,7 @@ public class DefaultMaintenanceInformati
     /**
      * Sets the types of resource and / or extents to which the maintenance information applies.
      *
-     * @param newValues The types of resource and / or extents to which the maintenance information applies.
+     * @param  newValues  the types of resource and / or extents to which the maintenance information applies.
      *
      * @since 0.5
      */
@@ -342,7 +343,7 @@ public class DefaultMaintenanceInformati
      * Returns the scope of data to which maintenance is applied.
      * This method fetches the values from the {@linkplain #getMaintenanceScopes() maintenance scopes}.
      *
-     * @return Scope of data to which maintenance is applied.
+     * @return scope of data to which maintenance is applied.
      *
      * @deprecated As of ISO 19115:2014, {@code getUpdateScopes()} and {@link #getUpdateScopeDescriptions()}
      *             were combined into {@link #getMaintenanceScopes()} in order to allow specifying a scope
@@ -378,7 +379,7 @@ public class DefaultMaintenanceInformati
      * Sets the scope of data to which maintenance is applied.
      * This method stores the values in the {@linkplain #getMaintenanceScopes() maintenance scopes}.
      *
-     * @param newValues The new update scopes.
+     * @param  newValues  the new update scopes.
      *
      * @deprecated As of ISO 19115:2014, replaced by {@link #setMaintenanceScopes(Collection)}.
      */
@@ -392,7 +393,7 @@ public class DefaultMaintenanceInformati
      * Returns additional information about the range or extent of the resource.
      * This method fetches the values from the {@linkplain #getMaintenanceScopes() maintenance scopes}.
      *
-     * @return Additional information about the range or extent of the resource.
+     * @return additional information about the range or extent of the resource.
      *
      * @deprecated As of ISO 19115:2014, {@link #getUpdateScopes()} and {@code getUpdateScopeDescriptions()}
      *             were combined into {@link #getMaintenanceScopes()} in order to allow specifying a scope
@@ -431,7 +432,7 @@ public class DefaultMaintenanceInformati
      * Sets additional information about the range or extent of the resource.
      * This method stores the values in the {@linkplain #getMaintenanceScopes() maintenance scopes}.
      *
-     * @param newValues The new update scope descriptions.
+     * @param  newValues  the new update scope descriptions.
      *
      * @deprecated As of ISO 19115:2014, replaced by {@link #setMaintenanceScopes(Collection)}.
      */
@@ -444,7 +445,7 @@ public class DefaultMaintenanceInformati
     /**
      * Returns information regarding specific requirements for maintaining the resource.
      *
-     * @return Information regarding specific requirements for maintaining the resource.
+     * @return information regarding specific requirements for maintaining the resource.
      */
     @Override
     @XmlElement(name = "maintenanceNote")
@@ -455,7 +456,7 @@ public class DefaultMaintenanceInformati
     /**
      * Sets information regarding specific requirements for maintaining the resource.
      *
-     * @param newValues The new maintenance notes.
+     * @param  newValues  the new maintenance notes.
      */
     public void setMaintenanceNotes(final Collection<? extends InternationalString> newValues) {
         maintenanceNotes = writeCollection(newValues, maintenanceNotes, InternationalString.class);
@@ -470,7 +471,7 @@ public class DefaultMaintenanceInformati
      * This change may be applied in GeoAPI 4.0.
      * </div>
      *
-     * @return Means of communicating with person(s) and organization(s) with responsibility
+     * @return means of communicating with person(s) and organization(s) with responsibility
      *         for maintaining the resource.
      */
     @Override
@@ -488,7 +489,7 @@ public class DefaultMaintenanceInformati
      * This change may be applied in GeoAPI 4.0.
      * </div>
      *
-     * @param newValues The new identification of person(s) and organization(s)
+     * @param  newValues  the new identification of person(s) and organization(s)
      *                  with responsibility for maintaining the resource.
      */
     public void setContacts(final Collection<? extends ResponsibleParty> newValues) {

Modified: sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/maintenance/DefaultScope.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/maintenance/DefaultScope.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/maintenance/DefaultScope.java [UTF-8] (original)
+++ sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/maintenance/DefaultScope.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -50,6 +50,7 @@ import static org.opengis.annotation.Spe
  * @version 0.5
  * @module
  */
+@SuppressWarnings("CloneableClassWithoutClone")                 // ModifiableMetadata needs shallow clones.
 @XmlType(name = "DQ_Scope_Type", propOrder = {
    "level",
    "extents",
@@ -86,7 +87,7 @@ public class DefaultScope extends ISOMet
     /**
      * Creates a scope initialized to the given level.
      *
-     * @param level The hierarchical level of the data specified by the scope.
+     * @param level  the hierarchical level of the data specified by the scope.
      */
     public DefaultScope(final ScopeCode level) {
         this.level = level;
@@ -97,7 +98,7 @@ public class DefaultScope extends ISOMet
      * 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(Scope)
      */
@@ -128,8 +129,8 @@ public class DefaultScope extends ISOMet
      *       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 DefaultScope castOrCopy(final Scope object) {
@@ -142,7 +143,7 @@ public class DefaultScope extends ISOMet
     /**
      * Returns the hierarchical level of the data specified by the scope.
      *
-     * @return Hierarchical level of the data, or {@code null}.
+     * @return hierarchical level of the data, or {@code null}.
      */
     @Override
     @XmlElement(name = "level", required = true)
@@ -153,7 +154,7 @@ public class DefaultScope extends ISOMet
     /**
      * Sets the hierarchical level of the data specified by the scope.
      *
-     * @param newValue The new level.
+     * @param  newValue  the new level.
      */
     public void setLevel(final ScopeCode newValue) {
         checkWritePermission();
@@ -163,7 +164,7 @@ public class DefaultScope extends ISOMet
     /**
      * Returns information about the spatial, vertical and temporal extents of the resource specified by the scope.
      *
-     * @return Information about the extent of the resource.
+     * @return information about the extent of the resource.
      *
      * @since 0.5
      */
@@ -214,7 +215,7 @@ public class DefaultScope extends ISOMet
     /**
      * Returns detailed descriptions about the level of the data specified by the scope.
      *
-     * @return Detailed description about the level of the data.
+     * @return detailed description about the level of the data.
      */
     @Override
     @XmlElement(name = "levelDescription")
@@ -225,7 +226,7 @@ public class DefaultScope extends ISOMet
     /**
      * Sets detailed descriptions about the level of the data specified by the scope.
      *
-     * @param newValues The new level description.
+     * @param  newValues  the new level description.
      */
     public void setLevelDescription(final Collection<? extends ScopeDescription> newValues) {
         levelDescription = writeCollection(newValues, levelDescription, ScopeDescription.class);

Modified: sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/maintenance/DefaultScopeDescription.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/maintenance/DefaultScopeDescription.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/maintenance/DefaultScopeDescription.java [UTF-8] (original)
+++ sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/maintenance/DefaultScopeDescription.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -62,6 +62,7 @@ import org.opengis.feature.type.FeatureT
  * @version 0.5
  * @module
  */
+@SuppressWarnings("CloneableClassWithoutClone")                 // ModifiableMetadata needs shallow clones.
 @XmlType(name = "MD_ScopeDescription_Type") // No need for propOrder since this structure is a union (see javadoc).
 @XmlRootElement(name = "MD_ScopeDescription")
 public class DefaultScopeDescription extends ISOMetadata implements ScopeDescription {
@@ -139,7 +140,7 @@ public class DefaultScopeDescription ext
      * {@linkplain #getAttributeInstances() attribute instances}
      * and {@linkplain #getOther() other}.</p>
      *
-     * @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(ScopeDescription)
      */
@@ -193,8 +194,8 @@ public class DefaultScopeDescription ext
      *       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 DefaultScopeDescription castOrCopy(final ScopeDescription object) {
@@ -229,8 +230,10 @@ public class DefaultScopeDescription ext
                        ? null : new ExcludedSet<E>(NAMES[code-1], NAMES[property-1]);
             }
         }
-        // Unconditionally create a new set, because the
-        // user may hold a reference to the previous one.
+        /*
+         * Unconditionally create a new set, because the
+         * user may hold a reference to the previous one.
+         */
         final Set<E> c = nonNullSet(null, type);
         property = code;
         this.value = c;
@@ -241,8 +244,8 @@ public class DefaultScopeDescription ext
      * Sets the properties identified by the {@code code} argument, if non-null and non-empty.
      * This discards any other properties.
      *
-     * @param caller The caller method, for logging purpose.
-     * @param code   The property which is going to be set.
+     * @param caller  the caller method, for logging purpose.
+     * @param code    the property which is going to be set.
      */
     private <E> void setProperty(final Set<? extends E> newValue, final Class<E> type, final byte code) {
         Set<E> c = null;
@@ -260,7 +263,7 @@ public class DefaultScopeDescription ext
     /**
      * Sends a warning if setting the value for the given property would overwrite an existing property.
      *
-     * @param code The property which is going to be set.
+     * @param  code  the property which is going to be set.
      */
     private void warningOnOverwrite(final byte code) {
         if (value != null && property != code) {
@@ -279,7 +282,7 @@ public class DefaultScopeDescription ext
      * “<cite>Administrative area A, B &amp; C</cite>” description.
      * </div>
      *
-     * @return Dataset to which the information applies, or {@code null}.
+     * @return dataset to which the information applies, or {@code null}.
      */
     @Override
     @XmlElement(name = "dataset")
@@ -294,7 +297,7 @@ public class DefaultScopeDescription ext
      * If and only if the {@code newValue} is non-null, then this method automatically
      * discards all other properties.
      *
-     * @param newValue The new dataset.
+     * @param  newValue  the new dataset.
      */
     public void setDataset(final String newValue) {
         checkWritePermission();
@@ -322,7 +325,7 @@ public class DefaultScopeDescription ext
      * The type of this property may be changed to {@code Set<CharSequence>} for ISO 19115:2014 conformance.
      * See <a href="http://jira.codehaus.org/browse/GEO-238">GEO-238</a> for more information.</div>
      *
-     * @return Feature types to which the information applies.
+     * @return feature types to which the information applies.
      */
     @Override
     public Set<FeatureType> getFeatures() {
@@ -340,7 +343,7 @@ public class DefaultScopeDescription ext
      * The type of this property may be changed to {@code Set<CharSequence>} for ISO 19115:2014 conformance.
      * See <a href="http://jira.codehaus.org/browse/GEO-238">GEO-238</a> for more information.</div>
      *
-     * @param newValues The new feature types.
+     * @param  newValues  the new feature types.
      */
     public void setFeatures(final Set<? extends FeatureType> newValues) {
         setProperty(newValues, FeatureType.class, FEATURES);
@@ -363,7 +366,7 @@ public class DefaultScopeDescription ext
      * The type of this property may be changed to {@code Set<CharSequence>} for ISO 19115:2014 conformance.
      * See <a href="http://jira.codehaus.org/browse/GEO-238">GEO-238</a> for more information.</div>
      *
-     * @return Attribute types to which the information applies.
+     * @return attribute types to which the information applies.
      */
     @Override
     public Set<AttributeType> getAttributes() {
@@ -381,7 +384,7 @@ public class DefaultScopeDescription ext
      * The type of this property may be changed to {@code Set<CharSequence>} for ISO 19115:2014 conformance.
      * See <a href="http://jira.codehaus.org/browse/GEO-238">GEO-238</a> for more information.</div>
      *
-     * @param newValues The new attribute types.
+     * @param  newValues  the new attribute types.
      */
     public void setAttributes(final Set<? extends AttributeType> newValues) {
         setProperty(newValues, AttributeType.class, ATTRIBUTES);
@@ -404,7 +407,7 @@ public class DefaultScopeDescription ext
      * The type of this property may be changed to {@code Set<CharSequence>} for ISO 19115:2014 conformance.
      * See <a href="http://jira.codehaus.org/browse/GEO-238">GEO-238</a> for more information.</div>
      *
-     * @return Feature instances to which the information applies.
+     * @return feature instances to which the information applies.
      */
     @Override
     public Set<FeatureType> getFeatureInstances() {
@@ -422,7 +425,7 @@ public class DefaultScopeDescription ext
      * The type of this property may be changed to {@code Set<CharSequence>} for ISO 19115:2014 conformance.
      * See <a href="http://jira.codehaus.org/browse/GEO-238">GEO-238</a> for more information.</div>
      *
-     * @param newValues The new feature instances.
+     * @param  newValues  the new feature instances.
      */
     public void setFeatureInstances(final Set<? extends FeatureType> newValues) {
         setProperty(newValues, FeatureType.class, FEATURE_INSTANCES);
@@ -445,7 +448,7 @@ public class DefaultScopeDescription ext
      * The type of this property may be changed to {@code Set<CharSequence>} for ISO 19115:2014 conformance.
      * See <a href="http://jira.codehaus.org/browse/GEO-238">GEO-238</a> for more information.</div>
      *
-     * @return Attribute instances to which the information applies.
+     * @return attribute instances to which the information applies.
      */
     @Override
     public Set<AttributeType> getAttributeInstances() {
@@ -463,7 +466,7 @@ public class DefaultScopeDescription ext
      * The type of this property may be changed to {@code Set<CharSequence>} for ISO 19115:2014 conformance.
      * See <a href="http://jira.codehaus.org/browse/GEO-238">GEO-238</a> for more information.</div>
      *
-     * @param newValues The new attribute instances.
+     * @param  newValues  the new attribute instances.
      */
     public void setAttributeInstances(final Set<? extends AttributeType> newValues) {
         setProperty(newValues, AttributeType.class, ATTRIBUTE_INSTANCES);
@@ -476,7 +479,7 @@ public class DefaultScopeDescription ext
      * The type of this property may be changed to {@link InternationalString} for ISO 19115:2014 conformance.
      * See <a href="http://jira.codehaus.org/browse/GEO-221">GEO-221</a> for more information.</div>
      *
-     * @return Class of information that does not fall into the other categories, or {@code null}.
+     * @return class of information that does not fall into the other categories, or {@code null}.
      */
     @Override
     @XmlElement(name = "other")

Modified: sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/quality/AbstractCompleteness.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/quality/AbstractCompleteness.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/quality/AbstractCompleteness.java [UTF-8] (original)
+++ sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/quality/AbstractCompleteness.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -42,6 +42,7 @@ import org.opengis.metadata.quality.Comp
  * @version 0.3
  * @module
  */
+@SuppressWarnings("CloneableClassWithoutClone")                 // ModifiableMetadata needs shallow clones.
 @XmlType(name = "AbstractDQ_Completeness_Type")
 @XmlRootElement(name = "DQ_Completeness")
 @XmlSeeAlso({
@@ -65,7 +66,7 @@ public class AbstractCompleteness extend
      * 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(Completeness)
      */
@@ -92,8 +93,8 @@ public class AbstractCompleteness extend
      *       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 AbstractCompleteness castOrCopy(final Completeness object) {

Modified: sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/quality/AbstractElement.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/quality/AbstractElement.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/quality/AbstractElement.java [UTF-8] (original)
+++ sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/quality/AbstractElement.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -64,6 +64,7 @@ import static org.apache.sis.util.collec
  * @version 0.3
  * @module
  */
+@SuppressWarnings("CloneableClassWithoutClone")                 // ModifiableMetadata needs shallow clones.
 @XmlType(name = "AbstractDQ_Element_Type", propOrder = {
     "namesOfMeasure",
     "measureIdentification",
@@ -183,8 +184,8 @@ public class AbstractElement extends ISO
         public Date get(final int index) {
             long date = date1;
             switch (index) {
-                case 1:  date = date2; // Fall through
-                case 0:  if (date != Long.MIN_VALUE) return new Date(date); // else fallthrough.
+                case 1:  date = date2;                                          // Fall through
+                case 0:  if (date != Long.MIN_VALUE) return new Date(date);     // else fallthrough.
                 default: throw new IndexOutOfBoundsException(Errors.format(Errors.Keys.IndexOutOfBounds_1, index));
             }
         }
@@ -213,7 +214,7 @@ public class AbstractElement extends ISO
         public Date remove(final int index) {
             final Date previous = get(index);
             switch (index) {
-                case 0: date1 = date2; // Fallthrough
+                case 0: date1 = date2;                      // Fallthrough
                 case 1: date2 = Long.MIN_VALUE; break;
             }
             modCount++;
@@ -534,7 +535,7 @@ public class AbstractElement extends ISO
      */
     public void setDates(final Collection<? extends Date> newValues) {
         checkWritePermission();
-        if (newValues != dates) { // Mandatory check for avoiding the call to 'dates.clear()'.
+        if (newValues != dates) {               // Mandatory check for avoiding the call to 'dates.clear()'.
             writeDates(newValues);
         }
     }

Modified: sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/quality/AbstractLogicalConsistency.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/quality/AbstractLogicalConsistency.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/quality/AbstractLogicalConsistency.java [UTF-8] (original)
+++ sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/quality/AbstractLogicalConsistency.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -45,6 +45,7 @@ import org.opengis.metadata.quality.Conc
  * @version 0.3
  * @module
  */
+@SuppressWarnings("CloneableClassWithoutClone")                 // ModifiableMetadata needs shallow clones.
 @XmlType(name = "AbstractDQ_LogicalConsistency_Type")
 @XmlRootElement(name = "DQ_LogicalConsistency")
 @XmlSeeAlso({
@@ -70,7 +71,7 @@ public class AbstractLogicalConsistency
      * 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(LogicalConsistency)
      */
@@ -97,8 +98,8 @@ public class AbstractLogicalConsistency
      *       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 AbstractLogicalConsistency castOrCopy(final LogicalConsistency object) {

Modified: sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/quality/AbstractPositionalAccuracy.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/quality/AbstractPositionalAccuracy.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/quality/AbstractPositionalAccuracy.java [UTF-8] (original)
+++ sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/quality/AbstractPositionalAccuracy.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -44,6 +44,7 @@ import org.opengis.metadata.quality.Rela
  * @version 0.3
  * @module
  */
+@SuppressWarnings("CloneableClassWithoutClone")                 // ModifiableMetadata needs shallow clones.
 @XmlType(name = "AbstractDQ_PositionalAccuracy_Type")
 @XmlRootElement(name = "DQ_PositionalAccuracy")
 @XmlSeeAlso({

Modified: sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/quality/AbstractResult.java
URL: http://svn.apache.org/viewvc/sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/quality/AbstractResult.java?rev=1779559&r1=1779558&r2=1779559&view=diff
==============================================================================
--- sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/quality/AbstractResult.java [UTF-8] (original)
+++ sis/trunk/core/sis-metadata/src/main/java/org/apache/sis/metadata/iso/quality/AbstractResult.java [UTF-8] Fri Jan 20 04:34:26 2017
@@ -44,6 +44,7 @@ import org.apache.sis.metadata.iso.ISOMe
  * @version 0.3
  * @module
  */
+@SuppressWarnings("CloneableClassWithoutClone")                 // ModifiableMetadata needs shallow clones.
 @XmlType(name = "AbstractDQ_Result_Type")
 @XmlRootElement(name = "DQ_Result")
 @XmlSeeAlso({
@@ -68,7 +69,7 @@ public class AbstractResult extends ISOM
      * 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(Result)
      */
@@ -95,8 +96,8 @@ public class AbstractResult extends ISOM
      *       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 AbstractResult castOrCopy(final Result object) {



Mime
View raw message