sis-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From desruisse...@apache.org
Subject svn commit: r1600526 - in /sis/ip-review: ./ rev/10796/ rev/20874/ rev/24485/ rev/24925/
Date Wed, 04 Jun 2014 22:26:14 GMT
Author: desruisseaux
Date: Wed Jun  4 22:26:14 2014
New Revision: 1600526

URL: http://svn.apache.org/r1600526
Log:
IP review.

Added:
    sis/ip-review/AffineTransform2D.xhtml   (with props)
    sis/ip-review/ConstantTransform1D.xhtml   (with props)
    sis/ip-review/ImmutableAffineTransform.xhtml   (with props)
    sis/ip-review/rev/10796/AffineTransform2D.xhtml   (with props)
    sis/ip-review/rev/10796/ImmutableAffineTransform.xhtml   (with props)
    sis/ip-review/rev/20874/AffineTransform2D.xhtml   (with props)
    sis/ip-review/rev/20874/ConstantTransform1D.xhtml   (with props)
    sis/ip-review/rev/20874/ImmutableAffineTransform.xhtml   (with props)
    sis/ip-review/rev/24485/ImmutableAffineTransform.xhtml   (with props)
    sis/ip-review/rev/24925/AffineTransform2D.xhtml   (with props)

Added: sis/ip-review/AffineTransform2D.xhtml
URL: http://svn.apache.org/viewvc/sis/ip-review/AffineTransform2D.xhtml?rev=1600526&view=auto
==============================================================================
--- sis/ip-review/AffineTransform2D.xhtml (added)
+++ sis/ip-review/AffineTransform2D.xhtml Wed Jun  4 22:26:14 2014
@@ -0,0 +1,72 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml">
+  <head>
+    <meta charset="UTF-8"/>
+    <title>AffineTransform2D history</title>
+    <style type="text/css" media="all">
+      @import url("./reports.css");
+    </style>
+  </head>
+  <body>
+    <div>
+      <h1>AffineTransform2D history</h1>
+  <p>Click on the commit message for inspecting the <code>diff</code> and how the code has been rewritten.</p>
+<p><b>Command line:</b></p>
+<blockquote><code>svn log -r31996:1 http://svn.osgeo.org/geotools/trunk/modules/library/referencing/src/main/java/org/geotools/referencing/operation/transform/AffineTransform2D.java</code></blockquote>
+<table>
+  <tr>
+    <th>Rev.</th>
+    <th>Date</th>
+    <th>Author</th>
+    <th class="last">Message</th>
+  </tr>
+<tr><td class="rev">31456</td><td>2008-09-08</td><td>desruisseaux</td><td>Fixed a StackOverflowError: AffineTransform2D.createTransformedShape was invoking itself indirectly when the given shape is not one of the recognized shapes like Rectangle2D.</td></tr>
+<tr><td class="rev">30641</td><td>2008-06-12</td><td>acuster</td><td>Copyright headers: lib/referencing, this time with feeling (and the el in Toolkit)</td></tr>
+<tr><td class="rev">30639</td><td>2008-06-12</td><td>acuster</td><td>Copyright headers: cleanup library/referencing</td></tr>
+<tr><td class="rev">30521</td><td>2008-06-05</td><td>acuster</td><td>Copyright: Update referencing headers</td></tr>
+<tr><td class="rev">30258</td><td>2008-05-08</td><td>acuster</td><td>Reshuffle the top level repo: drop uDig, move up trunk, tags, and branches.</td></tr>
+<tr><td class="rev">30257</td><td>2008-05-08</td><td>acuster</td><td>Move trunk/gt/ directory contents up to trunk/ and drop gt</td></tr>
+<tr><td class="rev">29725</td><td>2008-03-27</td><td>desruisseaux</td><td>Added a XAffineTransform.transform(..., Shape, ...) method trying to preserve the shape class.</td></tr>
+<tr><td class="rev">29422</td><td>2008-02-21</td><td>desruisseaux</td><td>Provides a convenience TileBuilder.setMosaicEnvelope(...) method for getting the 'gridToCRS' transform computed automatically for us, with usual assumption on axis flipping, etc. Users can control those assumptions by overriding the createGridToEnveloppeMapper(...) method.</td></tr>
+<tr><td class="rev">28922</td><td>2008-01-24</td><td>acuster</td><td>Bump the (at)since version to 2.5 since WKTParser was cut from 2.4</td></tr>
+<tr><td class="rev">28846</td><td>2008-01-20</td><td>desruisseaux</td><td>Covariant return type for inverse() method in MathTransform1D and MathTransform2D. Also javadoc fixes and Java 5 type safety. Changed ViewType to an enum and initial implementation of GridCoverageViews (not yet connected to GridCoverage2D).</td></tr>
+<tr><td class="rev">28817</td><td>2008-01-18</td><td>desruisseaux</td><td>Added constructor with 6-double values parameters.</td></tr>
+<tr><td class="rev">28540</td><td>2007-12-29</td><td>acuster</td><td>Hide buttons which are not yet used</td></tr>
+<tr><td class="rev">28500</td><td>2007-12-24</td><td>desruisseaux</td><td>Added a new GridGeometry implementation (ImageGeometry) has a lightweight placeholder for 'gridRange' as a Rectangle and 'gridToCRS' as an AffineTransform. Used by TileBuilder.</td></tr>
+<tr><td class="rev">28081</td><td>2007-11-27</td><td>desruisseaux</td><td>Java 5 constructs: replaced some StringBuffer by StringBuilder in order to avoid unecessary synchronisation; rely on auto-boxing in order to give a chance to the JVM to reuse its cached values.</td></tr>
+<tr><td class="rev">27575</td><td>2007-10-22</td><td>desruisseaux</td><td>Converted Java source files from ISO-LATIN-1 encoding to UTF-8 (GEOT-1516).</td></tr>
+<tr><td class="rev">24925</td><td>2007-03-27</td><td class="unav">jgarnett</td><td><a href="rev/24925/AffineTransform2D.xhtml">change to org.opengis.geometry</a></td></tr>
+<tr><td class="rev">22710</td><td>2006-11-12</td><td>desruisseaux</td><td>Review of coverage branch merge: In 'CategoryList', check for the 'no data' category name only if the geophysics value is some NaN value. Moved XAffineTransform to a public place and added methods for testing axis swapping, as a proposed replacement for 'GeneralGridGeometry.swapXY(CoordinateSystem)' (replacement not yet done; waiting for the okay from the mailing list). In GeneralGridGeometry, added methods for returning a math transform relative to some pixel corner instead of pixel center, as a proposed replacement for the 'getTransform(...)' public static methods (I would like to remove those static methods in order to keep the API a little bit more orthogonal).</td></tr>
+<tr><td class="rev">22327</td><td>2006-10-23</td><td>desruisseaux</td><td>GEOT-982: regroup 'module', 'plugin', 'ext' and 'unsupported' in a common directory.</td></tr>
+<tr><td class="rev">22315</td><td>2006-10-22</td><td>desruisseaux</td><td>Reorganisation of directory tree structure (GEOT-982) phase 1: moved 'src' to 'src/main/java'.</td></tr>
+<tr><td class="rev">21780</td><td>2006-10-03</td><td>desruisseaux</td><td>Removed methods from the referencing module that were deprecated in Geotools 2.3. This imply the replacement of 'CRS.transform' by 'CRS.findMathTransform' in the whole Geotools code base.</td></tr>
+<tr><td class="rev">20874</td><td>2006-08-07</td><td class="unav">jgarnett</td><td><a href="rev/20874/AffineTransform2D.xhtml">ip review</a></td></tr>
+<tr><td class="rev">17672</td><td>2006-01-19</td><td>desruisseaux</td><td>Added @source tag.</td></tr>
+<tr><td class="rev">17660</td><td>2006-01-18</td><td>desruisseaux</td><td>Fixed SVN attributes, including the addition of URL attribute.</td></tr>
+<tr><td class="rev">15350</td><td>2005-08-16</td><td>desruisseaux</td><td>Merged the 'split-main' branch to trunk (GEOT-662)</td></tr>
+<tr><td class="rev">15325</td><td>2005-08-13</td><td>desruisseaux</td><td>Created a matrix package on top of vecmath / Fixed 'getPositionalAccuracy' issues</td></tr>
+<tr><td class="rev">15256</td><td>2005-08-09</td><td>desruisseaux</td><td>Extraction of referencing module from main</td></tr>
+<tr><td class="rev">15255</td><td>2005-08-09</td><td>desruisseaux</td><td>Prepare branch for splitting main</td></tr>
+<tr><td class="rev">14934</td><td>2005-07-27</td><td>desruisseaux</td><td>Transformations between 3D ProjectedCRS should propagate the height (GEOT-637, 638 and 639)</td></tr>
+<tr><td class="rev">14834</td><td>2005-07-20</td><td>desruisseaux</td><td>Reorganized I18N resources</td></tr>
+<tr><td class="rev">13925</td><td>2005-05-30</td><td>desruisseaux</td><td>Added @since javadoc tag</td></tr>
+<tr><td class="rev">13922</td><td>2005-05-29</td><td>desruisseaux</td><td>Renaming of remainding GeoAPI implementations. Renaming is now finished.</td></tr>
+<tr><td class="rev">13836</td><td>2005-05-26</td><td>desruisseaux</td><td>Added 'Default' suffix to some CRS implementations: partial work</td></tr>
+<tr><td class="rev">13178</td><td>2005-04-19</td><td>desruisseaux</td><td>Various bug fixed related to GridCoverage and the new-legacy API bridge</td></tr>
+<tr><td class="rev">11248</td><td>2005-02-10</td><td>desruisseaux</td><td>Limited support for transformations between 3D CRS</td></tr>
+<tr><td class="rev">11001</td><td>2005-02-02</td><td>desruisseaux</td><td>Fixed wrong encoding in main/src</td></tr>
+<tr><td class="rev">10796</td><td>2005-01-28</td><td class="unav">dzwiers</td><td><a href="rev/10796/AffineTransform2D.xhtml">imports cleaned</a></td></tr>
+<tr><td class="rev">10228</td><td>2005-01-04</td><td>desruisseaux</td><td>Renamed 'getDimSource' as 'getSourceDimension'</td></tr>
+<tr><td class="rev">6657</td><td>2004-06-30</td><td>desruisseaux</td><td>Fine tuning getParameterValues() method</td></tr>
+<tr><td class="rev">6627</td><td>2004-06-29</td><td>desruisseaux</td><td>Finished refactoring of WKT formatting for MathTransform</td></tr>
+<tr><td class="rev">6536</td><td>2004-06-25</td><td>desruisseaux</td><td>Added SingleCRS class / Renamed ObliqueCartesianCS as AffineCS / Moved Position as super-interface for DirectPosition and Point</td></tr>
+<tr><td class="rev">6383</td><td>2004-06-11</td><td>desruisseaux</td><td>Search for parameters and operation methods by identifiers</td></tr>
+<tr><td class="rev">6375</td><td>2004-06-10</td><td>desruisseaux</td><td>Work on parameters handling</td></tr>
+<tr><td class="rev">6362</td><td>2004-06-08</td><td>desruisseaux</td><td>Initial work for an OperationProvider</td></tr>
+<tr><td class="rev">6333</td><td>2004-06-06</td><td>desruisseaux</td><td>Continuing CRS implementation</td></tr>
+<tr><td class="rev">6325</td><td>2004-06-04</td><td>desruisseaux</td><td>Added MathTransform implementations</td></tr>
+<tr><td class="rev">6301</td><td>2004-06-03</td><td>desruisseaux</td><td>Refactored basic transform in their own package</td></tr>
+</table>
+    </div>
+  </body>
+</html>

