sis-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From desruisse...@apache.org
Subject svn commit: r1600494 [2/2] - in /sis/ip-review: ./ rev/06405/ rev/07861/ rev/10796/ rev/20874/ rev/24485/ rev/24925/ rev/27154/
Date Wed, 04 Jun 2014 21:01:43 GMT
Added: sis/ip-review/rev/20874/PassThroughTransform.xhtml
URL: http://svn.apache.org/viewvc/sis/ip-review/rev/20874/PassThroughTransform.xhtml?rev=1600494&view=auto
==============================================================================
--- sis/ip-review/rev/20874/PassThroughTransform.xhtml (added)
+++ sis/ip-review/rev/20874/PassThroughTransform.xhtml Wed Jun  4 21:01:42 2014
@@ -0,0 +1,51 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml">
+  <head>
+    <meta charset="UTF-8"/>
+    <title>PassThroughTransform changes for revisions 20873:20874</title>
+    <style type="text/css" media="all">
+      @import url("../../reports.css");
+    </style>
+  </head>
+  <body>
+    <div>
+      <h1>PassThroughTransform changes for revisions 20873:20874</h1>
+<p>Changes in GeoTools header only. The removal of the "<cite>or (at your option)
any later version</cite>" clause
+must be keep in Geotk for every classes having contribution from a developer other than those
who accepted re-licensing.
+This header does not apply to Apache SIS, since the above-cited contributions are omitted.</p>
+<p><b>Command line:</b></p>
+<blockquote><code>svn diff --extensions "--unified --ignore-space-change --ignore-all-space
--ignore-eol-style" -r20873:20874 http://svn.osgeo.org/geotools/trunk/modules/library/referencing/src/main/java/org/geotools/referencing/operation/transform/PassThroughTransform.java</code></blockquote>
+<table class="changes">
+<tr><th>Revision 20873</th><th>Revision 20874</th></tr>
+<tr><td><pre>/*
+<span class="del"> * Geotools 2 - OpenSource mapping toolkit</span>
+<span class="del"> * (C) 2003, Geotools Project Managment Committee (PMC)</span>
+ * (C) 2001, Institut de Recherche pour le Développement
+ *
+ *    This library is free software; you can redistribute it and/or</pre></td>
+<td><pre>/*
+<span class="add"> *    GeoTools - OpenSource mapping toolkit</span>
+<span class="add"> *    http://geotools.org</span>
+<span class="add"> *</span>
+<span class="add"> *   (C) 2003-2006, Geotools Project Managment Committee (PMC)</span>
+ * (C) 2001, Institut de Recherche pour le Développement
+ *
+ *    This library is free software; you can redistribute it and/or</pre></td></tr>
+<tr><td><pre> *    but WITHOUT ANY WARRANTY; without even the implied warranty
of
+ *    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ *    Lesser General Public License for more details.
+<span class="del"> *</span>
+<span class="del"> *    You should have received a copy of the GNU Lesser General Public</span>
+<span class="del"> *    License along with this library; if not, write to the Free
Software</span>
+<span class="del"> *    Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307
 USA</span>
+ */
+package org.geotools.referencing.operation.transform;</pre></td>
+<td><pre> *    but WITHOUT ANY WARRANTY; without even the implied warranty of
+ *    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ *    Lesser General Public License for more details.
+ */
+package org.geotools.referencing.operation.transform;</pre></td></tr>
+</table>
+    </div>
+  </body>
+</html>

Propchange: sis/ip-review/rev/20874/PassThroughTransform.xhtml
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: sis/ip-review/rev/20874/PassThroughTransform.xhtml
------------------------------------------------------------------------------
    svn:mime-type = text/html

