[geos-commits] [SCM] GEOS branch master updated. 1bb072153d7bb195d5065416123de06c34b972a9

[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, master has been updated
       via  1bb072153d7bb195d5065416123de06c34b972a9 (commit)
       via  a19a9f44527d4be33992c17294b3579f8e85c792 (commit)
       via  7d8efa201d93f39f224758f5e2d15bf3401b1a8b (commit)
      from  c4aebbbc048618e75b39f6c8d85dce8e8216cf9f (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 1bb072153d7bb195d5065416123de06c34b972a9
Author: Even Rouault <even.rouault at spatialys.com>
Date:   Thu Sep 17 00:39:09 2020 +0200

    GEOSGeom_setPrecisionTest.cpp: fix warning about initialization of members not in the order of their declaration

diff --git a/tests/unit/capi/GEOSGeom_setPrecisionTest.cpp b/tests/unit/capi/GEOSGeom_setPrecisionTest.cpp
index c11c62b..13000d3 100644
--- a/tests/unit/capi/GEOSGeom_setPrecisionTest.cpp
+++ b/tests/unit/capi/GEOSGeom_setPrecisionTest.cpp
@@ -9,10 +9,10 @@ namespace tut {
 // Common data used in test cases.
 struct test_capigeosgeomsetprecision_data : public capitest::utility {
 
-    GEOSWKTWriter* wktw_;
-    GEOSGeometry* geom1_;
-    GEOSGeometry* geom2_;
-    GEOSGeometry* geom3_;
+    GEOSWKTWriter* wktw_ = nullptr;
+    GEOSGeometry* geom1_ = nullptr;
+    GEOSGeometry* geom2_ = nullptr;
+    GEOSGeometry* geom3_ = nullptr;
 
     GEOSGeometry*
     fromWKT(const char* wkt)
@@ -32,7 +32,6 @@ struct test_capigeosgeomsetprecision_data : public capitest::utility {
     }
 
     test_capigeosgeomsetprecision_data()
-        : geom1_(0), geom2_(0), geom3_(0), wktw_(0)
     {
         wktw_ = GEOSWKTWriter_create();
         GEOSWKTWriter_setTrim(wktw_, 1);

commit a19a9f44527d4be33992c17294b3579f8e85c792
Author: Even Rouault <even.rouault at spatialys.com>
Date:   Thu Sep 17 00:38:49 2020 +0200

    WKBReader.cpp: fix warning about signed vs unsigned comparison

diff --git a/src/io/WKBReader.cpp b/src/io/WKBReader.cpp
index c38c367..a2b21c9 100644
--- a/src/io/WKBReader.cpp
+++ b/src/io/WKBReader.cpp
@@ -409,7 +409,7 @@ WKBReader::readCoordinateSequence(int size)
     if(targetDim > inputDimension) {
         targetDim = inputDimension;
     }
-    for(unsigned int i = 0; i < size; i++) {
+    for(int i = 0; i < size; i++) {
         readCoordinate();
         for(unsigned int j = 0; j < targetDim; j++) {
             seq->setOrdinate(i, j, ordValues[j]);

commit 7d8efa201d93f39f224758f5e2d15bf3401b1a8b
Author: Even Rouault <even.rouault at spatialys.com>
Date:   Thu Sep 17 00:38:28 2020 +0200

    Orientation.cpp: fix warning about implicit cast of size_t to int

diff --git a/src/algorithm/Orientation.cpp b/src/algorithm/Orientation.cpp
index 64d3c2a..6fc701c 100644
--- a/src/algorithm/Orientation.cpp
+++ b/src/algorithm/Orientation.cpp
@@ -45,7 +45,7 @@ bool
 Orientation::isCCW(const geom::CoordinateSequence* ring)
 {
     // # of points without closing endpoint
-    int nPts = ring->size() - 1;
+    int nPts = static_cast<int>(ring->size()) - 1;
     // sanity check
     if (nPts < 3)
         throw util::IllegalArgumentException(

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

Summary of changes:
 src/algorithm/Orientation.cpp                 | 2 +-
 src/io/WKBReader.cpp                          | 2 +-
 tests/unit/capi/GEOSGeom_setPrecisionTest.cpp | 9 ++++-----
 3 files changed, 6 insertions(+), 7 deletions(-)


hooks/post-receive
-- 
GEOS