Propchange: sis/ip-review/AffineTransform2D.xhtml
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: sis/ip-review/AffineTransform2D.xhtml
------------------------------------------------------------------------------
    svn:mime-type = text/html

Added: sis/ip-review/ConstantTransform1D.xhtml
URL: http://svn.apache.org/viewvc/sis/ip-review/ConstantTransform1D.xhtml?rev=1600526&view=auto
==============================================================================
--- sis/ip-review/ConstantTransform1D.xhtml (added)
+++ sis/ip-review/ConstantTransform1D.xhtml Wed Jun  4 22:26:14 2014
@@ -0,0 +1,49 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml">
+  <head>
+    <meta charset="UTF-8"/>
+    <title>ConstantTransform1D history</title>
+    <style type="text/css" media="all">
+      @import url("./reports.css");
+    </style>
+  </head>
+  <body>
+    <div>
+      <h1>ConstantTransform1D history</h1>
+  <p>Click on the commit message for inspecting the <code>diff</code> and how the code has been rewritten.</p>
+<p><b>Command line:</b></p>
+<blockquote><code>svn log -r31996:1 http://svn.osgeo.org/geotools/trunk/modules/library/referencing/src/main/java/org/geotools/referencing/operation/transform/ConstantTransform1D.java</code></blockquote>
+<table>
+  <tr>
+    <th>Rev.</th>
+    <th>Date</th>
+    <th>Author</th>
+    <th class="last">Message</th>
+  </tr>
+<tr><td class="rev">30641</td><td>2008-06-12</td><td>acuster</td><td>Copyright headers: lib/referencing, this time with feeling (and the el in Toolkit)</td></tr>
+<tr><td class="rev">30639</td><td>2008-06-12</td><td>acuster</td><td>Copyright headers: cleanup library/referencing</td></tr>
+<tr><td class="rev">30521</td><td>2008-06-05</td><td>acuster</td><td>Copyright: Update referencing headers</td></tr>
+<tr><td class="rev">30258</td><td>2008-05-08</td><td>acuster</td><td>Reshuffle the top level repo: drop uDig, move up trunk, tags, and branches.</td></tr>
+<tr><td class="rev">30257</td><td>2008-05-08</td><td>acuster</td><td>Move trunk/gt/ directory contents up to trunk/ and drop gt</td></tr>
+<tr><td class="rev">28922</td><td>2008-01-24</td><td>acuster</td><td>Bump the (at)since version to 2.5 since WKTParser was cut from 2.4</td></tr>
+<tr><td class="rev">28540</td><td>2007-12-29</td><td>acuster</td><td>Hide buttons which are not yet used</td></tr>
+<tr><td class="rev">28520</td><td>2007-12-27</td><td>desruisseaux</td><td>Added @Override annotation for safety.</td></tr>
+<tr><td class="rev">27575</td><td>2007-10-22</td><td>desruisseaux</td><td>Converted Java source files from ISO-LATIN-1 encoding to UTF-8 (GEOT-1516).</td></tr>
+<tr><td class="rev">22327</td><td>2006-10-23</td><td>desruisseaux</td><td>GEOT-982: regroup 'module', 'plugin', 'ext' and 'unsupported' in a common directory.</td></tr>
+<tr><td class="rev">22315</td><td>2006-10-22</td><td>desruisseaux</td><td>Reorganisation of directory tree structure (GEOT-982) phase 1: moved 'src' to 'src/main/java'.</td></tr>
+<tr><td class="rev">20874</td><td>2006-08-07</td><td class="unav">jgarnett</td><td><a href="rev/20874/ConstantTransform1D.xhtml">ip review</a></td></tr>
+<tr><td class="rev">17672</td><td>2006-01-19</td><td>desruisseaux</td><td>Added @source tag.</td></tr>
+<tr><td class="rev">17660</td><td>2006-01-18</td><td>desruisseaux</td><td>Fixed SVN attributes, including the addition of URL attribute.</td></tr>
+<tr><td class="rev">16205</td><td>2005-10-11</td><td>desruisseaux</td><td>Cleaned javadoc warning. More work done on Maven 2 configuration (GEOT-691)</td></tr>
+<tr><td class="rev">15350</td><td>2005-08-16</td><td>desruisseaux</td><td>Merged the 'split-main' branch to trunk (GEOT-662)</td></tr>
+<tr><td class="rev">15325</td><td>2005-08-13</td><td>desruisseaux</td><td>Created a matrix package on top of vecmath / Fixed 'getPositionalAccuracy' issues</td></tr>
+<tr><td class="rev">15256</td><td>2005-08-09</td><td>desruisseaux</td><td>Extraction of referencing module from main</td></tr>
+<tr><td class="rev">15255</td><td>2005-08-09</td><td>desruisseaux</td><td>Prepare branch for splitting main</td></tr>
+<tr><td class="rev">13925</td><td>2005-05-30</td><td>desruisseaux</td><td>Added @since javadoc tag</td></tr>
+<tr><td class="rev">13836</td><td>2005-05-26</td><td>desruisseaux</td><td>Added 'Default' suffix to some CRS implementations: partial work</td></tr>
+<tr><td class="rev">6325</td><td>2004-06-04</td><td>desruisseaux</td><td>Added MathTransform implementations</td></tr>
+<tr><td class="rev">6301</td><td>2004-06-03</td><td>desruisseaux</td><td>Refactored basic transform in their own package</td></tr>
+</table>
+    </div>
+  </body>
+</html>