Added: sis/ip-review/rev/20874/ProjectiveTransform.xhtml
URL: http://svn.apache.org/viewvc/sis/ip-review/rev/20874/ProjectiveTransform.xhtml?rev=1600494&view=auto
==============================================================================
--- sis/ip-review/rev/20874/ProjectiveTransform.xhtml (added)
+++ sis/ip-review/rev/20874/ProjectiveTransform.xhtml Wed Jun  4 21:01:42 2014
@@ -0,0 +1,54 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml">
+  <head>
+    <meta charset="UTF-8"/>
+    <title>ProjectiveTransform changes for revisions 20873:20874</title>
+    <style type="text/css" media="all">
+      @import url("../../reports.css");
+    </style>
+  </head>
+  <body>
+    <div>
+      <h1>ProjectiveTransform changes for revisions 20873:20874</h1>
+<p>Changes in GeoTools header only. The removal of the "<cite>or (at your option)
any later version</cite>" clause
+must be keep in Geotk for every classes having contribution from a developer other than those
who accepted re-licensing.
+This header does not apply to Apache SIS, since the above-cited contributions are omitted.</p>
+<p><b>Command line:</b></p>
+<blockquote><code>svn diff --extensions "--unified --ignore-space-change --ignore-all-space
--ignore-eol-style" -r20873:20874 http://svn.osgeo.org/geotools/trunk/modules/library/referencing/src/main/java/org/geotools/referencing/operation/transform/ProjectiveTransform.java</code></blockquote>
+<table class="changes">
+<tr><th>Revision 20873</th><th>Revision 20874</th></tr>
+<tr><td><pre>/*
+<span class="del"> * Geotools 2 - OpenSource mapping toolkit</span>
+<span class="del"> * (C) 2003, Geotools Project Managment Committee (PMC)</span>
+ * (C) 2001, Institut de Recherche pour le Développement
+ *
+ *    This library is free software; you can redistribute it and/or</pre></td>
+<td><pre>/*
+<span class="add"> *    GeoTools - OpenSource mapping toolkit</span>
+<span class="add"> *    http://geotools.org</span>
+<span class="add"> *</span>
+<span class="add"> *   (C) 2003-2006, Geotools Project Managment Committee (PMC)</span>
+ * (C) 2001, Institut de Recherche pour le Développement
+ *
+ *    This library is free software; you can redistribute it and/or</pre></td></tr>
+<tr><td><pre>*    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
 See the GNU
+*    Lesser General Public License for more details.
+*
+<span class="del">*    You should have received a copy of the GNU Lesser General Public</span>
+<span class="del">*    License along with this library; if not, write to the Free Software</span>
+<span class="del">*    Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307
 USA</span>
+<span class="del">*</span>
+<span class="del">*</span>
+*    This package contains documentation from OpenGIS specifications.
+*    OpenGIS consortium's work is fully acknowledged here.
+*/</pre></td>
+<td><pre>*    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+*    Lesser General Public License for more details.
+*
+*    This package contains documentation from OpenGIS specifications.
+*    OpenGIS consortium's work is fully acknowledged here.
+*/</pre></td></tr>
+</table>
+    </div>
+  </body>
+</html>

Propchange: sis/ip-review/rev/20874/ProjectiveTransform.xhtml
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: sis/ip-review/rev/20874/ProjectiveTransform.xhtml
------------------------------------------------------------------------------
    svn:mime-type = text/html

