refactor(geometry): Port Point and Bounds classes to Kotlin

This commit modernizes the `Point` and `Bounds` classes by converting them from Java to idiomatic Kotlin. This change improves code conciseness, readability, and null safety.

Key changes include:
- The `Point` and `Bounds` classes are now written in Kotlin.
- `@JvmField` annotations are used on properties to maintain binary compatibility for Java consumers.
- New unit tests, `PointTest.kt` and `BoundsTest.kt`, have been added to ensure the correctness of the ported classes.

Author: dkhawk

library/src/main/java/com/google/maps/android/geometry/Bounds.kt

@@ -1,5 +1,5 @@
 /*
- * Copyright 2013 Google Inc.
+ * Copyright 2025 Google Inc.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -14,48 +14,40 @@
  * limitations under the License.
  */
 
-package com.google.maps.android.geometry;
+package com.google.maps.android.geometry
 
 /**
  * Represents an area in the cartesian plane.
  */
-public class Bounds {
-    public final double minX;
-    public final double minY;
-
-    public final double maxX;
-    public final double maxY;
-
-    public final double midX;
-    public final double midY;
-
-    public Bounds(double minX, double maxX, double minY, double maxY) {
-        this.minX = minX;
-        this.minY = minY;
-        this.maxX = maxX;
-        this.maxY = maxY;
-
-        midX = (minX + maxX) / 2;
-        midY = (minY + maxY) / 2;
-    }
-
-    public boolean contains(double x, double y) {
-        return minX <= x && x <= maxX && minY <= y && y <= maxY;
+class Bounds(
+    @JvmField val minX: Double,
+    @JvmField val maxX: Double,
+    @JvmField val minY: Double,
+    @JvmField val maxY: Double
+) {
+    @JvmField
+    val midX: Double = (minX + maxX) / 2
+
+    @JvmField
+    val midY: Double = (minY + maxY) / 2
+
+    fun contains(x: Double, y: Double): Boolean {
+        return minX <= x && x <= maxX && minY <= y && y <= maxY
     }
 
-    public boolean contains(Point point) {
-        return contains(point.x, point.y);
+    fun contains(point: Point): Boolean {
+        return contains(point.x, point.y)
     }
 
-    public boolean intersects(double minX, double maxX, double minY, double maxY) {
-        return minX < this.maxX && this.minX < maxX && minY < this.maxY && this.minY < maxY;
+    fun intersects(minX: Double, maxX: Double, minY: Double, maxY: Double): Boolean {
+        return minX < this.maxX && this.minX < maxX && minY < this.maxY && this.minY < maxY
     }
 
-    public boolean intersects(Bounds bounds) {
-        return intersects(bounds.minX, bounds.maxX, bounds.minY, bounds.maxY);
+    fun intersects(bounds: Bounds): Boolean {
+        return intersects(bounds.minX, bounds.maxX, bounds.minY, bounds.maxY)
     }
 
-    public boolean contains(Bounds bounds) {
-        return bounds.minX >= minX && bounds.maxX <= maxX && bounds.minY >= minY && bounds.maxY <= maxY;
+    fun contains(bounds: Bounds): Boolean {
+        return bounds.minX >= minX && bounds.maxX <= maxX && bounds.minY >= minY && bounds.maxY <= maxY
     }
-}
+}

library/src/main/java/com/google/maps/android/geometry/Point.kt

@@ -1,5 +1,5 @@
 /*
- * Copyright 2023 Google Inc.
+ * Copyright 2025 Google Inc.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -14,25 +14,32 @@
  * limitations under the License.
  */
 
-package com.google.maps.android.geometry;
+package com.google.maps.android.geometry
 
-import androidx.annotation.NonNull;
+open class Point(@JvmField val x: Double, @JvmField val y: Double) {
+    override fun toString(): String {
+        return "Point(x=$x, y=$y)"
+    }
+
+    override fun equals(other: Any?): Boolean {
+        if (this === other) return true
+        if (javaClass != other?.javaClass) return false
+
+        other as Point
 
-public class Point {
-    public final double x;
-    public final double y;
+        if (x != other.x) return false
+        if (y != other.y) return false
+
+        return true
+    }
 
-    public Point(double x, double y) {
-        this.x = x;
-        this.y = y;
+    override fun hashCode(): Int {
+        var result = x.hashCode()
+        result = 31 * result + y.hashCode()
+        return result
     }
 
-    @NonNull
-    @Override
-    public String toString() {
-        return "Point{" +
-                "x=" + x +
-                ", y=" + y +
-                '}';
+    fun copy(x: Double = this.x, y: Double = this.y): Point {
+        return Point(x, y)
     }
-}
+}

library/src/test/java/com/google/maps/android/geometry/BoundsTest.kt

@@ -0,0 +1,85 @@
+/*
+ * Copyright 2025 Google Inc.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package com.google.maps.android.geometry
+
+import org.junit.Assert.assertEquals
+import org.junit.Assert.assertFalse
+import org.junit.Assert.assertTrue
+import org.junit.Test
+
+class BoundsTest {
+
+    @Test
+    fun testInitialization() {
+        val bounds = Bounds(0.0, 10.0, 0.0, 20.0)
+        assertEquals(0.0, bounds.minX, 0.0)
+        assertEquals(10.0, bounds.maxX, 0.0)
+        assertEquals(0.0, bounds.minY, 0.0)
+        assertEquals(20.0, bounds.maxY, 0.0)
+        assertEquals(5.0, bounds.midX, 0.0)
+        assertEquals(10.0, bounds.midY, 0.0)
+    }
+
+    @Test
+    fun testContainsPoint() {
+        val bounds = Bounds(0.0, 10.0, 0.0, 10.0)
+        assertTrue(bounds.contains(Point(5.0, 5.0)))
+        assertTrue(bounds.contains(Point(0.0, 0.0)))
+        assertTrue(bounds.contains(Point(10.0, 10.0)))
+        assertFalse(bounds.contains(Point(11.0, 5.0)))
+        assertFalse(bounds.contains(Point(5.0, 11.0)))
+    }
+
+    @Test
+    fun testContainsCoordinates() {
+        val bounds = Bounds(0.0, 10.0, 0.0, 10.0)
+        assertTrue(bounds.contains(5.0, 5.0))
+        assertTrue(bounds.contains(0.0, 0.0))
+        assertTrue(bounds.contains(10.0, 10.0))
+        assertFalse(bounds.contains(11.0, 5.0))
+        assertFalse(bounds.contains(5.0, 11.0))
+    }
+
+    @Test
+    fun testIntersectsBounds() {
+        val bounds1 = Bounds(0.0, 10.0, 0.0, 10.0)
+        val bounds2 = Bounds(5.0, 15.0, 5.0, 15.0)
+        val bounds3 = Bounds(11.0, 20.0, 11.0, 20.0)
+        val bounds4 = Bounds(0.0, 10.0, 11.0, 20.0)
+
+        assertTrue(bounds1.intersects(bounds2))
+        assertFalse(bounds1.intersects(bounds3))
+        assertFalse(bounds1.intersects(bounds4))
+    }
+
+    @Test
+    fun testIntersectsCoordinates() {
+        val bounds = Bounds(0.0, 10.0, 0.0, 10.0)
+        assertTrue(bounds.intersects(5.0, 15.0, 5.0, 15.0))
+        assertFalse(bounds.intersects(11.0, 20.0, 11.0, 20.0))
+    }
+
+    @Test
+    fun testContainsBounds() {
+        val bounds1 = Bounds(0.0, 10.0, 0.0, 10.0)
+        val bounds2 = Bounds(2.0, 8.0, 2.0, 8.0)
+        val bounds3 = Bounds(5.0, 15.0, 5.0, 15.0)
+
+        assertTrue(bounds1.contains(bounds2))
+        assertFalse(bounds1.contains(bounds3))
+    }
+}

library/src/test/java/com/google/maps/android/geometry/PointTest.kt

@@ -0,0 +1,62 @@
+/*
+ * Copyright 2025 Google Inc.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package com.google.maps.android.geometry
+
+import org.junit.Assert.assertEquals
+import org.junit.Assert.assertNotEquals
+import org.junit.Test
+
+class PointTest {
+
+    @Test
+    fun testInitialization() {
+        val point = Point(1.0, 2.0)
+        assertEquals(1.0, point.x, 0.0)
+        assertEquals(2.0, point.y, 0.0)
+    }
+
+    @Test
+    fun testToString() {
+        val point = Point(1.0, 2.0)
+        assertEquals("Point(x=1.0, y=2.0)", point.toString())
+    }
+
+    @Test
+    fun testEqualsAndHashCode() {
+        val point1 = Point(1.0, 2.0)
+        val point2 = Point(1.0, 2.0)
+        val point3 = Point(2.0, 1.0)
+
+        assertEquals(point1, point2)
+        assertEquals(point1.hashCode(), point2.hashCode())
+
+        assertNotEquals(point1, point3)
+        assertNotEquals(point1.hashCode(), point3.hashCode())
+    }
+
+    @Test
+    fun testCopy() {
+        val original = Point(1.0, 2.0)
+        val copy = original.copy()
+        val modifiedCopy = original.copy(y = 3.0)
+
+        assertEquals(original, copy)
+        assertNotEquals(original, modifiedCopy)
+        assertEquals(1.0, modifiedCopy.x, 0.0)
+        assertEquals(3.0, modifiedCopy.y, 0.0)
+    }
+}