Propchange: sis/ip-review/ConstantTransform1D.xhtml
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: sis/ip-review/ConstantTransform1D.xhtml
------------------------------------------------------------------------------
    svn:mime-type = text/html

Added: sis/ip-review/ImmutableAffineTransform.xhtml
URL: http://svn.apache.org/viewvc/sis/ip-review/ImmutableAffineTransform.xhtml?rev=1600526&view=auto
==============================================================================
--- sis/ip-review/ImmutableAffineTransform.xhtml (added)
+++ sis/ip-review/ImmutableAffineTransform.xhtml Wed Jun  4 22:26:14 2014
@@ -0,0 +1,64 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml">
+  <head>
+    <meta charset="UTF-8"/>
+    <title>ImmutableAffineTransform history</title>
+    <style type="text/css" media="all">
+      @import url("./reports.css");
+    </style>
+  </head>
+  <body>
+    <div>
+      <h1>ImmutableAffineTransform history</h1>
+  <p>Click on the commit message for inspecting the <code>diff</code> and how the code has been rewritten.</p>
+<p><b>Command line:</b></p>
+<blockquote><code>svn log -r31996:1 http://svn.osgeo.org/geotools/trunk/modules/library/referencing/src/main/java/org/geotools/referencing/operation/matrix/XAffineTransform.java</code></blockquote>
+<table>
+  <tr>
+    <th>Rev.</th>
+    <th>Date</th>
+    <th>Author</th>
+    <th class="last">Message</th>
+  </tr>
+<tr><td class="rev">31486</td><td>2008-09-09</td><td>desruisseaux</td><td>Documentation adjustements (no code change).</td></tr>
+<tr><td class="rev">31456</td><td>2008-09-08</td><td>desruisseaux</td><td>Fixed a StackOverflowError: AffineTransform2D.createTransformedShape was invoking itself indirectly when the given shape is not one of the recognized shapes like Rectangle2D.</td></tr>
+<tr><td class="rev">30641</td><td>2008-06-12</td><td>acuster</td><td>Copyright headers: lib/referencing, this time with feeling (and the el in Toolkit)</td></tr>
+<tr><td class="rev">30639</td><td>2008-06-12</td><td>acuster</td><td>Copyright headers: cleanup library/referencing</td></tr>
+<tr><td class="rev">30521</td><td>2008-06-05</td><td>acuster</td><td>Copyright: Update referencing headers</td></tr>
+<tr><td class="rev">30258</td><td>2008-05-08</td><td>acuster</td><td>Reshuffle the top level repo: drop uDig, move up trunk, tags, and branches.</td></tr>
+<tr><td class="rev">30257</td><td>2008-05-08</td><td>acuster</td><td>Move trunk/gt/ directory contents up to trunk/ and drop gt</td></tr>
+<tr><td class="rev">29732</td><td>2008-03-28</td><td>desruisseaux</td><td>Added a case for java.awt.geom.Area.</td></tr>
+<tr><td class="rev">29725</td><td>2008-03-27</td><td>desruisseaux</td><td>Added a XAffineTransform.transform(..., Shape, ...) method trying to preserve the shape class.</td></tr>
+<tr><td class="rev">29422</td><td>2008-02-21</td><td>desruisseaux</td><td>Provides a convenience TileBuilder.setMosaicEnvelope(...) method for getting the 'gridToCRS' transform computed automatically for us, with usual assumption on axis flipping, etc. Users can control those assumptions by overriding the createGridToEnveloppeMapper(...) method.</td></tr>
+<tr><td class="rev">28922</td><td>2008-01-24</td><td>acuster</td><td>Bump the (at)since version to 2.5 since WKTParser was cut from 2.4</td></tr>
+<tr><td class="rev">28817</td><td>2008-01-18</td><td>desruisseaux</td><td>Added constructor with 6-double values parameters.</td></tr>
+<tr><td class="rev">28540</td><td>2007-12-29</td><td>acuster</td><td>Hide buttons which are not yet used</td></tr>
+<tr><td class="rev">28502</td><td>2007-12-25</td><td>desruisseaux</td><td>Optimizations.</td></tr>
+<tr><td class="rev">27983</td><td>2007-11-22</td><td>desruisseaux</td><td>Use more Java 5 constructs for safety.</td></tr>
+<tr><td class="rev">27972</td><td>2007-11-21</td><td>desruisseaux</td><td>More use of Java 5 generic. The generic spotted for us a bug in Citation.getIdentifiers(), which returns a set of Strings when it should returns a set of Identifiers. Not yet fixed because it will require a search in the whole GeoTools code base. I wish we used Generic types sooner for preventing this kind of bug...</td></tr>
+<tr><td class="rev">27575</td><td>2007-10-22</td><td>desruisseaux</td><td>Converted Java source files from ISO-LATIN-1 encoding to UTF-8 (GEOT-1516).</td></tr>
+<tr><td class="rev">26030</td><td>2007-06-26</td><td>desruisseaux</td><td>Mostly javadoc cleaning.</td></tr>
+<tr><td class="rev">24493</td><td>2007-02-17</td><td>desruisseaux</td><td>Cleaning in 'isIdentity(double)' methods. Note that AbstractMathTransform.isIdentity(double) may move in the LinearTransform interface, so we are marking them as deprecated for now and will create a new LinearTransform.isIdentity(double) method in a next commit.</td></tr>
+<tr><td class="rev">24485</td><td>2007-02-16</td><td class="unav">simboss</td><td><a href="rev/24485/ImmutableAffineTransform.xhtml">-first bunch of changes for the alignment of trunk with 2.3.x</a></td></tr>
+<tr><td class="rev">22997</td><td>2006-11-26</td><td>desruisseaux</td><td>Replaced the use of some deprecated methods (MonolineFormatter.initGeotools --> Logging.forceMonolineConsoleOutput; Utilities.unexpectedException --> Logging.unexpectedException; GeneralEnvelope.getCoordinateReferenceSystem --> Envelope.getCoordinateReferenceSystem).</td></tr>
+<tr><td class="rev">22711</td><td>2006-11-12</td><td>desruisseaux</td><td>Moved the GeneralGridGeometry.getEnvelope(...) static method as a GeneralEnvelope constructor.</td></tr>
+<tr><td class="rev">22710</td><td>2006-11-12</td><td>desruisseaux</td><td>Review of coverage branch merge: In 'CategoryList', check for the 'no data' category name only if the geophysics value is some NaN value. Moved XAffineTransform to a public place and added methods for testing axis swapping, as a proposed replacement for 'GeneralGridGeometry.swapXY(CoordinateSystem)' (replacement not yet done; waiting for the okay from the mailing list). In GeneralGridGeometry, added methods for returning a math transform relative to some pixel corner instead of pixel center, as a proposed replacement for the 'getTransform(...)' public static methods (I would like to remove those static methods in order to keep the API a little bit more orthogonal).</td></tr>
+<tr><td class="rev">22327</td><td>2006-10-23</td><td>desruisseaux</td><td>GEOT-982: regroup 'module', 'plugin', 'ext' and 'unsupported' in a common directory.</td></tr>
+<tr><td class="rev">22315</td><td>2006-10-22</td><td>desruisseaux</td><td>Reorganisation of directory tree structure (GEOT-982) phase 1: moved 'src' to 'src/main/java'.</td></tr>
+<tr><td class="rev">20874</td><td>2006-08-07</td><td class="unav">jgarnett</td><td><a href="rev/20874/ImmutableAffineTransform.xhtml">ip review</a></td></tr>
+<tr><td class="rev">17672</td><td>2006-01-19</td><td>desruisseaux</td><td>Added @source tag.</td></tr>
+<tr><td class="rev">17660</td><td>2006-01-18</td><td>desruisseaux</td><td>Fixed SVN attributes, including the addition of URL attribute.</td></tr>
+<tr><td class="rev">15350</td><td>2005-08-16</td><td>desruisseaux</td><td>Merged the 'split-main' branch to trunk (GEOT-662)</td></tr>
+<tr><td class="rev">15256</td><td>2005-08-09</td><td>desruisseaux</td><td>Extraction of referencing module from main</td></tr>
+<tr><td class="rev">15255</td><td>2005-08-09</td><td>desruisseaux</td><td>Prepare branch for splitting main</td></tr>
+<tr><td class="rev">14834</td><td>2005-07-20</td><td>desruisseaux</td><td>Reorganized I18N resources</td></tr>
+<tr><td class="rev">13925</td><td>2005-05-30</td><td>desruisseaux</td><td>Added @since javadoc tag</td></tr>
+<tr><td class="rev">11001</td><td>2005-02-02</td><td>desruisseaux</td><td>Fixed wrong encoding in main/src</td></tr>
+<tr><td class="rev">10796</td><td>2005-01-28</td><td class="unav">dzwiers</td><td><a href="rev/10796/ImmutableAffineTransform.xhtml">imports cleaned</a></td></tr>
+<tr><td class="rev">6410</td><td>2004-06-12</td><td class="unav">jgarnett</td><td><a href="rev/10796/ImmutableAffineTransform.xhtml">Added geoapi dependency to imagedatasource, and fixed spelling mistake in view.</a></td></tr>
+<tr><td class="rev">6209</td><td>2004-05-27</td><td>desruisseaux</td><td>Added Id property</td></tr>
+<tr><td class="rev">6150</td><td>2004-05-25</td><td>desruisseaux</td><td>Moved geometry related resources in the org.geotools.resources.geometry package</td></tr>
+</table>
+    </div>
+  </body>
+</html>