Added: sis/ip-review/rev/24485/LinearTransform1D.xhtml
URL: http://svn.apache.org/viewvc/sis/ip-review/rev/24485/LinearTransform1D.xhtml?rev=1600494&view=auto
==============================================================================
--- sis/ip-review/rev/24485/LinearTransform1D.xhtml (added)
+++ sis/ip-review/rev/24485/LinearTransform1D.xhtml Wed Jun  4 21:01:42 2014
@@ -0,0 +1,54 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml">
+  <head>
+    <meta charset="UTF-8"/>
+    <title>LinearTransform1D changes for revisions 24484:24485</title>
+    <style type="text/css" media="all">
+      @import url("../../reports.css");
+    </style>
+  </head>
+  <body>
+    <div>
+      <h1>LinearTransform1D changes for revisions 24484:24485</h1>
+<p>This change has been reverted.</p>
+<p><b>Command line:</b></p>
+<blockquote><code>svn diff --extensions "--unified --ignore-space-change --ignore-all-space
--ignore-eol-style" -r24484:24485 http://svn.osgeo.org/geotools/trunk/modules/library/referencing/src/main/java/org/geotools/referencing/operation/transform/LinearTransform1D.java</code></blockquote>
+<table class="changes">
+<tr><th>Revision 24484</th><th>Revision 24485</th></tr>
+<tr><td><pre> * Tests whether this transform does not move any points.
+ */
+public boolean isIdentity() {
+<span class="del">    return offset==0 &amp;&amp; scale==1;</span>
+}
+
+/**
+ * Gets the derivative of this transform at a point.  This implementation is different
+ * from the default {@link AbstractMathTransform#derivative} implementation in that no
+ * coordinate point is required and {@link Double#NaN} may be a legal output value for</pre></td>
+<td><pre> * Tests whether this transform does not move any points.
+ */
+public boolean isIdentity() {
+<span class="add">   return isIdentity(0);</span>
+}
+
+/**
+<span class="add"> * Tests whether this transform does not move any points.</span>
+<span class="add"> *</span>
+<span class="add"> * &lt;p&gt;</span>
+<span class="add"> * It uses the provided &lt;code&gt;tolerance&lt;/code&gt;
before perfoming</span>
+<span class="add"> * the check.</span>
+<span class="add"> * @snce 2.3.1</span>
+<span class="add"> */</span>
+<span class="add">public boolean isIdentity(double tolerance) {</span>
+<span class="add">    tolerance=Math.abs(tolerance);</span>
+<span class="add">    return Math.abs(offset)&lt;=tolerance&amp;&amp; Math.abs(scale-1)&lt;=tolerance;</span>
+<span class="add">}</span>
+<span class="add"></span>
+<span class="add">/**</span>
+ * Gets the derivative of this transform at a point.  This implementation is different
+ * from the default {@link AbstractMathTransform#derivative} implementation in that no
+ * coordinate point is required and {@link Double#NaN} may be a legal output value for</pre></td></tr>
+</table>
+    </div>
+  </body>
+</html>

Propchange: sis/ip-review/rev/24485/LinearTransform1D.xhtml
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: sis/ip-review/rev/24485/LinearTransform1D.xhtml
------------------------------------------------------------------------------
    svn:mime-type = text/html

Added: sis/ip-review/rev/24485/ProjectiveTransform.xhtml
URL: http://svn.apache.org/viewvc/sis/ip-review/rev/24485/ProjectiveTransform.xhtml?rev=1600494&view=auto
==============================================================================
--- sis/ip-review/rev/24485/ProjectiveTransform.xhtml (added)
+++ sis/ip-review/rev/24485/ProjectiveTransform.xhtml Wed Jun  4 21:01:42 2014
@@ -0,0 +1,54 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml">
+  <head>
+    <meta charset="UTF-8"/>
+    <title>ProjectiveTransform changes for revisions 24484:24485</title>
+    <style type="text/css" media="all">
+      @import url("../../reports.css");
+    </style>
+  </head>
+  <body>
+    <div>
+      <h1>ProjectiveTransform changes for revisions 24484:24485</h1>
+<p>This change has been removed. Since the translations and scale factors in a matrix
have very different magnitudes,
+a single tolerance factor is not sufficient. The Apache SIS approach is rather to use double-double
arithmetic in
+matrix operations in the hope to avoid the need for a tolerance threshold.</p>
+<p><b>Command line:</b></p>
+<blockquote><code>svn diff --extensions "--unified --ignore-space-change --ignore-all-space
--ignore-eol-style" -r24484:24485 http://svn.osgeo.org/geotools/trunk/modules/library/referencing/src/main/java/org/geotools/referencing/operation/transform/ProjectiveTransform.java</code></blockquote>
+<table class="changes">
+<tr><th>Revision 24484</th><th>Revision 24485</th></tr>
+<tr><td><pre>}
+
+/**
+ * Creates the inverse transform of this object.
+ */
+public MathTransform inverse() throws NoninvertibleTransformException {</pre></td>
+<td><pre>}
+
+/**
+<span class="add"> * Tests whether this transform does not move any points by employing
the provided tolerance.</span>
+<span class="add"> * @since 2.4</span>
+<span class="add"> */</span>
+<span class="add">public boolean isIdentity(double tolerance) {</span>
+<span class="add">    tolerance=Math.abs(tolerance);</span>
+<span class="add">    if (numRow != numCol) {</span>
+<span class="add">        return false;</span>
+<span class="add">    }</span>
+<span class="add">    int index=0;</span>
+<span class="add">    for (int j=0; j&lt;numRow; j++) {</span>
+<span class="add">        for (int i=0; i&lt;numCol; i++) {</span>
+<span class="add">            if (Math.abs(elt[index++] -(i==j ? 1 : 0))&gt;tolerance)
{</span>
+<span class="add">                return false;</span>
+<span class="add">            }</span>
+<span class="add">        }</span>
+<span class="add">    }</span>
+<span class="add">    return true;</span>
+<span class="add">}</span>
+<span class="add">/**</span>
+ * Creates the inverse transform of this object.
+ */
+public MathTransform inverse() throws NoninvertibleTransformException {</pre></td></tr>
+</table>
+    </div>
+  </body>
+</html>

Propchange: sis/ip-review/rev/24485/ProjectiveTransform.xhtml
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: sis/ip-review/rev/24485/ProjectiveTransform.xhtml
------------------------------------------------------------------------------
    svn:mime-type = text/html

Added: sis/ip-review/rev/24925/IdentityTransform.xhtml
URL: http://svn.apache.org/viewvc/sis/ip-review/rev/24925/IdentityTransform.xhtml?rev=1600494&view=auto
==============================================================================
--- sis/ip-review/rev/24925/IdentityTransform.xhtml (added)
+++ sis/ip-review/rev/24925/IdentityTransform.xhtml Wed Jun  4 21:01:42 2014
@@ -0,0 +1,37 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml">
+  <head>
+    <meta charset="UTF-8"/>
+    <title>IdentityTransform changes for revisions 24924:24925</title>
+    <style type="text/css" media="all">
+      @import url("../../reports.css");
+    </style>
+  </head>
+  <body>
+    <div>
+      <h1>IdentityTransform changes for revisions 24924:24925</h1>
+<p>This commit propagates the package names change which was done in GeoAPI.
+There is no way such change could be done differently,
+since not renaming the packages cause a compilation error.</p>
+<p><b>Command line:</b></p>
+<blockquote><code>svn diff --extensions "--unified --ignore-space-change --ignore-all-space
--ignore-eol-style" -r24924:24925 http://svn.osgeo.org/geotools/trunk/modules/library/referencing/src/main/java/org/geotools/referencing/operation/transform/IdentityTransform.java</code></blockquote>
+<table class="changes">
+<tr><th>Revision 24924</th><th>Revision 24925</th></tr>
+<tr><td><pre>import org.opengis.referencing.operation.Matrix;
+import org.opengis.referencing.operation.MathTransform;
+import org.opengis.referencing.operation.TransformException;
+<span class="del">import org.opengis.spatialschema.geometry.DirectPosition;</span>
+
+// Geotools dependencies
+import org.geotools.geometry.GeneralDirectPosition;</pre></td>
+<td><pre>import org.opengis.referencing.operation.Matrix;
+import org.opengis.referencing.operation.MathTransform;
+import org.opengis.referencing.operation.TransformException;
+<span class="add">import org.opengis.geometry.DirectPosition;</span>
+
+// Geotools dependencies
+import org.geotools.geometry.GeneralDirectPosition;</pre></td></tr>
+</table>
+    </div>
+  </body>
+</html>

Propchange: sis/ip-review/rev/24925/IdentityTransform.xhtml
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: sis/ip-review/rev/24925/IdentityTransform.xhtml
------------------------------------------------------------------------------
    svn:mime-type = text/html

Added: sis/ip-review/rev/24925/LinearTransform1D.xhtml
URL: http://svn.apache.org/viewvc/sis/ip-review/rev/24925/LinearTransform1D.xhtml?rev=1600494&view=auto
==============================================================================
--- sis/ip-review/rev/24925/LinearTransform1D.xhtml (added)
+++ sis/ip-review/rev/24925/LinearTransform1D.xhtml Wed Jun  4 21:01:42 2014
@@ -0,0 +1,37 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml">
+  <head>
+    <meta charset="UTF-8"/>
+    <title>LinearTransform1D changes for revisions 24924:24925</title>
+    <style type="text/css" media="all">
+      @import url("../../reports.css");
+    </style>
+  </head>
+  <body>
+    <div>
+      <h1>LinearTransform1D changes for revisions 24924:24925</h1>
+<p>This commit propagates the package names change which was done in GeoAPI.
+There is no way such change could be done differently,
+since not renaming the packages cause a compilation error.</p>
+<p><b>Command line:</b></p>
+<blockquote><code>svn diff --extensions "--unified --ignore-space-change --ignore-all-space
--ignore-eol-style" -r24924:24925 http://svn.osgeo.org/geotools/trunk/modules/library/referencing/src/main/java/org/geotools/referencing/operation/transform/LinearTransform1D.java</code></blockquote>
+<table class="changes">
+<tr><th>Revision 24924</th><th>Revision 24925</th></tr>
+<tr><td><pre>import org.opengis.referencing.operation.Matrix;
+import org.opengis.referencing.operation.NoninvertibleTransformException;
+import org.opengis.referencing.operation.TransformException;
+<span class="del">import org.opengis.spatialschema.geometry.DirectPosition;</span>
+
+// Geotools dependencies
+import org.geotools.referencing.operation.matrix.Matrix1;</pre></td>
+<td><pre>import org.opengis.referencing.operation.Matrix;
+import org.opengis.referencing.operation.NoninvertibleTransformException;
+import org.opengis.referencing.operation.TransformException;
+<span class="add">import org.opengis.geometry.DirectPosition;</span>
+
+// Geotools dependencies
+import org.geotools.referencing.operation.matrix.Matrix1;</pre></td></tr>
+</table>
+    </div>
+  </body>
+</html>

Propchange: sis/ip-review/rev/24925/LinearTransform1D.xhtml
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: sis/ip-review/rev/24925/LinearTransform1D.xhtml
------------------------------------------------------------------------------
    svn:mime-type = text/html

Added: sis/ip-review/rev/24925/PassThroughTransform.xhtml
URL: http://svn.apache.org/viewvc/sis/ip-review/rev/24925/PassThroughTransform.xhtml?rev=1600494&view=auto
==============================================================================
--- sis/ip-review/rev/24925/PassThroughTransform.xhtml (added)
+++ sis/ip-review/rev/24925/PassThroughTransform.xhtml Wed Jun  4 21:01:42 2014
@@ -0,0 +1,39 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml">
+  <head>
+    <meta charset="UTF-8"/>
+    <title>PassThroughTransform changes for revisions 24924:24925</title>
+    <style type="text/css" media="all">
+      @import url("../../reports.css");
+    </style>
+  </head>
+  <body>
+    <div>
+      <h1>PassThroughTransform changes for revisions 24924:24925</h1>
+<p>This commit propagates the package names change which was done in GeoAPI.
+There is no way such change could be done differently,
+since not renaming the packages cause a compilation error.</p>
+<p><b>Command line:</b></p>
+<blockquote><code>svn diff --extensions "--unified --ignore-space-change --ignore-all-space
--ignore-eol-style" -r24924:24925 http://svn.osgeo.org/geotools/trunk/modules/library/referencing/src/main/java/org/geotools/referencing/operation/transform/PassThroughTransform.java</code></blockquote>
+<table class="changes">
+<tr><th>Revision 24924</th><th>Revision 24925</th></tr>
+<tr><td><pre>import org.opengis.referencing.operation.Matrix;
+import org.opengis.referencing.operation.NoninvertibleTransformException;
+import org.opengis.referencing.operation.TransformException;
+<span class="del">import org.opengis.spatialschema.geometry.DirectPosition;</span>
+<span class="del">import org.opengis.spatialschema.geometry.MismatchedDimensionException;</span>
+
+// Geotools dependencies
+import org.geotools.geometry.GeneralDirectPosition;</pre></td>
+<td><pre>import org.opengis.referencing.operation.Matrix;
+import org.opengis.referencing.operation.NoninvertibleTransformException;
+import org.opengis.referencing.operation.TransformException;
+<span class="add">import org.opengis.geometry.DirectPosition;</span>
+<span class="add">import org.opengis.geometry.MismatchedDimensionException;</span>
+
+// Geotools dependencies
+import org.geotools.geometry.GeneralDirectPosition;</pre></td></tr>
+</table>
+    </div>
+  </body>
+</html>

Propchange: sis/ip-review/rev/24925/PassThroughTransform.xhtml
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: sis/ip-review/rev/24925/PassThroughTransform.xhtml
------------------------------------------------------------------------------
    svn:mime-type = text/html

Added: sis/ip-review/rev/27154/HEADER.html
URL: http://svn.apache.org/viewvc/sis/ip-review/rev/27154/HEADER.html?rev=1600494&view=auto
==============================================================================
--- sis/ip-review/rev/27154/HEADER.html (added)
+++ sis/ip-review/rev/27154/HEADER.html Wed Jun  4 21:01:42 2014
@@ -0,0 +1,17 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml">
+  <head>
+    <meta charset="UTF-8"/>
+    <title>Revision 27154</title>
+  </head>
+  <body>
+    <div>
+      <h1>Revision 27154</h1>
+<table>
+  <tr><td><b>Author:</b></td><td>jezekjan</td></tr>
+  <tr><td><b>Date:</b></td><td>2007-09-25</td></tr>
+  <tr><td><b>Message:</b></td><td>ProjectiveTransform2D
added (Thanks Martin :-) ).</td></tr>
+</table>
+    </div>
+  </body>
+</html>

