Author: desruisseaux Date: Sun Feb 1 22:17:37 2015 New Revision: 1656370 URL: http://svn.apache.org/r1656370 Log: Documentation updates. In particular explain better what a MathTransform is, and its relationship with Formula.
Modified: sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/operation/DefaultFormula.java sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/operation/transform/AbstractMathTransform.java sis/branches/JDK8/core/sis-utility/src/main/java/org/apache/sis/util/iso/AbstractFactory.java Modified: sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/operation/DefaultFormula.java URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/operation/DefaultFormula.java?rev=1656370&r1=1656369&r2=1656370&view=diff ============================================================================== --- sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/operation/DefaultFormula.java [UTF-8] (original) +++ sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/operation/DefaultFormula.java [UTF-8] Sun Feb 1 22:17:37 2015 @@ -33,11 +33,20 @@ import java.util.Objects; /** * Specification of the coordinate operation method formula. + * A formula may be {@linkplain #getFormula() given textually}, + * or may be a {@linkplain #getCitation() reference to a publication}. + * + * <p>{@code Formula} is for human reading. + * The object that actually does the work of applying formula to coordinate values is + * {@link org.opengis.referencing.operation.MathTransform}.</p> * * @author Martin Desruisseaux (Geomatys) * @version 0.5 * @since 0.5 * @module + * + * @see DefaultOperationMethod + * @see org.apache.sis.referencing.operation.transform.AbstractMathTransform */ public class DefaultFormula extends FormattableObject implements Formula, Serializable { /** Modified: sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/operation/transform/AbstractMathTransform.java URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/operation/transform/AbstractMathTransform.java?rev=1656370&r1=1656369&r2=1656370&view=diff ============================================================================== --- sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/operation/transform/AbstractMathTransform.java [UTF-8] (original) +++ sis/branches/JDK8/core/sis-referencing/src/main/java/org/apache/sis/referencing/operation/transform/AbstractMathTransform.java [UTF-8] Sun Feb 1 22:17:37 2015 @@ -47,18 +47,23 @@ import static org.apache.sis.util.Argume /** * Provides a default implementation for most methods required by the {@link MathTransform} interface. - * {@code AbstractMathTransform} provides a convenient base class from which transform implementations + * A {@code MathTransform} is an object that actually does the work of applying a + * {@linkplain org.apache.sis.referencing.operation.DefaultFormula formula} to coordinate values. + * The math transform does not know or care how the coordinates relate to positions in the real world. + * For example if an affine transform scales <var>z</var> values by a factor of 1000, + * then it could be converting metres to millimetres, or it could be converting kilometres to metres. + * + * <p>{@code AbstractMathTransform} provides a convenient base class from which {@code MathTransform} implementations * can be easily derived. It also defines a few additional SIS-specific methods for convenience of performance. + * The simplest way to implement this abstract class is to provide an implementation for the following methods only:</p> * - * <p>The simplest way to implement this abstract class is to provide an implementation for the following methods - * only:</p> * <ul> * <li>{@link #getSourceDimensions()}</li> * <li>{@link #getTargetDimensions()}</li> * <li>{@link #transform(double[], int, double[], int, boolean)}</li> * </ul> * - * However more performance may be gained by overriding the other {@code transform} methods as well. + * However more performance may be gained by overriding the other {@code transform(…)} methods as well. * * {@section Immutability and thread safety} * All Apache SIS implementations of {@code MathTransform} are immutable and thread-safe. Modified: sis/branches/JDK8/core/sis-utility/src/main/java/org/apache/sis/util/iso/AbstractFactory.java URL: http://svn.apache.org/viewvc/sis/branches/JDK8/core/sis-utility/src/main/java/org/apache/sis/util/iso/AbstractFactory.java?rev=1656370&r1=1656369&r2=1656370&view=diff ============================================================================== --- sis/branches/JDK8/core/sis-utility/src/main/java/org/apache/sis/util/iso/AbstractFactory.java [UTF-8] (original) +++ sis/branches/JDK8/core/sis-utility/src/main/java/org/apache/sis/util/iso/AbstractFactory.java [UTF-8] Sun Feb 1 22:17:37 2015 @@ -39,7 +39,7 @@ public abstract class AbstractFactory im /** * Returns the implementor of this factory, or {@code null} if unknown. * The default implementation tries to fetch this information from the - * manifest associated to the package of this class. + * manifest associated to the package of {@code this.getClass()}. * * @return The vendor for this factory implementation, or {@code null} if unknown. *