This is an automated email from the ASF dual-hosted git repository.

mattjuntunen pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-geometry.git

commit 6c90e34ff11fb9fa279d9b060abf70c14ce3cd2a
Author: XenoAmess <xenoam...@gmail.com>
AuthorDate: Sat Jul 4 09:47:06 2020 +0800

    change a field to local variable, as I didn't see anywhere using it.
---
 .../java/org/apache/commons/geometry/euclidean/threed/Planes.java  | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git 
a/commons-geometry-euclidean/src/main/java/org/apache/commons/geometry/euclidean/threed/Planes.java
 
b/commons-geometry-euclidean/src/main/java/org/apache/commons/geometry/euclidean/threed/Planes.java
index dfde9e3..721d729 100644
--- 
a/commons-geometry-euclidean/src/main/java/org/apache/commons/geometry/euclidean/threed/Planes.java
+++ 
b/commons-geometry-euclidean/src/main/java/org/apache/commons/geometry/euclidean/threed/Planes.java
@@ -786,9 +786,6 @@ public final class Planes {
         /** Base plane to extrude from. */
         private final EmbeddingPlane basePlane;
 
-        /** Extruded plane; this forms the end of the 3D region opposite the 
base plane. */
-        private final EmbeddingPlane extrudedPlane;
-
         /** Vector to extrude along; the extruded plane is translated from the 
base plane by this amount. */
         private final Vector3D extrusionVector;
 
@@ -809,7 +806,9 @@ public final class Planes {
                 final DoublePrecisionContext precision) {
 
             this.basePlane = basePlane;
-            this.extrudedPlane = basePlane.translate(extrusionVector);
+
+            // Extruded plane; this forms the end of the 3D region opposite 
the base plane.
+            EmbeddingPlane extrudedPlane = 
basePlane.translate(extrusionVector);
 
             if (basePlane.contains(extrudedPlane)) {
                 throw new IllegalArgumentException(

Reply via email to