sis-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From desruisse...@apache.org
Subject svn commit: r1718942 - in /sis/ip-review: ./ rev/05908/ rev/06275/ rev/07854/ rev/10796/ rev/20874/ rev/25050/ rev/25890/
Date Wed, 09 Dec 2015 19:18:34 GMT
Author: desruisseaux
Date: Wed Dec  9 19:18:34 2015
New Revision: 1718942

URL: http://svn.apache.org/viewvc?rev=1718942&view=rev
Log:
Added IP review for test case.

Added:
    sis/ip-review/GeodeticObjectFactoryTest.xhtml   (with props)
    sis/ip-review/rev/05908/
    sis/ip-review/rev/05908/GeodeticObjectFactoryTest.xhtml   (with props)
    sis/ip-review/rev/05908/HEADER.html   (with props)
    sis/ip-review/rev/06275/GeodeticObjectFactoryTest.xhtml   (with props)
    sis/ip-review/rev/07854/
    sis/ip-review/rev/07854/GeodeticObjectFactoryTest.xhtml   (with props)
    sis/ip-review/rev/07854/HEADER.html   (with props)
    sis/ip-review/rev/10796/GeodeticObjectFactoryTest.xhtml   (with props)
    sis/ip-review/rev/20874/GeodeticObjectFactoryTest.xhtml   (with props)
    sis/ip-review/rev/25050/GeodeticObjectFactoryTest.xhtml   (with props)
    sis/ip-review/rev/25890/GeodeticObjectFactoryTest.xhtml   (with props)

