sis-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From desruisse...@apache.org
Subject svn commit: r1600770 - /sis/ip-review/rev/27154/ProjectiveTransform2D.xhtml
Date Thu, 05 Jun 2014 21:24:39 GMT
Author: desruisseaux
Date: Thu Jun  5 21:24:39 2014
New Revision: 1600770

URL: http://svn.apache.org/r1600770
Log:
More details on the process followed for rewriting a code.

Modified:
    sis/ip-review/rev/27154/ProjectiveTransform2D.xhtml

Modified: sis/ip-review/rev/27154/ProjectiveTransform2D.xhtml
URL: http://svn.apache.org/viewvc/sis/ip-review/rev/27154/ProjectiveTransform2D.xhtml?rev=1600770&r1=1600769&r2=1600770&view=diff
==============================================================================
--- sis/ip-review/rev/27154/ProjectiveTransform2D.xhtml (original)
+++ sis/ip-review/rev/27154/ProjectiveTransform2D.xhtml Thu Jun  5 21:24:39 2014
@@ -14,6 +14,21 @@
 This is the smallest possible implementation of a two-dimensional <code>ProjectiveTransform</code>.
 There is no way to do the minimalist version otherwise,
 except for Javadoc and the <code>public</code> modifier of constructor.</p>
+
+<p>This contribution has been redone as below:</p>
+<ul>
+  <li>Create a new class for a 2D transform using the same naming convention than all
other classes in this package,
+      namely "BaseClassName + 2D".</li>
+  <li>Extend the base class, namely <code>ProjectiveTransform</code>.</li>
+  <li>The compiler complains that there is no default constructor to inherit from the
parent class.
+      Fix the compiler error by adding a constructor with the same parameter than the parent
constructor,
+      and just pass that parameter to the parent class constructor.</li>
+  <li>Implement <code>MathTransform2D</code>, which is the whole purpose
of this class.</li>
+</ul>
+<p>And we are done (ignoring all abstract methods that we need to implement).
+However while this contribution has been trivially be redone,
+we nevertheless keep the original author name for credit.</p>
+
 <p><b>Command line:</b></p>
 <blockquote><code>svn diff --extensions "--unified --ignore-space-change --ignore-all-space
--ignore-eol-style" -r27153:27154 http://svn.osgeo.org/geotools/trunk/modules/library/referencing/src/main/java/org/geotools/referencing/operation/transform/ProjectiveTransform2D.java</code></blockquote>
 <table class="changes">



Mime
View raw message