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


The following commit(s) were added to refs/heads/master by this push:
     new 385c02a  using enhanced for loop when possible
385c02a is described below

commit 385c02a9b2cc924ebdaacc8ba02f93b76cac31cc
Author: Matt Juntunen <[email protected]>
AuthorDate: Sat Jul 4 08:02:05 2020 -0400

    using enhanced for loop when possible
---
 .../java/org/apache/commons/geometry/euclidean/threed/Planes.java     | 4 ++--
 .../org/apache/commons/geometry/euclidean/testio/TestOBJWriter.java   | 4 ++--
 .../org/apache/commons/geometry/examples/io/threed/obj/OBJWriter.java | 4 ++--
 3 files changed, 6 insertions(+), 6 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 721d729..73ce61e 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
@@ -328,8 +328,8 @@ public final class Planes {
                         i, face.length));
             }
 
-            for (int j = 0; j < face.length; ++j) {
-                faceVertices.add(vertices.get(face[j]));
+            for (final int vertexIndex : face) {
+                faceVertices.add(vertices.get(vertexIndex));
             }
 
             polygons.add(convexPolygonFromVertices(
diff --git 
a/commons-geometry-euclidean/src/test/java/org/apache/commons/geometry/euclidean/testio/TestOBJWriter.java
 
b/commons-geometry-euclidean/src/test/java/org/apache/commons/geometry/euclidean/testio/TestOBJWriter.java
index 82c8f6a..9080802 100644
--- 
a/commons-geometry-euclidean/src/test/java/org/apache/commons/geometry/euclidean/testio/TestOBJWriter.java
+++ 
b/commons-geometry-euclidean/src/test/java/org/apache/commons/geometry/euclidean/testio/TestOBJWriter.java
@@ -243,9 +243,9 @@ public final class TestOBJWriter implements AutoCloseable {
 
         writer.write('f');
 
-        for (int i = 0; i < vertexIndices.length; ++i) {
+        for (final int vertexIndex : vertexIndices) {
             writer.write(SPACE);
-            writer.write(String.valueOf(vertexIndices[i] + vertexOffset));
+            writer.write(String.valueOf(vertexIndex + vertexOffset));
         }
 
         writer.write(lineSeparator);
diff --git 
a/commons-geometry-examples/examples-io/src/main/java/org/apache/commons/geometry/examples/io/threed/obj/OBJWriter.java
 
b/commons-geometry-examples/examples-io/src/main/java/org/apache/commons/geometry/examples/io/threed/obj/OBJWriter.java
index 3754ab8..736edcd 100644
--- 
a/commons-geometry-examples/examples-io/src/main/java/org/apache/commons/geometry/examples/io/threed/obj/OBJWriter.java
+++ 
b/commons-geometry-examples/examples-io/src/main/java/org/apache/commons/geometry/examples/io/threed/obj/OBJWriter.java
@@ -246,9 +246,9 @@ public final class OBJWriter implements AutoCloseable {
 
         writer.write(OBJConstants.FACE_KEYWORD);
 
-        for (int i = 0; i < vertexIndices.length; ++i) {
+        for (final int vertexIndex : vertexIndices) {
             writer.write(SPACE);
-            writer.write(String.valueOf(vertexIndices[i] + vertexOffset));
+            writer.write(String.valueOf(vertexIndex + vertexOffset));
         }
 
         writer.write(lineSeparator);

Reply via email to