Propchange: sis/ip-review/ImmutableAffineTransform.xhtml
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: sis/ip-review/ImmutableAffineTransform.xhtml
------------------------------------------------------------------------------
    svn:mime-type = text/html

Added: sis/ip-review/rev/10796/AffineTransform2D.xhtml
URL: http://svn.apache.org/viewvc/sis/ip-review/rev/10796/AffineTransform2D.xhtml?rev=1600526&view=auto
==============================================================================
--- sis/ip-review/rev/10796/AffineTransform2D.xhtml (added)
+++ sis/ip-review/rev/10796/AffineTransform2D.xhtml Wed Jun  4 22:26:14 2014
@@ -0,0 +1,93 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml">
+  <head>
+    <meta charset="UTF-8"/>
+    <title>AffineTransform2D changes for revisions 10795:10796</title>
+    <style type="text/css" media="all">
+      @import url("../../reports.css");
+    </style>
+  </head>
+  <body>
+    <div>
+      <h1>AffineTransform2D changes for revisions 10795:10796</h1>
+<p>Changes in this commit seem to be the result of some "<cite>auto reformat</cite>" tool execution.
+The <cite>Java</cite> - <cite>GeoAPI</cite> - <cite>GeoTools</cite> import ordering is altered,
+imports used only in Javadoc are lost and the encoding of non-ASCII characters is broken.
+This commit has been reverted, except for the removal of really unused imports which are keep removed.</p>
+<p><b>Command line:</b></p>
+<blockquote><code>svn diff --extensions "--unified --ignore-space-change --ignore-all-space --ignore-eol-style" -r10795:10796 http://svn.osgeo.org/geotools/trunk/modules/library/referencing/src/main/java/org/geotools/referencing/operation/transform/AffineTransform2D.java</code></blockquote>
+<table class="changes">
+<tr><th>Revision 10795</th><th>Revision 10796</th></tr>
+<tr><td><pre>/*
+ * Geotools 2 - OpenSource mapping toolkit
+ * (C) 2003, Geotools Project Managment Committee (PMC)
+<span class="del"> * (C) 2001, Institut de Recherche pour le Développement</span>
+ *
+ *    This library is free software; you can redistribute it and/or
+ *    modify it under the terms of the GNU Lesser General Public</pre></td>
+<td><pre>/*
+ * Geotools 2 - OpenSource mapping toolkit
+ * (C) 2003, Geotools Project Managment Committee (PMC)
+<span class="add"> * (C) 2001, Institut de Recherche pour le D�veloppement</span>
+ *
+ *    This library is free software; you can redistribute it and/or
+ *    modify it under the terms of the GNU Lesser General Public</pre></td></tr>
+<tr><td><pre>package org.geotools.referencing.operation.transform;
+
+// J2SE dependencies
+import java.awt.geom.Point2D;
+<span class="del">import java.awt.geom.AffineTransform;</span>
+import java.util.prefs.Preferences;
+
+<span class="del">// OpenGIS dependencies</span>
+<span class="del">import org.opengis.referencing.operation.Matrix;</span>
+import org.opengis.referencing.operation.MathTransform;
+import org.opengis.referencing.operation.MathTransform2D;
+import org.opengis.referencing.operation.NoninvertibleTransformException;
+import org.opengis.spatialschema.geometry.DirectPosition;
+<span class="del">import org.opengis.parameter.ParameterValueGroup;</span>
+
+<span class="del">// Geotools dependencies</span>
+<span class="del">import org.geotools.referencing.wkt.Formatter;</span>
+<span class="del">import org.geotools.resources.Formattable;</span>
+<span class="del">import org.geotools.resources.cts.Resources;</span>
+<span class="del">import org.geotools.resources.cts.ResourceKeys;</span>
+<span class="del">import org.geotools.resources.geometry.XAffineTransform;</span>
+<span class="del">import org.geotools.referencing.operation.LinearTransform;</span>
+<span class="del">import org.geotools.referencing.operation.GeneralMatrix;</span>
+<span class="del">import org.geotools.geometry.GeneralDirectPosition;</span>
+
+<span class="del"></span>
+/**
+ * Transforms two-dimensional coordinate points using an {@link AffineTransform}.
+ *</pre></td>
+<td><pre>package org.geotools.referencing.operation.transform;
+
+// J2SE dependencies
+<span class="add">import java.awt.geom.AffineTransform;</span>
+import java.awt.geom.Point2D;
+import java.util.prefs.Preferences;
+
+<span class="add">import org.geotools.geometry.GeneralDirectPosition;</span>
+<span class="add">import org.geotools.referencing.operation.GeneralMatrix;</span>
+<span class="add">import org.geotools.referencing.operation.LinearTransform;</span>
+<span class="add">import org.geotools.referencing.wkt.Formatter;</span>
+<span class="add">import org.geotools.resources.Formattable;</span>
+<span class="add">import org.geotools.resources.cts.ResourceKeys;</span>
+<span class="add">import org.geotools.resources.cts.Resources;</span>
+<span class="add">import org.geotools.resources.geometry.XAffineTransform;</span>
+<span class="add">import org.opengis.parameter.ParameterValueGroup;</span>
+import org.opengis.referencing.operation.MathTransform;
+import org.opengis.referencing.operation.MathTransform2D;
+<span class="add">import org.opengis.referencing.operation.Matrix;</span>
+import org.opengis.referencing.operation.NoninvertibleTransformException;
+import org.opengis.spatialschema.geometry.DirectPosition;
+
+
+/**
+ * Transforms two-dimensional coordinate points using an {@link AffineTransform}.
+ *</pre></td></tr>
+</table>
+    </div>
+  </body>
+</html>

Propchange: sis/ip-review/rev/10796/AffineTransform2D.xhtml
------------------------------------------------------------------------------
    svn:eol-style = native

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

Added: sis/ip-review/rev/10796/ImmutableAffineTransform.xhtml
URL: http://svn.apache.org/viewvc/sis/ip-review/rev/10796/ImmutableAffineTransform.xhtml?rev=1600526&view=auto
==============================================================================
--- sis/ip-review/rev/10796/ImmutableAffineTransform.xhtml (added)
+++ sis/ip-review/rev/10796/ImmutableAffineTransform.xhtml Wed Jun  4 22:26:14 2014
@@ -0,0 +1,71 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml">
+  <head>
+    <meta charset="UTF-8"/>
+    <title>ImmutableAffineTransform changes for revisions 6409:10796</title>
+    <style type="text/css" media="all">
+      @import url("../../reports.css");
+    </style>
+  </head>
+  <body>
+    <div>
+      <h1>ImmutableAffineTransform changes for revisions 6409:10796</h1>
+<p>Changes in this commit seem to be the result of some "<cite>auto reformat</cite>" tool execution.
+The <cite>Java</cite> - <cite>GeoAPI</cite> - <cite>GeoTools</cite> import ordering is altered,
+imports used only in Javadoc are lost and the encoding of non-ASCII characters is broken.
+This commit has been reverted, except for the removal of really unused imports which are keep removed.</p>
+<p><b>Command line:</b></p>
+<blockquote><code>svn diff --extensions "--unified --ignore-space-change --ignore-all-space --ignore-eol-style" -r6409:10796 http://svn.osgeo.org/geotools/trunk/modules/library/referencing/src/main/java/org/geotools/referencing/operation/matrix/XAffineTransform.java</code></blockquote>
+<table class="changes">
+<tr><th>Revision 6409</th><th>Revision 10796</th></tr>
+<tr><td><pre>/*
+ * Geotools 2 - OpenSource mapping toolkit
+ * (C) 2003, Geotools Project Managment Committee (PMC)
+<span class="del"> * (C) 2001, Institut de Recherche pour le Développement</span>
+ *
+ *    This library is free software; you can redistribute it and/or
+ *    modify it under the terms of the GNU Lesser General Public</pre></td>
+<td><pre>/*
+ * Geotools 2 - OpenSource mapping toolkit
+ * (C) 2003, Geotools Project Managment Committee (PMC)
+<span class="add"> * (C) 2001, Institut de Recherche pour le D�veloppement</span>
+ *
+ *    This library is free software; you can redistribute it and/or
+ *    modify it under the terms of the GNU Lesser General Public</pre></td></tr>
+<tr><td><pre>package org.geotools.resources.geometry;
+
+// J2SE dependencies
+import java.awt.geom.Point2D;
+import java.awt.geom.Rectangle2D;
+<span class="del">import java.awt.geom.AffineTransform;</span>
+<span class="del">import java.awt.geom.NoninvertibleTransformException;</span>
+
+<span class="del">// Geotools dependencies</span>
+import org.geotools.resources.XMath;</pre></td>
+<td><pre>package org.geotools.resources.geometry;
+
+// J2SE dependencies
+<span class="add">import java.awt.geom.AffineTransform;</span>
+<span class="add">import java.awt.geom.NoninvertibleTransformException;</span>
+import java.awt.geom.Point2D;
+import java.awt.geom.Rectangle2D;
+
+import org.geotools.resources.XMath;</pre></td></tr>
+<tr><td><pre> * defined to always throw an exception, then &lt;code&gt;XAffineTransform&lt;/code&gt;
+ * is immutable.
+ *
+<span class="del"> * @version $Id: XAffineTransform.java,v 1.8 2003/06/25 15:16:19 desruisseaux Exp $</span>
+ * @author Martin Desruisseaux
+ */
+public abstract class XAffineTransform extends AffineTransform {</pre></td>
+<td><pre> * defined to always throw an exception, then &lt;code&gt;XAffineTransform&lt;/code&gt;
+ * is immutable.
+ *
+<span class="add"> * @version $Id$</span>
+ * @author Martin Desruisseaux
+ */
+public abstract class XAffineTransform extends AffineTransform {</pre></td></tr>
+</table>
+    </div>
+  </body>
+</html>

Propchange: sis/ip-review/rev/10796/ImmutableAffineTransform.xhtml
------------------------------------------------------------------------------
    svn:eol-style = native

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

Added: sis/ip-review/rev/20874/AffineTransform2D.xhtml
URL: http://svn.apache.org/viewvc/sis/ip-review/rev/20874/AffineTransform2D.xhtml?rev=1600526&view=auto
==============================================================================
--- sis/ip-review/rev/20874/AffineTransform2D.xhtml (added)
+++ sis/ip-review/rev/20874/AffineTransform2D.xhtml Wed Jun  4 22:26:14 2014
@@ -0,0 +1,51 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml">
+  <head>
+    <meta charset="UTF-8"/>
+    <title>AffineTransform2D changes for revisions 20873:20874</title>
+    <style type="text/css" media="all">
+      @import url("../../reports.css");
+    </style>
+  </head>
+  <body>
+    <div>
+      <h1>AffineTransform2D 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/AffineTransform2D.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/AffineTransform2D.xhtml
------------------------------------------------------------------------------
    svn:eol-style = native

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

Added: sis/ip-review/rev/20874/ConstantTransform1D.xhtml
URL: http://svn.apache.org/viewvc/sis/ip-review/rev/20874/ConstantTransform1D.xhtml?rev=1600526&view=auto
==============================================================================
--- sis/ip-review/rev/20874/ConstantTransform1D.xhtml (added)
+++ sis/ip-review/rev/20874/ConstantTransform1D.xhtml Wed Jun  4 22:26:14 2014
@@ -0,0 +1,51 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml">
+  <head>
+    <meta charset="UTF-8"/>
+    <title>ConstantTransform1D changes for revisions 20873:20874</title>
+    <style type="text/css" media="all">
+      @import url("../../reports.css");
+    </style>
+  </head>
+  <body>
+    <div>
+      <h1>ConstantTransform1D 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/ConstantTransform1D.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/ConstantTransform1D.xhtml
------------------------------------------------------------------------------
    svn:eol-style = native

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

Added: sis/ip-review/rev/20874/ImmutableAffineTransform.xhtml
URL: http://svn.apache.org/viewvc/sis/ip-review/rev/20874/ImmutableAffineTransform.xhtml?rev=1600526&view=auto
==============================================================================
--- sis/ip-review/rev/20874/ImmutableAffineTransform.xhtml (added)
+++ sis/ip-review/rev/20874/ImmutableAffineTransform.xhtml Wed Jun  4 22:26:14 2014
@@ -0,0 +1,50 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml">
+  <head>
+    <meta charset="UTF-8"/>
+    <title>ImmutableAffineTransform changes for revisions 20873:20874</title>
+    <style type="text/css" media="all">
+      @import url("../../reports.css");
+    </style>
+  </head>
+  <body>
+    <div>
+      <h1>ImmutableAffineTransform 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/matrix/XAffineTransform.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"> *    (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.resources.geometry;</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.resources.geometry;</pre></td></tr>
+</table>
+    </div>
+  </body>
+</html>

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

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

Added: sis/ip-review/rev/24485/ImmutableAffineTransform.xhtml
URL: http://svn.apache.org/viewvc/sis/ip-review/rev/24485/ImmutableAffineTransform.xhtml?rev=1600526&view=auto
==============================================================================
--- sis/ip-review/rev/24485/ImmutableAffineTransform.xhtml (added)
+++ sis/ip-review/rev/24485/ImmutableAffineTransform.xhtml Wed Jun  4 22:26:14 2014
@@ -0,0 +1,324 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml">
+  <head>
+    <meta charset="UTF-8"/>
+    <title>ImmutableAffineTransform changes for revisions 24484:24485</title>
+    <style type="text/css" media="all">
+      @import url("../../reports.css");
+    </style>
+  </head>
+  <body>
+    <div>
+      <h1>ImmutableAffineTransform changes for revisions 24484:24485</h1>
+<p>This commit has been reverted:</p>
+<ul>
+  <li><code>final</code> keyword in static methods is not necessary.</li>
+  <li>A single tolerance factor for comparison of matrix coefficients does not work,
+  because scale and offset terms have very different magnitude (unless we use relative
+  tolerance, but this is not the case in this commit).</li>
+</ul>
+<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/matrix/XAffineTransform.java</code></blockquote>
+<table class="changes">
+<tr><th>Revision 24484</th><th>Revision 24485</th></tr>
+<tr><td><pre> */
+public abstract class XAffineTransform extends AffineTransform {
+    /**
+     * Serial number for interoperability with different versions.
+     */
+    private static final long serialVersionUID = 5215291166450556451L;</pre></td>
+<td><pre> */
+public abstract class XAffineTransform extends AffineTransform {
+    /**
+<span class="add">     * Redefines {@link #isIdentity()} default behaviour from {@link AffineTransform}</span>
+<span class="add">     * in order to keep into account roundings errors.</span>
+<span class="add">     *</span>
+<span class="add">     * &lt;p&gt;</span>
+<span class="add">     * For example if we have the following affine transnform</span>
+<span class="add">     *</span>
+<span class="add">     * 1.0000000000000000001   0               0</span>
+<span class="add">     * 0               0.999999999999999999999  0</span>
+<span class="add">     * 0               0                 1</span>
+<span class="add">     *</span>
+<span class="add">     * I would say that is pretty hard to state that is not the identity.</span>
+<span class="add">     *</span>
+<span class="add">     * &lt;P&gt;</span>
+<span class="add">     * Keep in mind that the original check is pretty simple, see below here:</span>
+<span class="add">     *</span>
+<span class="add">     *   return (state == APPLY_IDENTITY || (getType() == TYPE_IDENTITY));</span>
+<span class="add">     *</span>
+<span class="add">     *</span>
+<span class="add">     *  @param tr {@link AffineTransform} to be checked for identity.</span>
+<span class="add">     *  @param tolerance to use when checking for identity.</span>
+<span class="add">     *  return &lt;code&gt;true&lt;/code&gt; id this tranformation is *close* enough to the</span>
+<span class="add">     *       indentity, &lt;code&gt;false&lt;/code&gt; otherwise.</span>
+<span class="add">     *  @since 2.3.1</span>
+<span class="add">     */</span>
+<span class="add">    public final static boolean isIdentity(final AffineTransform tr,double tolerance) {</span>
+<span class="add">        tolerance=Math.abs(tolerance);</span>
+<span class="add">        boolean isIdentity=tr.isIdentity();</span>
+<span class="add">        if(!isIdentity){</span>
+<span class="add">           //get scale</span>
+<span class="add">         final double scale=getScale(tr);</span>
+<span class="add">         if(Math.abs(scale-1)&gt;tolerance)</span>
+<span class="add">          return false;</span>
+<span class="add">         //rotation</span>
+<span class="add">         final double rotation =XAffineTransform.getRotation(tr);</span>
+<span class="add">         if(Math.abs(rotation)&gt;tolerance)</span>
+<span class="add">          return false;</span>
+<span class="add">         //translations</span>
+<span class="add">         final double transX=tr.getTranslateX();</span>
+<span class="add">         if(Math.abs(transX)&gt;tolerance)</span>
+<span class="add">          return false;</span>
+<span class="add">         final double transY=tr.getTranslateY();</span>
+<span class="add">         if(Math.abs(transY)&gt;tolerance)</span>
+<span class="add">          return false;</span>
+<span class="add">         //shear</span>
+<span class="add">         final double shearX=tr.getShearX();</span>
+<span class="add">         if(Math.abs(shearX)&gt;tolerance)</span>
+<span class="add">          return false;</span>
+<span class="add">         final double shearY=tr.getShearY();</span>
+<span class="add">         if(Math.abs(shearY)&gt;tolerance)</span>
+<span class="add">          return false;</span>
+<span class="add"></span>
+<span class="add">         //scale NOT SURE WE NEED THIS</span>
+<span class="add">         final double scaleX=tr.getScaleX();</span>
+<span class="add">         if(Math.abs(scaleX-1)&gt;tolerance)</span>
+<span class="add">          return false;</span>
+<span class="add">         final double scaleY=tr.getScaleY();</span>
+<span class="add">         if(Math.abs(scaleY-1)&gt;tolerance)</span>
+<span class="add">          return false;</span>
+<span class="add">         return true;</span>
+<span class="add"></span>
+<span class="add"></span>
+<span class="add">           ////</span>
+<span class="add">           //</span>
+<span class="add">           // Proposed replacement less performace though</span>
+<span class="add">           //</span>
+<span class="add">           ////</span>
+<span class="add">//       final AffineTransform clonedTransformation=(AffineTransform) tr.clone();</span>
+<span class="add">//       XAffineTransform.round(clonedTransformation, tolerance);</span>
+<span class="add">//       if(clonedTransformation.isIdentity())</span>
+<span class="add">//          return true;</span>
+<span class="add">        }</span>
+<span class="add">       return isIdentity;</span>
+<span class="add">    }</span>
+<span class="add">    /**</span>
+<span class="add">     * Check whether or not this {@link XAffineTransform} is te identity by</span>
+<span class="add">     * using the provided &lt;code&gt;tolerance&lt;/code&gt;.</span>
+<span class="add">     *</span>
+<span class="add">     *</span>
+<span class="add">     * @param tolerance to use for this check.</span>
+<span class="add">     * @return true if the check succeeds, faflse otherwise.</span>
+<span class="add">     * @since 2.3.1</span>
+<span class="add">     */</span>
+<span class="add">    public boolean isIdentity(double tolerance) {</span>
+<span class="add">        return isIdentity(this, tolerance);</span>
+<span class="add">    }</span>
+<span class="add">    /**</span>
+     * Serial number for interoperability with different versions.
+     */
+    private static final long serialVersionUID = 5215291166450556451L;</pre></td></tr>
+<tr><td><pre>/**
+ * Tolerance value for floating point comparisons.
+ */
+<span class="del">private static final double EPS = 1E-6;</span>
+
+/**
+ * Constructs a new {@code XAffineTransform} that is a</pre></td>
+<td><pre>/**
+ * Tolerance value for floating point comparisons.
+ */
+<span class="add">public static final double EPS = 1E-6;</span>
+
+/**
+ * Constructs a new {@code XAffineTransform} that is a</pre></td></tr>
+<tr><td><pre> *
+ * @return The direct transform of the {@code bounds} rectangle.
+ */
+<span class="del">public static Rectangle2D transform(final AffineTransform transform,</span>
+                                    final Rectangle2D     bounds,
+                                    final Rectangle2D     dest)
+{</pre></td>
+<td><pre> *
+ * @return The direct transform of the {@code bounds} rectangle.
+ */
+<span class="add">public final static Rectangle2D transform(final AffineTransform transform,</span>
+                                    final Rectangle2D     bounds,
+                                    final Rectangle2D     dest)
+{</pre></td></tr>
+<tr><td><pre> * @return The inverse transform of the {@code bounds} rectangle.
+ * @throws NoninvertibleTransformException if the affine transform can't be inverted.
+ */
+<span class="del">public static Rectangle2D inverseTransform(final AffineTransform transform,</span>
+                                           final Rectangle2D     bounds,
+                                           final Rectangle2D     dest)
+        throws NoninvertibleTransformException</pre></td>
+<td><pre> * @return The inverse transform of the {@code bounds} rectangle.
+ * @throws NoninvertibleTransformException if the affine transform can't be inverted.
+ */
+<span class="add">public final static Rectangle2D inverseTransform(final AffineTransform transform,</span>
+                                           final Rectangle2D     bounds,
+                                           final Rectangle2D     dest)
+        throws NoninvertibleTransformException</pre></td></tr>
+<tr><td><pre> * @return The inverse transform of the {@code source} point.
+ * @throws NoninvertibleTransformException if the affine transform can't be inverted.
+ */
+<span class="del">public static Point2D inverseDeltaTransform(final AffineTransform transform,</span>
+                                            final Point2D         source,
+                                            final Point2D         dest)
+        throws NoninvertibleTransformException</pre></td>
+<td><pre> * @return The inverse transform of the {@code source} point.
+ * @throws NoninvertibleTransformException if the affine transform can't be inverted.
+ */
+<span class="add">public final static Point2D inverseDeltaTransform(final AffineTransform transform,</span>
+                                            final Point2D         source,
+                                            final Point2D         dest)
+        throws NoninvertibleTransformException</pre></td></tr>
+<tr><td><pre> * preserved, {@code -1} if the transform seems to swap axis to the (&lt;var&gt;y&lt;/var&gt;,
+ * &lt;var&gt;x&lt;/var&gt;) axis order, or {@code 0} if this method can not make a decision.
+ */
+<span class="del">public static int getSwapXY(final AffineTransform tr) {</span>
+    final int flip = getFlip(tr);
+    if (flip != 0) {
+        final double scaleX = getScaleX0(tr);</pre></td>
+<td><pre> * preserved, {@code -1} if the transform seems to swap axis to the (&lt;var&gt;y&lt;/var&gt;,
+ * &lt;var&gt;x&lt;/var&gt;) axis order, or {@code 0} if this method can not make a decision.
+ */
+<span class="add">public final static int getSwapXY(final AffineTransform tr) {</span>
+    final int flip = getFlip(tr);
+    if (flip != 0) {
+        final double scaleX = getScaleX0(tr);</pre></td></tr>
+<tr><td><pre> * @return An estimation of the rotation angle in radians, or {@link Double#NaN NaN}
+ *         if the angle can not be estimated.
+ */
+<span class="del">public static double getRotation(final AffineTransform tr) {</span>
+    final int flip = getFlip(tr);
+    if (flip != 0) {
+        final double scaleX = getScaleX0(tr);</pre></td>
+<td><pre> * @return An estimation of the rotation angle in radians, or {@link Double#NaN NaN}
+ *         if the angle can not be estimated.
+ */
+<span class="add">public final static double getRotation(final AffineTransform tr) {</span>
+    final int flip = getFlip(tr);
+    if (flip != 0) {
+        final double scaleX = getScaleX0(tr);</pre></td></tr>
+<tr><td><pre> * boolean flipped = (tr.{@linkplain #getType() getType()} &amp; {@linkplain #TYPE_FLIP}) != 0;
+ * &lt;/code&gt;&lt;/blockquote&gt;
+ */
+<span class="del">public static int getFlip(final AffineTransform tr) {</span>
+    final int scaleX = XMath.sgn(tr.getScaleX());
+    final int scaleY = XMath.sgn(tr.getScaleY());
+    final int shearX = XMath.sgn(tr.getShearX());</pre></td>
+<td><pre> * boolean flipped = (tr.{@linkplain #getType() getType()} &amp; {@linkplain #TYPE_FLIP}) != 0;
+ * &lt;/code&gt;&lt;/blockquote&gt;
+ */
+<span class="add">public final static int getFlip(final AffineTransform tr) {</span>
+    final int scaleX = XMath.sgn(tr.getScaleX());
+    final int scaleY = XMath.sgn(tr.getScaleY());
+    final int shearX = XMath.sgn(tr.getShearX());</pre></td></tr>
+<tr><td><pre> * effect of eventual flip and rotation. This factor is calculated by
+ * &lt;IMG src="{@docRoot}/org/geotools/display/canvas/doc-files/scaleX0.png"&gt;.
+ */
+<span class="del">public static double getScaleX0(final AffineTransform tr) {</span>
+    return XMath.hypot(tr.getScaleX(), tr.getShearX());
+}</pre></td>
+<td><pre> * effect of eventual flip and rotation. This factor is calculated by
+ * &lt;IMG src="{@docRoot}/org/geotools/display/canvas/doc-files/scaleX0.png"&gt;.
+ */
+<span class="add">public final static double getScaleX0(final AffineTransform tr) {</span>
+    return XMath.hypot(tr.getScaleX(), tr.getShearX());
+}</pre></td></tr>
+<tr><td><pre> * effect of eventual flip and rotation. This factor is calculated by
+ * &lt;IMG src="{@docRoot}/org/geotools/display/canvas/doc-files/scaleY0.png"&gt;.
+ */
+<span class="del">public static double getScaleY0(final AffineTransform tr) {</span>
+    return XMath.hypot(tr.getScaleY(), tr.getShearY());
+}</pre></td>
+<td><pre> * effect of eventual flip and rotation. This factor is calculated by
+ * &lt;IMG src="{@docRoot}/org/geotools/display/canvas/doc-files/scaleY0.png"&gt;.
+ */
+<span class="add">public final  static double getScaleY0(final AffineTransform tr) {</span>
+    return XMath.hypot(tr.getScaleY(), tr.getShearY());
+}</pre></td></tr>
+<tr><td><pre> * The way to compute such a "global" scale is somewhat arbitrary and may change
+ * in a future version.
+ */
+<span class="del">public static double getScale(final AffineTransform tr) {</span>
+    return 0.5 * (getScaleX0(tr) + getScaleY0(tr));
+}</pre></td>
+<td><pre> * The way to compute such a "global" scale is somewhat arbitrary and may change
+ * in a future version.
+ */
+<span class="add">public final static double getScale(final AffineTransform tr) {</span>
+    return 0.5 * (getScaleX0(tr) + getScaleY0(tr));
+}</pre></td></tr>
+<tr><td><pre> * @return   Affine transform of a zoom which leaves the
+ *          (&lt;var&gt;x&lt;/var&gt;,&lt;var&gt;y&lt;/var&gt;) coordinate unchanged.
+ */
+<span class="del">public static AffineTransform getScaleInstance(final double sx, final double sy,</span>
+                                               final double  x, final double  y) {
+    return new AffineTransform(sx, 0, 0, sy, (1-sx)*x, (1-sy)*y);
+}</pre></td>
+<td><pre> * @return   Affine transform of a zoom which leaves the
+ *          (&lt;var&gt;x&lt;/var&gt;,&lt;var&gt;y&lt;/var&gt;) coordinate unchanged.
+ */
+<span class="add">public final  static AffineTransform getScaleInstance(final double sx, final double sy,</span>
+                                               final double  x, final double  y) {
+    return new AffineTransform(sx, 0, 0, sy, (1-sx)*x, (1-sy)*y);
+}</pre></td></tr>
+<tr><td><pre> * nearest whole numbers. This rounding up is useful, for example, for
+ * speeding up image displays.  Above all, it is efficient when we know that
+ * a matrix has a chance of being close to the similarity matrix.
+ */
+<span class="del">public static void round(final AffineTransform tr) {</span>
+    double r;
+    final double m00, m01, m10, m11;
+<span class="del">    if (Math.abs((m00 = Math.rint(r = tr.getScaleX())) - r) &lt;= EPS &amp;&amp;</span>
+<span class="del">        Math.abs((m01 = Math.rint(r = tr.getShearX())) - r) &lt;= EPS &amp;&amp;</span>
+<span class="del">        Math.abs((m11 = Math.rint(r = tr.getScaleY())) - r) &lt;= EPS &amp;&amp;</span>
+<span class="del">        Math.abs((m10 = Math.rint(r = tr.getShearY())) - r) &lt;= EPS)</span>
+    {
+        if ((m00!=0 || m01!=0) &amp;&amp; (m10!=0 || m11!=0)) {
+<span class="del">            double m02=Math.rint(r=tr.getTranslateX()); if (!(Math.abs(m02-r)&lt;=EPS)) m02=r;</span>
+<span class="del">            double m12=Math.rint(r=tr.getTranslateY()); if (!(Math.abs(m12-r)&lt;=EPS)) m12=r;</span>
+            tr.setTransform(m00, m10, m01, m11, m02, m12);
+        }
+    }</pre></td>
+<td><pre> * nearest whole numbers. This rounding up is useful, for example, for
+ * speeding up image displays.  Above all, it is efficient when we know that
+ * a matrix has a chance of being close to the similarity matrix.
+<span class="add"> *</span>
+<span class="add"> * &lt;p&gt;</span>
+<span class="add"> * It is crucial to note that this method uses a default rounding threshold</span>
+<span class="add"> * whose value is hedl by the field {@link #EPS} which is {@value #EPS}.</span>
+ */
+<span class="add">public final static void round(final AffineTransform tr) {</span>
+<span class="add">  round(tr,EPS);</span>
+<span class="add">}</span>
+<span class="add"></span>
+<span class="add">/**</span>
+<span class="add"> * Checks whether the matrix coefficients are close to whole numbers.</span>
+<span class="add"> * If this is the case, these coefficients will be rounded up to the</span>
+<span class="add"> * nearest whole numbers. This rounding up is useful, for example, for</span>
+<span class="add"> * speeding up image displays.  Above all, it is efficient when we know that</span>
+<span class="add"> * a matrix has a chance of being close to the similarity matrix.</span>
+<span class="add"> */</span>
+<span class="add">public final static void round(final AffineTransform tr,final double CUSTOM_EPS) {</span>
+    double r;
+    final double m00, m01, m10, m11;
+<span class="add">    if (Math.abs((m00 = Math.rint(r = tr.getScaleX())) - r) &lt;= CUSTOM_EPS &amp;&amp;</span>
+<span class="add">        Math.abs((m01 = Math.rint(r = tr.getShearX())) - r) &lt;= CUSTOM_EPS &amp;&amp;</span>
+<span class="add">        Math.abs((m11 = Math.rint(r = tr.getScaleY())) - r) &lt;= CUSTOM_EPS &amp;&amp;</span>
+<span class="add">        Math.abs((m10 = Math.rint(r = tr.getShearY())) - r) &lt;= CUSTOM_EPS)</span>
+    {
+        if ((m00!=0 || m01!=0) &amp;&amp; (m10!=0 || m11!=0)) {
+<span class="add">            double m02=Math.rint(r=tr.getTranslateX()); if (!(Math.abs(m02-r)&lt;=CUSTOM_EPS)) m02=r;</span>
+<span class="add">            double m12=Math.rint(r=tr.getTranslateY()); if (!(Math.abs(m12-r)&lt;=CUSTOM_EPS)) m12=r;</span>
+            tr.setTransform(m00, m10, m01, m11, m02, m12);
+        }
+    }</pre></td></tr>
+</table>
+    </div>
+  </body>
+</html>

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

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

Added: sis/ip-review/rev/24925/AffineTransform2D.xhtml
URL: http://svn.apache.org/viewvc/sis/ip-review/rev/24925/AffineTransform2D.xhtml?rev=1600526&view=auto
==============================================================================
--- sis/ip-review/rev/24925/AffineTransform2D.xhtml (added)
+++ sis/ip-review/rev/24925/AffineTransform2D.xhtml Wed Jun  4 22:26:14 2014
@@ -0,0 +1,39 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml">
+  <head>
+    <meta charset="UTF-8"/>
+    <title>AffineTransform2D changes for revisions 24924:24925</title>
+    <style type="text/css" media="all">
+      @import url("../../reports.css");
+    </style>
+  </head>
+  <body>
+    <div>
+      <h1>AffineTransform2D 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/AffineTransform2D.java</code></blockquote>
+<table class="changes">
+<tr><th>Revision 24924</th><th>Revision 24925</th></tr>
+<tr><td><pre>import org.opengis.referencing.operation.MathTransform2D;
+import org.opengis.referencing.operation.Matrix;
+import org.opengis.referencing.operation.NoninvertibleTransformException;
+<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.MathTransform2D;
+import org.opengis.referencing.operation.Matrix;
+import org.opengis.referencing.operation.NoninvertibleTransformException;
+<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/AffineTransform2D.xhtml
------------------------------------------------------------------------------
    svn:eol-style = native

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



Mime
View raw message