Added: sis/ip-review/GeodeticObjectFactoryTest.xhtml
URL: http://svn.apache.org/viewvc/sis/ip-review/GeodeticObjectFactoryTest.xhtml?rev=1718942&view=auto
==============================================================================
--- sis/ip-review/GeodeticObjectFactoryTest.xhtml (added)
+++ sis/ip-review/GeodeticObjectFactoryTest.xhtml Wed Dec  9 19:18:34 2015
@@ -0,0 +1,82 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml">
+  <head>
+    <meta charset="UTF-8"/>
+    <title>GeodeticObjectFactoryTest history</title>
+    <style type="text/css" media="all">
+      @import url("./reports.css");
+    </style>
+  </head>
+  <body>
+    <div>
+      <h1>GeodeticObjectFactoryTest 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/test/java/org/geotools/referencing/FactoriesTest.java</code></blockquote>
+<table>
+  <tr>
+    <th>Rev.</th>
+    <th>Date</th>
+    <th>Author</th>
+    <th class="last">Message</th>
+  </tr>
+<tr><td class="rev">31148</td><td>2008-08-07</td><td>desruisseaux</td><td>Renamed
GenericName.name() as tip(). The old method still around as a deprecated one.</td></tr>
+<tr><td class="rev">31139</td><td>2008-08-06</td><td>desruisseaux</td><td>Avoid
usage of deprecated methods from the org.opengis.util.GenericName interface.</td></tr>
+<tr><td class="rev">31000</td><td>2008-07-10</td><td>desruisseaux</td><td>Avoid
usage of 2 deprecated constructors from CRSFactory. Their replacement were tricky and required
more sophesticated code in ReferencingFactoryContainer.toGeodetic3D.</td></tr>
+<tr><td class="rev">30760</td><td>2008-06-18</td><td>desruisseaux</td><td>Upgrated
from JSR-108 to JSR-275 (GEOT-1266). Some opportunist javadoc fixes and @Override annotations
in the process.</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">29845</td><td>2008-04-08</td><td>desruisseaux</td><td>More
aggressive simplification of MathTransforms during concatenation. It is probably related to
GEOT-1419, but I have not verified. This fix results in drastic performance improvement for
the cases where DefaultConcatenatedTransform previously failed to see concatenation that could
be simplified to identity transform. Those cases occured more often in 3D or 4D cases. It
may have some benefit in the pure 2D case as well, but probably less often.</td></tr>
+<tr><td class="rev">29768</td><td>2008-04-02</td><td>desruisseaux</td><td>Upgrated
the metadata and referencing tests to JUnit 4.</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">27995</td><td>2007-11-22</td><td>desruisseaux</td><td>More
generics type, and moved two methods from ReferencingFactoryContainer to GeoAPI MathTransformFactory
interface. Note that the build fails in the arcsde module for a reason that seems unrelated
to this work.</td></tr>
+<tr><td class="rev">27923</td><td>2007-11-18</td><td>desruisseaux</td><td>Applying
some more generic types, especially in the factory package. Required slight changes in the
way FactoryRegistry is used, as documented in http://docs.codehaus.org/display/GEOTOOLS/Fixing+compilation+errors</td></tr>
+<tr><td class="rev">25943</td><td>2007-06-20</td><td>desruisseaux</td><td>Minor
cleaning (mainly reformating).</td></tr>
+<tr><td class="rev">25890</td><td>2007-06-18</td><td class="unav">jgarnett</td><td><a
href="rev/25890/GeodeticObjectFactoryTest.xhtml">Changed from FactoryGroup to ReferencingFactoryContainer,
GEOT-1338</a></td></tr>
+<tr><td class="rev">25262</td><td>2007-04-23</td><td>desruisseaux</td><td>Finished
review of metadata and factory hints change. Fixed a potential glitch in parameter names handling
(some code were disabled in AbstractIdentifiedObject apparently in order to avoid a ClassCastException
caused by the spit of org.opengis.metadata.Identifier in two interfaces, Identifier and ReferenceIdentifier.
Fixed the exception and reenabled the code). Not sure yet what the consequence was. It may
help for the stereographic projection issue, but need to check.</td></tr>
+<tr><td class="rev">25050</td><td>2007-04-06</td><td class="unav">jgarnett</td><td><a
href="rev/25050/GeodeticObjectFactoryTest.xhtml">example of custom file based TestSutie,
copy JTS parsing code over to unsupported/geometry</a></td></tr>
+<tr><td class="rev">24576</td><td>2007-02-24</td><td>desruisseaux</td><td>Renamed
'StereographicFoo' classes as 'FooStereographic' for consistency with EPSG and OGC naming.
Note that 'ObliqueStereographic' is NOT the new name for 'StereographicOblique'; it is rather
the new name for 'StereographicDouble'. The old name was confusing since 'StereographicOblique'
was not for the EPSG 'Oblique_Stereographic' projection. The new naming scheme should match
more closely the common usage. In addition, also renamed some JUnit test classes in an attempt
to make clearer what they are testing.</td></tr>
+<tr><td class="rev">24402</td><td>2007-02-15</td><td>desruisseaux</td><td>Added
datum alias for WGS 84 (GEOT-409).</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">22317</td><td>2006-10-22</td><td>desruisseaux</td><td>Reorganisation
of directory tree structure (GEOT-982) phase 3: moved 'test' to 'src/test/java'.</td></tr>
+<tr><td class="rev">20874</td><td>2006-08-07</td><td class="unav">jgarnett</td><td><a
href="rev/20874/GeodeticObjectFactoryTest.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">16515</td><td>2005-10-25</td><td>desruisseaux</td><td>Made
the RegistryFactory more tolerant to failures during service initialization (GEOT-712)</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">14581</td><td>2005-06-22</td><td>desruisseaux</td><td>Implemented
South-Orientated transverse mercator / more GEOT-589 work</td></tr>
+<tr><td class="rev">14205</td><td>2005-06-09</td><td>desruisseaux</td><td>Updated
to GeoAPI 2.0 final release</td></tr>
+<tr><td class="rev">13907</td><td>2005-05-28</td><td>desruisseaux</td><td>More
additions of 'Default' prefix in CRS package</td></tr>
+<tr><td class="rev">13869</td><td>2005-05-27</td><td>desruisseaux</td><td>Added
'Default' or 'Abstract' prefix in CRS's 'datum' and 'cs' packages</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">12965</td><td>2005-04-05</td><td>desruisseaux</td><td>Merged
'factory-hints' branch r12393:12964 into the trunk. Added 'getImplementationHints()' method
after the merge into Factory implementations, in order to get Geotools to compile with Maven.</td></tr>
+<tr><td class="rev">12539</td><td>2005-03-22</td><td>desruisseaux</td><td>Moved
new datum aliases table to its right location (in ...referencing.factory package)</td></tr>
+<tr><td class="rev">12302</td><td>2005-03-16</td><td>desruisseaux</td><td>Enabled
the refactored DatumAliases as a replacement of the initial version</td></tr>
+<tr><td class="rev">12212</td><td>2005-03-15</td><td>desruisseaux</td><td>First
working version of DatumAliases as a DatumFactory</td></tr>
+<tr><td class="rev">11463</td><td>2005-02-17</td><td>desruisseaux</td><td>Remplaced
JDBC DriverManager by JDBC DataSource / Added EPSG FactoryForSQL</td></tr>
+<tr><td class="rev">11093</td><td>2005-02-03</td><td>desruisseaux</td><td>Roolback
to createParameterizedTransform without classification parameter</td></tr>
+<tr><td class="rev">11074</td><td>2005-02-03</td><td>desruisseaux</td><td>Revisited
CRSFactory.createProjectedCRS(...) API</td></tr>
+<tr><td class="rev">11008</td><td>2005-02-02</td><td>desruisseaux</td><td>Ask
OperationMethod to be specified explicitly</td></tr>
+<tr><td class="rev">10944</td><td>2005-02-01</td><td>desruisseaux</td><td>Bug
fix in ProjectedCRS: axis swapping and units conversion were not correctly handled</td></tr>
+<tr><td class="rev">10796</td><td>2005-01-28</td><td class="unav">dzwiers</td><td><a
href="rev/10796/GeodeticObjectFactoryTest.xhtml">imports cleaned</a></td></tr>
+<tr><td class="rev">10177</td><td>2005-01-03</td><td>desruisseaux</td><td>Tuned
JUnit tests for WKT parsing</td></tr>
+<tr><td class="rev">7854</td><td>2004-09-08</td><td class="unav">jgarnett</td><td><a
href="rev/07854/GeodeticObjectFactoryTest.xhtml">try and cleanup cs</a></td></tr>
+<tr><td class="rev">6729</td><td>2004-07-06</td><td>desruisseaux</td><td>Removed
some deprecated classes</td></tr>
+<tr><td class="rev">6275</td><td>2004-05-31</td><td class="unav">jgarnett</td><td><a
href="rev/06275/GeodeticObjectFactoryTest.xhtml">fixed format test problem</a></td></tr>
+<tr><td class="rev">6255</td><td>2004-05-31</td><td>desruisseaux</td><td>Bug
fix for horizontal and vertical line colinear with an axis</td></tr>
+<tr><td class="rev">5908</td><td>2004-05-23</td><td class="unav">jgarnett</td><td><a
href="rev/05908/GeodeticObjectFactoryTest.xhtml">cts-coordtrans moved to main</a></td></tr>
+<tr><td class="rev">5851</td><td>2004-05-23</td><td class="unav">jgarnett</td><td><a
href="rev/05908/GeodeticObjectFactoryTest.xhtml">Test is assumed to be the junit directory</a></td></tr>
+<tr><td class="rev">5774</td><td>2004-05-23</td><td class="unav">jgarnett</td><td><a
href="rev/05908/GeodeticObjectFactoryTest.xhtml">geotools-src serves as the start of our
new module directory (the data access modules will escape off to plugin directory soon)</a></td></tr>
+<tr><td class="rev">5769</td><td>2004-05-23</td><td class="unav">jgarnett</td><td><a
href="rev/05908/GeodeticObjectFactoryTest.xhtml">Project renamed to gt</a></td></tr>
+<tr><td class="rev">5619</td><td>2004-05-13</td><td class="unav">(pas
d'auteur)</td><td><a href="rev/05908/GeodeticObjectFactoryTest.xhtml">Move
repository under gt.</a></td></tr>
+<tr><td class="rev">3610</td><td>2003-08-04</td><td>desruisseaux</td><td>Testing
object creation</td></tr>
+</table>
+    </div>
+  </body>
+</html>

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

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

Added: sis/ip-review/rev/05908/GeodeticObjectFactoryTest.xhtml
URL: http://svn.apache.org/viewvc/sis/ip-review/rev/05908/GeodeticObjectFactoryTest.xhtml?rev=1718942&view=auto
==============================================================================
--- sis/ip-review/rev/05908/GeodeticObjectFactoryTest.xhtml (added)
+++ sis/ip-review/rev/05908/GeodeticObjectFactoryTest.xhtml Wed Dec  9 19:18:34 2015
@@ -0,0 +1,21 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml">
+  <head>
+    <meta charset="UTF-8"/>
+    <title>GeodeticObjectFactoryTest changes for revisions 5618:5908</title>
+    <style type="text/css" media="all">
+      @import url("../../reports.css");
+    </style>
+  </head>
+  <body>
+    <div>
+      <h1>GeodeticObjectFactoryTest changes for revisions 5618:5908</h1>
+<p>Commits cancel each other.</p>
+<p><b>Command line:</b></p>
+<blockquote><code>svn diff --extensions "--unified --ignore-space-change --ignore-all-space
--ignore-eol-style" -r5618:5908 http://svn.osgeo.org/geotools/trunk/modules/library/referencing/src/test/java/org/geotools/referencing/FactoriesTest.java</code></blockquote>
+<table class="changes">
+<tr><th>Revision 5618</th><th>Revision 5908</th></tr>
+</table>
+    </div>
+  </body>
+</html>

Propchange: sis/ip-review/rev/05908/GeodeticObjectFactoryTest.xhtml
------------------------------------------------------------------------------
    svn:eol-style = native

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

Added: sis/ip-review/rev/05908/HEADER.html
URL: http://svn.apache.org/viewvc/sis/ip-review/rev/05908/HEADER.html?rev=1718942&view=auto
==============================================================================
--- sis/ip-review/rev/05908/HEADER.html (added)
+++ sis/ip-review/rev/05908/HEADER.html Wed Dec  9 19:18:34 2015
@@ -0,0 +1,17 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml">
+  <head>
+    <meta charset="UTF-8"/>
+    <title>Revision 5908</title>
+  </head>
+  <body>
+    <div>
+      <h1>Revision 5908</h1>
+<table>
+  <tr><td><b>Author:</b></td><td>jgarnett</td></tr>
+  <tr><td><b>Date:</b></td><td>2004-05-23</td></tr>
+  <tr><td><b>Message:</b></td><td>cts-coordtrans moved
to main</td></tr>
+</table>
+    </div>
+  </body>
+</html>

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

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

Added: sis/ip-review/rev/06275/GeodeticObjectFactoryTest.xhtml
URL: http://svn.apache.org/viewvc/sis/ip-review/rev/06275/GeodeticObjectFactoryTest.xhtml?rev=1718942&view=auto
==============================================================================
--- sis/ip-review/rev/06275/GeodeticObjectFactoryTest.xhtml (added)
+++ sis/ip-review/rev/06275/GeodeticObjectFactoryTest.xhtml Wed Dec  9 19:18:34 2015
@@ -0,0 +1,36 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml">
+  <head>
+    <meta charset="UTF-8"/>
+    <title>GeodeticObjectFactoryTest changes for revisions 6274:6275</title>
+    <style type="text/css" media="all">
+      @import url("../../reports.css");
+    </style>
+  </head>
+  <body>
+    <div>
+      <h1>GeodeticObjectFactoryTest changes for revisions 6274:6275</h1>
+      <p>Does not apply to SIS since we do not use the SVN <code>$Id$</code>
property.</p>
+
+<p><b>Command line:</b></p>
+<blockquote><code>svn diff --extensions "--unified --ignore-space-change --ignore-all-space
--ignore-eol-style" -r6274:6275 http://svn.osgeo.org/geotools/trunk/modules/library/referencing/src/test/java/org/geotools/referencing/FactoriesTest.java</code></blockquote>
+<table class="changes">
+<tr><th>Revision 6274</th><th>Revision 6275</th></tr>
+<tr><td><pre>/**
+ * Test the creation of {@link CoordinateSystem} objects.
+ *
+<span class="del"> * @version $Id: CreationTest.java,v 1.1 2003/08/04 22:10:09 desruisseaux
Exp $</span>
+ */
+public class CreationTest extends TestCase {
+    /**</pre></td>
+<td><pre>/**
+ * Test the creation of {@link CoordinateSystem} objects.
+ *
+<span class="add"> * @version $Id$</span>
+ */
+public class CreationTest extends TestCase {
+    /**</pre></td></tr>
+</table>
+    </div>
+  </body>
+</html>

Propchange: sis/ip-review/rev/06275/GeodeticObjectFactoryTest.xhtml
------------------------------------------------------------------------------
    svn:eol-style = native

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

Added: sis/ip-review/rev/07854/GeodeticObjectFactoryTest.xhtml
URL: http://svn.apache.org/viewvc/sis/ip-review/rev/07854/GeodeticObjectFactoryTest.xhtml?rev=1718942&view=auto
==============================================================================
--- sis/ip-review/rev/07854/GeodeticObjectFactoryTest.xhtml (added)
+++ sis/ip-review/rev/07854/GeodeticObjectFactoryTest.xhtml Wed Dec  9 19:18:34 2015
@@ -0,0 +1,37 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml">
+  <head>
+    <meta charset="UTF-8"/>
+    <title>GeodeticObjectFactoryTest changes for revisions 7853:7854</title>
+    <style type="text/css" media="all">
+      @import url("../../reports.css");
+    </style>
+  </head>
+  <body>
+    <div>
+      <h1>GeodeticObjectFactoryTest changes for revisions 7853:7854</h1>
+<p>There is no functional change in this commit. SIS has something similar however,
with a different variable name.</p>
+
+<p><b>Command line:</b></p>
+<blockquote><code>svn diff --extensions "--unified --ignore-space-change --ignore-all-space
--ignore-eol-style" -r7853:7854 http://svn.osgeo.org/geotools/trunk/modules/library/referencing/src/test/java/org/geotools/referencing/FactoriesTest.java</code></blockquote>
+<table class="changes">
+<tr><th>Revision 7853</th><th>Revision 7854</th></tr>
+<tr><td><pre>final CoordinateSystemFactory factory = CoordinateSystemFactory.getDefault();
+
+final Ellipsoid airy1830;
+<span class="del">airy1830 = Ellipsoid.createEllipsoid("Airy1830", 6377563.396, 6356256.910,
Unit.METRE);</span>
+out.println();
+out.print("create Coodinate System....2: ");
+out.println(airy1830);</pre></td>
+<td><pre>final CoordinateSystemFactory factory = CoordinateSystemFactory.getDefault();
+
+final Ellipsoid airy1830;
+<span class="add">Unit m = Unit.METRE;</span>
+<span class="add">airy1830 = Ellipsoid.createEllipsoid("Airy1830", 6377563.396, 6356256.910,
m );</span>
+out.println();
+out.print("create Coodinate System....2: ");
+out.println(airy1830);</pre></td></tr>
+</table>
+    </div>
+  </body>
+</html>

Propchange: sis/ip-review/rev/07854/GeodeticObjectFactoryTest.xhtml
------------------------------------------------------------------------------
    svn:eol-style = native

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

Added: sis/ip-review/rev/07854/HEADER.html
URL: http://svn.apache.org/viewvc/sis/ip-review/rev/07854/HEADER.html?rev=1718942&view=auto
==============================================================================
--- sis/ip-review/rev/07854/HEADER.html (added)
+++ sis/ip-review/rev/07854/HEADER.html Wed Dec  9 19:18:34 2015
@@ -0,0 +1,17 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml">
+  <head>
+    <meta charset="UTF-8"/>
+    <title>Revision 7854</title>
+  </head>
+  <body>
+    <div>
+      <h1>Revision 7854</h1>
+<table>
+  <tr><td><b>Author:</b></td><td>jgarnett</td></tr>
+  <tr><td><b>Date:</b></td><td>2004-09-08</td></tr>
+  <tr><td><b>Message:</b></td><td>try and cleanup cs</td></tr>
+</table>
+    </div>
+  </body>
+</html>

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

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

Added: sis/ip-review/rev/10796/GeodeticObjectFactoryTest.xhtml
URL: http://svn.apache.org/viewvc/sis/ip-review/rev/10796/GeodeticObjectFactoryTest.xhtml?rev=1718942&view=auto
==============================================================================
--- sis/ip-review/rev/10796/GeodeticObjectFactoryTest.xhtml (added)
+++ sis/ip-review/rev/10796/GeodeticObjectFactoryTest.xhtml Wed Dec  9 19:18:34 2015
@@ -0,0 +1,93 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml">
+  <head>
+    <meta charset="UTF-8"/>
+    <title>GeodeticObjectFactoryTest changes for revisions 10795:10796</title>
+    <style type="text/css" media="all">
+      @import url("../../reports.css");
+    </style>
+  </head>
+  <body>
+    <div>
+      <h1>GeodeticObjectFactoryTest 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/test/java/org/geotools/referencing/FactoriesTest.java</code></blockquote>
+<table class="changes">
+<tr><th>Revision 10795</th><th>Revision 10796</th></tr>
+<tr><td><pre>package org.geotools.referencing;
+
+// J2SE dependencies and extensions
+<span class="del">import java.io.*;</span>
+import java.util.Map;
+<span class="del">import java.util.Collections;</span>
+import javax.units.SI;
+import javax.units.Unit;
+<span class="del">import javax.units.NonSI;</span>
+
+<span class="del">// JUnit dependencies</span>
+import junit.framework.Test;
+import junit.framework.TestCase;
+import junit.framework.TestSuite;
+
+<span class="del">// OpenGIS dependencies</span>
+<span class="del">import org.opengis.parameter.*;</span>
+<span class="del">import org.opengis.referencing.cs.*;</span>
+<span class="del">import org.opengis.referencing.crs.*;</span>
+<span class="del">import org.opengis.referencing.datum.*;</span>
+<span class="del">import org.opengis.referencing.operation.*;</span>
+import org.opengis.referencing.FactoryException;
+
+<span class="del">// Geotools dependencies</span>
+<span class="del">import org.geotools.resources.Arguments;</span>
+<span class="del">import org.geotools.referencing.FactoryFinder;</span>
+
+<span class="del"></span>
+/**
+ * Test the creation of {@link CoordinateReferenceSystem} objects.
+ *</pre></td>
+<td><pre>package org.geotools.referencing;
+
+// J2SE dependencies and extensions
+<span class="add">import java.io.PrintWriter;</span>
+<span class="add">import java.io.StringWriter;</span>
+<span class="add">import java.util.Collections;</span>
+import java.util.Map;
+<span class="add"></span>
+<span class="add">import javax.units.NonSI;</span>
+import javax.units.SI;
+import javax.units.Unit;
+
+import junit.framework.Test;
+import junit.framework.TestCase;
+import junit.framework.TestSuite;
+
+<span class="add">import org.geotools.resources.Arguments;</span>
+<span class="add">import org.opengis.parameter.ParameterValueGroup;</span>
+import org.opengis.referencing.FactoryException;
+<span class="add">import org.opengis.referencing.crs.CRSFactory;</span>
+<span class="add">import org.opengis.referencing.crs.CoordinateReferenceSystem;</span>
+<span class="add">import org.opengis.referencing.crs.GeographicCRS;</span>
+<span class="add">import org.opengis.referencing.crs.ProjectedCRS;</span>
+<span class="add">import org.opengis.referencing.cs.AxisDirection;</span>
+<span class="add">import org.opengis.referencing.cs.CSFactory;</span>
+<span class="add">import org.opengis.referencing.cs.CartesianCS;</span>
+<span class="add">import org.opengis.referencing.cs.EllipsoidalCS;</span>
+<span class="add">import org.opengis.referencing.datum.DatumFactory;</span>
+<span class="add">import org.opengis.referencing.datum.Ellipsoid;</span>
+<span class="add">import org.opengis.referencing.datum.GeodeticDatum;</span>
+<span class="add">import org.opengis.referencing.datum.PrimeMeridian;</span>
+<span class="add">import org.opengis.referencing.operation.MathTransform;</span>
+<span class="add">import org.opengis.referencing.operation.MathTransformFactory;</span>
+
+
+/**
+ * Test the creation of {@link CoordinateReferenceSystem} objects.
+ *</pre></td></tr>
+</table>
+    </div>
+  </body>
+</html>

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

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

Added: sis/ip-review/rev/20874/GeodeticObjectFactoryTest.xhtml
URL: http://svn.apache.org/viewvc/sis/ip-review/rev/20874/GeodeticObjectFactoryTest.xhtml?rev=1718942&view=auto
==============================================================================
--- sis/ip-review/rev/20874/GeodeticObjectFactoryTest.xhtml (added)
+++ sis/ip-review/rev/20874/GeodeticObjectFactoryTest.xhtml Wed Dec  9 19:18:34 2015
@@ -0,0 +1,50 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml">
+  <head>
+    <meta charset="UTF-8"/>
+    <title>GeodeticObjectFactoryTest changes for revisions 20873:20874</title>
+    <style type="text/css" media="all">
+      @import url("../../reports.css");
+    </style>
+  </head>
+  <body>
+    <div>
+      <h1>GeodeticObjectFactoryTest 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/test/java/org/geotools/referencing/FactoriesTest.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 Management Committee (PMC)</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>/*
+<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>
+ *
+ *    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> *    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;</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;</pre></td></tr>
+</table>
+    </div>
+  </body>
+</html>

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

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

Added: sis/ip-review/rev/25050/GeodeticObjectFactoryTest.xhtml
URL: http://svn.apache.org/viewvc/sis/ip-review/rev/25050/GeodeticObjectFactoryTest.xhtml?rev=1718942&view=auto
==============================================================================
--- sis/ip-review/rev/25050/GeodeticObjectFactoryTest.xhtml (added)
+++ sis/ip-review/rev/25050/GeodeticObjectFactoryTest.xhtml Wed Dec  9 19:18:34 2015
@@ -0,0 +1,75 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml">
+  <head>
+    <meta charset="UTF-8"/>
+    <title>GeodeticObjectFactoryTest changes for revisions 25049:25050</title>
+    <style type="text/css" media="all">
+      @import url("../../reports.css");
+    </style>
+  </head>
+  <body>
+    <div>
+      <h1>GeodeticObjectFactoryTest changes for revisions 25049:25050</h1>
+<p>
+  The only change in this commit is to rename <code>FactoryFinder</code> as <code>ReferencingFactoryFinder</code>.
+  We do not plan to port <code>[Referencing]FactoryFinder</code> in SIS, since
the plan is to replace them
+  by some dependency injection mechanism. In the particular case of test cases however SIS
uses something similar,
+  while much simpler: <code>DefaultFactories</code> (an internal class).
+</p>
+
+<p><b>Command line:</b></p>
+<blockquote><code>svn diff --extensions "--unified --ignore-space-change --ignore-all-space
--ignore-eol-style" -r25049:25050 http://svn.osgeo.org/geotools/trunk/modules/library/referencing/src/test/java/org/geotools/referencing/FactoriesTest.java</code></blockquote>
+<table class="changes">
+<tr><th>Revision 25049</th><th>Revision 25050</th></tr>
+<tr><td><pre>out.println("---------------------");
+out.println();
+out.println("create Coodinate Reference System....1: ");
+<span class="del">final         DatumFactory datumFactory = FactoryFinder.getDatumFactory
       (null);</span>
+<span class="del">final            CSFactory    csFactory = FactoryFinder.getCSFactory
          (null);</span>
+<span class="del">final           CRSFactory   crsFactory = FactoryFinder.getCRSFactory
         (null);</span>
+<span class="del">final MathTransformFactory    mtFactory = FactoryFinder.getMathTransformFactory(null);</span>
+
+final Ellipsoid airy1830;
+final Unit meters = SI.METER;</pre></td>
+<td><pre>out.println("---------------------");
+out.println();
+out.println("create Coodinate Reference System....1: ");
+<span class="add">final         DatumFactory datumFactory = ReferencingFactoryFinder.getDatumFactory
       (null);</span>
+<span class="add">final            CSFactory    csFactory = ReferencingFactoryFinder.getCSFactory
          (null);</span>
+<span class="add">final           CRSFactory   crsFactory = ReferencingFactoryFinder.getCRSFactory
         (null);</span>
+<span class="add">final MathTransformFactory    mtFactory = ReferencingFactoryFinder.getMathTransformFactory(null);</span>
+
+final Ellipsoid airy1830;
+final Unit meters = SI.METER;</pre></td></tr>
+<tr><td><pre>out.println();
+out.println("Testing classification names");
+out.println("----------------------------");
+<span class="del">final MathTransformFactory mtFactory = FactoryFinder.getMathTransformFactory(null);</span>
+final Collection methods = mtFactory.getAvailableMethods(Projection.class);
+for (final Iterator it=methods.iterator(); it.hasNext();) {
+    final OperationMethod    method = (OperationMethod) it.next();</pre></td>
+<td><pre>out.println();
+out.println("Testing classification names");
+out.println("----------------------------");
+<span class="add">final MathTransformFactory mtFactory = ReferencingFactoryFinder.getMathTransformFactory(null);</span>
+final Collection methods = mtFactory.getAvailableMethods(Projection.class);
+for (final Iterator it=methods.iterator(); it.hasNext();) {
+    final OperationMethod    method = (OperationMethod) it.next();</pre></td></tr>
+<tr><td><pre>for (int i=0; i&lt;3; i++) {
+    switch (i) {
+        case  0: factory = new DatumAliases(factory);           break;
+<span class="del">        case  1: factory = FactoryFinder.getDatumFactory(null); break;</span>
+        case  2: ((DatumAliases) factory).freeUnused();         break;
+        default: throw new AssertionError(); // Should not occurs.
+    }</pre></td>
+<td><pre>for (int i=0; i&lt;3; i++) {
+    switch (i) {
+        case  0: factory = new DatumAliases(factory);           break;
+<span class="add">        case  1: factory = ReferencingFactoryFinder.getDatumFactory(null);
break;</span>
+        case  2: ((DatumAliases) factory).freeUnused();         break;
+        default: throw new AssertionError(); // Should not occurs.
+    }</pre></td></tr>
+</table>
+    </div>
+  </body>
+</html>

Propchange: sis/ip-review/rev/25050/GeodeticObjectFactoryTest.xhtml
------------------------------------------------------------------------------
    svn:eol-style = native

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

Added: sis/ip-review/rev/25890/GeodeticObjectFactoryTest.xhtml
URL: http://svn.apache.org/viewvc/sis/ip-review/rev/25890/GeodeticObjectFactoryTest.xhtml?rev=1718942&view=auto
==============================================================================
--- sis/ip-review/rev/25890/GeodeticObjectFactoryTest.xhtml (added)
+++ sis/ip-review/rev/25890/GeodeticObjectFactoryTest.xhtml Wed Dec  9 19:18:34 2015
@@ -0,0 +1,82 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml">
+  <head>
+    <meta charset="UTF-8"/>
+    <title>GeodeticObjectFactoryTest changes for revisions 25889:25890</title>
+    <style type="text/css" media="all">
+      @import url("../../reports.css");
+    </style>
+  </head>
+  <body>
+    <div>
+      <h1>GeodeticObjectFactoryTest changes for revisions 25889:25890</h1>
+<p>
+  This commit contains a renaming of <code>FactoryGroup</code> class as <code>ReferencingFactoryContainer</code>.
+  This change will not apply to SIS since we plan to avoid that class.
+  The <code>Hints</code> will not be ported neither.
+</p>
+<p><b>Command line:</b></p>
+<blockquote><code>svn diff --extensions "--unified --ignore-space-change --ignore-all-space
--ignore-eol-style" -r25889:25890 http://svn.osgeo.org/geotools/trunk/modules/library/referencing/src/test/java/org/geotools/referencing/FactoriesTest.java</code></blockquote>
+<table class="changes">
+<tr><th>Revision 25889</th><th>Revision 25890</th></tr>
+<tr><td><pre>import java.util.Arrays;
+import java.util.Collection;
+import java.util.Collections;
+import java.util.Iterator;
+import java.util.Map;
+import javax.units.NonSI;</pre></td>
+<td><pre>import java.util.Arrays;
+import java.util.Collection;
+import java.util.Collections;
+<span class="add">import java.util.HashMap;</span>
+import java.util.Iterator;
+import java.util.Map;
+import javax.units.NonSI;</pre></td></tr>
+<tr><td><pre>import org.opengis.util.ScopedName;
+
+// Geotools dependencies
+import org.geotools.referencing.factory.DatumAliases;
+import org.geotools.referencing.factory.FactoryGroup;
+import org.geotools.referencing.factory.GeotoolsFactory;
+import org.geotools.referencing.cs.DefaultCartesianCS;
+import org.geotools.referencing.cs.DefaultEllipsoidalCS;
+import org.geotools.referencing.datum.DefaultEllipsoid;</pre></td>
+<td><pre>import org.opengis.util.ScopedName;
+
+// Geotools dependencies
+<span class="add">import org.geotools.factory.Hints;</span>
+import org.geotools.referencing.factory.DatumAliases;
+import org.geotools.referencing.factory.FactoryGroup;
+import org.geotools.referencing.factory.GeotoolsFactory;
+<span class="add">import org.geotools.referencing.factory.ReferencingFactoryContainer;</span>
+import org.geotools.referencing.cs.DefaultCartesianCS;
+import org.geotools.referencing.cs.DefaultEllipsoidalCS;
+import org.geotools.referencing.datum.DefaultEllipsoid;</pre></td></tr>
+<tr><td><pre>out.println(cartCS); // No WKT for coordinate systems
+
+final ProjectedCRS projCRS;
+<span class="del">projCRS = new FactoryGroup(datumFactory, csFactory, crsFactory, mtFactory).</span>
+<span class="del">     createProjectedCRS(name("Great_Britian_National_Grid"), geogCRS,
null, param, cartCS);</span>
+out.println();
+out.println("create Coodinate System....9: ");
+out.println(projCRS.toWKT());</pre></td>
+<td><pre>out.println(cartCS); // No WKT for coordinate systems
+
+final ProjectedCRS projCRS;
+<span class="add"></span>
+<span class="add">Map hints = new HashMap();</span>
+<span class="add">hints.put( Hints.DATUM_FACTORY, datumFactory );</span>
+<span class="add">hints.put( Hints.CS_FACTORY, csFactory );</span>
+<span class="add">hints.put( Hints.CRS_FACTORY, crsFactory );</span>
+<span class="add">hints.put( Hints.MATH_TRANSFORM_FACTORY, mtFactory );</span>
+<span class="add"></span>
+<span class="add">ReferencingFactoryContainer container = new ReferencingFactoryContainer(
new Hints(hints));</span>
+<span class="add">projCRS = container.createProjectedCRS(name("Great_Britian_National_Grid"),
geogCRS, null, param, cartCS);</span>
+<span class="add"></span>
+out.println();
+out.println("create Coodinate System....9: ");
+out.println(projCRS.toWKT());</pre></td></tr>
+</table>
+    </div>
+  </body>
+</html>

Propchange: sis/ip-review/rev/25890/GeodeticObjectFactoryTest.xhtml
------------------------------------------------------------------------------
    svn:eol-style = native

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




Mime
View raw message