Propchange: sis/ip-review/rev/27154/HEADER.html
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: sis/ip-review/rev/27154/HEADER.html
------------------------------------------------------------------------------
    svn:mime-type = text/html

Added: sis/ip-review/rev/27154/ProjectiveTransform.xhtml
URL: http://svn.apache.org/viewvc/sis/ip-review/rev/27154/ProjectiveTransform.xhtml?rev=1600494&view=auto
==============================================================================
--- sis/ip-review/rev/27154/ProjectiveTransform.xhtml (added)
+++ sis/ip-review/rev/27154/ProjectiveTransform.xhtml Wed Jun  4 21:01:42 2014
@@ -0,0 +1,57 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml">
+  <head>
+    <meta charset="UTF-8"/>
+    <title>ProjectiveTransform changes for revisions 24924:27154</title>
+    <style type="text/css" media="all">
+      @import url("../../reports.css");
+    </style>
+  </head>
+  <body>
+    <div>
+      <h1>ProjectiveTransform changes for revisions 24924:27154</h1>
+<p>The main change is the creation of a <code>MathTransform2D</code> implementation
for the two-dimensional case.
+The same pattern is applied in other classes (Identity, Molodensky, basically every transform
that may have arbitrary
+dimension). So we can copy-paste-edit the same code from other classes. Anyway, the code
is written differently in
+Geotk / Apache SIS (using a <code>if</code> statement instead than <code>switch</code>,
but this is very minor details).</p>
+<p><b>Command line:</b></p>
+<blockquote><code>svn diff --extensions "--unified --ignore-space-change --ignore-all-space
--ignore-eol-style" -r24924:27154 http://svn.osgeo.org/geotools/trunk/modules/library/referencing/src/main/java/org/geotools/referencing/operation/transform/ProjectiveTransform.java</code></blockquote>
+<table class="changes">
+<tr><th>Revision 24924</th><th>Revision 27154</th></tr>
+<tr><td><pre>import org.opengis.referencing.operation.Matrix;
+import org.opengis.referencing.operation.NoninvertibleTransformException;
+import org.opengis.referencing.operation.OperationMethod;
+<span class="del">import org.opengis.spatialschema.geometry.DirectPosition;</span>
+
+// Geotools dependencies
+import org.geotools.metadata.iso.citation.Citations;</pre></td>
+<td><pre>import org.opengis.referencing.operation.Matrix;
+import org.opengis.referencing.operation.NoninvertibleTransformException;
+import org.opengis.referencing.operation.OperationMethod;
+<span class="add">import org.opengis.geometry.DirectPosition;</span>
+
+// Geotools dependencies
+import org.geotools.metadata.iso.citation.Citations;</pre></td></tr>
+<tr><td><pre>            }
+        }
+    }
+<span class="del">    return new ProjectiveTransform(matrix);</span>
+}
+
+/**
+ * Creates a transform for the specified matrix as a Java2D object.</pre></td>
+<td><pre>            }
+        }
+    }
+<span class="add">    switch (dimension) {</span>
+<span class="add">        case 2:  return new ProjectiveTransform2D(matrix);</span>
+<span class="add">        default: return new ProjectiveTransform  (matrix);</span>
+}
+<span class="add">}</span>
+
+/**
+ * Creates a transform for the specified matrix as a Java2D object.</pre></td></tr>
+</table>
+    </div>
+  </body>
+</html>

Propchange: sis/ip-review/rev/27154/ProjectiveTransform.xhtml
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: sis/ip-review/rev/27154/ProjectiveTransform.xhtml
------------------------------------------------------------------------------
    svn:mime-type = text/html

Added: sis/ip-review/rev/27154/ProjectiveTransform2D.xhtml
URL: http://svn.apache.org/viewvc/sis/ip-review/rev/27154/ProjectiveTransform2D.xhtml?rev=1600494&view=auto
==============================================================================
--- sis/ip-review/rev/27154/ProjectiveTransform2D.xhtml (added)
+++ sis/ip-review/rev/27154/ProjectiveTransform2D.xhtml Wed Jun  4 21:01:42 2014
@@ -0,0 +1,61 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml">
+  <head>
+    <meta charset="UTF-8"/>
+    <title>ProjectiveTransform2D changes for revisions 27153:27154</title>
+    <style type="text/css" media="all">
+      @import url("../../reports.css");
+    </style>
+  </head>
+  <body>
+    <div>
+      <h1>ProjectiveTransform2D changes for revisions 27153:27154</h1>
+<p>Deleting this contribution and rewriting it can only result in the same starting
point.
+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><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">
+<tr><th>Revision 27154</th></tr>
+<tr>
+<td><pre><span class="add">/*</span>
+<span class="add"> *    GeoTools - OpenSource mapping toolkit</span>
+<span class="add"> *    http://geotools.org</span>
+<span class="add"> *    (C) 2002-2007, GeoTools Project Managment Committee (PMC)</span>
+<span class="add"> *</span>
+<span class="add"> *    This library is free software; you can redistribute it and/or</span>
+<span class="add"> *    modify it under the terms of the GNU Lesser General Public</span>
+<span class="add"> *    License as published by the Free Software Foundation;</span>
+<span class="add"> *    version 2.1 of the License.</span>
+<span class="add"> *</span>
+<span class="add"> *    This library is distributed in the hope that it will be useful,</span>
+<span class="add"> *    but WITHOUT ANY WARRANTY; without even the implied warranty
of</span>
+<span class="add"> *    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
GNU</span>
+<span class="add"> *    Lesser General Public License for more details.</span>
+<span class="add"> */</span>
+<span class="add">package org.geotools.referencing.operation.transform;</span>
+<span class="add"></span>
+<span class="add">import org.opengis.referencing.operation.MathTransform2D;</span>
+<span class="add">import org.opengis.referencing.operation.Matrix;</span>
+<span class="add"></span>
+<span class="add"></span>
+<span class="add">/**</span>
+<span class="add"> * Projective transform in 2D case.</span>
+<span class="add"> *</span>
+<span class="add"> * @source $URL$</span>
+<span class="add"> * @version $Id$</span>
+<span class="add"> * @author Jan Jezek</span>
+<span class="add"> */</span>
+<span class="add">final class ProjectiveTransform2D extends ProjectiveTransform implements
MathTransform2D {</span>
+<span class="add">    /**</span>
+<span class="add">     * Creates projective transform from a matrix.</span>
+<span class="add">     */</span>
+<span class="add">    public ProjectiveTransform2D(Matrix matrix) {</span>
+<span class="add">        super(matrix);</span>
+<span class="add">    }</span>
+<span class="add">}</span></pre></td></tr>
+</table>
+    </div>
+  </body>
+</html>

Propchange: sis/ip-review/rev/27154/ProjectiveTransform2D.xhtml
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: sis/ip-review/rev/27154/ProjectiveTransform2D.xhtml
------------------------------------------------------------------------------
    svn:mime-type = text/html



Mime
View raw message