perf(street): use PreparedGeometry instead of RelateNG for area containment

The containsSegment check in PreparedAreaGroup used RelateNG to prepare and
reuse a spatial index over the area-group polygon. Benchmarks (see PR #7715
discussion) show the classic PreparedGeometry API is faster than RelateNG for
this contains predicate, and the sibling areasCrossedBy crossing test already
uses PreparedGeometry. Switch containsSegment to PreparedGeometry.contains for
consistency and performance.

This restores the pre-#7696 semantics of plain Polygon.contains (which can
throw TopologyException on invalid polygons) but keeps the prepared-index reuse
that #7696 introduced.

Author: vpaturet

street/src/main/java/org/opentripplanner/street/linking/PreparedAreaGroup.java

@@ -7,8 +7,6 @@
 import org.locationtech.jts.geom.LineString;
 import org.locationtech.jts.geom.prep.PreparedGeometry;
 import org.locationtech.jts.geom.prep.PreparedGeometryFactory;
-import org.locationtech.jts.operation.relateng.RelateNG;
-import org.locationtech.jts.operation.relateng.RelatePredicate;
 import org.opentripplanner.street.geometry.GeometryUtils;
 import org.opentripplanner.street.model.edge.Area;
 import org.opentripplanner.street.model.edge.AreaGroup;
@@ -36,7 +34,7 @@ final class PreparedAreaGroup {
   private static final double AREA_INTERSECTION_SHRINKING = 0.0001;
 
   private final AreaGroup areaGroup;
-  private RelateNG prepared;
+  private PreparedGeometry preparedGroup;
   private PreparedGeometry[] preparedAreas;
 
   PreparedAreaGroup(AreaGroup areaGroup) {
@@ -52,14 +50,14 @@ AreaGroup areaGroup() {
    * shrunk slightly at both ends first, so that a segment connecting two boundary points is not
    * rejected by floating point imprecision at the boundary. Reuses a cached prepared index across
    * calls. A plain {@code Polygon.contains(...)} would rebuild a sweep-line index over the polygon
-   * boundary on every call; {@link RelateNG} builds it once and is also tolerant of invalid /
-   * self-intersecting OSM polygons (it does not throw {@code TopologyException}).
+   * boundary on every call; a {@link PreparedGeometry} builds it once and reuses it, matching the
+   * indexed {@link #areasCrossedBy} crossing test on this same group.
    */
   boolean containsSegment(Coordinate from, Coordinate to) {
-    if (prepared == null) {
-      prepared = RelateNG.prepare(areaGroup.getGeometry());
+    if (preparedGroup == null) {
+      preparedGroup = PreparedGeometryFactory.prepare(areaGroup.getGeometry());
     }
-    return prepared.evaluate(createShrunkLine(from, to), RelatePredicate.contains());
+    return preparedGroup.contains(createShrunkLine(from, to));
   }
 
   /**