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

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

commit 632fd3535c1df7de09f7c877b5e9d1abfc543bc2
Author: Alex Herbert <[email protected]>
AuthorDate: Mon Sep 22 14:07:58 2025 +0100

    pmd fix: use final
---
 .../apache/commons/geometry/core/internal/AbstractBucketPointMap.java | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/commons-geometry-core/src/main/java/org/apache/commons/geometry/core/internal/AbstractBucketPointMap.java
 
b/commons-geometry-core/src/main/java/org/apache/commons/geometry/core/internal/AbstractBucketPointMap.java
index 0094ccf3..c4452bbe 100644
--- 
a/commons-geometry-core/src/main/java/org/apache/commons/geometry/core/internal/AbstractBucketPointMap.java
+++ 
b/commons-geometry-core/src/main/java/org/apache/commons/geometry/core/internal/AbstractBucketPointMap.java
@@ -441,7 +441,7 @@ public abstract class AbstractBucketPointMap<P extends 
Point<P>, V>
         final double aDist = a != null ? a.getDistance() : nullDistance;
         final double bDist = b != null ? b.getDistance() : nullDistance;
 
-        int cmp = Double.compare(aDist, bDist);
+        final int cmp = Double.compare(aDist, bDist);
         if (cmp == 0 &&
                 a != null &&
                 b != null) {
@@ -1368,7 +1368,7 @@ public abstract class AbstractBucketPointMap<P extends 
Point<P>, V>
             for (int i = 0; i < childCount; ++i) {
                 final BucketNode<P, V> child = node.children.get(i);
                 if (child != null) {
-                    Iterator<Entry<P, V>> childIt = 
findIteratorRecursive(child, currentOffset);
+                    final Iterator<Entry<P, V>> childIt = 
findIteratorRecursive(child, currentOffset);
                     if (childIt != null) {
                         return childIt;
                     }

Reply via email to