Author: desruisseaux
Date: Tue Jul 19 12:09:57 2016
New Revision: 1753370
URL: http://svn.apache.org/viewvc?rev=1753370&view=rev
Log:
Add IP review for DefaultPassThroughOperation.
Added:
sis/ip-review/DefaultPassThroughOperation.xhtml (with props)
sis/ip-review/rev/06405/DefaultPassThroughOperation.xhtml (with props)
sis/ip-review/rev/10796/DefaultPassThroughOperation.xhtml (with props)
sis/ip-review/rev/20874/DefaultPassThroughOperation.xhtml (with props)
Added: sis/ip-review/DefaultPassThroughOperation.xhtml
URL:
http://svn.apache.org/viewvc/sis/ip-review/DefaultPassThroughOperation.xhtml?rev=1753370&view=auto
==============================================================================
--- sis/ip-review/DefaultPassThroughOperation.xhtml (added)
+++ sis/ip-review/DefaultPassThroughOperation.xhtml Tue Jul 19 12:09:57 2016
@@ -0,0 +1,58 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml";>
+ <head>
+ <meta charset="UTF-8"/>
+ <title>DefaultPassThroughOperation history</title>
+ <style type="text/css" media="all">
+ @import url("./reports.css");
+ </style>
+ </head>
+ <body>
+ <div>
+ <h1>DefaultPassThroughOperation 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/DefaultPassThroughOperation.java</code></blockquote>
+<table>
+ <tr>
+ <th>Rev.</th>
+ <th>Date</th>
+ <th>Author</th>
+ <th class="last">Message</th>
+ </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">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">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">29753</td><td>2008-04-01</td><td>desruisseaux</td><td>Applied
parameterized types as a side-effect of hunting for a
DimensionMismatchException during reprojection.</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">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">24607</td><td>2007-02-26</td><td>desruisseaux</td><td>Avoid
UnformattableObjectException for 'LOCAL_CS' and 'VERT_CS' WKT elements. Try to
provides some hints about the cause for the other cases of unformattable
WKT.</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/DefaultPassThroughOperation.xhtml">ip review</a></td></tr>
+<tr><td
class="rev">19557</td><td>2006-05-18</td><td>desruisseaux</td><td>AuthorityBackedFactory
now take inverse transform in account. Improved pseudo-WKT formatting for
CoordinateOperation.</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">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">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">11146</td><td>2005-02-07</td><td>desruisseaux</td><td>CoordinateOperationFactory
refactoring</td></tr>
+<tr><td
class="rev">11144</td><td>2005-02-07</td><td>desruisseaux</td><td>Fixed
GEOT-400 (create appropriate instance of Operation subclasses)</td></tr>
+<tr><td class="rev">11140</td><td>2005-02-06</td><td>desruisseaux</td><td>Work
in progres for GEOT-400 (CoordinateOperationFactory should create appropriate
subclass of CoordinateOperation)</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/DefaultPassThroughOperation.xhtml">imports cleaned</a></td></tr>
+<tr><td class="rev">6440</td><td>2004-06-13</td><td>desruisseaux</td><td>Added
GeocentricTransform / Added registry service in TransformFactory / Refactored
JUnit tests</td></tr>
+<tr><td class="rev">6405</td><td>2004-06-12</td><td
class="unav">jgarnett</td><td><a
href="rev/06405/DefaultPassThroughOperation.xhtml">Made PassThroughTransform
public constructor, and changed PassThroughOpperation to call theconstructor
with the parameters in the correct order. I assume someone forgot a
commit?</a></td></tr>
+<tr><td class="rev">6343</td><td>2004-06-07</td><td>desruisseaux</td><td>More
ports of legacy CTS to new CRS</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>
+</table>
+ </div>
+ </body>
+</html>
Propchange: sis/ip-review/DefaultPassThroughOperation.xhtml
------------------------------------------------------------------------------
svn:eol-style = native
Propchange: sis/ip-review/DefaultPassThroughOperation.xhtml
------------------------------------------------------------------------------
svn:mime-type = text/html
Added: sis/ip-review/rev/06405/DefaultPassThroughOperation.xhtml
URL:
http://svn.apache.org/viewvc/sis/ip-review/rev/06405/DefaultPassThroughOperation.xhtml?rev=1753370&view=auto
==============================================================================
--- sis/ip-review/rev/06405/DefaultPassThroughOperation.xhtml (added)
+++ sis/ip-review/rev/06405/DefaultPassThroughOperation.xhtml Tue Jul 19
12:09:57 2016
@@ -0,0 +1,39 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml";>
+ <head>
+ <meta charset="UTF-8"/>
+ <title>DefaultPassThroughOperation changes for revisions 6404:6405</title>
+ <style type="text/css" media="all">
+ @import url("../../reports.css");
+ </style>
+ </head>
+ <body>
+ <div>
+ <h1>DefaultPassThroughOperation changes for revisions 6404:6405</h1>
+
+<p>Change of argument order only. Reverting this change cause a compilation
error, which can be fixed only by applying the same change again.</p>
+
+<p><b>Command line:</b></p>
+<blockquote><code>svn diff --extensions "--unified --ignore-space-change
--ignore-all-space --ignore-eol-style" -r6404:6405
http://svn.osgeo.org/geotools/trunk/modules/library/referencing/src/main/java/org/geotools/referencing/operation/DefaultPassThroughOperation.java</code></blockquote>
+<table class="changes">
+<tr><th>Revision 6404</th><th>Revision 6405</th></tr>
+<tr><td><pre> final int
numTrailingOrdinates)
+{
+ ensureNonNull("operation", operation);
+<span class="del"> return new
PassThroughTransform(operation.getMathTransform(),</span>
+<span class="del"> firstAffectedOrdinate,
numTrailingOrdinates);</span>
+}
+
+/**</pre></td>
+<td><pre> final int
numTrailingOrdinates)
+{
+ ensureNonNull("operation", operation);
+<span class="add"> return new
PassThroughTransform(firstAffectedOrdinate,</span>
+<span class="add"> operation.getMathTransform(),
numTrailingOrdinates);</span>
+}
+
+/**</pre></td></tr>
+</table>
+ </div>
+ </body>
+</html>
Propchange: sis/ip-review/rev/06405/DefaultPassThroughOperation.xhtml
------------------------------------------------------------------------------
svn:eol-style = native
Propchange: sis/ip-review/rev/06405/DefaultPassThroughOperation.xhtml
------------------------------------------------------------------------------
svn:mime-type = text/html
Added: sis/ip-review/rev/10796/DefaultPassThroughOperation.xhtml
URL:
http://svn.apache.org/viewvc/sis/ip-review/rev/10796/DefaultPassThroughOperation.xhtml?rev=1753370&view=auto
==============================================================================
--- sis/ip-review/rev/10796/DefaultPassThroughOperation.xhtml (added)
+++ sis/ip-review/rev/10796/DefaultPassThroughOperation.xhtml Tue Jul 19
12:09:57 2016
@@ -0,0 +1,65 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml";>
+ <head>
+ <meta charset="UTF-8"/>
+ <title>DefaultPassThroughOperation changes for revisions
10795:10796</title>
+ <style type="text/css" media="all">
+ @import url("../../reports.css");
+ </style>
+ </head>
+ <body>
+ <div>
+ <h1>DefaultPassThroughOperation 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/DefaultPassThroughOperation.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) 2004, Geotools Project Managment Committee (PMC)
+<span class="del"> * (C) 2004, 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) 2004, Geotools Project Managment Committee (PMC)
+<span class="add"> * (C) 2004, 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>// J2SE dependencies
+import java.util.Map;
+
+<span class="del">// OpenGIS dependencies</span>
+import org.opengis.referencing.operation.Operation;
+<span class="del">import
org.opengis.referencing.operation.MathTransform;</span>
+<span class="del">import
org.opengis.referencing.crs.CoordinateReferenceSystem;</span>
+
+<span class="del">// Geotools dependencies</span>
+<span class="del">import
org.geotools.referencing.operation.transform.PassThroughTransform;</span>
+
+<span class="del"></span>
+/**
+ * A pass-through operation specifies that a subset of a coordinate tuple is
subject to a specific
+ * coordinate operation.</pre></td>
+<td><pre>// J2SE dependencies
+import java.util.Map;
+
+<span class="add">import
org.geotools.referencing.operation.transform.PassThroughTransform;</span>
+<span class="add">import
org.opengis.referencing.crs.CoordinateReferenceSystem;</span>
+<span class="add">import
org.opengis.referencing.operation.MathTransform;</span>
+import org.opengis.referencing.operation.Operation;
+
+
+/**
+ * A pass-through operation specifies that a subset of a coordinate tuple is
subject to a specific
+ * coordinate operation.</pre></td></tr>
+</table>
+ </div>
+ </body>
+</html>
Propchange: sis/ip-review/rev/10796/DefaultPassThroughOperation.xhtml
------------------------------------------------------------------------------
svn:eol-style = native
Propchange: sis/ip-review/rev/10796/DefaultPassThroughOperation.xhtml
------------------------------------------------------------------------------
svn:mime-type = text/html
Added: sis/ip-review/rev/20874/DefaultPassThroughOperation.xhtml
URL:
http://svn.apache.org/viewvc/sis/ip-review/rev/20874/DefaultPassThroughOperation.xhtml?rev=1753370&view=auto
==============================================================================
--- sis/ip-review/rev/20874/DefaultPassThroughOperation.xhtml (added)
+++ sis/ip-review/rev/20874/DefaultPassThroughOperation.xhtml Tue Jul 19
12:09:57 2016
@@ -0,0 +1,65 @@
+<!DOCTYPE html>
+<html xmlns="http://www.w3.org/1999/xhtml";>
+ <head>
+ <meta charset="UTF-8"/>
+ <title>DefaultPassThroughOperation changes for revisions
20873:20874</title>
+ <style type="text/css" media="all">
+ @import url("../../reports.css");
+ </style>
+ </head>
+ <body>
+ <div>
+ <h1>DefaultPassThroughOperation 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/DefaultPassThroughOperation.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) 2004, Geotools Project Managment Committee
(PMC)</span>
+ * (C) 2004, Institut de Recherche pour le Développement
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+<span class="del"> * License as published by the Free Software Foundation;
either</span>
+<span class="del"> * version 2.1 of the License, or (at your option) any
later version.</span>
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of</pre></td>
+<td><pre>/*
+<span class="add"> * GeoTools - OpenSource mapping toolkit</span>
+<span class="add"> * http://geotools.org</span>
+<span class="add"> * (C) 2004-2006, GeoTools Project Managment Committee
(PMC)</span>
+ * (C) 2004, Institut de Recherche pour le Développement
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+<span class="add"> * License as published by the Free Software
Foundation;</span>
+<span class="add"> * version 2.1 of the License.</span>
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty
of</pre></td></tr>
+<tr><td><pre>* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See
the GNU
+* Lesser General Public License for more details.
+*
+<span class="del">* You should have received a copy of the GNU Lesser
General Public</span>
+<span class="del">* License along with this library; if not, write to the
Free Software</span>
+<span class="del">* Foundation, Inc., 59 Temple Place, Suite 330, Boston,
MA 02111-1307 USA</span>
+<span class="del">*</span>
+<span class="del">*</span>
+* This package contains documentation from OpenGIS specifications.
+* OpenGIS consortium's work is fully acknowledged here.
+*/</pre></td>
+<td><pre>* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+* Lesser General Public License for more details.
+*
+* This package contains documentation from OpenGIS specifications.
+* OpenGIS consortium's work is fully acknowledged here.
+*/</pre></td></tr>
+</table>
+ </div>
+ </body>
+</html>
Propchange: sis/ip-review/rev/20874/DefaultPassThroughOperation.xhtml
------------------------------------------------------------------------------
svn:eol-style = native
Propchange: sis/ip-review/rev/20874/DefaultPassThroughOperation.xhtml
------------------------------------------------------------------------------
svn:mime-type = text/html
