[geos-commits] [SCM] GEOS branch 3.12 updated. 51c3b3d8a74864e7b05916c27e57ed95036301b4

[git at osgeo.org]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "GEOS".

The branch, 3.12 has been updated
       via  51c3b3d8a74864e7b05916c27e57ed95036301b4 (commit)
       via  52a86996371fa11d791919f2635aefadde292a3b (commit)
      from  0402c861bd32fdb58e6fa9c4a8574eac95520238 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 51c3b3d8a74864e7b05916c27e57ed95036301b4
Author: Martin Davis <mtnclimb at gmail.com>
Date:   Tue Nov 5 16:26:54 2024 -0800

    Update NEWS

diff --git a/NEWS.md b/NEWS.md
index ee77dbf89..c85d46c92 100644
--- a/NEWS.md
+++ b/NEWS.md
@@ -12,6 +12,7 @@
   - DouglasPeuckerLineSimplifier, avoid crash with Point input and NaN tolerance (GH-1078, Dan Baston)
   - GEOSLineSubstring, avoid crash with NaN length fraction (GH-1077, Dan Baston)
   - MinimumClearance, avoid crash on NaN inputs (GH-1079, Dan Baston)
+  - Fix TopologyPreservingSimplifier/TaggedLineString to avoid jumping components (JTS-1096, Martin Davis)
 
 ## Changes in 3.12.2
 2024-06-05

commit 52a86996371fa11d791919f2635aefadde292a3b
Author: Martin Davis <mtnclimb at gmail.com>
Date:   Tue Nov 5 14:45:36 2024 -0800

    Fix TaggedLineString as per JTS 1096

diff --git a/src/simplify/TaggedLineString.cpp b/src/simplify/TaggedLineString.cpp
index e0dc40ddb..9a6d09324 100644
--- a/src/simplify/TaggedLineString.cpp
+++ b/src/simplify/TaggedLineString.cpp
@@ -169,7 +169,7 @@ TaggedLineString::extractCoordinates(
 
     if(size) {
         for(std::size_t i = 0; i < size; i++) {
-            TaggedLineSegment* seg = segs[i];
+                TaggedLineSegment* seg = segs[i];
             assert(seg);
             pts->add(seg->p0);
         }
@@ -185,7 +185,6 @@ TaggedLineString::extractCoordinates(
 const Coordinate&
 TaggedLineString::getCoordinate(std::size_t i) const
 {
-
     return parentLine->getCoordinateN(i);
 }
 
@@ -198,6 +197,10 @@ TaggedLineString::size() const
 const Coordinate&
 TaggedLineString::getComponentPoint() const
 {
+    //-- when simplified use a valid coordinate
+    if (resultSegs.size() > 0) {
+        return resultSegs[0]->p0;
+    }
     return getParentCoordinates()->getAt(1);
 }
 

-----------------------------------------------------------------------

Summary of changes:
 NEWS.md                           | 1 +
 src/simplify/TaggedLineString.cpp | 7 +++++--
 2 files changed, 6 insertions(+), 2 deletions(-)


hooks/post-receive
-- 
GEOS